[Chapter05] 실무에서 작성하는 코드에서는 이미 책임중심 설계가 이뤄지고 있는 걸까요?

[JisuPark-dev]

spring프로젝트의 경우 대부분 controller, service, repository로 계층화된 코드를 작성하는게 원칙인데 만약 이에 따를 경우 책임이 어느정도 분리된 코드라고 봐야할 까요? 아니면 좀 더 책임이 분리된 코드를 만들려면 어떻게 해야하는 걸까요? 최근 작성한 코드를 리팩토링해보면서 예시를 들어봐도 좋을 것 같습니다.

[pythonstrup]

• 표현 계층, 로직 작성 계층, 데이터 접근 계층이라는 책임이 분리된 거라고 볼 수 있겠네요. 하지만 이것만으로 책임 중심 설계라고 말할 순 없을 것 같아요.
• 거의 대부분의 책임과 역할은 비즈니스 로직에서 나온다고 생각해요. 도메인이 중요한 이유도 그것 때문이고요.
  • 표현 계층과 데이터 접근 계층은 거의 DTO로 데이터를 전달하고 전달받는 정도의 일만 수행!
• Service 코드는 객체들이 서로 메시지를 교환할 수 있는 판을 깔아주는 정도라고 생각합니다!

책임이 분리된 코드

• 책임을 분리하려면 결국 객체를 나눠야 합니다.
• 객체에 너무 책임이 많다고 느낀다면 새로운 도메인 객체를 만들어 책임을 나눠주면 됩니다!
• 서비스의 책임을 도메인 객체로 나누지 않고, 또 다른 서비스 객체에게 나누는 건 절차지향적인 코드를 그냥 반으로 똑 잘라 놓은 거라고 생각해요! (물론 서비스를 분리함으로써 코드 가독성이 좋아지는 등의 이점은 있겠지만요!)

1급 컬렉션

• 저는 최근에 컬렉션 때문에 서비스 계층의 코드가 상당히 지저분해진다는 생각이 들었어요.
  • 컬렉션을 위한 모든 로직을 서비스에서 구현하거나 단일 도메인 객체에 static 메소드를 두는 식으로 구현했었는데요. 이 방식이 적절한 책임 분리 방식은 아니라고 생각했어요.
• 저는 이번에 서포터즈 기능을 만들면서 1급 컬렉션 객체를 사용해 컬렉션 로직을 객체 위임해봤어요.
• 코드는 내일 보여드릴게요!

[JisuPark-dev]

public class ReservationService{
  public void requestReservation(HospitalJobDto hospitalJobDto) {
          private final HospitalService hospitalService;
          private final ConsultGroupSerivce consultGroupSerivce;
          private final ConsultService consultService;
          private final FavoriteService favoriteService;

          // 병원 주소록 조회 및 생성 or 업데이트
          hospitalService.hospitalPatientGenerator(hospitalJobDto);

          // 진료 그룹 INSERT
          consultGroupSerivce.insertConsultGroup(hospitalJobDto);

          // 진료 INSERT
          consultService.insertConsult(hospitalJobDto);

          // 병원 즐겨찾기 추가
          favoriteService.addFavoriteHospital(hospitalJobDto);
      }
}

이런 코드는 그럼 서비스에 서비스를 붙인 것 뿐일까요. 만약 여기서 정말로 책임 지향으로 가고 싶다면, 각각의 도메인 객체가 책임을 가져가는게 맞는걸까요.

가령

public class ReservationService {
    private Hospital hospital;
    private ConsultGroup consultGroup;
    private Consult consult;
    private Favorite favorite;

    @Override
    public void requestReservation(HospitalJobDto hospitalJobDto) {

        // 병원 주소록 조회 및 생성 or 업데이트
        hospital.create(hospitalJobDto);

        // 진료 그룹 INSERT
        consultGroup.insert(hospitalJobDto);

        // 진료 INSERT
        consult.insert(hospitalJobDto);

        // 병원 즐겨찾기 추가
        favorite.add(hospitalJobDto);
    }
}

이런식으로 말이죠