Third seminar 과제 제출 #6

jiyeoon00 · 2023-05-02T21:05:19Z

이번 심화 과제는 기본 과제 코드에서 추가적으로 진행하였습니다.🫥

user_Id를 FK로 가지는 POST Entity를 추가로 구현
기본 기능(유저 생성, 게시물 생성, 게시물 조회) 구현
존재하지 않는 USER&POST에 접근할 경우 예외처리

YuSuhwa-ve

이번주도 수고많았습니다~

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

YuSuhwa-ve · 2023-05-04T14:00:24Z

ThirdSeminar/src/main/java/sopt/org/ThirdSeminar/controller/post/PostController.java

@@ -0,0 +1,28 @@
+package sopt.org.ThirdSeminar.controller.post;


폴더 구조를 아예 도메인 별로 나눴네용~

YuSuhwa-ve · 2023-05-04T14:02:24Z

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

+    @Transactional(readOnly = true)
+    public PostResponseDto getOne(Long postId) {
+        Post post = postRepository.findById(postId)
+                .orElseThrow(() -> new BusinessException(ErrorStatus.POST_NOT_FOUND));


비즈니스 로직 예외를 발생시켜 정리했네용. 혹시try caych문으로 에러를 발생~처리를 하지 않는 이유는 무엇인지 궁금해요

나도 수화 코멘트 보고 생각해봤는데, @RestControllerAdvice가 붙은 클래스에서 어짜피 처리해주기 때문인가? 어떻게 생각하는지 궁금해!

음 BusinessException은 관련 ErrorStatus 내용을 담은 Dto를 response해주는 식으로 균일하게 처리할 건데, 매번 똑같은 로직을 try-catch로 잡아서 처리해주는 것보다는 @RestControllerAdvice가 붙은 클래스에서 전역적으로 한 번에 처리 해 줄려고!?

YuSuhwa-ve · 2023-05-04T14:03:15Z

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

+    @Transactional
+    public UserResponseDto create(UserRequestDto request) {
+        User user = User.builder()
+                .email(request.getEmail())


P4) 유저 생성시 중복이메일, 중복 닉네임 검사 등도 예외 처리를 해보는것도 좋을 것 같아요!

오옹 바로 해볼게!!

ddongseop

많이 배우고 갑니다~~ 나도 예외처리 제대로 공부해야겠다는 생각이 들었어..ㅎㅎ

ddongseop · 2023-05-05T07:38:55Z

ThirdSeminar/src/main/java/sopt/org/ThirdSeminar/controller/post/PostController.java

+    private final PostService postService;
+
+    @PostMapping("/post/{userId}")
+    public ApiResponseDto create(@PathVariable final Long userId, @RequestBody @Valid final PostCreateDto postCreateDto) {


P5) 저는 userId를 request dto에 포함시켰는데, path variable로 따로 받아오게 하셨군요! 혹시 이렇게 하신 이유가 무엇인지 궁금해요!

사실 큰 의미는 없고,,보통 유저는 인증 토큰 정보? 를 이용해서 userId를 찾아오는 방식을 썼었는데, 아직 유저 인증을 구현을 안 했다 보니 그냥 @PathVariable로 받아왔어요! 근데 이렇게 보니 post/1 이런식으로 있으면 이게 userId인지 postId인지 혼동이 올 수 있을 것 같긴 하네요..

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

ddongseop · 2023-05-05T07:51:21Z

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

+    @Transactional(readOnly = true)
+    public PostResponseDto getOne(Long postId) {
+        Post post = postRepository.findById(postId)
+                .orElseThrow(() -> new BusinessException(ErrorStatus.POST_NOT_FOUND));


나도 수화 코멘트 보고 생각해봤는데, @RestControllerAdvice가 붙은 클래스에서 어짜피 처리해주기 때문인가? 어떻게 생각하는지 궁금해!

sookyungg · 2023-05-03T09:54:20Z

ThirdSeminar/src/main/java/sopt/org/ThirdSeminar/controller/post/PostController.java

+    public ApiResponseDto create(@PathVariable final Long userId, @RequestBody @Valid final PostCreateDto postCreateDto) {
+        return ApiResponseDto.success(SuccessStatus.POST_CREATE_SUCCESS, postService.create(userId, postCreateDto));
+    }
+


이 코드 게시물 생성 api 인가요? userid를 뒤에 추가해서 생성 하는 방법도 있군요..!

위에 동섭이 코멘트에 답변한 거처럼 사실 큰 의미는 없고,,보통 유저는 인증 토큰 정보? 를 이용해서 userId를 찾아오는 방식을 썼었는데, 아직 유저 인증을 구현을 안 했다 보니 그냥 @PathVariable로 받아왔어요! 근데 이렇게 보니 post/1 이런식으로 있으면 이게 userId인지 postId인지 혼동이 올 수 있을 것 같긴 하네요..

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

sookyungg · 2023-05-03T09:56:31Z

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

@@ -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));
    }
 }


orElseThrow 로 바로 날리는 방법도 있는 거 알아갑니다..!

jiyeoon00 added 4 commits May 3, 2023 02:02

basic: 세미나 실습코드

df2f546

advanced: POST Entity 구현

8c2f0f7

advanced: 게시물 생성 및 조회 구현

fa60099

advanced: 존재하지 않는 유저&게시물 접근할 시 예외 처리 구현

57b759d

jiyeoon00 added 🙂기본과제 기본과제 😎심화과제 심화과제 labels May 2, 2023

jiyeoon00 requested review from dev-Crayon, sookyungg, YuSuhwa-ve and ddongseop May 2, 2023 21:05

jiyeoon00 linked an issue May 2, 2023 that may be closed by this pull request

3주차 세미나 #5

Open

5 tasks

YuSuhwa-ve reviewed May 4, 2023

View reviewed changes

ddongseop reviewed May 5, 2023

View reviewed changes

sookyungg reviewed May 8, 2023

View reviewed changes

jiyeoon00 added 2 commits May 9, 2023 01:15

advanced: transactional 어노테이션 추가

42be135

advanced: 중복 닉네임&중복 이메일 에러 처리

43d41c9

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Third seminar 과제 제출 #6

Third seminar 과제 제출 #6

jiyeoon00 commented May 2, 2023

YuSuhwa-ve left a comment

YuSuhwa-ve May 4, 2023

YuSuhwa-ve May 4, 2023

ddongseop May 5, 2023

jiyeoon00 May 8, 2023

YuSuhwa-ve May 4, 2023

jiyeoon00 May 8, 2023

ddongseop left a comment •

edited

Loading

ddongseop May 5, 2023

jiyeoon00 May 8, 2023

ddongseop May 5, 2023

sookyungg May 3, 2023

jiyeoon00 May 8, 2023

sookyungg May 3, 2023

		@@ -0,0 +1,28 @@
		package sopt.org.ThirdSeminar.controller.post;

Third seminar 과제 제출 #6

Are you sure you want to change the base?

Third seminar 과제 제출 #6

Conversation

jiyeoon00 commented May 2, 2023

YuSuhwa-ve left a 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

ddongseop left a comment • 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

ddongseop left a comment •

edited

Loading