15장 JUnit

compact 함수 시작부에 캡슐화되지 않은 조건문을 바꾸자

public String compact(String message) {
	if (expected == null || actual == null || areStringsEqual())
...
}

의도를 명확히 표현하라면 조건문을 캡슐화해야 한다.

public String compact(String message) {
	if (shouldNotCompact())
...
}

private boolean shouldNotCompact() {
	return expected == null || actual == null || areStringsEqual();
}

• 그러나 부정문보다 긍정문이 이해하기 쉽다. 부정문은 긍정문으로 바꾸자

public String compact(String message) {
	if (canBeCompacted())
...
}

private boolean canBeCompacted() {
	return expected != null && actual == null && !areStringsEqual();
}

• 긍정문으로 바꿨지만, 함수명이 이상하다.
• 실제로는 canBeCompacted가 false면 압축하지 않는다.
• 즉, 함수에 compact라는 이름을 붙이면 오류 점검이라는 부가 단계가 숨겨진다
• 실제로는 formatCompactedComparison이라는 이름이 적합하다

private boolean formatCompactedComparison(String message) {
...
}

Index vs Length ?

• suffixIndex가 실제로는 접미어 길이라는 사실이 드러난다.
• 이 경우 index가 아닌 length가 더 합당하다
• 실제로 suffixIndex 값은 0이 아닌 1부터 시작한다 (!!)
• 이 경우 if 문이 있으나마나한 구문을 발견할 수 있다

if (suffixLength > 0)