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

안녕하세요!

영한님 강의를 들으면서 추가적으로 RestTemplate에 대해 알게되어 지금까지 배운 것을 복습할 겸 api와 DTO 통신에 대해 연습하고 있었습니다.

DTO를 반환할 때, 그냥 주기보다는 Result<T>의 data같은 속성에 담아서 주는 것이 바람직하다고 하여 그 응용을 연습해 볼겸 Result<T>에 status 속성을 추가해보았습니다.

status 속성은 외부 api에서 조건을 충족하는 api를 찾아 와서 반환할 DTO가 있을 경우 "success"를, 조건을 충족하는 api를 찾지 못하여 HttpClientErrorException 예외가 발생할 경우 "fail"을 담아서 반환하고자 하였습니다.

아래의 코드를 통해 구현은 성공하였으나, Service 클래스에서 try-catch문을 사용하는게 좀 찜찜하여 혹시 try-catch문 없이 구현하는게 더 좋을지 아니면 그냥 사용해도 좋은지가 궁금하여 질문을 하게 되었습니다.

(status가 "fail"이면 data는 null을 반환하게끔 설계해보았습니다)

@RestController
@RequiredArgsConstructor
@RequestMapping("/summoners")
public class SummonerController {

    private final SummonerService summonerService;

    @GetMapping
    public ResponseEntity<Result<SummonerDTO>> findSummonerApi(@RequestParam String name) {
        SummonerDTO summonerDTO = summonerService.findSummonerApi(name);

        if (summonerDTO == null) {
            return ResponseEntity.ok().body(new Result<>("fail", null));
        }
        return ResponseEntity.ok().body(new Result<>("success", summonerDTO));
    }
}

@Slf4j
@Service
@RequiredArgsConstructor
public class SummonerService {

    @Value("${apiKey}")
    private String apiKey;

    private final SummonerRepository summonerRepository;
    private final RestTemplate restTemplate;

    // riot에서 소환사 정보 api를 받아오는 메서드
    public SummonerDTO findSummonerApi(String name) {
        String url = "https://kr.api.riotgames.com/lol/summoner/v4/summoners/by-name/{summonerName}";

        HttpHeaders headers = new HttpHeaders();
        headers.set("X-Riot-Token", apiKey);

        try {
            return restTemplate.exchange(url, HttpMethod.GET, new HttpEntity<>(headers), SummonerDTO.class, name).getBody();
        } catch (HttpClientErrorException e) {
            return null;
        }
    }
}

Controller와 Service의 코드는 위와 같습니다. (restTemplate은 별도의 설정파일에 빈으로 구현하였습니다)