멘토링 정리 - JH 멘토님
2월 9일 오전 10시 30분
< 깃허브 Readme 부분 >
프로젝트를 포트폴리오로 제출하면 깃허브까지 확인할 수 있으니 참고할 수 있는 자료들을 더 많이 추가하면 좋겠습니다.
지금도 잘 정리했지만 개발하면서 생겼던 이슈들이나 트러블 슈팅, API 문서 같은 것도 추가로 정리하면 좋을 것 같습니다.
시간이 조금 지나고 리펙토링이나 추가 기능들을 구현하면서 정리하면 프로젝트를 마무리하고 돌아보는 좋은 시간이 될 것 입니다.
(참고하면 좋을 자료들)
https://github.com/woowacourse-teams
< 소스코드 리뷰 >
부분 부분 사용하지 않는 변수들이 있는데 전체적으로 리펙토링하면서 정리하면 좋을 것 같습니다.
그리고 지금 로그아웃 부분에 HttpServletRequest가 서비스로 넘어오고 있는데 HTTP 관련 부분은 컨트롤러에서 처리하는 것이 더 좋을 것 같습니다. 이 부분과 쿠키 관련 부분에 대해서 추가로 고민이 더 필요합니다.
이메일 부분은 인터페이스로 가져가도 좋을 것 같습니다. 지금 상황에서는 문제 없지만 이메일에서 알림톡이나 SNS, 문자 등으로 방법을 바꿀 수도 있기 때문에, 이렇게 서비스를 운영하면서 변경 가능성이 있는 부분들은 인터페이스로 가져가면 이후 수정과 유지보수를 더 쉽게 할 수 있습니다.
code.equals(verificationCode);
이런 코드에서 지금 NullPointerException이 발생할 수 있습니다. code 가 null 인 경우가 생길 수 있기 때문인데, verificationCode 부분은 null 이 아니기 때문에 null 이 아님이 확정된 변수를 앞에 적으면 NullPointerException을 해결할 수 있습니다.
null이_아닌_변수.equals(null이어도_상관_없음) (이렇게)
IDE에서 코드 옆에 오류 원인 출력되는거 “Inspection Lens” 플러그인 사용하고 있습니다.
코드 중에 객체를 두번 저장하는 부분이 있는데 한번만 될 수 있도록 수정하면 좋겠습니다.
< 피피티 리뷰 >
요구사항 분석 등의 부분에 글로 많이 되어 있는데 플로우 차트 같이 각 기능들을 그림으로 정리하면 이해가 더 잘 될 것 같습니다.
API 명세서에 부분부분 URL이 “/api/**” 이 없는 부분이 있습니다.
개발 중에 수정했다면 명세서도 같이 수정하면 좋겠습니다.
