Feature/create letter(wr9-22) #29

Kojinyoung7220 · 2025-02-19T03:33:12Z

Pull Request Template

📝 PR 설명

Letter 엔티티 설계, enum 값 설정, LetterController , LetterService 로직 구현
CreateLetterRequest, LetterResponse DTO 구현,
LettersControllerTest, LetterServiceTest 테스트 구현
local 환경, test 환경 각각 mysql, h2로 db 분리

🔍 주요 변경사항

application.yml 파일 on-profile: test 부분 제거
application-test.yml 추가 후 인메모리 h2 db 세팅, 테스트 코드에 어노테이션 추가

📸 스크린샷 (선택사항)

UI 변경사항이 있다면 스크린샷을 첨부해주세요.

🧪 테스트

단위 테스트 추가/수정
테스트 커버리지 확인
소나클라우드 품질 게이트 통과

✅ 체크리스트

코드 컨벤션을 준수했나요?
불필요한 코드나 주석이 없나요?
브랜치 네이밍 컨벤션을 따랐나요?
관련 문서를 업데이트했나요?

🤝 관련 이슈

closes #이슈번호

yhd1101 · 2025-02-19T04:55:09Z

src/main/java/io/crops/warmletter/domain/letter/controller/LetterController.java

별거는 아니지만.. 메서드명을 단수로 바꿔보시는건 어떨까요??

넵 고치겠습니다

yhd1101 · 2025-02-19T04:55:24Z

src/main/java/io/crops/warmletter/domain/letter/entity/LetterMatching.java

BaseTimeEntity가 있는데 상속을 안 받은 이유가 있으실까요?
createdAt이랑 updatedAt 필드가 중복 선언된 것 같아서 보강 차 질문드립니다!

아 LetterMatching 이랑 LetterMatchingTimeout은 뼈대만 잡고 아직 구현은 안했습니다

yhd1101 · 2025-02-19T05:02:16Z

src/main/java/io/crops/warmletter/domain/letter/service/LetterService.java

메서드명 write()를 createLetter()와 같이 통일하는 건 어떨까요?

현재 Letter.builder() 호출이 if-else로 나뉘어 있어서 공통 부분도 중복으로 작성된 것 같아요.

공통되는 필드 설정은 먼저 한 번만 작성하고,
달라지는 부분만 if-else로 처리하면 어떨까 하는게 제생각입니다..

더 깔끔해질거 같습니다 고치겠습니다~

leeys9423 · 2025-02-19T05:06:26Z

src/main/java/io/crops/warmletter/domain/letter/controller/LetterControllerDocs.java

+public interface LetterControllerDocs {
+}


이 인터페이스는 무슨 역할을 하는 파일일까요?!

컨트롤러에 Swagger 어노테이션이 여기저기 붙어 있는 것을 피하기 위해서 일단 만들어 뒀습니다

leeys9423 · 2025-02-19T05:08:08Z

src/main/java/io/crops/warmletter/domain/letter/dto/response/LetterResponse.java

+ * }
+ */
+@Getter
+public class LetterResponse {


편지를 보낸 후의 Response라면 좀 더 객체명을 명확하게 표현하면 좋을 것 같아요!(ex. CreateLetterResponse)

편지 조회 생성 수정 등에서 공통으로 사용하실 것이라면 냅둬도 상관 없을 것 같습니다!

편지를 보낸 후의 Response라서 객체명 변경하겠습니다~

leeys9423 · 2025-02-19T05:12:10Z

src/main/java/io/crops/warmletter/domain/letter/dto/response/LetterResponse.java

+    @Builder
+    public LetterResponse(Long letterId, Long writerId, Long receiverId, Long parentLetterId, String title, String content, Category category, PaperType paperType, FontType font, DeliveryStatus deliveryStatus, LocalDateTime deliveryStartedAt, LocalDateTime deliveryCompletedAt) {
+        this.letterId = letterId;
+        this.writerId = writerId;
+        this.receiverId = receiverId;
+        this.parentLetterId = parentLetterId;
+        this.title = title;
+        this.content = content;
+        this.category = category;
+        this.paperType = paperType;
+        this.font = font;
+        this.deliveryStatus = deliveryStatus;
+        this.deliveryStartedAt = deliveryStartedAt;
+        this.deliveryCompletedAt = deliveryCompletedAt;
+    }


모든필드를 기준으로 생성자를 만들어서 빌더패턴을 적용시키려고 하고 있군요!

이럴 경우에는 클래스 자체에 @builder를 적용시키면 이 부분은 생략되어도 될 것 같아요.

다만 여기서 생각해봐야할 것은 아마 생성자의 역할은 편지 엔티티를 Response로 변환해주기 위해 사용하는 느낌으로 만드신것 같아요!(아직 전체적인 코드를 아직 보진 않았습니다)

저의 예상이 맞다면 정적 팩토리 메서드 패턴 public static LetterResponse from(Letter letter) { ... } 이런식으로 만들 것 같아요(from이라는 명칭은 일반적으로 많이 사용되고 있는 메서드 명이라서 저렇게 적어봤습니다.)

아 습관처럼 만들어 버렸네요 고치겠습니다~

leeys9423 · 2025-02-19T05:14:34Z

src/main/java/io/crops/warmletter/domain/letter/dto/response/LetterResponse.java

+    public static LetterResponse fromEntity(Letter letter) {
+        return LetterResponse.builder()
+                .letterId(letter.getId())
+                .writerId(letter.getWriterId())
+                .receiverId(letter.getReceiverId())
+                .parentLetterId(letter.getParentLetterId())
+                .title(letter.getTitle())
+                .content(letter.getContent())
+                .category(letter.getCategory())
+                .paperType(letter.getPaperType())
+                .font(letter.getFontType())
+                .deliveryStatus(letter.getDeliveryStatus())
+                .deliveryStartedAt(letter.getDeliveryStartedAt())
+                .deliveryCompletedAt(letter.getDeliveryCompletedAt())
+                .build();
+    }


아 밑에 이렇게 적어주셨구나.... 이미 잘 사용하고 계셨군요...
그럼 생성자 부분은 지우고 차라리 클래스자체에 @builder 라는 어노테이션을 적용시키면 생성자 부분을 아예 삭제 시킬 수 있습니다~

넵 확인했습니다

leeys9423 · 2025-02-19T05:15:43Z

src/main/java/io/crops/warmletter/domain/letter/entity/LetterMatching.java

+@Getter
+@NoArgsConstructor(access = AccessLevel.PUBLIC)
+@Table(name = "letter_matchings")
+public class LetterMatching {


BaseTimeEntity 누락 되어있네여

넵 확인했습니다 감사합니다

leeys9423 · 2025-02-19T05:16:59Z

src/main/java/io/crops/warmletter/domain/letter/entity/LetterMatchingTimeout.java

ERD 기준으로 아직 필드들이 다 설정 안되어있는 것 같네요!

추후에 해당 기능 만드실때 하려고 하신거겠죠??

넵 LetterMatching 이랑 LetterMatchingTimeout은 뼈대만 잡고 아직 구현은 안했습니다

# Conflicts: # src/main/java/io/crops/warmletter/global/config/SecurityConfig.java

Kojinyoung7220 added 18 commits February 18, 2025 12:41

feat: 초기 구조 설정 (WR9-22)

a2f5696

feat: dto, enum 추가 (WR9-22)

e97b9f8

feat: CreateLetterRequest 검증 추가 (WR9-22)

ca1b5bc

feat: LettersService 편지 생성 로직 (WR9-22)

93ce1c8

feat: LettersControllerTest 추가 (WR9-28)

426b675

refactor: LetterMatching, LetterMatchingTimeout 엔티티명 수정, 테이블명 수정

edaf9d5

refactor: LettersMatching, LetterMatchingTimeouts 삭제 (WR9-29)

1e59a80

refactor: 클래스명 수정 (WR9-22)

47e80fb

remove: 변경 전 자바 코드 제거 (WR9-22)

2070330

refactor: LetterService 로직 변경 (WR9-30)

8bf2446

test: content 필수값 검증 테스트 추가 (WR9-28)

49f401a

feat: LetterResponse dto 생성, 임시 저장 (WR9-32)

50ae491

fix: csrf 토큰 문제 active 변경으로 해결, local과 test 환경 db 분리 작업 (WR9-27)

615e8f3

feat: 서비스단 로직 변경, dto에 팩토리 메서드 추가, 공통응답 return 추가 (WR9-30)

d36bb79

test: 편지 작성 로직에 대한 서비스 테스트 추가, 컨트롤러 테스트 검증 추가 (WR9-28)

9c778a6

feat: letter 엔티티 db 테이블명 변경 (WR9-29)

5d86ce4

refactor: enum 값 변경, CreateLetterRequest에 NoArgsConstructor 추가 (WR9-29)

bccf771

test: test 코드 enum 값 변경 (WR9-28)

b690ed8

Kojinyoung7220 added ⚙ Setting 개발 환경 세팅 ✨ Feature 기능 개발 ✅ Test test 관련(storybook, jest...) labels Feb 19, 2025

Kojinyoung7220 self-assigned this Feb 19, 2025

Kojinyoung7220 linked an issue Feb 19, 2025 that may be closed by this pull request

[WR9-22] 편지 생성 #20

Closed

10 tasks

Kojinyoung7220 requested a review from yhd1101 February 19, 2025 03:33

Merge branch 'dev' into feature/create-letter(WR9-22)

e779992

yhd1101 reviewed Feb 19, 2025

View reviewed changes

leeys9423 reviewed Feb 19, 2025

View reviewed changes

fix: banned_word -> bad_word 문구 수정

9bc6d50

Kojinyoung7220 changed the title ~~Feature/create letter(wr9 22)~~ Feature/create letter(wr9-22) Feb 19, 2025

refactor: pr후 수정사항 변경

4020abc

Kojinyoung7220 added 2 commits February 19, 2025 15:07

refactor: CreateLetterRequest AccessLevel 수정, 변수 복수형 수정 (WR9-22)

9d38f60

Merge branch 'dev' into feature/create-letter(WR9-22)

fdc2ee9

# Conflicts: # src/main/java/io/crops/warmletter/global/config/SecurityConfig.java

Kojinyoung7220 merged commit 7cf999a into dev Feb 19, 2025
1 check failed

Kojinyoung7220 deleted the feature/create-letter(WR9-22) branch February 19, 2025 06:31

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/create letter(wr9-22) #29

Feature/create letter(wr9-22) #29

Kojinyoung7220 commented Feb 19, 2025 •

edited

Loading

yhd1101 Feb 19, 2025

Kojinyoung7220 Feb 19, 2025

yhd1101 Feb 19, 2025

Kojinyoung7220 Feb 19, 2025

yhd1101 Feb 19, 2025

Kojinyoung7220 Feb 19, 2025

leeys9423 Feb 19, 2025

Kojinyoung7220 Feb 19, 2025

leeys9423 Feb 19, 2025

Kojinyoung7220 Feb 19, 2025

leeys9423 Feb 19, 2025 •

edited

Loading

Kojinyoung7220 Feb 19, 2025

leeys9423 Feb 19, 2025

Kojinyoung7220 Feb 19, 2025

leeys9423 Feb 19, 2025

Kojinyoung7220 Feb 19, 2025

leeys9423 Feb 19, 2025

Kojinyoung7220 Feb 19, 2025

Feature/create letter(wr9-22) #29

Feature/create letter(wr9-22) #29

Conversation

Kojinyoung7220 commented Feb 19, 2025 • edited Loading

Pull Request Template

📝 PR 설명

🔍 주요 변경사항

📸 스크린샷 (선택사항)

🧪 테스트

✅ 체크리스트

🤝 관련 이슈

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

leeys9423 Feb 19, 2025 • 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

Choose a reason for hiding this comment

Kojinyoung7220 commented Feb 19, 2025 •

edited

Loading

leeys9423 Feb 19, 2025 •

edited

Loading