-
Notifications
You must be signed in to change notification settings - Fork 117
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Spring Core] 한경준 미션 제출합니다. #269
base: hkjbrian
Are you sure you want to change the base?
Conversation
# Conflicts: # src/main/java/roomescape/SpringConfig.java # src/main/java/roomescape/controller/ReservationController.java # src/main/resources/application.properties # src/main/resources/schema.sql # src/test/java/roomescape/MissionStepTest.java
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이번주 과제도 고생하셨습니다!
아직 스프링이 익숙하지 않아서 경준님 코드를 보면서 많이 배울 수 있었습니다!
전체적으로 스터디 때 받은 피드백을 고려하신 것 같아 리뷰하기에 수월했습니다.
앞으로 함께 열심히 공부해 보아요 👍
try { | ||
jdbcTemplate.update(sql, id); | ||
} catch (DataAccessException e) { | ||
throw new ReservationExceptionHandler.NoReservationException("존재하지 않는 시간입니다."); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
시간에 대한 예외 처리를 다루는 클래스를 따로 만드는 게 더 좋을 것 같아요!
@@ -0,0 +1,7 @@ | |||
package roomescape.domain.dto; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dto를 잘 사용해주신 것 같아요 👍
return new ResponseTime(id, requestTime.time()); | ||
} | ||
|
||
public List<ResponseTime> findAll() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sql문을 잘 활용하신 것 같아요!
안녕하세요 리뷰어님 개인 사정으로 현재 step 9까지만 진행된 상태로 step 10이 완료되면 다시 말씀드리겠습니다..!
최대한 빠르게 완료해보도록 하겠습니다 죄송합니다..!
커밋 범위 : step 8
커밋 범위 : step 9 ~ step 10