인프런 커뮤니티 질문&답변

안녕하세요 호돌맨님. 강의 잘 보고 있습니다!

저도 처음에 볼 때 이해하기 힘들어서 수강생분들의 질문과 호돌맨님의 답변을 보며

해당 패턴을 사용하는 목적과 동작 방식을 이해할 수 있었습니다.

더 나아가서, '혹시 다른 방법은 없을까?' 고민하면서 만들어 봤습니다...

업무때문에 바쁘신 줄은 알지만, 나중에라도 보게 되신다면 피드백 부탁드리고 싶습니다ㅎㅎ

@Getter
@Entity
public class Post {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Long id;

    private String title;

    @Lob
    private String content;

    @Builder
    public Post(String title, String content) {
        this.title = title;
        this.content = content;
    }

    /**
     * 첫번째 방법
     *   - 문제1
     *     .title(postEdit.getTitle() != null ? postEdit.getTitle() : title)
     *     에서
     *     'postEdit.getTitle() != null' 을 어떤 개발자가 'postEdit.getContent() != null' 로 몰래 바꾼다면?
     *
     *   - 문제2
     *     가독성이 안 좋아서, 문제1의 경우에 원인을 빠르게 찾기도 힘듬
     */
    public void edit1(PostEdit postEdit) {
        PostEdit fixedPostEdit = PostEdit.builder()
                .title(postEdit.getTitle() != null ? postEdit.getTitle() : title)
                .content(postEdit.getContent() != null ? postEdit.getContent() : content)
                .build();

        title = fixedPostEdit.getTitle();
        content = fixedPostEdit.getContent();
    }

    /**
     * 두번째 방법
     *   - PostEditor의 방식처럼 원본 데이터(Post)의 값을 먼저 넣고,
     *     변경 데이터(PostEdit)의 값으로 덮어쓰는 방식
     *
     *     .title(title) 또는
     *     .title(postEdit.getTitle())
     *     처럼 키워드가(title, content, ...)가 바로 옆에 붙어있어서
     *     첫번째 방법보다 알아보기 쉬움
     *     -> 에러 발생 시, 문제 원인을 찾기 쉬움
     */
    public void edit2(PostEdit postEdit) {
        PostEdit.PostEditBuilder editBuilder = PostEdit.builder()
                .title(title)
                .content(content);

        PostEdit fixedPostEdit = editBuilder
                .title(postEdit.getTitle())
                .content(postEdit.getContent())
                .build();

        title = fixedPostEdit.getTitle();
        content = fixedPostEdit.getContent();
    }

}