reafactor(community): 커뮤니티 게시글 조회 페이징 처리

Author: ayoung-dev

src/main/java/com/somemore/community/repository/board/CommunityBoardRepository.java

@@ -2,16 +2,17 @@
 
 import com.somemore.community.domain.CommunityBoard;
 import com.somemore.community.repository.mapper.CommunityBoardView;
+import org.springframework.data.domain.Page;
+import org.springframework.data.domain.Pageable;
 
-import java.util.List;
 import java.util.Optional;
 import java.util.UUID;
 
 public interface CommunityBoardRepository {
     CommunityBoard save(CommunityBoard communityBoard);
     Optional<CommunityBoard> findById(Long id);
-    List<CommunityBoardView> getCommunityBoards();
-    List<CommunityBoardView> findByWriterId(UUID writerId);
+    Page<CommunityBoardView> findCommunityBoards(Pageable pageable);
+    Page<CommunityBoardView> findByWriterId(UUID writerId, Pageable pageable);
     boolean existsById(Long id);
     default boolean doesNotExistById(Long id) {
         return !existsById(id);

src/main/java/com/somemore/community/repository/board/CommunityBoardRepositoryImpl.java

@@ -8,6 +8,9 @@
 import com.somemore.community.domain.QCommunityBoard;
 import com.somemore.volunteer.domain.QVolunteer;
 import lombok.RequiredArgsConstructor;
+import org.springframework.data.domain.Page;
+import org.springframework.data.domain.Pageable;
+import org.springframework.data.support.PageableExecutionUtils;
 import org.springframework.stereotype.Repository;
 
 import java.util.List;
@@ -21,15 +24,16 @@ public class CommunityBoardRepositoryImpl implements CommunityBoardRepository {
     private final JPAQueryFactory queryFactory;
     private final CommunityBoardJpaRepository communityBoardJpaRepository;
 
+    private static final QCommunityBoard communityBoard = QCommunityBoard.communityBoard;
+    private static final QVolunteer volunteer = QVolunteer.volunteer;
+
     @Override
     public CommunityBoard save(CommunityBoard communityBoard) {
         return communityBoardJpaRepository.save(communityBoard);
     }
 
     @Override
     public Optional<CommunityBoard> findById(Long id) {
-        QCommunityBoard communityBoard = QCommunityBoard.communityBoard;
-
         return Optional.ofNullable(queryFactory
                 .selectFrom(communityBoard)
                 .where(communityBoard.id.eq(id)
@@ -38,18 +42,30 @@ public Optional<CommunityBoard> findById(Long id) {
     }
 
     @Override
-    public List<CommunityBoardView> getCommunityBoards() {
-        return getCommunityBoardsQuery()
+    public Page<CommunityBoardView> findCommunityBoards(Pageable pageable) {
+        List<CommunityBoardView> content = getCommunityBoardsQuery()
                 .where(QCommunityBoard.communityBoard.deleted.eq(false))
                 .fetch();
+
+        JPAQuery<Long> countQuery = queryFactory
+                .select(communityBoard.count())
+                .from(communityBoard);
+
+        return PageableExecutionUtils.getPage(content, pageable, countQuery::fetchOne);
     }
 
     @Override
-    public List<CommunityBoardView> findByWriterId(UUID writerId) {
-        return getCommunityBoardsQuery()
+    public Page<CommunityBoardView> findByWriterId(UUID writerId, Pageable pageable) {
+        List<CommunityBoardView> content = getCommunityBoardsQuery()
                 .where(QCommunityBoard.communityBoard.writerId.eq(writerId)
                         .and(QCommunityBoard.communityBoard.deleted.eq(false)))
                 .fetch();
+
+        JPAQuery<Long> countQuery = queryFactory
+                .select(communityBoard.count())
+                .from(communityBoard);
+
+        return PageableExecutionUtils.getPage(content, pageable, countQuery::fetchOne);
     }
 
     @Override
@@ -58,9 +74,6 @@ public boolean existsById(Long id) {
     }
 
     private JPAQuery<CommunityBoardView> getCommunityBoardsQuery() {
-        QCommunityBoard communityBoard = QCommunityBoard.communityBoard;
-        QVolunteer volunteer = QVolunteer.volunteer;
-
         return queryFactory
                 .select(Projections.constructor(CommunityBoardView.class,
                         communityBoard,

src/main/java/com/somemore/community/service/board/CommunityBoardQueryService.java

@@ -2,18 +2,19 @@
 
 import com.somemore.community.domain.CommunityBoard;
 import com.somemore.community.repository.mapper.CommunityBoardView;
-import com.somemore.community.dto.response.CommunityBoardGetDetailResponseDto;
-import com.somemore.community.dto.response.CommunityBoardGetResponseDto;
+import com.somemore.community.dto.response.CommunityBoardDetailResponseDto;
+import com.somemore.community.dto.response.CommunityBoardResponseDto;
 import com.somemore.community.repository.board.CommunityBoardRepository;
 import com.somemore.community.usecase.board.CommunityBoardQueryUseCase;
 import com.somemore.global.exception.BadRequestException;
 import lombok.RequiredArgsConstructor;
+import org.springframework.data.domain.Page;
+import org.springframework.data.domain.PageRequest;
+import org.springframework.data.domain.Pageable;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
-import java.util.List;
 import java.util.UUID;
-import java.util.function.Function;
 
 import static com.somemore.global.exception.ExceptionMessage.NOT_EXISTS_COMMUNITY_BOARD;
 
@@ -23,29 +24,26 @@
 public class CommunityBoardQueryService implements CommunityBoardQueryUseCase {
 
     private final CommunityBoardRepository communityBoardRepository;
+    private static final int PAGE_SIZE = 10;
 
     @Override
-    public List<CommunityBoardGetResponseDto> getCommunityBoards() {
-        List<CommunityBoardView> boards = communityBoardRepository.getCommunityBoards();
-        return mapEntitiesToDtos(boards, CommunityBoardGetResponseDto::fromEntity);
+    public Page<CommunityBoardResponseDto> getCommunityBoards(int page) {
+        Pageable pageable = PageRequest.of(page, PAGE_SIZE);
+        Page<CommunityBoardView> boards = communityBoardRepository.findCommunityBoards(pageable);
+        return boards.map(CommunityBoardResponseDto::from);
     }
 
     @Override
-    public List<CommunityBoardGetResponseDto> getCommunityBoardsByWriterId(UUID writerId) {
-        List<CommunityBoardView> boards = communityBoardRepository.findByWriterId(writerId);
-        return mapEntitiesToDtos(boards, CommunityBoardGetResponseDto::fromEntity);
+    public Page<CommunityBoardResponseDto> getCommunityBoardsByWriterId(UUID writerId, int page) {
+        Pageable pageable = PageRequest.of(page, PAGE_SIZE);
+        Page<CommunityBoardView> boards = communityBoardRepository.findByWriterId(writerId ,pageable);
+        return boards.map(CommunityBoardResponseDto::from);
     }
 
     @Override
-    public CommunityBoardGetDetailResponseDto getCommunityBoardDetail(Long id) {
+    public CommunityBoardDetailResponseDto getCommunityBoardDetail(Long id) {
         CommunityBoard board = communityBoardRepository.findById(id)
                 .orElseThrow(() -> new BadRequestException(NOT_EXISTS_COMMUNITY_BOARD.getMessage()));
-        return CommunityBoardGetDetailResponseDto.fromEntity(board);
-    }
-
-    private <T, R> List<R> mapEntitiesToDtos(List<T> entities, Function<T, R> mapper) {
-        return entities.stream()
-                .map(mapper)
-                .toList();
+        return CommunityBoardDetailResponseDto.from(board);
     }
 }

src/main/java/com/somemore/community/usecase/board/CommunityBoardQueryUseCase.java

@@ -1,13 +1,13 @@
 package com.somemore.community.usecase.board;
 
-import com.somemore.community.dto.response.CommunityBoardGetDetailResponseDto;
-import com.somemore.community.dto.response.CommunityBoardGetResponseDto;
+import com.somemore.community.dto.response.CommunityBoardDetailResponseDto;
+import com.somemore.community.dto.response.CommunityBoardResponseDto;
+import org.springframework.data.domain.Page;
 
-import java.util.List;
 import java.util.UUID;
 
 public interface CommunityBoardQueryUseCase {
-    List<CommunityBoardGetResponseDto> getCommunityBoards();
-    List<CommunityBoardGetResponseDto> getCommunityBoardsByWriterId(UUID writerId);
-    CommunityBoardGetDetailResponseDto getCommunityBoardDetail(Long id);
+    Page<CommunityBoardResponseDto> getCommunityBoards(int page);
+    Page<CommunityBoardResponseDto> getCommunityBoardsByWriterId(UUID writerId, int page);
+    CommunityBoardDetailResponseDto getCommunityBoardDetail(Long id);
 }