1차 프로젝트 리팩토링

• 예정일 : 2022/8/2(화)

리팩토링 목록

• 리팩토링 - 내일(8/2) 피처 개발 완료하고 밤중으로 생각해오기
  • Giver, Service
    • TeamService의 findById, TeamGiverService findById 중복
    • TeamMemberService TeamMemberGiverService의 exists 중복
  • 네이밍 (DTO, Update**, **Update, 체크 로직, entity에 로직넣는 것)
  • 테스트 네이밍 testXXXX, XXXXTest , XXXXX
  • 테스트 코드 private
  • 클래스 private 함수는 맨 밑으로 내리는가?
    • 혜빈 : 공통 함수는 맨 밑으로 내리고 ~ 하나의 메서드에서만 사용하는건 그 메서드 바로 밑 !
    • 클린 코드에 나왔었다고.. 사용되는 메서드의 바로 밑에 사용하는 것이 가독성이 좋다. (추천)
    • 진형 : 혜빈님 말씀 처럼 여러 함수에서 사용되는 함수는 맨밑으로 내리고 하나의 메서드에서만 사용되면 바로 밑에 두는것도 좋은 방법인 것 같습니다.
    • 곽씌 : 부속품 같은 메서드는 사용 메서드 밑에, 공용 함수는 밑으로
  • 컨벤션도 확인해주세요 ~

Giver 의존성 문제

혜 빈그래프 - a4 용지가 없어요 기부해주세요 👍 👍 👍

• Giver, Service

  • TeamService의 findById, TeamGiverService findById 중복
  • TeamMemberService TeamMemberGiverService의 exists 중복

  형욱

  • Facade 해야겠따.

  진형

  • Facade를 해도되고 Giver를 해도된다.

DTO 네이밍

• 생성에 쓰는 경우 Create, Update, 등의 키워드를 넣을 건지

  • 엘리하이해 : 무슨 말인지 잘 모르겠어용..

• @Schema (Swagger) 어노테이션은 클라이언트에게 나가는 경우에만 명세

  • 혜빈
  • 동운

• inner class인 경우에 response 이름을 제외해도 되지 않은지?

  

  • 동운
    • Response 제외 찬성
  • 혜빈
    • 중복이니 제외 해도 무방
  • 진형
    • Response 제외 찬성
  • 형욱
    • Response 제외 찬성
  • 병연
    • 찬성

• dto 변수명 어떻게 할것인지 ??

  

패키지 관련

• Team 도메인 관련은 모두 Team으로 묶어야 되지 않을런지?

  

  • 혜빈 : 동운님이 복수 괜찮으시면 저는 좋습니당