Skip to content
This repository was archived by the owner on Jan 11, 2026. It is now read-only.

[REFACTOR] 스터디 DTO 코드 정리#18

Merged
dvlp-sy merged 8 commits intodevelopfrom
SPOT-288/refactor
Jun 5, 2025
Merged

[REFACTOR] 스터디 DTO 코드 정리#18
dvlp-sy merged 8 commits intodevelopfrom
SPOT-288/refactor

Conversation

@dvlp-sy
Copy link
Collaborator

@dvlp-sy dvlp-sy commented Jun 5, 2025

#️⃣ 연관된 이슈

ex) #이슈 번호, #이슈 링크


🔎 작업 내용

기능에서 어떤 부분이 구현되었는지 설명해주세요.

  • 스터디 Request DTO 정리 -> StudyMemberRequestDTO, BaseStudySearchRequestDTO, StudySearchRequestDTO, StudySearchRequestWithThemeDTO (Search 관련된 부분은 따로 건들지 않았습니다~!)
  • 스터디 Response DTO 정리 -> StudyResponseDTO, StudyMemberResponseDTO, StudyImageResponseDTO, SearchResponseDTO

📷 스크린샷 (선택)

작업한 결과물에 대한 간단한 스크린샷을 올려주세요.


💬리뷰 요구사항 (선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.

@dvlp-sy dvlp-sy requested a review from msk226 June 5, 2025 02:37
@dvlp-sy dvlp-sy self-assigned this Jun 5, 2025
@dvlp-sy dvlp-sy added the ♻️ refactor Code Refactoring label Jun 5, 2025
@dvlp-sy dvlp-sy linked an issue Jun 5, 2025 that may be closed by this pull request
Copy link
Member

@msk226 msk226 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

항상 고생 많으시네요! 저도 종강하고 빠르게 다시 진행할게요!

수고하셨습니다~!

Comment on lines +121 to +133
@Getter
@RequiredArgsConstructor(access = AccessLevel.PRIVATE)
@Builder(access = AccessLevel.PRIVATE)
private static class PostingMemberDTO {

private final Long memberId;
private final String name;
private final String profileImage;

public static PostingMemberDTO toDTO(Member member) {
return PostingMemberDTO.builder()
.memberId(member.getId())
.name(member.getName())
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

생성자 Private 해놓고 정적 팩토리 메서드 쓰는거 좋네요!

@dvlp-sy dvlp-sy merged commit 4cc685f into develop Jun 5, 2025
1 check passed
@dvlp-sy dvlp-sy deleted the SPOT-288/refactor branch June 5, 2025 13:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

♻️ refactor Code Refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 스터디 DTO 코드 정리

2 participants