[5기 김남규, 박유진] Spring Boot JPA 게시판 구현 미션 제출합니다. #269

eugene225 · 2023-11-23T02:36:05Z

📌 과제 설명

Spring Boot JPA - Rest API를 강의를 듣고, 게시판 구현 미션 수행

👩‍💻 요구 사항과 구현 내용

SpringDataJPA 를 설정한다.

datasource : h2

엔티티를 구성한다

회원(User)
- id (PK) (auto increment)
- name
- age
- hobby
- created_at
게시글(Post)
- id (PK) (auto increment)
- title
- content
- created_at
회원과 게시글에 대한 연관관계를 설정한다.
- 회원과 게시글은 1:N 관계이다.
게시글 Repository를 구현한다. (PostRepository)

API를 구현한다.

페이징 조회 (GET "/posts") : getAll
단건 조회 (GET "/posts/{id}") : findById
게시글 작성 (POST "/posts") : create
게시글 수정 (POST "/posts/{id}") : update

문서화

REST-DOCS를 이용해서 문서화한다.

✅ 피드백 반영사항

✅ PR 포인트 & 궁금한 점

GeneratedValue 전략 : Sequence vs Identity

sequence (H2 ... )
- sequence object 를 통해서 id 값 부여 → unique 값을 생성
DB에 들어가기 전에 id 값이 발급되기 때문에 쓰기 지연이 가능
- identity (MySQL ... )
- 데이터베이스 안에 insert 된 후 id 값이 부여 → 쓰기 지연 불가능

`CascadeType.ALL` + `orphanRemoval=true`

부모 엔티티를 통해서 자식 엔티티의 생명주기를 관리
cascadeType.ALL : persist + remove -> 단순하게 연관관계를 끊었을 경우 고아객체가 된 자식 엔티티가 삭제되지 않아서 발생할 수 있는 문제를 orpahnRemoval=true 옵션을 통해 해결

Entity생성 시 기본 생성자를 필요로 하는 이유

JPA의 지연로딩에 사용되는 Proxy 객체를 생성할 때에 Reflection API 가 활용
Reflection API : 객체 추정을 위해서는 기본생성자를 필요로 한다.
Proxy 객체를 기본 생성하려면 기본 생성자를 필요로 한다.

궁금한 점

restdocs 사용 시, page 정보 필요한 것들만 내보내도록 수정하는 방법이 swagger로 변경을 해야만 해결이 되는건지 궁금합니다.

SangHyunGil

구현하고 반영해주시느라 고생많으셨어요.
따로 코멘트가 없어서 완료된걸로 이해했어요. 다음부터는 완료되면 따로 코멘트 남겨주시면 좋을 것 같아요.
리뷰 남겨놓은 부분은 고민해보시고 남겨주시면 좋을 것 같고 머지해주셔도 좋습니다 👍

SangHyunGil · 2023-12-11T15:14:47Z

src/main/java/com/prgrms/dev/springbootboardjpa/controller/PostController.java

@@ -38,8 +38,7 @@ public ApiResponse<PostDto> create(@RequestBody PostRequestDto postRequestDto, @
    } // body에서 전부 받는 방식으로

    //수정
-    // post, put, patch ... HTTP 메서드 - 멱등성
-    @PostMapping("/{id}")
+    @PutMapping("/{id}")


SangHyunGil · 2023-12-11T15:17:03Z

src/main/java/com/prgrms/dev/springbootboardjpa/dto/PostCreateRequestDto.java

+    private void validateNull(String input) {
+        if(input.isEmpty() || input.isBlank()) throw new NullPointerException("NPE");


검증좋네요 👍
나중에 title / content에서 길이와 같은 개별 검증이 들어갈 수 있을 것 같기도 해서 각각 분리해보는걸 고려해봐도 좋을 것 같아요 👍

SangHyunGil · 2023-12-11T15:21:06Z

src/main/java/com/prgrms/dev/springbootboardjpa/exception/GlobalExceptionHandler.java


-@RestControllerAdvice
+@RestControllerAdvice(basePackageClasses = PostController.class)


패키지는 공통 exception이지만 포스트 컨트롤러에 대해서만 적용하신 이유가 있나요?

SangHyunGil · 2023-12-11T15:22:23Z

src/main/java/com/prgrms/dev/springbootboardjpa/exception/GlobalExceptionHandler.java

-    public ApiResponse<String> notFoundHandler(ChangeSetPersister.NotFoundException e) {
-        return ApiResponse.fail(NOT_FOUND, e.getMessage());
+    public ResponseEntity<String> notFoundHandler(ChangeSetPersister.NotFoundException e) {
+        return ResponseEntity.status(NOT_FOUND).body(e.getMessage());


HttpStatusCode를 활용하는걸 고려해보세요.

SangHyunGil · 2023-12-11T15:22:28Z

src/main/java/com/prgrms/dev/springbootboardjpa/exception/GlobalExceptionHandler.java

-        return ApiResponse.fail(INTERNAL_SERVER_ERROR, e.getMessage());
-    } // 다음 타자 .................
+    public ResponseEntity<String> internalServerErrorHandler(Exception e) {
+        return ResponseEntity.status(INTERNAL_SERVER_ERROR).body(e.getMessage());


HttpStatusCode를 활용하는걸 고려해보세요.

SangHyunGil · 2023-12-11T15:29:29Z

src/main/java/com/prgrms/dev/springbootboardjpa/domain/user/User.java

    private void validateAge(int age) {
-        if(age <= 0 || age >= 200) throw new NullPointerException("NPE");
+        if(age <= 0 || age >= 200) throw new IllegalArgumentException("나이는 1~199세 사이의 값 이어야 합니다.");


관련 검증을 Age 자체에 넘길 수 있도록 값객체로 선언할 수 있을 것 같아요.
나중에 고민해보시면 좋을 것 같아요 👍

SangHyunGil · 2023-12-11T16:23:22Z

src/main/java/com/prgrms/dev/springbootboardjpa/controller/PostController.java

+
+    //페이지 조회 -> get
+    @GetMapping
+    public ResponseEntity<Page<PostDto>> getAll(Pageable pageable) {


pageable로 받는 부분을 수정해보면 좋을 것 같네요 👍

SangHyunGil · 2023-12-11T16:38:46Z

src/main/java/com/prgrms/dev/springbootboardjpa/converter/Converter.java

+import com.prgrms.dev.springbootboardjpa.dto.UserDto;
+import org.springframework.stereotype.Component;
+
+@Component


꼭 빈으로 등록되어야할까요?

eugene225 and others added 23 commits November 20, 2023 22:19

init commit

5d1b20c

Chore : 프로젝트 환경설정

e67814f

Feat : entity 생성 - User, Post

71f3790

Fix : 연관관계 매핑 오류 해결

3bc557b

Feat : Post 관련 Dto 구현

b0c5ecb

Feat : PostRepository 구현

426d642

Refactor : 사용하지 않는 애노테이션 제거

7fbe52e

Feat : UserRepository 구현

91b8736

Feat : UserDto 구현

ec97bec

Feat : PostService 구현

5f485c8

Feat : PostController 구현

462b862

Refactor : PostDto에 UserDto 추가

1c49847

Feat : setUser, update 메서드 추가

bc065c9

Feat : Converter 구현

f4d7a9b

Refactor : update Api 수정

afa8a7d

Test : PostService 테스트

8bfc7d2

Feat : ApiReponse 생성 및 postController 적용

ff17326

Chore : restdocs 플러그인 추가

95e91f8

Test : PostController 테스트 및 문서화 작업

d15af2a

Feat : GlobalExceptionHandler 구현

33b67e2

Docs : api 문서화

c008314

Feat : postController ApiResponse 적용

268882d

Test : findById 실패 테스트 추가

cc0e870

eugene225 requested review from SangHyunGil and moosongsong November 23, 2023 02:36

eugene225 added 🔥 매운맛 Team-Dozzi 💧 순한맛 and removed 🔥 매운맛 💧 순한맛 labels Nov 23, 2023

eugene225 added the 🔥 매운맛 label Nov 23, 2023

eugene225 assigned eugene225 and GiHoo Nov 23, 2023

GiHoo and others added 18 commits December 4, 2023 00:50

1차 피드백 수정

63ed5a5

1차 피드백 수정

35628e6

Refactor: HTTP 메서드 변경 (POST -> PUT)

b01f6ed

Refactor: create를 위한 로직 수정(dto 생성 및 converter 추가)

57127d9

Refactor: update를 위한 로직 수정(null 체크)

4cafb55

Refactor: update를 위한 로직 수정

f6a301b

Remove: ApiResponse 제거

0b112f1

Refactor : ApiResponse -> ResponseEntity 변경

0f91afa

Refactor : @requiredargsconstructor 로 변경

6599c3c

Test: 변경한 Dto 반영

bdda4e9

Test: 변경한 Dto 반영

055fdd0

Docs: Test를 위한 환경 설정 추가

71d2f67

Refactor: BaseEntity abstract class 변경

226bcac

Feat: CustomException 생성

2068b99

Refactor: CustomException 적용

100260e

Refactor: CustomException 적용

e4bc9c1

Refactor: update 로직 변경

bd2b1ca

Comment: 1차 코드리뷰 주석 삭제

858c1ac

SangHyunGil approved these changes Dec 12, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[5기 김남규, 박유진] Spring Boot JPA 게시판 구현 미션 제출합니다. #269

[5기 김남규, 박유진] Spring Boot JPA 게시판 구현 미션 제출합니다. #269

eugene225 commented Nov 23, 2023 •

edited

SangHyunGil left a comment

SangHyunGil Dec 11, 2023

SangHyunGil Dec 11, 2023

SangHyunGil Dec 11, 2023

SangHyunGil Dec 11, 2023

SangHyunGil Dec 11, 2023

SangHyunGil Dec 11, 2023

SangHyunGil Dec 11, 2023

SangHyunGil Dec 11, 2023

		private void validateNull(String input) {
		if(input.isEmpty() \|\| input.isBlank()) throw new NullPointerException("NPE");


		@RestControllerAdvice
		@RestControllerAdvice(basePackageClasses = PostController.class)

[5기 김남규, 박유진] Spring Boot JPA 게시판 구현 미션 제출합니다. #269

Are you sure you want to change the base?

[5기 김남규, 박유진] Spring Boot JPA 게시판 구현 미션 제출합니다. #269

Conversation

eugene225 commented Nov 23, 2023 • edited

📌 과제 설명

👩‍💻 요구 사항과 구현 내용

SpringDataJPA 를 설정한다.

엔티티를 구성한다

API를 구현한다.

문서화

✅ 피드백 반영사항

✅ PR 포인트 & 궁금한 점

GeneratedValue 전략 : Sequence vs Identity

CascadeType.ALL + orphanRemoval=true

Entity생성 시 기본 생성자를 필요로 하는 이유

궁금한 점

SangHyunGil 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

Choose a reason for hiding this comment

Choose a reason for hiding this comment

eugene225 commented Nov 23, 2023 •

edited

`CascadeType.ALL` + `orphanRemoval=true`