Merge pull request #37 from LikeLion-12th-SKHU/develop

Develop -> main push

src/main/java/org/likelion/likelion_12th_team05/common/EntityFinder.java

@@ -0,0 +1,15 @@
+package org.likelion.likelion_12th_team05.common;
+
+import org.likelion.likelion_12th_team05.common.error.ErrorCode;
+import org.likelion.likelion_12th_team05.common.exception.NotFoundException;
+
+import java.util.Optional;
+
+public class EntityFinder {
+
+    // 엔티티 not found 처리
+    public static <T> T findByIdOrThrow(Optional<T> optionalEntity, ErrorCode errorCode) {
+        return optionalEntity.orElseThrow(() ->
+                new NotFoundException(errorCode, errorCode.getMessage()));
+    }
+}

src/main/java/org/likelion/likelion_12th_team05/common/advice/CustomExceptionAdvice.java

@@ -0,0 +1,20 @@
+package org.likelion.likelion_12th_team05.common.advice;
+
+import org.likelion.likelion_12th_team05.common.exception.NotFoundException;
+import org.likelion.likelion_12th_team05.config.ApiResponseTemplate;
+import org.springframework.http.HttpStatus;
+import org.springframework.http.ResponseEntity;
+import org.springframework.web.bind.annotation.ControllerAdvice;
+import org.springframework.web.bind.annotation.ExceptionHandler;
+
+@ControllerAdvice
+public class CustomExceptionAdvice {
+
+    // 엔티티 not found 에러 처리 핸들러
+    @ExceptionHandler(NotFoundException.class)
+    public ResponseEntity<ApiResponseTemplate<Void>> handleNotFoundException(NotFoundException ex) {
+        ApiResponseTemplate<Void> errorResponse = ApiResponseTemplate.errorResponse(ex.getErrorCode());
+        return new ResponseEntity<>(errorResponse, HttpStatus.NOT_FOUND);
+    }
+
+}

src/main/java/org/likelion/likelion_12th_team05/curation/api/CurationController.java

@@ -32,7 +32,7 @@ public class CurationController {
     private final LocationService locationService;
     private final UserService userService;
 
-    @Operation(summary = "모든 사용자가 큐레이션 6개씩 조회", description = "모든 사용자가 큐레이션을 6개씩 조회합니다.")
+    @Operation(summary = "모든 사용자가 큐레이션 6개씩 조회", description = "모든 사용자가 산책로 지도 페이지에서 큐레이션을 한 페이지 당 6개씩 조회합니다.")
     @ApiResponses(value = {
             @ApiResponse(responseCode = "200", description = "응답 생성에 성공하였습니다."),
             @ApiResponse(responseCode = "400", description = "잘못된 요청입니다."),
@@ -57,7 +57,7 @@ public ApiResponseTemplate<CurationListResDto> curationFindAll (
         return ApiResponseTemplate.successResponse(curationListResDto, SuccessCode.GET_SUCCESS);
     }
 
-    @Operation(summary = "인증된 사용자가 큐레이션 생성", description = "인증된 사용자가 큐레이션을 생성합니다.")
+    @Operation(summary = "인증된 사용자가 큐레이션 생성", description = "인증된 사용자가 산책로 생성 페이지에서 큐레이션을 생성합니다.")
     @ApiResponses(value = {
             @ApiResponse(responseCode = "200", description = "응답 생성에 성공하였습니다."),
             @ApiResponse(responseCode = "400", description = "잘못된 요청입니다."),
@@ -71,7 +71,7 @@ public ApiResponseTemplate<CurationInfoResDto> curationSave(@RequestBody Curatio
         return ApiResponseTemplate.successResponse(curationInfoResDto, SuccessCode.CURATION_SAVE_SUCCESS);
     }
 
-    @Operation(summary = "인증된 사용자가 큐레이션 수정", description = "인증된 사용자가 큐레이션을 수정(큐레이션 제목, 설명)합니다.")
+    @Operation(summary = "인증된 사용자가 큐레이션 수정", description = "인증된 사용자가 산책로 지도 페이지에서 큐레이션을 수정(큐레이션 제목, 설명)합니다.")
     @ApiResponses(value = {
             @ApiResponse(responseCode = "200", description = "응답 생성에 성공하였습니다."),
             @ApiResponse(responseCode = "400", description = "잘못된 요청입니다."),
@@ -100,7 +100,7 @@ public ApiResponseTemplate<SuccessCode> curationDelete(@PathVariable("curationId
         return ApiResponseTemplate.successWithNoContent(SuccessCode.GET_SUCCESS);
     }
 
-    @Operation(summary = "모든 사용자가 큐레이션 검색", description = "모든 사용자가 큐레이션을 검색합니다.")
+    @Operation(summary = "모든 사용자가 큐레이션 검색", description = "모든 사용자가 산책로 지도 페이지에서 큐레이션을 검색합니다.")
     @ApiResponses(value = {
             @ApiResponse(responseCode = "200", description = "응답 생성에 성공하였습니다."),
             @ApiResponse(responseCode = "400", description = "잘못된 요청입니다."),
@@ -113,10 +113,7 @@ public ApiResponseTemplate<CurationListResDto> searchCurations(@RequestParam Str
         return ApiResponseTemplate.successResponse(searchResults, SuccessCode.GET_SUCCESS);
     }
 
-    // - 중, 조회-
-    // 랜딩 페이지 - 게시글을 많이 작성한 5명의 큐레이터 조회 가능
-
-    @Operation(summary = "모든 사용자가 좋아요 순으로 정렬된 큐레이션 6개 조회", description = "모든 사용자가 좋아요 순으로 정렬된 큐레이션을 조회합니다.")
+    @Operation(summary = "모든 사용자가 좋아요 순으로 정렬된 큐레이션 6개 조회", description = "모든 사용자가 랜딩페이지에서 좋아요 순으로 정렬된 6개의 큐레이션을 조회합니다.")
     @ApiResponses(value = {
             @ApiResponse(responseCode = "200", description = "응답 생성에 성공하였습니다."),
             @ApiResponse(responseCode = "400", description = "잘못된 요청입니다."),
@@ -128,4 +125,17 @@ public ApiResponseTemplate<CurationListResDto> findTop6ByOrderByLikeCountDesc()
         CurationListResDto curationListResDto = curationService.findTop6ByOrderByLikeCountDesc();
         return ApiResponseTemplate.successResponse(curationListResDto, SuccessCode.GET_SUCCESS);
     }
+
+    @Operation(summary = "모든 사용자가 최신순으로 정렬된 큐레이션 6개 조회", description = "모든 사용자가 랜딩페이지에서 최신순으로 정렬된 6개의 큐레이션을 조회합니다.")
+    @ApiResponses(value = {
+            @ApiResponse(responseCode = "200", description = "응답 생성에 성공하였습니다."),
+            @ApiResponse(responseCode = "400", description = "잘못된 요청입니다."),
+            @ApiResponse(responseCode = "500", description = "서버 내부 오류입니다.")
+    })
+    @GetMapping("/recent")
+    public ApiResponseTemplate<CurationListResDto> findTop6byOrderByCreateDateDesc() {
+        CurationListResDto curationListResDto = curationService.findTop6byOrderByCreateDateDesc();
+        return ApiResponseTemplate.successResponse(curationListResDto, SuccessCode.GET_SUCCESS);
+    }
+
 }

src/main/java/org/likelion/likelion_12th_team05/curation/api/dto/response/CurationInfoResDto.java

@@ -4,19 +4,22 @@
 import org.likelion.likelion_12th_team05.curation.domain.Curation;
 
 import java.time.LocalDate;
+import java.time.LocalDateTime;
 
 @Builder
 public record CurationInfoResDto(
         Long id,
         String name,
         String content,
+        LocalDateTime createDate,
         Integer likeCount
 ) {
     public static CurationInfoResDto from(Curation curation) {
         return CurationInfoResDto.builder()
                 .id(curation.getId())
                 .name(curation.getName())
                 .content(curation.getContent())
+                .createDate(curation.getCreateDate())
                 .likeCount(curation.getLikeCount())
                 .build();
     }

src/main/java/org/likelion/likelion_12th_team05/curation/application/CurationService.java

@@ -1,6 +1,7 @@
 package org.likelion.likelion_12th_team05.curation.application;
 
 import lombok.RequiredArgsConstructor;
+import org.likelion.likelion_12th_team05.common.EntityFinder;
 import org.likelion.likelion_12th_team05.common.error.ErrorCode;
 import org.likelion.likelion_12th_team05.common.exception.NotFoundException;
 import org.likelion.likelion_12th_team05.curation.api.dto.request.CurationUpdateReqDto;
@@ -29,10 +30,9 @@ public class CurationService {
     // 인증된 사용자 - 큐레이션 생성
     @Transactional
     public CurationInfoResDto curationSave(CurationSaveReqDto curationSaveReqDto, Principal principal) {
-        String email = principal.getName();
-        User user = userRepository.findByEmail(email).orElseThrow(
-                () -> new NotFoundException(ErrorCode.USER_NOT_FOUND_EXCEPTION
-                        , ErrorCode.USER_NOT_FOUND_EXCEPTION.getMessage()));
+        Long id = Long.parseLong(principal.getName());  // user repo에서 userId로 바꿔야 함
+        User user = getUserById(id);
+
         Curation curation = curationSaveReqDto.toEntity(user);
         curationRepository.save(curation);
 
@@ -42,19 +42,15 @@ public CurationInfoResDto curationSave(CurationSaveReqDto curationSaveReqDto, Pr
     // 인증된 사용자 - 큐레이션 수정
     @Transactional
     public CurationInfoResDto curationUpdate(Long curationId, CurationUpdateReqDto curationUpdateReqDto, Principal principal) {
-        String email = principal.getName();
-        User user = userRepository.findByEmail(email).orElseThrow(
-                () -> new NotFoundException(ErrorCode.USER_NOT_FOUND_EXCEPTION
-                        , ErrorCode.USER_NOT_FOUND_EXCEPTION.getMessage()));
+        Long id = Long.parseLong(principal.getName());  // user repo에서 userId로 바꿔야 함
+        User user = getUserById(id);
 
-        Curation curation = curationRepository.findById(curationId).orElseThrow(
-                () -> new NotFoundException(ErrorCode.CURATIONS_NOT_FOUND_EXCEPTION
-                                , ErrorCode.CURATIONS_NOT_FOUND_EXCEPTION.getMessage()));
+        Curation curation = getCurationById(curationId);
 
         // 수정 권한 확인 -- 토큰 발급의 주체가 email이기에 email로 사용자 확인을 하였으나
-        // userid로 판단 하는 것이 더 좋지 않을까 생각함. 일단 crud와 로그인 연결 테스트를 위해 email로 테스트 함 -- 상의 필요
-        String LoginEmail = principal.getName();
-        if (!email.equals(LoginEmail)) {
+        // userid로 판단 하는 것이 더 좋지 않을까 생각함. 일단 crud와 로그인 연결 테스트를 위해 email로 테스트 함 -> 테스트 성공 -> userId로 바꾸기로 함
+        String LoginId = principal.getName();
+        if (!id.equals(LoginId)) {
             throw new NotFoundException(ErrorCode.NO_AUTHORIZATION_EXCEPTION
                     , ErrorCode.NO_AUTHORIZATION_EXCEPTION.getMessage());
         }
@@ -67,8 +63,7 @@ public CurationInfoResDto curationUpdate(Long curationId, CurationUpdateReqDto c
     // 인증된 사용자 - 큐레이션 삭제
     @Transactional
     public void curationDelete(Long curationId, Principal principal) {
-        Curation curation = curationRepository.findById(curationId)
-                .orElseThrow(() -> new IllegalArgumentException("해당 큐레이션이 없습니다. id=" + curationId));
+        Curation curation = getCurationById(curationId);
 
         // 삭제 권한 확인
         Long id = Long.parseLong(principal.getName());
@@ -102,7 +97,7 @@ public CurationListResDto curationFindAll(Pageable pageable) {
         return CurationListResDto.from(curationInfoResDtoList);
     }
 
-    // 모든 사용자 - 큐레이션 좋아요 많은 순으로 6개만 조회
+    // 모든 사용자 - 랜딩 페이지 - 큐레이션 좋아요 많은 순으로 6개만 조회
     @Transactional
     public CurationListResDto findTop6ByOrderByLikeCountDesc() {
         List<Curation> curations = curationRepository.findTop6ByOrderByLikeCountDesc();
@@ -112,4 +107,24 @@ public CurationListResDto findTop6ByOrderByLikeCountDesc() {
         return CurationListResDto.from(curationInfoResDtoList);
     }
 
+    // 모든 사용자 - 랜딩페이지 - 큐레이션 최신(createDate desc) 순으로 6개만 조회
+    @Transactional
+    public CurationListResDto findTop6byOrderByCreateDateDesc() {
+        List<Curation> curations = curationRepository.findTop6byOrderByCreateDateDesc();
+        List<CurationInfoResDto> curationInfoResDtoList = curations.stream()
+                .map(CurationInfoResDto::from)
+                .toList();
+        return CurationListResDto.from(curationInfoResDtoList);
+    }
+
+    // 반복되는 예외 반환 메서드 추출 => 공통 예외 처리로 일관성 높이기 위함 => entityfinder로 중앙 관리
+    private Curation getCurationById(Long curationId) {
+        return EntityFinder.findByIdOrThrow(curationRepository.findById(curationId)
+                , ErrorCode.CURATIONS_NOT_FOUND_EXCEPTION);
+    }
+
+    private User getUserById(Long userId) {
+        return EntityFinder.findByIdOrThrow(userRepository.findById(userId)
+                , ErrorCode.USER_NOT_FOUND_EXCEPTION);
+    }
 }

src/main/java/org/likelion/likelion_12th_team05/curation/domain/repository/CurationRepository.java

@@ -20,4 +20,6 @@ public interface CurationRepository extends JpaRepository<Curation, Long> {
     List<Curation> findTop6ByOrderByLikeCountDesc();
 
     // 최신순 조회
+    @Query(value = "SELECT * FROM curation ORDER BY create_date DESC limit 6", nativeQuery = true)
+    List<Curation> findTop6byOrderByCreateDateDesc();
 }

src/main/java/org/likelion/likelion_12th_team05/like/application/LikeService.java

@@ -1,6 +1,7 @@
 package org.likelion.likelion_12th_team05.like.application;
 
 import lombok.RequiredArgsConstructor;
+import org.likelion.likelion_12th_team05.common.EntityFinder;
 import org.likelion.likelion_12th_team05.common.error.ErrorCode;
 import org.likelion.likelion_12th_team05.common.exception.NotFoundException;
 import org.likelion.likelion_12th_team05.curation.domain.Curation;
@@ -13,7 +14,6 @@
 import org.springframework.transaction.annotation.Transactional;
 import org.springframework.web.bind.annotation.PathVariable;
 
-import java.io.IOException;
 import java.security.Principal;
 import java.util.List;
 
@@ -27,14 +27,10 @@ public class LikeService {
 
     @Transactional
     public void likeSave(@PathVariable("curationId") Long curationId, Principal principal) {
-        Curation curation = curationRepository.findById(curationId).orElseThrow(
-                () -> new NotFoundException(ErrorCode.CURATIONS_NOT_FOUND_EXCEPTION,
-                        ErrorCode.CURATIONS_NOT_FOUND_EXCEPTION.getMessage()));
+        Curation curation = getCurationById(curationId);
 
-        String email = principal.getName();
-        User user = userRepository.findByEmail(email).orElseThrow(
-                () -> new NotFoundException(ErrorCode.USER_NOT_FOUND_EXCEPTION,
-                        ErrorCode.USER_NOT_FOUND_EXCEPTION.getMessage()));
+        Long id = Long.parseLong(principal.getName());
+        User user = getUserById(id);
 
         Like like = Like.of(curation, user);
         likeRepository.save(like);
@@ -46,14 +42,11 @@ public void likeSave(@PathVariable("curationId") Long curationId, Principal prin
     }
 
     @Transactional
-    public void likeDelete(@PathVariable("curationId") Long curationId, Principal principal) throws IOException {
-        Curation curation = curationRepository.findById(curationId).orElseThrow(
-                () -> new NotFoundException(ErrorCode.CURATIONS_NOT_FOUND_EXCEPTION,
-                        ErrorCode.CURATIONS_NOT_FOUND_EXCEPTION.getMessage()));
-        String email = principal.getName();
-        User user = userRepository.findByEmail(email).orElseThrow(
-                () -> new NotFoundException(ErrorCode.USER_NOT_FOUND_EXCEPTION,
-                        ErrorCode.USER_NOT_FOUND_EXCEPTION.getMessage()));
+    public void likeDelete(@PathVariable("curationId") Long curationId, Principal principal) {
+        Curation curation = getCurationById(curationId);
+
+        Long id = Long.parseLong(principal.getName());
+        User user = getUserById(id);
 
         // 큐레이션에 좋아요가 존재하는지 확인
         List<Like> existingLikes = likeRepository.findByCurationAndUser(curation, user);
@@ -68,4 +61,15 @@ public void likeDelete(@PathVariable("curationId") Long curationId, Principal pr
         curation.deleteLikeCount();
         curationRepository.save(curation);
     }
+
+    // 반복되는 예외 반환 메서드 추출 => 공통 예외 처리로 일관성 높이기 위함 => entityfinder로 중앙 관리
+    private Curation getCurationById(Long curationId) {
+        return EntityFinder.findByIdOrThrow(curationRepository.findById(curationId)
+                , ErrorCode.CURATIONS_NOT_FOUND_EXCEPTION);
+    }
+
+    private User getUserById(Long userId) {
+        return EntityFinder.findByIdOrThrow(userRepository.findById(userId)
+                , ErrorCode.USER_NOT_FOUND_EXCEPTION);
+    }
 }