Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DTO에 중복되는 생성자를 제거한다. #424

Open
2 tasks
gusah009 opened this issue Sep 20, 2022 · 0 comments
Open
2 tasks

DTO에 중복되는 생성자를 제거한다. #424

gusah009 opened this issue Sep 20, 2022 · 0 comments
Labels
📑 chore 오타 수정, 빌드 업무 수정, 패키지 매니저 수정 등 잡다한 수정 및 추가사항 👀 good first issue Good for newcomers

Comments

@gusah009
Copy link
Member

사전 수행 issue

없음.

이슈 내용

@Getter
@Setter
@NoArgsConstructor // Json을 위해 필요하지만 중복!
@AllArgsConstructor // Json을 위해 필요하지만 중복!
@Builder
@JsonInclude(Include.NON_NULL)
public class MemberDto {

위와 같이 저희 프로젝트에선 대부분 DTO에 빌더와 함께 NoArgsCon~, AllArgsCon~ 생성자를 사용하고 있습니다.

빌더는 많은 인자를 받을 때 훌륭한 가독성과 여러 문제를 해결하면서 객체를 생성해줍니다.

빌더가 하나의 생성자 역할을 하기 때문에 @NoArgs~@AllArgs~ 어노테이션은 필요 없을 것 같은데, Jackson 라이브러리에서 Json을 생성하려면 꼭 위의 두 어노테이션이 필요합니다.

하지만 여러 생성자를 열어놓는 것은 바람직하지 않기 때문에, 빌더를 사용하는 DTO에선 빌더만 열어놓으려고 합니다.

물론, 빌더를 사용하지 않는 코드는 건드릴 필요가 없습니다.

이에 이 사이트를 참고해서 생성자를 하나만 열어놓는 방식을 사용하도록 모든 DTO를 수정하고자 합니다.

  • 해당 블로그를 참고하여 팀원들과 어떤 방식을 사용하면 좋을 지 논의

    • 논의 후 해당 블로그를 참고하여 생성자를 하나만 열어놓는 방식을 사용하도록 모든 DTO를 수정
@gusah009 gusah009 added 📑 chore 오타 수정, 빌드 업무 수정, 패키지 매니저 수정 등 잡다한 수정 및 추가사항 👀 good first issue Good for newcomers labels Sep 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📑 chore 오타 수정, 빌드 업무 수정, 패키지 매니저 수정 등 잡다한 수정 및 추가사항 👀 good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant