※ 구현하며 신경 쓴 부분

◎ 공통 Constants 값 정리

• 다른 Class들에서 공통적으로 사용하는 상수는 따로 선언하고 같이 관리할 필요가 있다고 생각함
  • 만약 Class마다 클래스 상수로 선언하여 사용한다면, 추후 수정시에 일일히 다 찾아 수정해주어야 하기 때문이다.
• 그래서 Constants class를 만들어 public static final 로 값을 만들어 사용함
  • static : 객체 생성할 필요 없이 사용하기 위해
  • final : 런타임 중 해당 값의 변경을 막기 위해
  • 이후 다른 class에서도 Constants의 field 값을 사용함
• 변경 전후 코드
  • 변경 전
  • 변경 후

◎ InputViewData

• InputView의 기능은 공통적인 순서로 진행한다.

  • 입력 요청 문구 출력
  • 입력을 받고, 입력 형식 확인
  • 입력 형식이 일치하지 않으면 에러 메세지 출력

• 따라서 같은 형식으로 진행되나 바뀌는 데이터는 다음과 같다.

  • 입력 요청 문구
  • 형식이 올바른지 확인하는 메서드(함수)
  • 에러 문구

• InputViewData 통해 세가지 정보를 가지고 있게 함

  • InputViewData

• 사용한 기술들

  • 함수형 인터페이스, 람다식

    • 람다식과 함수형 인터페이스를 통해 메서드(함수) 자체를 인자로 가지는 효과가 있음

  • 문자열 형식 지정 : String.format()

    • 가변적인 String을 훨씬 보기 좋게 작성할 수 있다.

    // 변경 이전
    "[ERROR] " + Constants.MIN_LENGTH + " 이상 " + Constants.MAX_LENGTH + " 이하의 숫자만 입력할 수 있습니다."
    
    // 변경 이후
    String.format("[Error] %s 이상 %s 이하의 숫자만 입력할 수 있습니다.", Constants.MIN_LENGTH, Constants.MAX_LENGTH)
    

• 구현

  • InputView
  • readBridgeSize() , readMoving() , readGameCommand() 가 InputViewData 만 변경되고 같은 메서드에서 실행이 된다.
  • 코드의 중복을 많이 줄일 수 있었다.

※ 리팩토링

◎ 테스트 코드 분리

• 과제에서 제공한 ApplicationTest를 참고하여, 테스트, 데이터 부분과 도움을 주기 위한 도구 두 부분으로 파일을 분리함
• 변경 전후 코드 (해당 커밋)
  • 변경 전
  • 변경 후