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

    public void writeIncome(IncomeWriteDto incomeWriteDto) {
        // Repository에서 Entity를 찾고, Domain으로 변환한다.
        Member findMember = memberRepository.findById(incomeWriteDto.getMid()).orElseThrow(
                () -> new CustomApiException("사용자가 없습니다.")
        );
        MemberDomain memberDomain = new MemberDomain(findMember);

        // 잔여금을 계산한다.
        memberDomain.calculateRest(memberDomain.getRest() + incomeWriteDto.getIncomeMoney());


        // 년, 월을 계산한다.
        String month = incomeWriteDto.getMonth();
        String year = month.substring(0,4);

        Income income = Income.builder()
                            .incomeMoney(incomeWriteDto.getIncomeMoney())
                            .incomeReason(incomeWriteDto.getIncomeReason())
                            .year(year)
                            .month(month)
                            .member(findMember)
                            .build();

        incomeRepository.save(income);
    }

위 코드의 경우
영속성 객체와 도메인 객체를 분리했고,
도메인 객체 내부에 매서드를 만들어
비즈니스 로직을 구현했습니다.

이 때, calculateRest() 매서드 후
영속성 객체의 값이 변경되어야 하는데,

어떻게 영속성 객체의 값을 변경할 지
잘 모르겠습니다.

findMember.setRest()를 사용하자니
DDD에 어긋나는 것 같고,

아니면 calculateRest() 파라미터에
findMember를 전달해주고
이 내부에서 처리를 해야하는 걸까요?

AI에 코드와 관련된 질문 주셨기에 미약하게나마 아래와 같이 피드백 드립니다. 저라면 아래와 같이 변경할 듯합니다.

public void writeIncome(IncomeWriteDto incomeWriteDto) {
    MemberDomain memberDomain = memberRepository.getById(incomeWriteDto.getMid()).
    memberDomain.addRest(incomeWriteDto.getIncomeMoney());
    memberRepository.update(memberDomain);
    Income income = Income.builder()
        .incomeMoney(incomeWriteDto.getIncomeMoney())
        .incomeReason(incomeWriteDto.getIncomeReason())
        .year(incomeWriteDto.getYear())
        .month(incomeWriteDto.getMonth())
        .member(findMember)
        .build();
    incomeRepository.save(income);
}

member가 JPA엔티티라고 가정했습니다.

1. new MemberDomain(member) 처럼 작성한다면 MemberDomain 이 영속성 객체에 의존하게 됩니다. 생성자가 영속성 객체를 알게되니까요. 그러면 도메인 레이어는 영속성 객체에 의존하지 않는다는 설계의 목적을 달성할 수 없게 됩니다.

2. getById와 findById를 구분해서 사용하여 서비스 코드에 findById.orElseThrow가 계속 나오는 상황을 방지할 것 같습니다. 이에 대한 책임(orElseThrow)은 Repository로 옮길 것 같습니다.

3. memberDomain.calculateRest의 의도가 무엇인지 정확히 와닿지 않습니다. 추측하건데 이 메서드의 호출 결과로 도메인 내부 값이 변경되고 JPA의 변경 감지로 값이 변경되길 원하셨던 것 같습니다. 이런 의도라고 생각하고 그렇다면 관련해서 아래와 같은 답변을 드립니다.

   1. memberDomain.calculateRest(incomeWriteDto.getIncomeMoney())가 되면 될 것 같습니다. 자기 자신의 값을 외부에서 가져다 계산해서 넣어줄 이유가 없습니다. 내부적으로 주입받은 파라미터 값과 본인이 들고 있는 값을 더해 변경하도록 하면 됩니다.

   2. 영속성 객체와 도메인 모델을 분리하기로 결정한 순간 JPA의 변경 감지 기능은 더이상 못 쓰게 됩니다. (제가 아는 선에서 답변드리는 겁니다. 분명 더 나은 해법을 갖고 계신 분이 계실 수 있습니다.)

   3. 변경 감지로 코드를 짜는 코드가 좋은 코드인지 저는 잘 모르겠습니다. 그렇게 생각하는 이유는 의도가 안보이기 때문입니다. 저는 코드를 보면서 calculateRest의 의도가 무엇인지 알 수 없었습니다. 질문자님이 작성해 주신 코드의 의도를 읽은 후에야 ‘이 코드로 인해 member 모델의 값이 변경되길 원하는 거구나’라는 것을 알았습니다. 심지어 아직도 이게 의도하신 것이 맞는지 모릅니다. 그렇게 추측만 할 뿐입니다. 이러한 코드가 누적되면 서비스의 의도가 보이지 않게 된다고 생각합니다. 다른 사람이 짠 서비스 호출 결과 데이터 변경으로 인해 영속성 데이터의 값이 변경되지 않을지를 걱정하며 코드를 짜야 합니다. 그렇게 되면 결국 그러한 확신을 얻기 위해 하위 호출되는 메서드가 값을 변경하고 있지는 않은지 모두 까봐야 합니다. 영속성 데이터의 변경을 원하신다면 그 코드가 차라리 보이는 게 낫다고 생각합니다.

4. 년 월을 계산하는 책임은 incomeWriteDto에게 넘길 것 같습니다. IncomeWriteDto가 아래와 같은 메서드를 갖고 있으면 됩니다. (month라는 변수가 yyyy-mm 처럼 오는 것 같은데, 만약 그렇다면 변수 이름이 잘못된 듯 하네요)

   public String getYear() {
       return month.substring(0,4);
   }
   

답변이 도움되셨길 바랍니다. :)