Feedback01
- [x]
ValidationMethods에서 ID가 null인지 검증하는 3가지 코드 합치기
- 수정 전
- 수정 후
Feedback02
- [x] 이 메서드들은 하나의 메서드로 둘 중 하나만, 둘 다도 변경 가능하게 구현해 보시겠어요?
Optional.ofNullable()과 같은 기능으로 구현할 수 있어요.
JCFUserService 수정 전
JCFUserService 수정 후
JCFChannelService 수정 전
JCFChannelService 수정 후
Feedback03
- [x] isPrivate보다는 channelType로 하시고 ENUM으로 구현하는게 좋겠어요!
- 수정 전
- 수정 후
Feedback04
- [x] 여긴 가독성을 위해 스위치문을 사용하는게 더 좋겠어요
- 수정 전
- 수정 후
Feedback05
- [x] 전체 코드에서 파라미터로 객체를 받지 말고 id 같은 식별자로 받기