feat: 공지 CRUD API 구현 #323

kdkdhoho · 2024-10-30T07:38:30Z

Description

공지 CRUD API를 구현했습니다.

Relation Issues

close 게시글(공지) 관련 API 구현 #312

정확한 원인은 파악하지 못했지만, 추측컨데 AcceptanceTest와 IntegrationTest의 테스트 DB가 공유되면서 외래키 제약 조건으로 발생한 것으로 예상 (#312)

pparkjs

생각보다 공지가 구현할게 많았네요!!
고생하셨어요 ㅎㅎ 깔끔하게 잘 구현했네요

제가 코멘트 단부분 한 번 고민하시고 답변 주세용~~~

pparkjs · 2024-10-30T07:43:02Z

src/test/java/com/listywave/common/IntegrationTest.java

+    @Autowired
+    protected HistoryRepository historyRepository;
+    @Autowired
+    protected UserReactionRepository userReactionRepository;
+    @Autowired
+    protected ReactionStatsRepository reactionStatsRepository;

    protected User dh, js, ej, sy;
    protected ListEntity list;

    @BeforeEach
    void setUp() {
+        historyRepository.deleteAll();
+        userReactionRepository.deleteAllInBatch();
+        reactionStatsRepository.deleteAllInBatch();


엇 그럼 이거 그 카톡으로 말했던 문제 이거로 해결이 된건가요?

네엡 해결은 됐습니다만.. 원인은 정확히 모르겠네요 ㅜ

pparkjs · 2024-10-30T07:45:18Z

src/main/java/com/listywave/notice/application/domain/NoticeType.java

+public enum NoticeType {
+
+    NEWS(1, "소식"),
+    EVENT(2, "이벤트"),
+    TIP(3, "팁"),
+    ;
+
+    private final int code;
+    private final String viewName;
+
+    public static NoticeType codeOf(int code) {
+        return Arrays.stream(NoticeType.values())
+                .filter(noticeType -> noticeType.getCode() == code)
+                .findFirst()
+                .orElseThrow(() -> new CustomException(NOT_EXIST_CODE, NOT_EXIST_CODE.getDetail() + " 입력값: " + code));


pparkjs · 2024-10-30T08:00:00Z

src/main/java/com/listywave/notice/application/converter/ContentTypeConverter.java

@@ -0,0 +1,20 @@
+package com.listywave.notice.application.converter;
+


개행 제거!

원래 package 아래는 개행 한 줄이 들어갑니다 😃 (출처)

pparkjs · 2024-10-30T08:19:00Z

src/main/java/com/listywave/notice/application/domain/Notice.java

+    public void addContents(List<NoticeContent> contents) {
+        this.contents.addAll(contents);
+    }


아직도 햇갈리는거 contents에 데잍터 바인딩 시켰으면 NoticeContent에서도 Notice 양방향으로 채워줘야하는 거 아니였어요? 이게 아직도 햇갈리네 ㅠ

연관관계의 주인이 NoticeContent이니까 NoticeContent에도 Notice를 참조하도록 하는 게 맞습니다!

Notice#addContents()를 호출하는 NoticeService#create()를 살펴보면,
List<NoticeContent>를 만들 때 객체 참조를 할당해주고 있습니다!

아아 DTO에서 해주는군요 ㅎㅎ 확인했슴다!

pparkjs · 2024-10-30T08:22:37Z

src/main/java/com/listywave/notice/presentation/NoticeController.java

+            @RequestBody NoticeCreateRequest request
+    ) {
+        Long id = noticeService.create(request);
+        return ResponseEntity.created(URI.create("/admin/notices/" + id)).build();


혹시 이게 그 RestFul API 설계 원칙에 HATEOAS 방식을 사용한 건가요? 응답에 uri 넣어주는

이는 HTTP - 201 Created 상태 코드의 명세에 따른 것입니다~!

pparkjs · 2024-10-30T08:29:44Z

src/main/java/com/listywave/notice/application/service/dto/NoticeFindResponse.java

+        BesideNoticeDto nextNotice
+) {
+
+    public static NoticeFindResponse of(Notice notice, Notice prevNotice, Notice nextNotice) {


평소에 하시던 것 처럼 prevNotice와 nextNotice는 nullable 하니 @nullable 어노테이션으로 명확하게 구분지어 놓는 것도 좋을 거 같아요!

빠트렸네요! 감사합니다 👍

pparkjs · 2024-10-30T08:45:50Z

src/main/java/com/listywave/notice/repository/NoticeRepository.java

+    @Query("""
+            select n
+            from Notice n
+            join NoticeContent nc on nc.notice = n
+            where n.id = :noticeId
+            """)
+    Notice findOne(Long noticeId);


요거 fetch join을 통해 사전에 매핑해가지고 와서 n+1 방지하는걸 추천합니다!

과거 커밋을 보신 것 같습니다!
fetch 적용했습니다 ㅎㅎ

pparkjs · 2024-10-30T08:55:06Z

src/main/java/com/listywave/notice/presentation/NoticeController.java

+    @PatchMapping("/admin/notices/{noticeId}")
+    ResponseEntity<Void> updateExposure(@PathVariable Long noticeId) {
+        noticeService.updateExposure(noticeId);
+        return ResponseEntity.noContent().build();


저번에 말했던 분리한 거 👍🏻

pparkjs · 2024-10-30T08:57:50Z

src/test/java/com/listywave/notice/application/service/NoticeServiceTest.java

+        NoticeCreateRequest request1 = new NoticeCreateRequest(
+                1,
+                "첫 번째 공지입니다",
+                "첫 번째 공지에요",
+                List.of(
+                        new ContentDto(1, "subtitle", "소제목입니다", null, null, null),
+                        new ContentDto(2, "body", "본문입니다", null, null, null),
+                        new ContentDto(3, "image", "이미지입니다", "https://image.com", null, null),
+                        new ContentDto(4, "button", "버튼입니다", null, "버튼 이름", "https://buttonLink.com"),
+                        new ContentDto(5, "note", "유의사항입니다", null, null, null)
+                )
+        );


이거 공지 통합테스트에서 공통으로 많이 쓰이는 거 같은데 번째를 인자로 받는 메서드로 하나 빼는 거 어떨까요!?

pparkjs

ㅎㅎ 반영하느라 수고하셨어요 approve 드릴게요!!

pparkjs · 2024-10-30T09:22:47Z

src/test/java/com/listywave/notice/application/service/NoticeServiceTest.java

+        NoticeCreateRequest request1 = createNoticeCreateRequest(1);
+        NoticeCreateRequest request2 = createNoticeCreateRequest(2);
+        NoticeCreateRequest request3 = createNoticeCreateRequest(3);


kdkdhoho added 8 commits October 29, 2024 14:58

test: 전체 테스트가 실패하는 문제 해결

4e3af74

정확한 원인은 파악하지 못했지만, 추측컨데 AcceptanceTest와 IntegrationTest의 테스트 DB가 공유되면서 외래키 제약 조건으로 발생한 것으로 예상 (#312)

test: 요청 주제(토픽) 초기 생성 시 isExposed를 항상 true로 변경함에 따른 테스트 코드 수정 (#312)

8d5ece2

feat: 공지 카테고리(타입) 조회 API 구현 (#312)

4c39059

feat: Notice 도메인 객체 생성 (#312)

58e606e

feat: 공지 생성 API 구현 (#312)

d59695a

feat: 사용자용, 관리자용 공지 전체 조회 및 공지 상세 조회 API 구현 (#312)

3de2eed

feat: 공지 수정 API 구현 (#312)

2063ce6

feat: 공지 삭제 API 구현 (#312)

1c6b03e

kdkdhoho added the 기능 label Oct 30, 2024

kdkdhoho self-assigned this Oct 30, 2024

kdkdhoho requested a review from pparkjs as a code owner October 30, 2024 07:38

kdkdhoho linked an issue Oct 30, 2024 that may be closed by this pull request

게시글(공지) 관련 API 구현 #312

Closed

9 tasks

fix: 관리자 존재 체크 로직 제거 (#312)

8989a84

pparkjs reviewed Oct 30, 2024

View reviewed changes

kdkdhoho added 2 commits October 30, 2024 18:12

refactor: 빠진 @nullable 추가 (#312)

6a7d204

test: NoticeCreateRequest 생성 부분 리팩터링 (#312)

dffe121

pparkjs approved these changes Oct 30, 2024

View reviewed changes

kdkdhoho merged commit 29fdecd into dev Oct 30, 2024
1 check passed

kdkdhoho deleted the feat/312 branch October 30, 2024 09:23

kdkdhoho mentioned this pull request Nov 1, 2024

공지 이미지 업로드 API 구현 #325

Closed

1 task

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: 공지 CRUD API 구현 #323

feat: 공지 CRUD API 구현 #323

kdkdhoho commented Oct 30, 2024

pparkjs left a comment

pparkjs Oct 30, 2024

kdkdhoho Oct 30, 2024

pparkjs Oct 30, 2024

pparkjs Oct 30, 2024

kdkdhoho Oct 30, 2024 •

edited

Loading

pparkjs Oct 30, 2024

kdkdhoho Oct 30, 2024

pparkjs Oct 30, 2024

pparkjs Oct 30, 2024

kdkdhoho Oct 30, 2024 •

edited

Loading

pparkjs Oct 30, 2024

kdkdhoho Oct 30, 2024

pparkjs Oct 30, 2024

kdkdhoho Oct 30, 2024

pparkjs Oct 30, 2024

pparkjs Oct 30, 2024

pparkjs left a comment

pparkjs Oct 30, 2024

		@@ -0,0 +1,20 @@
		package com.listywave.notice.application.converter;

feat: 공지 CRUD API 구현 #323

feat: 공지 CRUD API 구현 #323

Conversation

kdkdhoho commented Oct 30, 2024

Description

Relation Issues

pparkjs 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

kdkdhoho Oct 30, 2024 • 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

kdkdhoho Oct 30, 2024 • 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

pparkjs left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

kdkdhoho Oct 30, 2024 •

edited

Loading

kdkdhoho Oct 30, 2024 •

edited

Loading