프로그래밍을 하다보면 인스턴스가 이미 가지고 있는 특정 데이터를 활용해 인스턴스의 상태를 변경하는 경우가 종종 있습니다. 인스턴스의 상태를 변경할 때 여러가지 방식으로 구현하는 것을 종종 확인할 수 있다. 예를 들어 다음과 같은 요구사항이 있다고 가정하자.

• 텍스트에 url이 포함되어 있는지 확인한다.
• url이 포함되어 있다면 url에 포함되어 있는 이미지를 찾아 썸네일, 제목, 내용을 추출해 온다.

일반적으로 사용자가 입력하는 데이터와 도메인 모델 클래스가 분리되어 있을 경우 명확하게 역할히 분리되는데 중복을 제거하기 위해 둘의 역할을 한 클래스에서 같이 사용하다보니 이 같은 상황이 종종 발생한다. 위 요구사항에 대한 구현코드는 대략적으로 다음과 같다.

public class SmallTalk {
    private String talk;


    private SiteSummary siteSummary;


    [...]


    public String getTalk() {
        return talk;
    }


    public void setTalk(String talk) {
        this.talk = talk;
    }


    public String getTime() {
        if (TimeUtils.diffDay(new Date(), this.createdDate) == 0) {
            return TimeUtils.agoTime(this.createdDate);
        }
        return "아주 오래전...";
    }


    public SiteSummary getSiteSummary() {
        return siteSummary;
    }


    public void setSiteSummary(SiteSummary siteSummary) {
        this.siteSummary = siteSummary;
    }
}

위와 같이 setter와 getter를 가지는 클래스를 생성한다. 위 요구사항을 만족하려면 talk 데이터를 활용해 SiteSummary를 생성하기 위해 필요한 데이터를 생성해야 한다. SiteSummary는 url에 대한 썸네일 정보, 제목, 내용을 포함하는 클래스이다.

public class SiteSummary implements Serializable {


    private static final long serialVersionUID = -7183093514202968034L;


    private String title;
    private String contents;
    private String thumbnailImage;
    private String targetUrl;


    public SiteSummary(String title, String contents, String thumbnailImage, String targetUrl) {
        this.title = title;
        this.contents = contents;
        this.thumbnailImage = thumbnailImage;
        this.setTargetUrl(targetUrl);
    }


    public SiteSummary(Document doc, String targetUrl) {
        this.title = doc.title();
        this.thumbnailImage = SiteImage.getImage(targetUrl, doc);
        this.contents = SiteContents.getContents(doc);
        this.setTargetUrl(targetUrl);
    }


    [...]
}

이와 같이 한 인스턴스의 특정 데이터를 활용해 이 인스턴스의 상태를 변경해야 되는 요구사항은 자주 발생한다. 이 요구사항은 다음과 같이 구현할 수 있다.

public class SmallTalkService { 
    public List<SmallTalk> getLastTalks() {
        Page<SmallTalk> page = smallTalkRepository.findAll(getPager());
        List<SmallTalk> orgSmallTalks = page.getContent();
        
        List<SmallTalk> smallTalks = Lists.newArrayList();
        for (SmallTalk smallTalk : orgSmallTalks) {
            smallTalk.setSiteSummary(summaryService.findOneThumbnail(smallTalk.getUrlInTalk()));
            smallTalks.add(smallTalk);
        }
        return smallTalks;
    }
}

public class SummaryService {
    public SiteSummary findOneThumbnail(String url) {
        [...]
    }
}

위 소스 코드를 보면 SmallTalk에서 url을 추출해서 summaryService에 역할을 위임해 SiteSummary를 생성한 SmallTalk 클래스의 setSiteSummary() 메소드를 활용해 인스턴스의 상태를 변경하고 있다.

내가 만나본 대부분의 개발자는 위와 같이 구현하는 경우가 일반적이다. 하지만 뭔가 찜찜하다. 위 방식 말고 다른 방식으로 개선한다면 어떻게 개선할 수 있을까?

PS. 이 글에 대한 요구사항은 메인 화면의 수다양 구현 중 발생한 소스 코드의 일부분이다. 요구사항은 수다양 기능을 통해 확인할 수 있다.