작성
·
207
1
CQS 관점으로 상품 서비스를 조회 서비스와 명령 서비스를 구분했습니다.
근데 1안, 2안, 3안 중 어느것이 나은지 모르겠습니다.
또는 1~3안이 아니더라도 더 좋은 방법이 있으면 알려주실 수 있나요?
1안
📌 ItemCommandService
public Long save(Item item) {
if (itemJpaRepository.existsByInfoCode(item.getCode())) {
throw new IllegalStateException(ErrorCode.ITEM_DUPLICATE_CHECK_ERROR.getMessage());
}
return itemJpaRepository.save(item).getId();
}
📌 Item 컨트롤러
@PostMapping
public ResponseEntity<ItemResponse> register(
@Valid @RequestBody ItemRequest request
) {
Item item = request.toEntity();
Long itemId = itemCommandService.save(item);
Item findItem = itemFindService.findOne(itemId);
ItemResponse response = ItemResponse.toDto(findItem);
URI location = ServletUriComponentsBuilder.fromCurrentRequest()
.path("/{id}")
.buildAndExpand(response.getId())
.toUri();
return ResponseEntity.created(location)
.body(response);
}
1안은 저장할 때 반환 타입을 Long으로 하는데 대신 컨트롤러에서 저장도 하고 조회도 하니깐
뭔가 비즈니스 로직이 컨트롤러쪽에도 있는 듯한 느낌이 듭니다.
2안
📌 ItemCommandService
public Item save(Item item) {
if (itemJpaRepository.existsByInfoCode(item.getCode())) {
throw new IllegalStateException(ErrorCode.ITEM_DUPLICATE_CHECK_ERROR.getMessage());
}
return itemJpaRepository.save(item);
}
📌 Item 컨트롤러
@PostMapping
public ResponseEntity<ItemResponse> register(
@Valid @RequestBody ItemRequest request
) {
Item item = request.toEntity();
Item savedItem = itemCommandService.save(item);
ItemResponse response = ItemResponse.toDto(savedItem);
URI location = ServletUriComponentsBuilder.fromCurrentRequest()
.path("/{id}")
.buildAndExpand(response.getId())
.toUri();
return ResponseEntity.created(location)
.body(response);
}
2안은 상품 저장 메서드의 반환 값을 Item으로 하니깐 컨트롤러가 간단해보여서 좋은데
대신 저장하는 save() 메서드의 반환값이 Item이니깐 저장도 되고, 조회도 되는 것 같아서
CQS가 잘 안지켜지는 것 같습니다.
3안
📌 ItemCommandService
public Long save(Item item) {
if (itemJpaRepository.existsByInfoCode(item.getCode())) {
throw new IllegalStateException(ErrorCode.ITEM_DUPLICATE_CHECK_ERROR.getMessage());
}
return itemJpaRepository.save(item).getId();
}
public Item register(Item item) {
Long itemId = save(item);
return itemFindService.findOne(itemId);
}
📌 Item 컨트롤러
@PostMapping
public ResponseEntity<ItemResponse> register(
@Valid @RequestBody ItemRequest request
) {
Item item = request.toEntity();
Item savedItem = itemCommandService.register(item);
ItemResponse response = ItemResponse.toDto(savedItem);
URI location = ServletUriComponentsBuilder.fromCurrentRequest()
.path("/{id}")
.buildAndExpand(response.getId())
.toUri();
return ResponseEntity.created(location)
.body(response);
}
3안은 상품을 저장하는 save() 메서드의 반환 값을 Long 타입으로 하고
register() 메서드를 만들어서 저장 및 조회를 할 수 있게 했습니다.
근데 3안도 뭔가 마음에 드는 것 같진 않아요...
더 좋은 방법이 어떤게 있을까요,,?