Skip to content

[Spring Core] 박채연 미션 제출합니다. #498

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

Open
wants to merge 3 commits into
base: qkrcodus
Choose a base branch
from

Conversation

qkrcodus
Copy link

@qkrcodus qkrcodus commented Jul 4, 2025

안녕하세요 ! 이번에도 리뷰 잘 부탁드립니다 🙇‍♂️

미션 회고

  • 이전 미션에서 Reservation 도메인에서 필드 유효성 검사를 할 때,
각 검증 조건마다 다른 예외 메시지를 반환할 수 있도록 validate() 메서드를 만들었다.
    이번 미션에서도 이와 유사하게 Time 객체를 검증했다.

  • js파일에서 선택된 시간의 값을 time이 아닌 timeId로 수정해 ReservationRequest DTO가 해당 값을 받을 수 있게 수정했다.
    이로 인해 기존 예약 추가 API 스펙에 맞춰서 요청을 보낼 경우 에러가 발생한다.

  • Reservation 엔티티가 Time 엔티티와 연관 관계를 갖도록 수정하며, reservation 테이블에 time_id 외래 키를 추가하고, ReservationRepository에서는 예약 정보를 조회할 때 INNER JOIN을 사용했다.
    이러한 객체 간 연관 관계가 생기면서 변경이 필요한 부분을 수정했다.

@qkrcodus qkrcodus changed the title [Spring Core] - 박채연 미션 제출합니다. [Spring Core] 박채연 미션 제출합니다. Jul 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant