Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#330 바코드 식품 조회 Utils 분리 #345

Merged
merged 5 commits into from
Aug 1, 2024

Conversation

sojungpp
Copy link
Member

@sojungpp sojungpp commented Aug 1, 2024

📍 이슈 번호


🛠️ 작업 내용

1. 바코드 API 관련 내용은 FoodBarcodeUtils로 따로 분리했습니다 ! 170b66a


🎸 기타 사항

  • 외부API 관련 방법이 HttpURLConnection(현재 사용), RestTemplate, WebClient 3가지 방법 찾아봤는데,
    HttpURLConnection이 속도가 가장 빠르지만 Response를 객체로 받아오지 못해서 지금처럼 일일히 파싱 작업이 필요한 것 같습니다.
    장단점 생각해봤을 때, 우선 속도가 가장 중요하다고 판단해서 일단 현상태 유지했는데, 추후 WebClient로 변경하는게 좋을지 어떤 방법이 좋을지 고민이 됩니답 !
  • Food는 일단 끝 !

@sojungpp sojungpp added the ♻️ refactor 리팩토링 label Aug 1, 2024
@sojungpp sojungpp self-assigned this Aug 1, 2024
Copy link
Member

@psyeon1120 psyeon1120 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WebClient를 사용하면 응답이랑 똑같은 형태의 dto를 만들어놨을 때 파싱없이 바로 dto로 받을 수 있는건가요??

@psyeon1120 psyeon1120 merged commit 6b7656a into develop Aug 1, 2024
1 check passed
@sojungpp
Copy link
Member Author

sojungpp commented Aug 1, 2024

WebClient를 사용하면 응답이랑 똑같은 형태의 dto를 만들어놨을 때 파싱없이 바로 dto로 받을 수 있는건가요??

이런 형식으로 Dto 매핑할 수 있는 것 같아요!

 Mono<MyDto> responseMono = webClient.get()
        .uri(url)
        .retrieve()
        .bodyToMono(MyDto.class);

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[refactor] Food 클린 코드 적용 및 성능 개선
2 participants