멋쟁이사자처럼 동아리/코드 리뷰7 코드리뷰 리뷰어 : 이동호 Spring JDBC 안녕하세요 지은님 ^^ 요번 리뷰어 이동호입니다.지은님께서 궁금하셨던 부분 2가지를 중점적으로 코드 리뷰를 진행했습니다.혹시 틀렸던 부분이나, 궁금하신 부분들은 댓글로 남겨주시면 피드백 혹은 답변해드리겠습니다. :)PlusUltraCode 지은님의 대부분의 예외처리를 해당 객체 안에서 예외 처리 함수를 만들고 진행하셨습니다. 만약 예외 처리가 한두 개가 아닌 많아지게 될 경우, 해당 객체는 해야 할 일들이 있는데 예외 함수에 가려질 수 있다고 생각해요. 예외 처리 함수들을 따로 클래스 객체로 만들어서 사용하는 게 더 좋아 보입니다. 백엔드 팀 리더분들의 코드를 살펴봤는데 Record를 사용하여 예외 처리를 진행한 것을 봤습니다.jelee2555 감사합니다! 그 부분 참고하여 커밋해보겠습니다.src/mai.. 2024. 6. 22. 코드 리뷰 리뷰어 : 이동호 로또 미션 안녕하세요 정현님, 코드 리뷰어 이동호입니다.과제하느라 고생이 많으셨어요. 코드 리뷰를 진행하겠습니다. ^^PlusUltraCode 정현씨의 대부분의 예외처리를 해당 객체 안에서 예외 처리 함수를 만들고 진행하셨습니다. 만약 예외 처리가 한두 개가 아닌 많아지게 될 경우, 해당 객체는 정말 해야 하는 일들이 있는데 예외 함수에 가려질 수 있다고 생각해요. 예외 처리 함수들을 따로 클래스 객체로 만들어서 사용하시는 게 더 좋아 보입니다. 백엔드 팀 리더분들의 코드를 살펴봤는데 Record라는 것을 사용하여 예외 처리를 진행한 걸 봤습니다.Anhye0n 감사합니다! 그 부분 참고하여 커밋해보겠습니다.src/main/java/domain/GenerateLotto.javaPlusUltraCode ArrayLis.. 2024. 6. 22. 코드리뷰 리뷰어 : 이동호 Spring MVC 안녕하세요 정현님! 미션 고생하셨어요. :)코드가 전체적으로 깔끔했습니다. 객체 분리도 잘 되어있습니다. 예외 상황 또한 잘 정리하셨어요 :)[질문] 예외 처리를 하였지만 이 예외처리들을 어떻게 효율적으로 관리할 수 있을지, learning-test에 있는 코드처럼 global 예외처리 파일을 만들어 처리하는 것이 좋은 방법인지에 대하여 고민 중입니다. 혹시 이 부분에 대해 의견이 있으시다면 공유해주세요.리뷰어 의견 저는 Spring 웹에서 발생하는 예외들은 하나의 클래스 안에서 전역적으로 오류를 관리하는 게 좋다고 생각해요. 이러한 부분이 Spring의 매력이라고 생각합니다. 그 외의 오류(데이터베이스 오류 및 Spring 핵심 모듈)는 각각의 파일들을 만들어 예외 상황을 정리하는 편입니다. 예를 들어.. 2024. 6. 22. 코드리뷰 Spring Core 안녕하세요 동호님. 저도 미션을 수행하다가 이 문제를 겪어서 혹시 도움이 될까 적어봅니다!아마도 클라이언트에서 time : Long의 형식으로 Request를 보내는 것 같아요. reservation RequestDTO의 timeId -> time으로 필드 명 바꾸시면 해결될 것 같습니다!sangu1026: 안녕하세요 동호님, 우선 미션하시느라 고생 많으셨습니다 👏 저도 테스트가 모두 통과하여서 잘 몰랐었는데 실제로 reservation을 등록하는 페이지에서 등록이 잘 안되는 문제가 발생하더라구요...저도 ReservationRequestDto에서 timeId로 time의 id를 받았었는데 알고 보니 time으로 받아야지 컨트롤러에서 @RequestBody로 ReservationRequestDto에 매.. 2024. 6. 22. 이전 1 2 다음
