[4기 윤영운, 김별] Spring Boot JPA 게시판 구현 미션 제출합니다. #258

byeolhaha · 2023-08-06T15:49:04Z

📌 과제 설명

SpringDataJPA 를 설정한다.

datasource : h2 or mysql

엔티티를 구성한다

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

API를 구현한다.

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

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

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

게시글 조회

게시글 조회할 때 제목과 내용으로 검색이 가능하도록 필터 기능을 넣었습니다.
N+1 문제가 발생되는 것을 개선하여 PostRepository에서 Join문을 이용하였습니다.

@Query("SELECT p FROM Post p " +
        "JOIN FETCH p.member " +
        "WHERE (p.title LIKE %:title%) " +
        "AND (p.content LIKE %:content%)")
 Slice<Post> findPostAllByFilter(@Param("title") String title,
                                @Param("content") String content,
                                Pageable pageable);

페이징 조회 (GET "/posts")
- Slice를 이용하여 무한스크롤로 구현하였습니다.
게시글 수정 요구 사항의 경우 "POST "로 되어 있었으나 PATCH로 수정하였습니다.

로그인 기능이 구현되지 않아서 AuditorAware 커스텀하여 CreatedBy에 관리자 이름이 들어가도록 하였습니다.

@EnableJpaAuditing
@Configuration
@Component
public class AuditorAwareConfig implements AuditorAware<String>, DateTimeProvider {

 @Override
 public Optional<String> getCurrentAuditor() {
     return Optional.of("별앤영");
  }

  @Override
  public Optional<TemporalAccessor> getNow() {
     return Optional.of(LocalDateTime.now());
   }
 }

REST-DOCS를 이용해서 문서화를 진행하였습니다.

- feat : restdoc 기능 구현 - test : controllerTest 구현 - refactor : findAllBy 관련 406 버그 수정

refactor: findPostAllByFilter메서드 join을 활용하는 방식으로 변경

@Valid

- nullable에서 NotNull로 수정 - post set 메서드 네임 변경 - controller '@Valid' 어노테이션 추가하여 null 방지

wonu606 · 2023-08-07T08:56:49Z

build.gradle

+tasks.named('asciidoctor') {
+	inputs.dir snippetsDir
+	attributes 'snippets': snippetsDir
+	dependsOn test
+}


테스트 코드 수정시 resources/static/docs/index.html이 업그레이드 되나요?

wonu606 · 2023-08-07T08:58:41Z

src/main/java/com/example/jpaboard/member/domain/Member.java

+    private String name;
+
+    @Embedded
+    private Age age;


원시값 포장하여 저장하는 방식 좋은 것 같습니다!👍

wonu606 · 2023-08-07T09:00:41Z

src/main/java/com/example/jpaboard/member/service/MemberService.java

+    public FindMemberResponse findById(Long id) {
+        Member member = memberRepository.findById(id).orElseThrow(() -> new EntityNotFoundException("존재하지 않은 고객입니다."));
+
+        return new FindMemberResponse(member);


FindMemberResponse 해당 클래스가 함수명처럼 느껴지는 것 같습니다.

MemberRetrieveResponse 이건 어떰요?

wonu606 · 2023-08-07T09:02:59Z

src/main/java/com/example/jpaboard/post/controller/dto/SaveApiRequest.java

+
+import jakarta.validation.constraints.NotNull;
+
+public record SaveApiRequest(@NotNull(message = "memberId 값이 입력되지 않았습니다.") Long memberId,


상세 메시지를 작성해놓으면 에러 찾을 때도 좋을 것 같네용👍

�제가 원한게 이거였어요.

근데 NotBlank가 더 맞음 ㅎㅎ

String에 한정해서는요.

wonu606 · 2023-08-07T09:10:05Z

src/main/java/com/example/jpaboard/post/controller/PostController.java

+public class PostController {
+
+    private final PostService postService;
+    private final PostApiMapper postApiMapper;


Api는 controller를 나타내는 걸까요?

넵 맞습니다!!

wonu606 · 2023-08-07T09:12:07Z

src/main/java/com/example/jpaboard/post/controller/PostController.java

+        URI location = ServletUriComponentsBuilder.fromCurrentRequest()
+                .path("/{id}")
+                .buildAndExpand(saveResponse.postId())
+                .toUri();


오 이런 방식이 있군요!👍

wonu606 · 2023-08-07T09:24:56Z

src/test/java/com/example/jpaboard/post/controller/PostControllerTest.java

+    void findAllByFilter() throws Exception {
+        FieldDescriptor[] responseFields = new FieldDescriptor[] {
+                subsectionWithPath("result").description("Result details"),
+                fieldWithPath("result.content[]").description("The list of content items"),
+                subsectionWithPath("result.pageable").description("Pagination information"),
+                fieldWithPath("result.pageable.sort.empty").description("Indicates if the sort is empty"),
+                fieldWithPath("result.pageable.sort.sorted").description("Indicates if the sort is sorted"),
+                fieldWithPath("result.pageable.sort.unsorted").description("Indicates if the sort is unsorted"),
+                fieldWithPath("result.pageable.offset").description("Offset value for pagination"),
+                fieldWithPath("result.pageable.pageSize").description("Page size for pagination"),
+                fieldWithPath("result.pageable.pageNumber").description("Page number"),
+                fieldWithPath("result.pageable.unpaged").description("Indicates if the page is unpaged"),
+                fieldWithPath("result.pageable.paged").description("Indicates if the page is paged"),
+                fieldWithPath("result.size").description("The page size"),
+                fieldWithPath("result.number").description("The current page number"),
+                fieldWithPath("result.sort.empty").description("Indicates if the sort is empty"),
+                fieldWithPath("result.sort.sorted").description("Indicates if the sort is sorted"),
+                fieldWithPath("result.sort.unsorted").description("Indicates if the sort is unsorted"),
+                fieldWithPath("result.first").description("Indicates if this is the first page"),
+                fieldWithPath("result.last").description("Indicates if this is the last page"),
+                fieldWithPath("result.numberOfElements").description("Number of elements in the current page"),
+                fieldWithPath("result.empty").description("Indicates if the content is empty"),
+                fieldWithPath("resultCode").description("The result code of the response"),
+                fieldWithPath("resultMsg").description("The result message of the response")
+        };


클라이언트에 이 정보들이 필요한가용?

restdocs의 경우 필드 하나하나 맞춰줘야 해서 넣게 되었습니다. ✏️

wonu606 · 2023-08-07T09:27:09Z

src/main/java/com/example/jpaboard/post/service/PostService.java

+                .orElseThrow(() -> new EntityNotFoundException("해당 post가 존재하지 않습니다."));
+        Long postOwnerId = findPost.getMember().getId();
+
+        if (isNotOwner(request.memberId(), postOwnerId)){


오 이런 로직 좋네용👍

어순에 맞게 코드를 작성 해주는게 제일 좋긴해요.

thePost.isOwner(memberId) 처럼.

wonu606 · 2023-08-07T09:40:59Z

src/main/java/com/example/jpaboard/post/service/PostService.java

+        FindMemberResponse findMember = memberService.findById(request.memberId());
+        Post post = mapper.to(request, findMember);


Member entity가 생성되는 거 같습니다.
그럼 Post에서 저장하는 Member, Entity로 쓰이는 Member 두가지 의미로 사용되는 거 같은데, 괜찮을까요?

wonu606 · 2023-08-07T09:43:04Z

영운님 별님 코드 잘 보았습니다!
페어프로그래밍이라 서로 의견을 조율해 나가는 과정이 쉽진 않으셨을텐데,
수고하셨습니다!!😁😁😁
저희 프로젝트엔 적용하지 못한 로직들이 많은 것 같아 배우고 가는 시간이었습니다.

onetuks · 2023-08-07T12:02:03Z

src/main/java/com/example/jpaboard/global/BaseEntity.java

+    @CreatedDate
+    @DateTimeFormat(pattern = "yyyy-MM-dd/HH:mm:ss")


만약 밀리세컨드를 표시하고 싶지 않다면, @column 어노테이션을 이용해서 타입을 명시해줄 수도 있어요.

JaeyoungAhn

영운님, 별님 코드 잘 보았습니다!

여러 상황들을 대비하기 위해 코드가 세부적으로 작성되었다는게 느껴집니다!

리뷰를 하면서 오히려 제가 많이 배울 수 있는 시간이었습니다.

페어프로그래밍 하신다고 수고 많으셨습니다!

JaeyoungAhn · 2023-08-07T12:37:46Z

src/main/java/com/example/jpaboard/post/service/PostService.java

+
+    @Transactional
+    public PostResponse updatePost(Long id, UpdateRequest request) {
+        Post findPost = postRepository.findById(id)


findMember의 find가 동사처럼 보이는데 더 좋은 변수명은 없을까용?

JaeyoungAhn · 2023-08-07T12:58:32Z

src/main/java/com/example/jpaboard/global/GlobalRestExceptionHandler.java

+        for (FieldError fieldError : bindingResult.getFieldErrors()) {
+            stringBuilder.append(fieldError.getField()).append(":");
+            stringBuilder.append(fieldError.getDefaultMessage());
+            stringBuilder.append(", ");
+        }


BindException 발생 시 세부적인 내용을 알려주는 코드 좋습니다!

JaeyoungAhn · 2023-08-07T13:15:50Z

src/main/java/com/example/jpaboard/post/domain/PostRepository.java

악의적인 유저가 Filter를 무시하고 모든 데이터들을 불러오는 것은 어려울까요? 만약 가능하다면 이를 예방할 수 있는 방법들이 있을까요?

onetuks

영운님, 별님
과제하시느라 고생하셨습니다.

강의 내용을 바탕으로 연관된 걸 많이 공부하신 느낌이 드네요.

전반적으로 코드도 깔끔했던 것 같아요.

고생하셨습니다~~😄

onetuks · 2023-08-07T12:05:26Z

src/main/java/com/example/jpaboard/global/GlobalRestExceptionHandler.java

+        StringBuilder resultStringBuilder = getResultStringBuilder(e);
+        int statusCode = HttpStatus.BAD_REQUEST.value();
+
+        return ResponseEntity.status(statusCode).body(getErrorResponse(statusCode, resultStringBuilder.toString(), request.getRequestURI()));


메소드 체이닝 마다 라인을 바꿔주면 한눈에 어떤 응답을 하는지 보기 좋을 것 같아요

return ResponseEntity .status(statusCode) .body(getErrorResponse( statusCode, resultStringBuilder.toString(), request.getRequestURI()));

status(BAD_REQUEST) 하면 안되나요?ㅠ statusCode 한번 보러가야 하잖아요 ㅠ

onetuks · 2023-08-07T12:06:37Z

src/main/java/com/example/jpaboard/global/SliceResponse.java

+public class SliceResponse<T> extends ApiResponse {
+
+    private Slice<T> data;
+
+    public SliceResponse(Slice<T> data, SuccessCode successCode) {


무한 스크롤을 생각하셨다니! 👍👍

onetuks · 2023-08-07T12:07:45Z

src/main/java/com/example/jpaboard/member/domain/Age.java

+@Embeddable
+public class Age {
+
+    private static final int AGE_MIN = 0;


이런 디테일이 프로그램을 좋게 만드는 것 같아요

onetuks · 2023-08-07T12:09:53Z

src/main/java/com/example/jpaboard/post/controller/PostController.java

+        return postResponseSliceResponse;
+    }
+
+    @PatchMapping("/{id}")


기본값이 json이긴 하지만, consumes 같은 메소드를 사용해서 명시해주는 것도 좋을 것 같아요.

onetuks · 2023-08-07T12:12:46Z

src/test/java/com/example/jpaboard/post/service/PostServiceTest.java

+    private void setupData() {
+        Member member1 = new Member("김별", new Age(27), "락 부르기");
+        Member member2 = new Member("윤영운", new Age(27), "저글링 돌리기");
+        Member member3 = new Member("박세영", new Age(27), "산책");


과제하시면서 제 생각이 나다니... 감동입니다👍

onetuks · 2023-08-08T11:50:53Z

src/main/java/com/example/jpaboard/global/ErrorResponse.java

+
+import java.time.LocalDateTime;
+
+public record ErrorResponse(int statusCode,


int 타입의 원시 http 상태코드는 버전 6 이후로 deprecated 된다고 해요.
HttpStatus enum 클래스를 써보시면 좋을 것 같습니다.

onetuks · 2023-08-08T11:54:46Z

src/main/java/com/example/jpaboard/global/exception/PermissionDeniedEditException.java

@@ -0,0 +1,9 @@
+package com.example.jpaboard.global.exception;
+
+public class PermissionDeniedEditException extends RuntimeException {


EntityNotFound 처럼 무엇에 대한 것인지 먼저 나오면 좋을 것 같아요.
EditPermissionDeniedException 추천합니다!

좋은 네이밍 추천 감사합니다ㅎㅎ

onetuks · 2023-08-08T11:57:40Z

src/test/java/com/example/jpaboard/post/controller/PostControllerTest.java

+                        requestFields(requestFields),
+                        responseFields(responseFields)


preprocess(prettyPrint()) 로 예쁘게 표현 가능하다고 해요!

KimMinheee

영운님, 별님 JPA 게시판 과제 구현하시느라 정말 고생 많으셨습니다 :)
코드가 깔끔해서 정말 쉽게 읽을 수 있었던 것 같아요 👍🏼
궁금했던 부분에 대해 겹치는 코멘트는 중복으로 달지 않고 다른분들 코멘트에 이모지👀를 달아두었고 제가 추가로 궁금했던 부분에 코멘트를 달아놨습니다 :)
코드 읽으면서 많이 배웠습니다! 고생하셨습니다 !

KimMinheee · 2023-08-09T03:46:16Z

src/main/java/com/example/jpaboard/JpaboardApplication.java

+@SpringBootApplication
+@EnableJpaAuditing(
+		auditorAwareRef = "auditorAwareConfig",
+		dateTimeProviderRef = "auditorAwareConfig"
+)


dateTimeProviderRef라는 속성이 있었군요! 하나 배워갑니다~
Jpa와 관련된 속성 혹은 어노테이션들은 별도의 JPA 관련 config 파일들을 두어 관리할 수 있을 것 같습니다! :)

KimMinheee · 2023-08-09T03:50:13Z

src/main/java/com/example/jpaboard/member/domain/Member.java

+    @NotNull
+    private String name;


name에 NotBlank가 아닌 NotNull로 validation을 거신 이유가 궁금합니다! :)
"" 나 " "를 허용함을 염두에 두신것이라면 NotNull이 맞을 것 같네요

다만, @NotNull 어노테이션은 db의 ddl에 notnull이 적용되지만, @notblank는 적용되지 않으니 이 부분을 고려하여 DTO에도 적절히 validation을 추가하면 좋을 듯 합니다.
nullable = false와 @NotNull 비교

KimMinheee · 2023-08-09T04:33:50Z

src/test/java/com/example/jpaboard/member/service/MemberServiceTest.java

+    @Autowired
+    MemberRepository memberRepository;
+
+    @Autowired
+    MemberService memberService;
+    Long id;


접근제한자가 추가되면 좋을 것 같습니다 😊

KimMinheee · 2023-08-09T04:44:03Z

src/main/java/com/example/jpaboard/member/domain/Member.java

+    @Embedded
+    private Age age;
+
+    private String hobby;


회원의 hobby에도 필드 검증이 추가되면 좋을 듯 합니다!

KimMinheee · 2023-08-09T05:09:02Z

src/main/java/com/example/jpaboard/global/SliceResponse.java

+
+public class SliceResponse<T> extends ApiResponse {
+
+    private Slice<T> data;


무한스크롤까지 구현하셨군요!! 최고최고👍🏼

WooSungHwan

과제하시느라 수고많으셨습니다 ㅎㅎ 테스트코드도 많이 작성하셨고 코드도 괜찮은데 기본적으로 있어야할 부분이 빠진게 좀 있어서 그 부분 채워졌다 싶을떄 approve 드릴게요! 리뷰내용이 도움이 되었으면 좋겠습니다.

WooSungHwan · 2023-08-10T11:44:21Z

src/main/java/com/example/jpaboard/JpaboardApplication.java

+		auditorAwareRef = "auditorAwareConfig",
+		dateTimeProviderRef = "auditorAwareConfig"
+)
+public class JpaboardApplication {


클래스명 camel 아닌듯

@byeolhaha 해당 부분 해결이 안됐네요.

WooSungHwan · 2023-08-10T11:44:48Z

src/main/java/com/example/jpaboard/config/AuditorAwareConfig.java

+
+@EnableJpaAuditing
+@Configuration
+@Component


component는 필요없지 않을까요?ㅎㅎ

configuration과 component가 합쳐져 있는 것 같은데 분리하셔요.

WooSungHwan · 2023-08-10T11:48:52Z

src/main/java/com/example/jpaboard/global/GlobalRestExceptionHandler.java

+    @ExceptionHandler({IllegalArgumentException.class, MissingRequestHeaderException.class, HttpMessageNotReadableException.class,
+            HttpClientErrorException.class, NoHandlerFoundException.class})
+    public ResponseEntity<ErrorResponse> handleBadRequestException(HttpServletRequest request, Exception e) {
+        int statusCode = HttpStatus.BAD_REQUEST.value();
+
+        return ResponseEntity.status(statusCode).body(getErrorResponse(statusCode, e.getMessage(), request.getRequestURI()));
+    }


적어도 switch 문이라도 있거나 instanceof라도 있어서 어떤 exception이 인스턴스로 넘어왔는지 log는 찍어줄 수 있을 것 같아요. 이후처리에 대한 대비를 하거나요. 근데 그럴거면 굳이 이렇게 합쳐놓을 필요가 애초에 없겠죠 ㅎㅎㅎ 이렇게 퉁처리는 장점이 될 수 없습니다.

WooSungHwan · 2023-08-10T11:49:36Z

src/main/java/com/example/jpaboard/global/GlobalRestExceptionHandler.java

+        StringBuilder resultStringBuilder = getResultStringBuilder(e);
+        int statusCode = HttpStatus.BAD_REQUEST.value();
+
+        return ResponseEntity.status(statusCode).body(getErrorResponse(statusCode, resultStringBuilder.toString(), request.getRequestURI()));


status(BAD_REQUEST) 하면 안되나요?ㅠ statusCode 한번 보러가야 하잖아요 ㅠ

WooSungHwan · 2023-08-10T11:49:55Z

src/main/java/com/example/jpaboard/global/GlobalRestExceptionHandler.java

+    @ExceptionHandler(Exception.class)
+    public ResponseEntity<Void> handleException(HttpServletRequest request, Exception e) {
+        logger.error("Sever Exception: ", e.getMessage());
+
+        return ResponseEntity.ok().build();
+    }


Exception인데 왜 ok죠?

WooSungHwan · 2023-08-10T11:54:41Z

src/main/java/com/example/jpaboard/post/controller/dto/SaveApiRequest.java

+
+import jakarta.validation.constraints.NotNull;
+
+public record SaveApiRequest(@NotNull(message = "memberId 값이 입력되지 않았습니다.") Long memberId,


�제가 원한게 이거였어요.

WooSungHwan · 2023-08-10T11:54:54Z

src/main/java/com/example/jpaboard/post/controller/dto/SaveApiRequest.java

+
+import jakarta.validation.constraints.NotNull;
+
+public record SaveApiRequest(@NotNull(message = "memberId 값이 입력되지 않았습니다.") Long memberId,


근데 NotBlank가 더 맞음 ㅎㅎ

WooSungHwan · 2023-08-10T11:55:05Z

src/main/java/com/example/jpaboard/post/controller/dto/SaveApiRequest.java

+
+import jakarta.validation.constraints.NotNull;
+
+public record SaveApiRequest(@NotNull(message = "memberId 값이 입력되지 않았습니다.") Long memberId,


String에 한정해서는요.

WooSungHwan · 2023-08-10T11:57:48Z

src/main/java/com/example/jpaboard/post/domain/Post.java

+    @ManyToOne
+    private Member member;


관계 맺을거면 확실하게!

lazy인지 제약조건 뭐있는지 어디 컬럼으로 걸건지 등등

WooSungHwan · 2023-08-10T12:00:01Z

src/main/java/com/example/jpaboard/post/service/PostService.java

+                .orElseThrow(() -> new EntityNotFoundException("해당 post가 존재하지 않습니다."));
+        Long postOwnerId = findPost.getMember().getId();
+
+        if (isNotOwner(request.memberId(), postOwnerId)){


어순에 맞게 코드를 작성 해주는게 제일 좋긴해요.

thePost.isOwner(memberId) 처럼.

WooSungHwan

수고 많으셨습니다 ㅎㅎ 코드가 깔끔합니다!
그런데 facade 같은 경우 service 보다는 별도 패키지로 둬주는게 좋을 것 같아요. 우선 service layer보다는 더 앞단에 있는 layer이기 때문에 service는 아니라고 판단해야 할 것 같습니다.

WooSungHwan · 2023-08-17T12:24:01Z

src/main/java/com/example/jpaboard/JpaboardApplication.java

+		auditorAwareRef = "auditorAwareConfig",
+		dateTimeProviderRef = "auditorAwareConfig"
+)
+public class JpaboardApplication {


@byeolhaha 해당 부분 해결이 안됐네요.

devYSK · 2023-08-27T14:27:57Z

src/main/java/com/example/jpaboard/global/ApiResponse.java

+    private int resultCode;
+    private String resultMsg;
+
+    public ApiResponse(final T result, SuccessCode successCode) {


왜 result만 final인가요?ㅎㅎ

devYSK · 2023-08-27T14:29:52Z

src/main/java/com/example/jpaboard/member/domain/Member.java

+
+    public Member(Name name, Age age, String hobby) {
+        this.name = name;
+        this.age = age;
+        this.hobby = hobby;
+    }
+
+    public Member(Long id, Name name, Age age, String hobby) {
+        this.id = id;
+        this.name = name;
+        this.age = age;
+        this.hobby = hobby;
+    }
+


VO로 잘 구현했지만, null 방어가 아쉽습니다! 실수를 방지할 수 있으면 방지하는게 좋은것같아요

devYSK · 2023-08-27T14:30:12Z

src/main/java/com/example/jpaboard/member/domain/Name.java

+
+    public Name(String value) {
+        validateName(value);
+        this.value = value;
+    }
+
+    public void changeName(String value) {


풍부하네요.

devYSK · 2023-08-27T14:30:38Z

src/main/java/com/example/jpaboard/member/domain/Name.java

+
+    private String value;
+


컬럼이 될 존재이고 nonnull이면

@Column(nullable=false)

는 어떨까요?

devYSK · 2023-08-27T14:31:01Z

src/main/java/com/example/jpaboard/member/service/MemberService.java

+
+    public MemberFindResponse findById(Long id) {
+        Member member = memberRepository.findById(id).orElseThrow(() -> new EntityNotFoundException("존재하지 않은 고객입니다."));
+
+        return new MemberFindResponse(member);
+    }


@Transactional(readonly=true)

devYSK · 2023-08-27T14:36:57Z

src/main/java/com/example/jpaboard/post/service/PostService.java

+
+        if (findPost.isNotOwner(request.memberId())){
+            throw new UnauthorizedEditException("해당 게시글을 수정할 권한이 없습니다.");
+        }


좋네요 도메인 로직.

이정도면 다음것도 고민해볼수 있겠어요

findPost.checkUpdate(memberId); // 내부적으로 throw하여 로직 응집

devYSK · 2023-08-27T14:37:22Z

src/main/java/com/example/jpaboard/rpc/RpcInput.java

+@RequiredArgsConstructor
+public class RpcInput {
+
+    private final PostService postService;
+
+    public void create(Map<String, String> request) {
+
+        var title = request.get("title");
+
+        var content = request.get("content");
+
+        Post.create(title, content, null);
+    }
+


이건모에용 ㅎㅎ

devYSK · 2023-08-27T14:37:40Z

src/test/java/com/example/jpaboard/member/domain/AgeTest.java

+    @Test
+    @DisplayName("나이에 대해서 음수를 입력한 경우 예외를 던진다.")
+    void member_MinusAge_throwsException() {


생성할수 없다도 좋겠어요

devYSK · 2023-08-27T14:38:25Z

src/test/java/com/example/jpaboard/post/controller/PostControllerTest.java

+                subsectionWithPath("result").description("Result details"),
+                fieldWithPath("result.content[]").description("The list of content items"),
+                subsectionWithPath("result.pageable").description("Pagination information"),
+                fieldWithPath("result.pageable.sort.empty").description("Indicates if the sort is empty"),
+                fieldWithPath("result.pageable.sort.sorted").description("Indicates if the sort is sorted"),
+                fieldWithPath("result.pageable.sort.unsorted").description("Indicates if the sort is unsorted"),
+                fieldWithPath("result.pageable.offset").description("Offset value for pagination"),
+                fieldWithPath("result.pageable.pageSize").description("Page size for pagination"),
+                fieldWithPath("result.pageable.pageNumber").description("Page number"),
+                fieldWithPath("result.pageable.unpaged").description("Indicates if the page is unpaged"),
+                fieldWithPath("result.pageable.paged").description("Indicates if the page is paged"),
+                fieldWithPath("result.size").description("The page size"),
+                fieldWithPath("result.number").description("The current page number"),
+                fieldWithPath("result.sort.empty").description("Indicates if the sort is empty"),
+                fieldWithPath("result.sort.sorted").description("Indicates if the sort is sorted"),
+                fieldWithPath("result.sort.unsorted").description("Indicates if the sort is unsorted"),
+                fieldWithPath("result.first").description("Indicates if this is the first page"),
+                fieldWithPath("result.last").description("Indicates if this is the last page"),
+                fieldWithPath("result.numberOfElements").description("Number of elements in the current page"),
+                fieldWithPath("result.empty").description("Indicates if the content is empty"),
+                fieldWithPath("resultCode").description("The result code of the response"),
+                fieldWithPath("resultMsg").description("The result message of the response")
+        };


외국계 회사군요

devYSK · 2023-08-27T14:38:52Z

src/test/java/com/example/jpaboard/post/service/PostServiceTest.java

+
+    @Autowired
+    PostService postService;
+    @Autowired
+    PostRepository postRepository;
+    @Autowired
+    MemberRepository memberRepository;
+
+
+    Long setupPostId1;
+
+    Long setupPostId2;
+    Long setupMemberId2;
+
+    Long setupMemberId3;
+


테스트코드도 이쁘게 관리하면 좋을것같아요

byeolhaha and others added 16 commits August 3, 2023 21:38

git init

53a2006

git init packages

ae776a9

feat : ã�Member ë관련 기능 구현

140f343

feat: post crud service 구현

fabdb67

Merge remote-tracking branch 'origin/main' into yw

d0465b8

feat: PostController, PostServiceTest 일부 구현

72e7ea8

요구사항 반영

a187d7e

- feat : restdoc 기능 구현 - test : controllerTest 구현 - refactor : findAllBy 관련 406 버그 수정

feat: PostServiceTest 완성, GlobalRestExceptionHandler 구현

a61bed8

feat : BaseEntity 구현 및 Custom AuditorAware 구현

f3f1e86

fix : conflict 해결

171a2e1

refactor: findPostAllByFilter메서드 join을 활용하는 방식으로 변경

9aeab27

Merge pull request #1 from byeolhaha/yw

0e70d21

refactor: findPostAllByFilter메서드 join을 활용하는 방식으로 변경

refactor: member와 post 조인 쿼리 변경

26232b4

refactor

613c8db

- nullable에서 NotNull로 수정 - post set 메서드 네임 변경 - controller '@Valid' 어노테이션 추가하여 null 방지

style : 코드 정렬 및 불필요한 import 삭제

fbb4744

gitignore yml 파일 포함

bd3b4e4

byeolhaha added 🔥 매운맛 Team-Blackdog labels Aug 6, 2023

byeolhaha requested review from WooSungHwan and devYSK August 6, 2023 15:49

byeolhaha assigned byeolhaha and young970 Aug 6, 2023

byeolhaha requested review from KimMinheee, JaeyoungAhn, wonu606 and onetuks and removed request for KimMinheee August 6, 2023 15:54

wonu606 reviewed Aug 7, 2023

View reviewed changes

wonu606 approved these changes Aug 7, 2023

View reviewed changes

onetuks reviewed Aug 7, 2023

View reviewed changes

JaeyoungAhn approved these changes Aug 7, 2023

View reviewed changes

onetuks self-requested a review August 8, 2023 11:45

onetuks approved these changes Aug 8, 2023

View reviewed changes

KimMinheee approved these changes Aug 9, 2023

View reviewed changes

WooSungHwan requested changes Aug 10, 2023

View reviewed changes

byeolhaha added 2 commits August 12, 2023 20:25

refactor : 코드리뷰 반영

93c70bc

feat : 퍼사드 패턴 구현

5f93988

byeolhaha requested a review from WooSungHwan August 14, 2023 16:04

WooSungHwan approved these changes Aug 17, 2023

View reviewed changes

devYSK approved these changes Aug 27, 2023

View reviewed changes


		import jakarta.validation.constraints.NotNull;

		public record SaveApiRequest(@NotNull(message = "memberId 값이 입력되지 않았습니다.") Long memberId,

		FindMemberResponse findMember = memberService.findById(request.memberId());
		Post post = mapper.to(request, findMember);

		@CreatedDate
		@DateTimeFormat(pattern = "yyyy-MM-dd/HH:mm:ss")


		import java.time.LocalDateTime;

		public record ErrorResponse(int statusCode,

		@@ -0,0 +1,9 @@
		package com.example.jpaboard.global.exception;

		public class PermissionDeniedEditException extends RuntimeException {


		public class SliceResponse<T> extends ApiResponse {

		private Slice<T> data;

[4기 윤영운, 김별] Spring Boot JPA 게시판 구현 미션 제출합니다. #258

Are you sure you want to change the base?

[4기 윤영운, 김별] Spring Boot JPA 게시판 구현 미션 제출합니다. #258

Conversation

byeolhaha commented Aug 6, 2023

📌 과제 설명

SpringDataJPA 를 설정한다.

엔티티를 구성한다

API를 구현한다.

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

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

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

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

byeolhaha Aug 8, 2023 • edited

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

wonu606 commented Aug 7, 2023

onetuks Aug 7, 2023 • edited

Choose a reason for hiding this comment

JaeyoungAhn 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

onetuks 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

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KimMinheee left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KimMinheee Aug 9, 2023 • edited

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

WooSungHwan 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

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

WooSungHwan 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

byeolhaha Aug 8, 2023 •

edited

onetuks Aug 7, 2023 •

edited

KimMinheee Aug 9, 2023 •

edited