HubInventoryAdapter.decreaseHubInventory()의 response.getData() NullPointerException 방어 처리

[coderabbitai]

개요

PR #9 코드 리뷰 중 발견된 잠재적 NullPointerException 문제를 후속 처리합니다.

관련 링크

• PR: 4 feature createorder #9
• 리뷰 코멘트: 4 feature createorder #9 (comment)
• 요청자: @hil679

문제 설명

HubInventoryAdapter.java의 decreaseHubInventory() 메서드에서 hubInventoryFeignClient.decreaseHubInventory() 호출 후 response.getData().items().stream()을 바로 호출하고 있습니다.

• response.getData()가 null일 경우 NPE 발생 가능
• response.getData().items()가 null일 경우 NPE 발생 가능

수정 방향

response.getData() 및 response.getData().items()에 대한 null 체크를 추가하거나, Optional을 활용하여 방어 처리 후 의미 있는 예외를 던지도록 개선합니다.

CommonResponse<HubInventoryResponseDto.Decrease> response = hubInventoryFeignClient.decreaseHubInventory(hubInventoryDto);

if (response.getData() == null || response.getData().items() == null) {
    throw new IllegalStateException("Hub inventory decrease response is invalid");
}

return response.getData().items().stream()
    .collect(Collectors.toMap(
        HubInventoryResponseDto.Decrease.InventoryResult::productId,
        HubInventoryResponseDto.Decrease.InventoryResult::hubInventoryId
    ));