Feat/#25 title

[ahah525]

저번주, 이번주 구현 내용입니다. 구현 기능 관련해서는 아래 일지를 참고해주시면 감사하겠습니다!!!

• 11

• 25

• 32

• 3week_23.02.02(목)~23.02.08(수) 일지

• 4week_23.02.09(목)~23.02.15(수) 일지

[ahah525]

작성해주신 코드 잘봤습니다!!

코드 가독성이 매우 좋아서 리뷰하기에도 굉장히 편했습니다 👍 특히 회원관련 로직들은 jwt, session 등을 활용해서 코드를 파악하기 힘들줄 알았는데 코드를 처음 보는 사람도 로직을 파악할 수 있도록 잘 작성해주셔서, 작성해주신 회원 도메인에 대해 날잡고 코드를 분석하고 싶다는 생각이 들었습니다!! 저도 승연님처럼 가독성이 좋은 코드를 작성하려고 노력해야겠네요 😄

추가적으로, 다음과 같이 예외가 발생했을 때 작성해주신 코드를

throw new RuntimeException("memberTitle 없음");

저희가 전부터 작성했던 RuntimeException 포멧인 BusinessException 클래스를 활용하는건 어떨까 싶습니다!!

고생많으셨고 코드 보면서 많이 배웠습니다 감사합니다 👍

범서님 코드리뷰 정말 꼼꼼하게 남겨주셔서 감사합니다!!! 덕분에 저도 많이 배우네요ㅋㅋ 제가 코드를 짤 때 가장 신경쓰는 부분 중 하나가 가독성입니다..ㅎㅎ 물론 코드 자체만으로도 읽기 쉬운 코드면 정말 좋겠지만.. 사실 그러기가 쉽지 않죠ㅎㅎ 협업을 하고 서로 코드리뷰를 하면서 많이 느꼈겠지만 생각보다 남이 짠 코드를 이해하는게 쉽진 않더라구요ㅠ 이게 코드 자체의 가독성이 떨어진다기 보다는 다른 도메인 API 개발에 대한 기본 지식이나 흐름에 대한 이해가 부족하기 때문에 더 어렵다는 생각을 했습니다

특히 인증 관련 부분은 제가 짠 코드지만 며칠 지나니까 또 기억이 희미해져서 대충 흐름을 주석으로 작성해두지 않으면 안되겠더라구요ㅠㅠ 사실 인증 구현 관련해서 글로 정리하려고 했는데 이것저것 할게 너무 많아서 계속 미루게되네요..😅

말씀하신 예외를 던지는 코드같은 경우에는 제가 매번 예외 클래스를 만들기는 애매해서 아무생각없이 RuntimeException()을 던졌는데, 비즈니스 로직에서 발생하는 예외는 모두 BusinessException() 으로 대체해서 사용하면 될까요??!!

[ahah525]

카카오로그인과 JWT를 구현하시면서 보안과 편의성을 어떻게 하면 둘 다 잡을 수 있을까 라는 고민을 많이 하신 게 느껴집니다! 구현 일지를 정말 꼼꼼하게 정리해주셔서 카카오 로그인과 JWT를 같이 사용했을 경우 작동 원리를 깊게 파악할 수 있었습니다 저도 많이 배우고 갑니다 감사합니다 👍 그리고 개인적으로 API 문서 작성하신 걸 보니, 저희도 이미 노션으로 API 문서를 관리하고는 있지만 더 깔끔하고 응답 샘플까지 보여줄 수 있는 API 문서 툴을 도입하면 좋겠다는 생각이 드네요! 제가 감명깊게 본 또 다른 API 문서는 gitbook을 이용한 API 문서였는데 다음 회의 때 같이 얘기해보면 좋을 것 같아요!

정말 정말 고생 많으셨습니다 🥹👍

칭찬의 말씀 정말 감사합니다!!😄 민지님이 말씀하신 것처럼 기존의 API 문서에는 간단한 정보(METHOD, URI, 설명)만 나타나있어 실제 API 호출을 테스트할 때 참고할 수 있는 더 자세한 문서툴의 필요성을 느끼게 되었습니다. 그리고 작성한 코드를 머지하기 이전 회의시간에 팀원과 공유의 목적도 있었습니다. 추후 프론트 협업이 확정된다면 API 문서화가 더 필수적인 상황이라 저 또한 API 문서화 툴을 도입하는 것에 대해 이야기를 꺼내보려고 했었습니다! 말씀해주신 자료도 잘 보았습니다ㅎㅎ 다른 분들께도 해당 내용을 공유하고 다음 회의 때까지 시간이 조금 있으니 여러 툴들을 더 조사해는 것도 좋을 것 같아요!!