16장 SerialDate 리팩터링(363~369)

논리적 의존성 제거하기

• 논리적으로 의존한다면 물리적으로도 의존해야한다
• DayDate에 getDayOfWeekForOrdinalZero라는 추상 메서드를 구현하고 SpreadsheetDate에서 Day.SATURDAY를 반환하도록 구현
• getDayofWeek 메서드를 DayDate로 옮긴 후 getOrdinalDay와 getDayOfWeekForOrdinalZero를 호출하게 변경

public Day getDayOfWeek(){
	Day startingDay = getDayOfWeekForOrdinalZero();
	int startingOffset = startingDay.index - Day.SUNDAY.index;
	return Day.make((getOrdinalDay() + startingOffset) % 7 + 1);
}

if 문 연쇄를 enum으로 옮겨 연쇄 없애기

public enum DateInterval{
	Open {
		public boolean isIn(int d, int left, int right) {
			return d > left && d < right;
		}
	},
	CLOSED_LEFT {
		public boolean isIn(int d, int left, int right) {
			return d >= left && d < right;
		}
	},
	CLOSED_RIGHT {
		public boolean isIn(int d, int left, int right) {
			return d > left && d <= right;
		}
	},
	CLOSED {
		public boolean isIn(int d, int left, int right) {
			return d >= left && d <= right;
		}
	};

	public abstract boolean isIn(int d, int left, int right);
}

public boolean isInRange(DayDate d1, DayDate d2, DateInterval interval) {
	int left = Math.min(d1.getOrdinalDay(), d2.getOrdinalDay());
	int right = Math.max(d1.getOrdinalDay(), d2.getOrdinalDay());
	return interval.isIn(getOrdinalDay(), left, right);
}

결론!

• 보이스카우트 규칙을 지켰다
• 테스트 커버리지가 증가했다
• 버그 몇 가지를 고쳤다
• 코드 크기가 줄었고 코드가 더 명확해졌다