Third seminar 과제 제출

ThirdSeminar/src/main/java/sopt/org/ThirdSeminar/common/advice/ControllerExceptionAdvice.java

@@ -6,6 +6,7 @@
 import org.springframework.web.bind.annotation.ResponseStatus;
 import org.springframework.web.bind.annotation.RestControllerAdvice;
 import sopt.org.ThirdSeminar.common.dto.ApiResponseDto;
+import sopt.org.ThirdSeminar.exception.BusinessException;
 import sopt.org.ThirdSeminar.exception.ErrorStatus;
 
 @RestControllerAdvice
@@ -17,6 +18,11 @@ public class ControllerExceptionAdvice {
     @ResponseStatus(HttpStatus.BAD_REQUEST)
     @ExceptionHandler(MethodArgumentNotValidException.class)
     protected ApiResponseDto handleMethodArgumentNotValidException(final MethodArgumentNotValidException e) {
-        return ApiResponseDto.error(ErrorStatus.VALIDATION_REQUEST_MISSING_EXCEPTION);
+        return ApiResponseDto.error(ErrorStatus.VALIDATION_EXCEPTION);
+    }
+
+    @ExceptionHandler(BusinessException.class)
+    protected ApiResponseDto handleBusinessException(final BusinessException e) {
+        return ApiResponseDto.error(e.getErrorStatus());
     }
 }

ThirdSeminar/src/main/java/sopt/org/ThirdSeminar/exception/BusinessException.java

@@ -0,0 +1,10 @@
+package sopt.org.ThirdSeminar.exception;
+
+import lombok.Getter;
+import lombok.RequiredArgsConstructor;
+
+@Getter
+@RequiredArgsConstructor
+public class BusinessException extends RuntimeException {
+    private final ErrorStatus errorStatus;
+}

ThirdSeminar/src/main/java/sopt/org/ThirdSeminar/exception/ErrorStatus.java

@@ -28,6 +28,12 @@ public enum ErrorStatus {
     INTERNAL_SERVER_ERROR(HttpStatus.INTERNAL_SERVER_ERROR, "예상치 못한 서버 에러가 발생했습니다."),
     BAD_GATEWAY_EXCEPTION(HttpStatus.BAD_GATEWAY, "일시적인 에러가 발생하였습니다.\n잠시 후 다시 시도해주세요!"),
     SERVICE_UNAVAILABLE_EXCEPTION(HttpStatus.SERVICE_UNAVAILABLE, "현재 점검 중입니다.\n잠시 후 다시 시도해주세요!"),
+
+    /*
+    NOT_FOUND
+     */
+    USER_NOT_FOUND(HttpStatus.NOT_FOUND, "해당 유저가 존재하지 않습니다."),
+    POST_NOT_FOUND(HttpStatus.NOT_FOUND, "해당 게시물이 존재하지 않습니다."),
     ;
 
     private final HttpStatus httpStatus;

ThirdSeminar/src/main/java/sopt/org/ThirdSeminar/service/PostService.java

@@ -7,6 +7,8 @@
 import sopt.org.ThirdSeminar.controller.post.dto.response.PostResponseDto;
 import sopt.org.ThirdSeminar.domain.Post;
 import sopt.org.ThirdSeminar.domain.User;
+import sopt.org.ThirdSeminar.exception.BusinessException;
+import sopt.org.ThirdSeminar.exception.ErrorStatus;
 import sopt.org.ThirdSeminar.infrastructure.PostRepository;
 
 @Service
@@ -33,7 +35,8 @@ public PostResponseDto create(Long userId, PostCreateDto postCreateDto) {
 
     @Transactional(readOnly = true)
     public PostResponseDto getOne(Long postId) {
-        Post post = postRepository.findById(postId).orElseThrow();
+        Post post = postRepository.findById(postId)
+                .orElseThrow(() -> new BusinessException(ErrorStatus.POST_NOT_FOUND));
 
         return PostResponseDto.of(post.getTitle(), post.getContent(), post.getUser().getNickname());
     }

ThirdSeminar/src/main/java/sopt/org/ThirdSeminar/service/UserService.java

@@ -6,6 +6,8 @@
 import sopt.org.ThirdSeminar.controller.user.dto.request.UserRequestDto;
 import sopt.org.ThirdSeminar.controller.user.dto.response.UserResponseDto;
 import sopt.org.ThirdSeminar.domain.User;
+import sopt.org.ThirdSeminar.exception.BusinessException;
+import sopt.org.ThirdSeminar.exception.ErrorStatus;
 import sopt.org.ThirdSeminar.infrastructure.UserRepository;
 
 @Service
@@ -28,6 +30,7 @@ public UserResponseDto create(UserRequestDto request) {
     }
 
     public User findUserById(Long userId) {
-        return userRepository.findById(userId).orElseThrow();
+        return userRepository.findById(userId)
+                .orElseThrow(() -> new BusinessException(ErrorStatus.USER_NOT_FOUND));
     }
 }