주특기 Lv.4 기술 매니저님 코드 리뷰
1. README.md를 너무 잘 작성하셨습니다. Use Case Diagram, API 명세서, ERD 그리고 추가 설명까지 너무 잘 썼습니다.
2. UserController api에 'users'가 있어야합니다. 그런데 이유를 들어보니 security 때문에 그랬다고 하시니, 이 컨트롤러가 로그인, 로그아웃과 관련된 컨트롤러더라구요. 이런 경우엔 UserController가 아니라 AuthController 이런 식으로 짓는게 좋을 것 같아요.
3. 보통 가독성 때문에 stream을 연속으로 잘 쓰진 않아요. 피하는게 좋을 것 같아요.
4. 쿠팡피셜 좋아요와 좋아요 취소는 하나의 api로 하는게 아니라 나누는게 좋다고 했습니다.
5. 전체적으로 너무 잘하셨습니다.
'항해99' 카테고리의 다른 글
항해99 주특기 프로젝트 (2일차) (0) | 2024.01.13 |
---|---|
항해99 주특기 프로젝트 (1일차) (0) | 2024.01.12 |
항해99 주특기 대기 (1일차) (0) | 2024.01.10 |
항해99 주특기 Spring Lv.5 (1일차) (0) | 2024.01.10 |
항해99 주특기 Spring Lv.4 (1일차) (1) | 2024.01.08 |