CoCo, 코드 리팩토링

코드 리팩토링

• 코드 리팩토링 하는 시간을 가졌다.
  먼저, 본인과 팀원들의 코드를 리뷰 & 질문하고, 조금 더 나은방식으로 코드를 짜보기 위해서 튜터님들의 의견을 반영하고 찾아보면서 수정했다. 우여곡절이 있었고 아직도 해결되지 않은 문제가 있지만, 한발자국 더 나아간 것 같다고 생각한다.

Member

⬇︎⬇︎⬇︎⬇︎

• return null을 null값으로 처리하지 말고, Optional.empty()을 이용해서 처리하라는 튜터님의 말씀이 있어서 수정했다.

  

• 참고한 링크 : https://mangkyu.tistory.com/70

MessageService

• 원래 중복검사가 싫어서

Member member = memberDetails.getMember();

로 간단하게 처리했는데, 영속성이 없어서 DB에 저장할 수가 없음. 어쩔 수 없이 중복검사해서 영속성 부여해줌.

• Optional로 처리한 이유는, optional로 안하고 repository에서 찾아오면 null값을 orElse(null) 또는 orElseThrow로 예외를 처리해줘야함
  ↳ orElseThrow로 예외를 던지면 무조건 500번 에러가 뜨기 때문에 상황에 맞게 예외처리하려고 바꾼 것.
• Member를 가져올 때 Optional을 안쓸 수 있는 방법은 없는지....중복검사 안할 순 없는지 알아봐야겠다.