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

[refactor] 여행지 리뷰(PlaceReview) 기능 추가 및 리팩토링 #96

Merged
merged 6 commits into from
Jan 23, 2024

Conversation

meena2003
Copy link
Member

🎯 목적

  • 새 기능 (New Feature)

  • 리팩토링 (Refactoring)

  • 간략한 설명:
    : 여행지 리뷰 기능 추가 및 리팩토링을 진행했습니다.


🛠 작성/변경 사항

- 여행지 리뷰 일부 삭제 성공 시 207 상태코드에서 200 상태 코드로 변경

  • 프론트엔드와 논의한 결과, 여행지 리뷰가 일부 성공하더라도 정책적으로 200 코드를 반환하기로 합의했습니다.
  • controller 계층 코드가 훨씬 깔끔해졌습니다.

- 여행지 리뷰 목록 및 My 여행지 리뷰 목록 조회에서 DTO 변경

  • 기존 버전은 Page 인터페이스를 그대로 반환했기 때문에 불필요한 정보까지 전달했습니다.
  • Page 인터페이스에서 중요한 정보만 추출해서 PlaceReviewListResponseDto 로 반환합니다.
    • Long totalCount : 조회한 총 개수
    • int nowPageNumber : 현재 페이지 넘버
    • boolean isFirst : 현재 페이지가 처음인지 여부
    • boolean isLast : 현재 페이지가 마지막인지 여부
    • List<PlaceReviewResponseDto> placeReviews : PlaceReview 리스트

- 여행지 리뷰 등록 및 수정 시 이미지 중복 등록으로 인한 포인트 부정 적립 방지 기능 추가

  • 여행지 리뷰를 등록할 때 이미지를 등록하지 않았다가, 수정 API를 통해 처음 이미지를 등록할 시 포인트 +1점 적립합니다.
  • 만약 수정 API에서 이미지를 삭제해 null로 만든 후, 다시 이미지를 등록하면 부정 적립으로 판단하고 포인트를 적립하지 않습니다.

- 여행지 리뷰 수정 시 타인의 리뷰 수정하지 못하도록 예외 발생 구현

  • 만약 로그인한 사용자가 작성하지 않은 리뷰를 수정하려고 할 시 UserInvalidAccessException 예외가 발생합니다.

- 여행지 리뷰 등록 시 비영속 상태인 member를 영속화 하기 위해 merge() 대신 memberRepository에서 꺼내오는 방법으로 변경

- 기타 코드 인라인 정리, 주석 및 import 문 정리


🔗 관련 이슈

@meena2003 meena2003 added enhancement New feature or request feat 기능을 추가합니다. labels Jan 21, 2024
@meena2003 meena2003 self-assigned this Jan 21, 2024
Copy link

Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 5554f7f. ± Comparison against base commit 3ea19ed.

Copy link
Contributor

@jo0oy jo0oy left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@meena2003 meena2003 linked an issue Jan 22, 2024 that may be closed by this pull request
@meena2003 meena2003 merged commit ad8cd22 into develop Jan 23, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feat 기능을 추가합니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[refactor] 여행지 리뷰(PlaceReview) 코드 리팩토링
2 participants