게시글 검색 최신순, 인기순 #447

oxix97 · 2024-04-29T04:12:35Z

개요

새로 게시글 검색 (인기순) API를 작성하지 않고 기존 로직에 수정을 하였습니다.

Controller

(require = false) Integer likeCount를 Param값으로 추가적으로 받습니다.

Service

-> lastLikeCount == null 인 경우 최신순으로 정렬(ID 내림차순)
-> lastLikeCount != null 인 경우 �좋아요 순으로 정렬 (좋아요 순 내림차순, ID 내림 차순)

�Reader

-> cursor == 0 인 경우 정렬 기준에 맞게 최대 10개 가져옴
-> cursor != 0 인 경우 커서보다 작은 ID를 정렬 기준에 맞게 가져옴

closed #437

PR 유형

어떤 변경 사항이 있나요?

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

커밋 메시지 컨벤션에 맞게 작성했습니다. Commit message convention 참고.
변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

LikeCount 값이 null 이 아닌 경우 : 인기순으로 검색된 내용 조회 LikeCount 값이 null 인 경우 : 최신순으로 검색된 내용 조회

like Count 유무에 따른 최신순 / 인기순 분기처리

… into refactor/Kernel360#437-post-search

GBGreenBravo · 2024-04-30T00:55:58Z

orury-domain/src/main/java/org/orury/domain/post/infrastructure/PostReaderImpl.java

        return (cursor.equals(NumberConstants.FIRST_CURSOR))
-                ? postRepository.findByTitleContainingOrContentContainingOrderByIdDesc(searchWord, searchWord, pageable)
+                ? postRepository.findByTitleContainingOrContentContainingOrderByLikeCountDesc(searchWord, pageable)
+                : postRepository.findByLikeCountLessThanAndTitleContainingOrLikeCountLessThanAndContentContainingOrderByLikeCountDesc(likeCount, searchWord, likeCount, searchWord, pageable);


이 로직에 cursor로 들어오는 id도 고려돼야하지 않을까요?
예를 들어 검색어를 포함하고 "좋아요 5개 눌린" 게시글이 13개고, 10개가 조회된다면
다음에는 나머지 3개가 조회되고 좋아요 4개 이하를 탐색해야 하지만,
현재의 코드에서는 위 예시의 3개 게시글이 누락되는 상황이 발생할 것으로 예상됩니다!

좋아요 5개 이하, 다음 커서로 조회 하면 나머지 3개 포함해서 출력되지 않나요?

우선 지금 코드에서 좋아요수 이하가 아닌 미만으로 설정돼 있습니다. 이하로 하시려면 LessThan 뒤에 Equal 붙이셔야 해요~

그리고 나머지 3개는 현재 커서 기준(좋아요수)으로는 탐색이 아마 안 될 겁니다..! api 요청해보시면 아마 확인하실 수 있을 거에요!
findBy
LikeCountLessThanEqualAndTitleContaining
Or
LikeCountLessThanEqualAndContentContaining
OrderByLikeCountDesc
이렇게 Equal 붙여서 구현돼 있더라도 이전에 조회한 결과랑 똑같은 10개를 가져오고, 나머지 3개는 조회가 안 될 겁니다..

음..그렇네요 이런 케이스는 어떻게 처리하지..?

고유값이 없기 때문에 좋아요 검색에 한해서는 페이지당 게시글 갯수제한을 없애는것말곤...잘 떠오르질 않네요

페이지네이션을 안 한다면 뭔가 컨벤션이 복잡해져서 저희 측에서 구현하는 게 맞을 듯 합니다..!
아직도 정복하지 못한 QueryDsl 이슈를 회피해서 최대한 쥐어짜내보면, 아래와 같은 방식으로 구현가능할 듯 합니다.

FirstCursor인 경우에 ->
기존의 findByTitleContainingOrContentContainingOrderByLikeCountDesc 뒤에 CreatedAtDesc를 추가해서
findByTitleContainingOrContentContainingOrderByLikeCountDescCreatedAtDesc로 구현하여, 좋아요수가 같은 경우는 최신순으로 정렬하도록 세팅.

FirstCursor가 아닌 경우에 ->
(1) cursor(마지막으로 조회한 게시글 id) 게시글의 좋아요 수와 같은 좋아요 수를 가진 posts 우선 조회
-findBy TitleContainingAndLikeCountEquals Or ContentContainingAndLikeCountEquals OrderByCreatedAtDesc

(2-1) 1번에서 조회된 posts의 개수가 POST_PAGINATION_SIZE와 같으면 posts 바로 return

(2-2) 1번에서 조회된 posts의 개수가 POST_PAGINATION_SIZE 미만이면, 그만큼 PageRequest의 사이즈를 줄여서 현재 좋아요수 미만의 조회 실행
-findBy LikeCountLessThanAndTitleContaining Or LikeCountLessThanAndContentContaining OrderByLikeCountDescCreatedAtDesc
후 1의 결과물과 병합하여 return

orury-client/src/main/java/org/orury/client/post/interfaces/PostController.java

kkkapuq · 2024-04-30T08:53:26Z

orury-domain/src/main/java/org/orury/domain/post/infrastructure/PostReaderImpl.java

        return (cursor.equals(NumberConstants.FIRST_CURSOR))
-                ? postRepository.findByTitleContainingOrContentContainingOrderByIdDesc(searchWord, searchWord, pageable)
+                ? postRepository.findByTitleContainingOrContentContainingOrderByLikeCountDesc(searchWord, pageable)
+                : postRepository.findByLikeCountLessThanAndTitleContainingOrLikeCountLessThanAndContentContainingOrderByLikeCountDesc(likeCount, searchWord, likeCount, searchWord, pageable);


음..그렇네요 이런 케이스는 어떻게 처리하지..?

kkkapuq · 2024-04-30T08:56:39Z

orury-domain/src/main/java/org/orury/domain/post/infrastructure/PostReaderImpl.java

        return (cursor.equals(NumberConstants.FIRST_CURSOR))
-                ? postRepository.findByTitleContainingOrContentContainingOrderByIdDesc(searchWord, searchWord, pageable)
+                ? postRepository.findByTitleContainingOrContentContainingOrderByLikeCountDesc(searchWord, pageable)
+                : postRepository.findByLikeCountLessThanAndTitleContainingOrLikeCountLessThanAndContentContainingOrderByLikeCountDesc(likeCount, searchWord, likeCount, searchWord, pageable);


고유값이 없기 때문에 좋아요 검색에 한해서는 페이지당 게시글 갯수제한을 없애는것말곤...잘 떠오르질 않네요

GBGreenBravo · 2024-05-07T01:58:56Z

orury-client/src/main/java/org/orury/client/post/application/PostFacade.java

+                PageRequest.of(cursor.intValue(), POST_PAGINATION_SIZE, Sort.by(Sort.Order.desc("likeCount"), Sort.Order.desc("createdAt")))
+                : PageRequest.of(cursor.intValue(), POST_PAGINATION_SIZE, Sort.by(Sort.Order.desc("createdAt")));


cursor.intValue를 PageRequest에 담으면 비정상적으로 조회되지 않나요?
커서가 페이지 번호를 의미하는 게 아니라 마지막으로 조회된 게시글의 id를 가져온 거라, 예상과 다르게 동작할 것으로 예측합니다!

cursor에다가 페이지 번호를 0부터 담으면 정상적으로 작동하는거 같은데, 혹시 이전 코드에서 cursor가 어떤 의미 역할을 맡고 있었나요?

페이지네이션 방식의 차이로 보입니다! 다른 페이지네이션에서의 커서는 전부 조회된 목록에서 가장 마지막으로 조회한 게시글/암장/댓글의 id를 받고 있었습니다. 해당 링크 참고하시면 도움될 듯 합니다.

그런데 이렇게 바뀌는 거라면, 프론트에서 전달하는 값도 달라지기 때문에 프론트와 협의가 필요하고,
추가로 구분을 위해서 파라미터명도 cursor -> page로의 변경도 필요할 듯 합니다!

orury-client/src/main/java/org/orury/client/post/application/PostServiceImpl.java

null이 아닌 정수형의 likeCount가 있으면 인기순 조회라는 가정으로 수정

… into refactor/Kernel360#437-post-search

kkkapuq · 2024-05-08T09:34:53Z

orury-domain/src/main/java/org/orury/domain/post/infrastructure/PostRepository.java

+    @Query("SELECT p FROM post p WHERE LOWER(p.title) LIKE CONCAT('%',LOWER(:keyword),'%') OR LOWER(p.content) LIKE CONCAT('%',LOWER(:keyword),'%') ORDER BY p.likeCount DESC, p.id DESC")
+    List<Post> findBySearchWordOrderByLikeCountDesc(String keyword, Pageable pageable);
+
+    @Query("SELECT p FROM post p WHERE (LOWER(p.title) LIKE CONCAT('%',LOWER(:keyword),'%') OR LOWER(p.content) LIKE CONCAT('%',LOWER(:keyword),'%')) AND (p.likeCount < :likeCount OR (p.likeCount = :likeCount AND p.id < :cursor)) ORDER BY p.likeCount DESC, p.id DESC")
+    List<Post> findBySearchWordOrderByLikeCountDescWithCursor(String keyword, Long cursor, int likeCount, Pageable pageable);


요게 핵심인것같은데, 인기순으로 정렬하고, 좋아요 수가 동일하다면 최신순으로 내부 정렬시킨다음, 그 커서값을 기준으로 다음 페이지를 조회하는 건가요?

가독성 높이기 위해서 아래처럼 줄바꿈하는 건 어떨까요?

SELECT p
FROM post p
WHERE (LOWER(p.title) LIKE CONCAT('%',LOWER(:keyword),'%') OR LOWER(p.content) LIKE CONCAT('%',LOWER(:keyword),'%'))
AND (p.likeCount < :likeCount OR (p.likeCount = :likeCount AND p.id < :cursor))
ORDER BY p.likeCount DESC, p.id DESC

형준님이 말씀하신게 정확합니다

줄바꿈 바로 반영합니다~

저도 가독성땜시 민협님 의견에 찬성합니다람쥐

GBGreenBravo · 2024-05-08T09:54:45Z

orury-domain/src/main/java/org/orury/domain/post/infrastructure/PostRepository.java

+    @Query("SELECT p FROM post p WHERE LOWER(p.title) LIKE CONCAT('%',LOWER(:keyword),'%') OR LOWER(p.content) LIKE CONCAT('%',LOWER(:keyword),'%') ORDER BY p.likeCount DESC, p.id DESC")
+    List<Post> findBySearchWordOrderByLikeCountDesc(String keyword, Pageable pageable);
+
+    @Query("SELECT p FROM post p WHERE (LOWER(p.title) LIKE CONCAT('%',LOWER(:keyword),'%') OR LOWER(p.content) LIKE CONCAT('%',LOWER(:keyword),'%')) AND (p.likeCount < :likeCount OR (p.likeCount = :likeCount AND p.id < :cursor)) ORDER BY p.likeCount DESC, p.id DESC")
+    List<Post> findBySearchWordOrderByLikeCountDescWithCursor(String keyword, Long cursor, int likeCount, Pageable pageable);


가독성 높이기 위해서 아래처럼 줄바꿈하는 건 어떨까요?

SELECT p
FROM post p
WHERE (LOWER(p.title) LIKE CONCAT('%',LOWER(:keyword),'%') OR LOWER(p.content) LIKE CONCAT('%',LOWER(:keyword),'%'))
AND (p.likeCount < :likeCount OR (p.likeCount = :likeCount AND p.id < :cursor))
ORDER BY p.likeCount DESC, p.id DESC

kkkapuq

고생하셨습니다! 어떻게 짜야될지 막막했는데 이런식으로 풀어주셨군요 감사합니다

kkkapuq · 2024-05-08T11:11:08Z

orury-domain/src/main/java/org/orury/domain/post/infrastructure/PostRepository.java

+    @Query("SELECT p FROM post p WHERE LOWER(p.title) LIKE CONCAT('%',LOWER(:keyword),'%') OR LOWER(p.content) LIKE CONCAT('%',LOWER(:keyword),'%') ORDER BY p.likeCount DESC, p.id DESC")
+    List<Post> findBySearchWordOrderByLikeCountDesc(String keyword, Pageable pageable);
+
+    @Query("SELECT p FROM post p WHERE (LOWER(p.title) LIKE CONCAT('%',LOWER(:keyword),'%') OR LOWER(p.content) LIKE CONCAT('%',LOWER(:keyword),'%')) AND (p.likeCount < :likeCount OR (p.likeCount = :likeCount AND p.id < :cursor)) ORDER BY p.likeCount DESC, p.id DESC")
+    List<Post> findBySearchWordOrderByLikeCountDescWithCursor(String keyword, Long cursor, int likeCount, Pageable pageable);


저도 가독성땜시 민협님 의견에 찬성합니다람쥐

oxix97 added 7 commits April 29, 2024 13:01

Refactor : 게시판 컨트롤러 수정

2031431

LikeCount 값이 null 이 아닌 경우 : 인기순으로 검색된 내용 조회 LikeCount 값이 null 인 경우 : 최신순으로 검색된 내용 조회

Refactor : 게시판 서비스 수정

1d3c361

like Count 유무에 따른 최신순 / 인기순 분기처리

Feature : 인기순, 최신순 JPA Query 작성

5594010

Refactor : 게시판 Facade 수정

dcb1a53

Chore : 테스트 코드 대응

f738f7f

Test : 게시판 검색 최신순, 인기순 코드 작성

5ba16e0

Merge branch 'develop' of https://github.com/Kernel360/f1-Orury-Backend…

a721c6c

… into refactor/Kernel360#437-post-search

oxix97 added 💻 BE 백엔드 🔨 Refactor 코드를 리팩토링한 경우 labels Apr 29, 2024

oxix97 self-assigned this Apr 29, 2024

oxix97 requested review from kkkapuq and GBGreenBravo as code owners April 29, 2024 04:12

GBGreenBravo requested changes Apr 30, 2024

View reviewed changes

kkkapuq requested a review from c0olcode April 30, 2024 08:48

kkkapuq requested changes Apr 30, 2024

View reviewed changes

oxix97 added 4 commits May 7, 2024 10:00

Chore : 테스트 코드 대응

47116d4

Refactor : �Page 객체로 반환

1eb49ec

Refactor : Pageable 객체를 받아 처리

562331f

Refactor : 좋아요 순, 최신 순 여부 확인을 위한 isLiked

602d8bc

GBGreenBravo reviewed May 7, 2024

View reviewed changes

oxix97 added 3 commits May 8, 2024 18:15

Refactor : 커서 기반 인기순, 최신순 조회(Controller)

538abe8

null이 아닌 정수형의 likeCount가 있으면 인기순 조회라는 가정으로 수정

Test : 테스트 코드 대응

71d6dea

Merge branch 'develop' of https://github.com/Kernel360/f1-Orury-Backend…

1e81a01

… into refactor/Kernel360#437-post-search

oxix97 requested review from kkkapuq and GBGreenBravo May 8, 2024 09:28

kkkapuq reviewed May 8, 2024

View reviewed changes

GBGreenBravo approved these changes May 8, 2024

View reviewed changes

Chore : Query 줄바꿈

bee299d

kkkapuq approved these changes May 8, 2024

View reviewed changes

oxix97 merged commit b2f7196 into Kernel360:develop May 8, 2024
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

게시글 검색 최신순, 인기순 #447

게시글 검색 최신순, 인기순 #447

oxix97 commented Apr 29, 2024 •

edited

Loading

GBGreenBravo Apr 30, 2024 •

edited

Loading

oxix97 Apr 30, 2024

GBGreenBravo Apr 30, 2024

kkkapuq Apr 30, 2024

kkkapuq Apr 30, 2024

GBGreenBravo Apr 30, 2024 •

edited

Loading

kkkapuq Apr 30, 2024

kkkapuq Apr 30, 2024

GBGreenBravo May 7, 2024

oxix97 May 7, 2024

GBGreenBravo May 7, 2024

kkkapuq May 8, 2024

GBGreenBravo May 8, 2024

oxix97 May 8, 2024

oxix97 May 8, 2024

kkkapuq May 8, 2024

GBGreenBravo May 8, 2024

kkkapuq left a comment

kkkapuq May 8, 2024

		PageRequest.of(cursor.intValue(), POST_PAGINATION_SIZE, Sort.by(Sort.Order.desc("likeCount"), Sort.Order.desc("createdAt")))
		: PageRequest.of(cursor.intValue(), POST_PAGINATION_SIZE, Sort.by(Sort.Order.desc("createdAt")));

게시글 검색 최신순, 인기순 #447

게시글 검색 최신순, 인기순 #447

Conversation

oxix97 commented Apr 29, 2024 • edited Loading

개요

Controller

Service

�Reader

PR 유형

PR Checklist

GBGreenBravo Apr 30, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

GBGreenBravo Apr 30, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

kkkapuq left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

oxix97 commented Apr 29, 2024 •

edited

Loading

GBGreenBravo Apr 30, 2024 •

edited

Loading

GBGreenBravo Apr 30, 2024 •

edited

Loading