chwijuntime / chwijuntimeserver Goto Github PK

            Tag tag = tagRepository.findByTagName(i);
            if(tag == null) {
                throw new NotFoundTagException();
            }

이 코드가 이곳 저곳 있는데 Optional로 처리해서 orElseThrow를 해도 되지 않을까요?

• Header에 Authorization 이 없는 경우에 Exception 처리는 어떻게 되는 걸까요?
• jwtToken 값을 substring 해서 사용 하는데, jwt Token을 그냥 사용해도 되는 건가요??
  • 이 부분은 JwtTokenProvider 클래스 확인 하였습니다.
• userEmail 을 Token 에서 가져왔다고 가정하더라도 User 정보에 해당 Email이 존재하지 않을 경우에도 Exception 처리가 필요해 보이네요.

Exception 마다 Class를 만들고 있는데요,
성격이 비슷한 Exception 끼리 묶고 자세한 내용은 Enum 을 만들어서 사용하는건 어떨까요??

예를들어, DataNotFoundException 을 만들고, 자세한 내용은 DataNotFoundType 이를 Enum으로 관리하는거죠.
방법이 정해진 것은 아니지만 관련 내용은 검색해보시면 자료 많이 나올거에요.

74~78라인까지 findByApplicationEmploymentStatus() 함수에 파라미터로 enum을 넣고 있는데 if문에 비교한 enum값과 동일한 값을 넣고있는데, 그냥 status 변수를 넘겨도 될 것 같습니다.