sihyun10 / FE_PS_Final_Refactor

몰리턴 기업 연계 프로젝트 코드리뷰 및 리팩토링
0 stars 0 forks source link

api 및 id 체크 로직 리팩토링 #16

Closed 05Castle closed 1 year ago

05Castle commented 1 year ago

PR check list

PR type

Summary

ApartDataApi의 코드 개선 UploadApi의 필요없는 함수 삭제 id 체크하여 데이터 가져오는 컴포넌트들의 로직 중복 개선

Describe your changes

ApartData는 .then 메서를 try-catch문으로 변경하여 가독성을 높이고, 코드 depth를 줄였다.

UploadApi에서 instance 선언을 위해 createAxiosInstance라는 함수와 interceptors 설정을 위해 setupResponseInterceptor함수를 실행하고 있다. 직업 instance와 interceptors를 선언하면 되므로 두 함수를 삭제하여 불필요한 과정을 줄였다.

PropertyInfo, 시세 테이블, pdf 요약부 테이블, Location, MarketPrice에서 현재 파라미터의 id를 체크하고 해당하는 데이터를 가져오는 로직을 사용하고 있다. 해당 로직이 각 컴포넌트에서 중복되어 사용되므로 이를 util 함수로 만들어 재사용성을 높이고 불필요한 코드 중복을 방지하였다.

Issue number and link

14 #15