DAY 32

🔖 오늘 읽은 범위 : 15. JUnit 들여다보기(334~342p)

---

🤓 책에서 기억하고 싶은 내용

시간 결합을 외부에 노출

• findCommonSuffix 에는 숨겨진 시간적인 결합이 존재
• findCommandSuffix는 findCommonPrefix가 prefixIndex를 계산한다는 사실에 의존하므로 잘못된 순서로 호출하면 디버깅이 어려워짐
• findCommonSuffix를 수정하여 prefixIndex를 인수로 넘기자

private void compactExpectedAndActual() {
    prefixIndex = findCommonPrefix();
    suffixIndex = findCommonSuffix(prefixIndex);
    compactExpected = compactString(expected);
    compactActual = compactString(actual);
}
private int findCommonSuffix(int prefixIndex) {
    int expectedSuffix = expected.length() - 1;
    int actualSuffix = actual.length() - 1;
    for (; actualSuffix >= prefixIndex && expectedSuffix >= prefixIndex; actualSuffix--, expectedSuffix--) {
        if (expected.charAt(expectedSuffix) != actual.charAt(actualSuffix))
            break;
    }
    return expected.length() - expectedSuffix;
}

• 불필요하게 인수를 추가한 느낌이 있다 다시 수정해보자

private void compactExpectedAndActual() {
    findCommonPrefixAndSuffix();
    compactExpected = compactString(expected);
    compactActual = compactString(actual);
}
private void findCommonPrefixAndSuffix() {
    findCommonPrefix();
    int expectedSuffix = expected.length() - 1;
    int actualSuffix = actual.length() - 1;
    for (; actualSuffix >= prefixIndex && expectedSuffix >= prefixIndex; actualSuffix--, expectedSuffix--) {
        if (expected.charAt(expectedSuffix) != actual.charAt(actualSuffix)) break;
    }
    suffixIndex = expected.length() - expectedSuffix;
}
private void findCommonPrefix() {
    prefixIndex = 0;
    int end = Math.min(expected.length(), actual.length());
    for (; prefixIndex < end; prefixIndex++)
        if (expected.charAt(prefixIndex) != actual.charAt(prefixIndex)) break;
}

• findCommonPrefixAndSuffix로 묶어 findCommonPrefix를 먼저 호출

private void findCommonPrefixAndSuffix() {
    findCommonPrefix();
    int suffixLength = 1;
    for (; !suffixOverlapsPrefix(suffixLength); suffixLength++) {
        if (charFromEnd(expected, suffixLength) != charFromEnd(actual, suffixLength))
            break;
    }
    suffixIndex = suffixLength;
}
private char charFromEnd(String s, int i) {
    return s.charAt(s.length() - i);
}
private boolean suffixOverlapsPrefix(int suffixLength) {
    return actual.length() - suffixLength < prefixLength ||
        expected.length() - suffixLength < prefixLength;
}

• findCommonPrefixAndSuffix 를 정리하였더니 suffixIndex 가 실제로는 0이 아닌 1부터 시작하는 ‘길이’ → suffixLength 로 변경하자

public class ComparisonCompactor { ...
    private int suffixLength;
    ...
    private void findCommonPrefixAndSuffix() {
        findCommonPrefix();
        suffixLength = 0;
        for (; !suffixOverlapsPrefix(suffixLength); suffixLength++) {

            if (charFromEnd(expected, suffixLength) != charFromEnd(actual, suffixLength))
                break;
        }
    }
    private char charFromEnd(String s, int i) {
        return s.charAt(s.length() - i - 1);
    }
    private boolean suffixOverlapsPrefix(int suffixLength) {
            return actual.length() - suffixLength <= prefixLength ||
                expected.length() - suffixLength <= prefixLength;
        }
        ...
        private String compactString(String source) {
            String result =
                DELTA_START +
                source.substring(prefixLength, source.length() - suffixLength) + DELTA_END;
            if (prefixLength > 0)
                result = computeCommonPrefix() + result;
            if (suffixLength > 0)
                result = result + computeCommonSuffix();
            return result;
        }
        ...
        private String computeCommonSuffix() {
            int end = Math.min(expected.length() - suffixLength + contextLength, expected.length());
            return
            expected.substring(expected.length() - suffixLength, end) + (expected.length() - suffixLength <
                expected.length() - contextLength ?
                ELLIPSIS : "");
        }
}

• computeCommonSuffix 에서 +1을 없앰
• charFromEnd에 -1 추가
• suffixOverlapsPrefix 에 ≤ 사용

private String compactString(String source) {
    return
    computeCommonPrefix() +
        DELTA_START +
        source.substring(prefixLength, source.length() - suffixLength) + DELTA_END +
        computeCommonSuffix();
}

• 불필요한 if 문 제거 if (suffixLength > 0)
  • suffixLength가 1 감소했으므로 ≥ 로 고쳐야 하지만 if문은 길이가 0인 접미어를 걸러내 첨부하지 않으므로 if문이 의미가 없어 제거

번복

• 리팩터링 하다보면 원래 했던 변경을 되돌리는 경우가 흔하다. 원래 수많은 시행착오를 반복하는 작업이다

결론

• 우수한 모듈일지라도 개선이 불필요한 모듈은 없다
• 코드를 처음보다 더 깨끗하게 만드는 책임은 우리 모두에게 있다

🤔 떠오르는 생각

🔎 질문

📝 소감 3줄 요약

• 리팩터링은 원래 번복도 하고 수많은 시행착오를 반복하는 작업이다
• 개선이 불필요한 모듈은 없다 우수한 모듈일지라도.
• 코드를 처음보다 더 깨끗하게 만드는 책임은 우리 모두에게 있다