Skip to content
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

[LIME-50] 아이템 목록 조회 정렬 조건 추가 #21

Merged
merged 3 commits into from
Jan 24, 2024

Conversation

Curry4182
Copy link
Contributor

@Curry4182 Curry4182 commented Jan 23, 2024

📌 PR 종류

어떤 종류의 PR인지 아래 항목 중에 체크 해주세요.

  • 🐛 버그 수정
  • ✨ 기능 추가
  •  테스트 추가
  • 🎨 코드 스타일 변경 (formatting, local variables)
  • 🔨 리팩토링 (기능 변경 X)
  • 💚 빌드 관련 수정
  • 📝 문서 내용 수정
  • 그 외, 어떤 종류인지 기입 바람:

📌 어떤 기능이 추가 되었나요?

아이템 목록 조회 정렬 조건 추가 😊❤

Issue Number

LIME -50

기능 설명

추가 된 정렬 조건❤😊

REVIEW_COUNT_DESC(아이템 리뷰 많은 순 정렬),
REVIEW_RATING_DESC(아이템 리뷰 평점 높은 순 정렬),
NEWEST(최신 순 정렬),
PRICE_HIGH_TO_LOW(아이템 가격 높은 순 정렬),
PRICE_LOW_TO_HIGH(아이템 가격 낮은 순 정렬);

정렬 기능 코드 설명❤😊 233c9d6.

  • 정렬할 때 review 테이블의 정보가 필요하여 item id를 기준으로 item과 review를 join 하였습니다.
  • item에 대해 review가 있는 경우 join을 하고 없는 경우에도 item을 조회할 수 있도록 leftjoin을 추가하였습니다
  • item id 기준으로 group by 하여 집계 함수를 사용하였고 이를 기준으로 정렬 하도록 하였습니다.

정렬 조건 없는 경우 최신순으로 반환 ❤😊 2cf3c5b

  • 정렬 조건이 없는 경우 아이템이 추가 된 순서대로 반환 됩니다.

📌 기존에 있던 기능에 영향을 주나요?

  • 아니요

@Curry4182 Curry4182 added the enhancement New feature or request label Jan 23, 2024
@Curry4182 Curry4182 self-assigned this Jan 23, 2024
Copy link
Member

@Yiseull Yiseull left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~!

@HandmadeCloud
Copy link
Member

수고하셨습니다~

@Curry4182 Curry4182 merged commit 2968119 into main Jan 24, 2024
@Curry4182 Curry4182 deleted the LIME-50-BE-item-sort-condition branch January 24, 2024 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants