🚀 2단계 - 지하철 노선 관리 (지하철 노선 생성)

[jeongwon-iee]

안녕하세요. PR 요청이 많이 늦어졌습니다.

아직 미션을 다 완료하진 않았는데, 코드 변경이 많아져서 리뷰요청 먼저 드려보려고 합니다...!

특별히 리뷰 받고 싶은 내용은 아래와 같습니다.

테스트 코드 작성

• StationFixtures를 만들어 subwayStation 생성한 방법이 적절한지
• 테스트 메서드 레벨에 @DirtiesContext 사용한 것이 적절한지

기능 구현

• 코드작성 순서가 아래와 같았는데 적절할까요?
  • 인수테스트 작성 → 도메인 작성 → 컨트롤러 작성하려니 필요해 dto 작성 → 컨트롤러 작성 → 서비스 작성하려니 인터페이스가 있으면 좋을 것 같아 인터페이스 작성 → 서비스 작성
• 도메인에서 필드를 private으로 선언했는데, 이 때 setter는 어느 범위까지 만들어야할까요? 일단 getter만 열어놓고 필요할 때마다 setter를 만드는 것이 적절할까요?
• dto에서 필드를 private final로 닫아놓은 것

구조

• request/response/domain 으로 패키지를 분리한 것.
• 지하철 역이든 노선이든 구간이든 C,R,U,D 로직이 있으니 인터페이스 만들면 좋을 것 같아 인터페이스 추가한 것
• 그리고 마지막으로 전체적으로 코드를 리팩터링한 것이 미션의 목표와 같은 방향성으로 가고 있는지 염려됩니다.

기능 구현은 좀 더 속도를 내볼게요 ..!
감사합니다 :)

[boorownie]

코드 작업 단위가 크면 막막할 수 있는데 중간에 리뷰요청 잘 해주셨어요 👍
질문 주신 부분 위주로 코멘트 남겼고 추가로 몇가지 더 남겼으니 이어서 진행해주세요!

질문

코드작성 순서가 아래와 같았는데 적절할까요?

• 네 인수 테스트를 통해 요구사항 이해를 한 후 기능 구현은 필요한 순서에 따라 진행 되는 것 좋습니다!
• 순서가 정해진 건 없고 TDD를 해 가는데 있어서 편한 방식을 사용하시면 됩니다.
• 개인적으로는 인수 테스트 -> 컨트롤러 -> 서비스 테스트 -> 서비스 순으로 가거나 인수 테스트 -> 컨트롤러 -> 도메인 테스트 -> 도메인 순으로 가는걸 선호합니다.
• DTO는 굳이 테스트를 만들지 않고 필요할 때 바로 바로 만들어줍니다.

setter 작성 범위

• 이러한 이유로 인해 저는 가급적이면 setter를 선호하지 않습니다.
• 필요할 때 getter를 만들고 객체의 상태를 변경해주어야 한다면 setter 메서드를 만들기 보다는 setter와 다른 이름(상태를 변경하는 의도나 목적이 드러나는)의 메서드를 만들어 줄 것 같습니다!

[boorownie]

강의에서 다루었는데 테스트 환경을 초기화 해주는 방법은 여러 방법이 있습니다.
DirtiesContext를 사용해도 초기화해주는데 큰 문제는 없지만
인수 테스트가 많아지면 속도가 느려질 수 있어서 가급적 사용하지 않고
다른 방법으로 사용하는 걸 권장하고 있습니다.

아래의 방법 등 "인수 테스트 초기화" 키워드로 찾아보시면 좋을 것 같아요!
https://newwisdom.tistory.com/95

[boorownie]

this를 붙이는 건 의미상 큰 차이는 없지만
저는 개인적으로 가독성을 위해 생성자가 아니면 제거하는 편 입니다.

[jeongwon-iee]

저는 private 멤버에는 this를 붙이는 패턴을 가지고 있었는데 말씀 주신 덕에 다시 한 번 생각해보게 되네요! 이 기회에 this를 왜 쓰는지 짚고 넘어가야하겠습니다...

[boorownie]

서비스 레이어의 인터페이스 분리 시도 👍
하지만 큰 목적 없이 반복적으로 인터페이스를 분리하신거라면 바로 구현체를 만들어도 좋다고 생각합니다.

아래 링크 참고해주세요!
https://velog.io/@hsw0194/Spring-Boot%EC%97%90%EC%84%9C-interface%EB%A5%BC-%EC%82%AC%EC%9A%A9%ED%95%B4%EC%95%BC-%ED%95%A0%EA%B9%8C

[jeongwon-iee]

StaionService, LineService, StationService 모두 동일하게 생성, 조회, 목록, 삭제 메서드가 있기에
SubwayService로 인터페이스를 만들어놓고 재사용하는게 유지보수 측면에서 좋을 것 같다고 생각했습니다 ☻