1차 프로젝트 리팩토링
리팩토링 목록
- 리팩토링 - 내일(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를 해도되고 Giver를 해도된다.
DTO 네이밍
-
생성에 쓰는 경우 Create, Update, 등의 키워드를 넣을 건지
-
@Schema
(Swagger) 어노테이션은 클라이언트에게 나가는 경우에만 명세
-
inner class인 경우에 response 이름을 제외해도 되지 않은지?

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

패키지 관련