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

 

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

  

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