본문 바로가기

전체 글406

코드리뷰 리뷰어 : 이동호 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.
코드리뷰 Lotto mission 안녕하세요 동호님 :) 미션 수행하느라 수고 많으셨어요 👍코드를 천천히 읽어보았는데 실제 고객이 로또를 사는 것처럼 미션을 구현하신 게 인상 깊었어요. 코드 전체적으로 객체가 해야 하는 일과 가져야 할 정보를 통제하려고 노력하신 것 같고, 예외처리나 테스트를 보면서 많은 시간을 들여서 꼼꼼하게 구현하셨다고 느꼈습니다.동호님이 요청하신 피드백 사항원시값 포장: 중요하고 안전하게 다뤄져야 하는 값은 객체로 포장하는 게 좋다고 생각합니다. [객체가 많아져도 좋아요 👍] 다만, BonusBall 객체 혹은 Lotto를 나타내는 객체는 구성 요소인 Integer가 모두 1에서 45 사이의 범위인지 확인하기 때문에 유효성 검사의 코드가 중복되고 있었습니다. Integer가 아닌 LottoNumber라는 객체를 .. 2024. 6. 22.
코드리뷰 Spring MVC 코드 리뷰 요약 및 권장사항HomeController.java리뷰어 의견:hkjbrian: @GetMapping을 통해 기본 페이지를 설정해주신 것 같아요. 물론 가능한 방법이지만, 스프링은 정적(static) 경로에서 index를 찾고 그 후 템플릿(templates) 경로에서 index를 가장 먼저 찾게 되어있다고 배웠던 내용을 활용하는 것도 좋을 것 같아요!PlusUltraCode: 네, 참고할게요. 감사합니다.ReservationController.java리뷰어 의견 1:hkjbrian: Reservation 클래스를 활용하여 Post 기능을 구현하셨군요. dto를 활용하면 오류가 날 확률이 줄어들고 가독성이 더 좋아질 것 같아요. dto를 활용하여 RequestReservation과 같은 이름으.. 2024. 6. 22.

티스토리툴바