이런 경우에 클래스 분리를 어떻게 해야 하나요?

Question

숙박업체를 등록하는 메소드가 있습니다.해당 메소드는 숙박업체(house)와 객실(room) 그리고 옵션(option), 숙박업체 이미지(house_image), 객실 이미지(room_image) 등 여러 개의 테이블에 create 작업을 수행해야 합니다.이때 /api/houses URL을 통해 POST 요청이 들어왔을 때Conrtoller에서는 다음과 같이 houseService를 호출하도록 했습니다.houseService.createHouse(houseReq, houseImages, roomImages);이때 하나의 createHouse에는 아래 작업들이 하나의 메소드 안에서 수행됩니다.House savedHouse &#61; houseRepository.save(toSaveHouse); houseOptionRepository.saveAll(toSaveOptions); houseAddressRepository.save(toSaveAddress); houseImageRepository.saveAll(toSaveHouseImages); Room savedRoom &#61; roomRepository.save(toSaveRoom); roomImageRepository.saveAll(toSaveRoomImages); 이때 너무 하나의 메소드에 많은 작업이 몰려있는 것 같아 객체단위로 분류해줄까 싶었습니다.그러나 문제점은 Controller에서 여러 service를 호출함에 있어서, Controller에서 반환받는 값을 Entity로 받게 되는 문제점입니다.위 부분을 어떻게 처리하는 것이 좋을지 궁금합니다.Room, RoomImage, Address를 저장하는 기능은 각각 RoomService, AddressService 등으로 옮겨놓은 후, HouseService에서 RoomService, AddressService를 호출해서 저장하는 방법Controller에서 각각의 Service를 호출하는 대신 Entity로 받지 않고 DTO로 받는 방법 (이럴 경우 Controller -> Service로 넘기면서 DTO-> Entity 변환을 수행한 다음 다시 Service -> Controller 과정에서 Entity -> DTO를 수행해야 하기 때문에 너무 비효율적이라 생각이 들어서 패스하려 합니다.)하나의 HouseService에 여러 메소드로 구분하기 (1번과 차이점은 1번은 각각의 기능이 각각의 Service들에게 선언되어 있지만, 3번은 HouseService 내에 메소드 단위로 선언되는 것에 차이가 있음)일단 생각한 방법은 위 3가지입니다.혹여나 제가 생각한 방법 말고도 좋은 방법이 있다면 알려주시면 감사하겠습니다.

메이슨 리 · Answer

1) 업무적으로 데이터의 일관성을 보장하기 위해서는 트랜잭션 경계 설정을 해야 할 텐데, 스프링은 보통 서비스(Service)에서 설정 하는거죠? 이 부분은 항상 고려하시는거죠? 에러가 났는데 이건 들어가고 저건 안들어 가면 아주 골치아프겠죠?2) 지금 createHouse 메서드가 나쁜가요? 전 모르겠네요. 만약 독립적으로 Room, RoomImage, Address...처리해야 하는 업무가 생긴다면 필요한 그때 해당 서비스(Service)를 만드시면 되지 않을까요?*전 SI개발이 주 업이라 보통 JPA로 개발하지는 않아서 JPA 뭐가 다른지는 정확히 모르겠습니다만 제가 했다면 뭐 그냥 저렇게 할꺼 같아요. 그리고 단독으로 Room, RoomImage, Address ...등을 CRUD 할 일이 있다면 그 때 서비스 만들어서 사용하고, 없으면 그냥 안만들꺼 같습니다.

kiel · Answer

하나의 메쏘드에 너무 많은 작업이 몰려있느냐 를 검토 X생성하는 작업에서 같이 묶여서 생성되어야 하는 데이터인가 검토 O저장하는 entity 개수가 중요한게 아니라 같이 묶여야 하는 작업이냐를 검토해보세요.

아직도개발중 · Answer

일단 트랜잭션 처리를 위해 하나의 서비스에서 가져가는게 맞을 것 같구요.controller -> main service -> sub service (room, roomimage, address)재사용성 생각 하시는 거면 좀 더 서비스를 분리하셔도 나쁘진 않습니다.예를 들어 주소만 수정하거나 추가하거나 이런다고 하면 주소 등록, 수정, 추가, 삭제가 있는 서비스를 별도로 분리해서 관리하고 그 서비스를 main service에서 create house 하실 때 사용하시는거구요.그런게 아닐 경우 지금 그대로 사용하셔도 나쁘지 않습니다.

이런 경우에 클래스 분리를 어떻게 해야 하나요?

3개의 답변

Q&A 목록