◎ BridgeGame isContinue()
- 어긋난 요구사항
- 함수(또는 메서드)의 길이가 10 라인을 넘어가지 않도록 구현한다.
- 변경 내용
isContinue()
에서 예외적인 상황을 처리하는 부분은 두고, route와 bridge를 비교하는 부분을 따로 메서드로 분리함
- Naming은 길이 올바른지 확인하는 메서드는
isCorrectRoute()
, 한 칸이 올바른지 확인하는 메서드는 isCorrectRouteSpace()
로 명명함
- 수정 전후 비교 (해당 커밋)
◎ OutputView
- 반영하지 않은 피드백
- 상수(static final)를 만들고 이름을 부여해 이 변수의 역할이 무엇인지 의도를 드러내라.
- 변경 내용
- OutputView에서 출력해야 하는 String을 전부 상수로 선언하여 사용함
- OutputView에서 사용하는 상수가 많아져 OutputViewText로 분리하고 이를 extends하여 사용함
- 출력 방식 변경 시에
System.*out*.println(string)
부분을 전부 변경해야 하므로 print(String string)
메서드로 사용함
- 수정 전후 비교 (해당 커밋)