TDD, 클린 코드 with Java 18기: 사다리타기 - FP, OOP(게임 실행)

yshjft·2024년 5월 22일
0

사다리타기 - FP, OOP(게임 실행)

리뷰 내용

🤔 리뷰 링크

사다리타기 - FP, OOP: 3단계 - 사다리(게임 실행) #2109


🤔 정적 팩토리 메서드 사용에 관하여

리뷰어

  • 테스트 코드에서만 사용되는 정적 팩토리 메서드라면 테스트 패키지쪽에 별도로 함수를 만드는게 어떨지?
  • 일반적으로 생성자에서 로직을 처리하면 생성 이외 책임을 추가로 갖기 때문에 데이터 변환 가공 등의 작업이 있는 경우 정적 팩터리에서 처리하고 생성자를 호출하도록 구현하는 경우가 많다. 따라서 생성자 파라미터와 정적 팩터리가 받는 파라미터가 동일하고 바이패스 하는 형태라서 정적 팩터리가 없이 생성자를 노출해도 괜찮을거 같다.

  • 다른 단계의 미션에서 정적 팩토리 메서드를 사용하는게 어떨지 피드백을 받았기에 처음에는 조금 고민이 되었습니다. 물론 피드백이 문제가 아니라 그저 기준 없이 좋구나하고 막 사용한 저의 문제였습니다. 그래서 아래 처럼 정적 팩토리 메서드 사용에 대한 나름의 기준을 정해봤습니다.
    • 객체 생성을 위해 값을 포맷해야 한다면 정적 팩토리 메서드를 사용을 고려하자
    • 만약 정적 팩토리 메서드에서 어떠한 포맷도 없이 단순히 값을 생성자로 전달하는 바이패스 역할을 한다면 생성자를 사용하자
    • 테스트 코드에서만 사용되는 정적 팩토리 메서드라면 해당 메서드는 테스트 패키지 쪽으로 분리하자
profile
꾸준히 나아가자 🐢

0개의 댓글