Skip to content

♻️ refactor[#58] refactoring order api#61

Merged
sohyun119 merged 11 commits intodevfrom
feature/58-refactoring-order-api
May 21, 2025
Merged

♻️ refactor[#58] refactoring order api#61
sohyun119 merged 11 commits intodevfrom
feature/58-refactoring-order-api

Conversation

@sohyun119
Copy link
Member

✨ 변경 타입

  • 신규 기능 추가/수정
  • 버그 수정
  • 리팩토링
  • 설정
  • 비기능 (주석 등 기능에 영향을 주지 않음)

✅ 체크리스트

  • 코드 작성 가이드라인을 준수했는가?
  • 변경 사항에 대한 테스트를 완료했는가?
  • 문서 변경이 필요한 경우, 해당 내용을 반영했는가?

🚀 변경 내용

  • 커밋내용 참고 부탁드려용

* QClass 새로 생성
* 패키지 구조 변경
* Service 예외처리 방식 수정
* 특정 음식점 리뷰 조회시 : 평균 별점값 반환 추가
@sohyun119 sohyun119 added the 🔨 Refactor 코드 리팩토링 label Mar 29, 2025
@sohyun119 sohyun119 self-assigned this Mar 29, 2025
sohyun119 added 8 commits May 21, 2025 16:29
- OrderItem이 FK 컬럼 가지게 설정
- Order는 mappedBy로 객체 지향적으로 리스트만 관리
- 양방향 연관관계를 사용하여 가격 총 합 등을 보다 쉽게 관리 할 수 있도록 함
- Order 에서 totalPrice 를 관리함으로써 객체지향적으로 리팩토링
- Order <-> OrderItem 양방향 관계를 설정하여 객체지향적으로 리팩토링
Copy link
Contributor

@mummumm mummumm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니당

Copy link
Member

@chokyuseong00 chokyuseong00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다

@sohyun119 sohyun119 merged commit e9809a6 into dev May 21, 2025
@sohyun119 sohyun119 deleted the feature/58-refactoring-order-api branch May 21, 2025 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 Refactor 코드 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants