인프런 영문 브랜드 로고
인프런 영문 브랜드 로고

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

terry9611님의 프로필 이미지
terry9611

작성한 질문수

실전! 스프링 부트와 JPA 활용1 - 웹 애플리케이션 개발

CQS에 대해 질문 드립니다.

작성

·

206

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안도 뭔가 마음에 드는 것 같진 않아요...

  

더 좋은 방법이 어떤게 있을까요,,?

 

 

 

 

 

 

 

답변 1

0

안녕하세요. terry9611님, 공식 서포터즈 David입니다.

1안 정도면 잘 구현하신듯합니다.

사실 1안 컨트롤러 쪽에서는 비즈니스 로직이라고 할 게 없습니다.

감사합니다.

terry9611님의 프로필 이미지
terry9611

작성한 질문수

질문하기