240604 TIL

이번 개인별 Spring 숙련 과제를 하는데 받은 피드백들이다.

 

- @Getter만 있는 Dto에 필드들은 final 처리해주면 좋습니다.

- LoginResponseDto에 msg 필드는 불필요한 것으로 보이는데, 사용하지 않는다면 삭제하는게 좋습니다.

- 강의자료에 이렇게 구현되어 있긴한데, UserRoleEnum 뒤에 굳이 Enum 이란 postfix를 붙일 필요는 없습니다.

- 강의자료에 이렇게 구현되어 있긴한데, UserRoleEnum 내부에 Authority 클래스는 외부에서 사용하지 않는다면 private로 변경하면 좋을 것 같습니다.

- git log에 몇 단계를 작업했는지도 추가해주면 좋을 것 같아요.

 

앞으로 final처리와 private 사용에 대해 조금더 고민해 봐야 할것같다.