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.

Sign up for GitHub

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

Jump to bottom

[Bug]: 모든 온라인API 응답 파싱 오류 찾아서 수정 #187

Open
3 of 4 tasks
pknujsp opened this issue Jul 25, 2023 · 0 comments
Open
3 of 4 tasks

[Bug]: 모든 온라인API 응답 파싱 오류 찾아서 수정 #187

pknujsp opened this issue Jul 25, 2023 · 0 comments
Assignees
@pknujsp
Labels
박준성 버그 Something isn't working 안드로이드 🔨refactor🔨 코드 리팩터링!

Comments

@pknujsp
Copy link
Collaborator

pknujsp commented Jul 25, 2023 •
edited
Loading

📑에러 내용

공공데이터포털 api응답시 json값 파싱이 제대로 되지 않고 오류가 발생하는 경우가 있습니다.

📑에러 원인

  • null로 인한 오류
  • 응답에 json,xml 두 형식이 섞여있는 api에서 알수없는 오류 분석해야 합니다

🔲작업 목록

  • 한개씩 테스트 하면서 수정
  • Retrofit에 CallAdapter추가 하여 일괄처리 하도록 기능 등록

✔️참고 사항(Optional)

No response

🕜예상 소요 시간(Optional)

No response
@pknujsp pknujsp added 버그 Something isn't working 박준성 안드로이드 🔨refactor🔨 코드 리팩터링! fix labels Jul 25, 2023
pknujsp added a commit that referenced this issue Jul 25, 2023
@pknujsp
#187 의약품 상세 정보 조회시 응답 행 개수 1개로 수정
3dc1e40
tgyuuAn added a commit that referenced this issue Jul 25, 2023
@tgyuuAn
#187 refactor null 아닌지 체크하는 구문 함수로 빼서 사용할 경우 !! 연산자가 강요되기 때문에 != null…
cf727bf 
… 로 변경
tgyuuAn added a commit that referenced this issue Jul 25, 2023
@tgyuuAn
#187 refactor viewModel에서 context 참조하고 있던 거 Mapper로 로직 옮겨서 해결
48ac015
tgyuuAn added a commit that referenced this issue Jul 25, 2023
@tgyuuAn
#187 setOnMoreClickListner 데이터바인딩으로 변경
f79155f
tgyuuAn added a commit that referenced this issue Jul 25, 2023
@tgyuuAn
#187 setOnMoreClickListner 데이터바인딩으로 변경
2a84447
pknujsp added a commit that referenced this issue Jul 26, 2023
@pknujsp
#187 의약품 허가 상세 정보 응답시 String으로 먼저 받고 오류 처리한 뒤 MedicineDetailInfoRespo…
4e77dc3 
…nse로 파싱하는 방식으로 수정
@pknujsp pknujsp mentioned this issue Jul 26, 2023
Merged
pknujsp added a commit that referenced this issue Aug 1, 2023
@pknujsp
#187 의약품 상세 정보 응답 시 재심사 날짜로 인해 발생하는 데이터 오류 수정, UI에 표시할 데이터 구조 수정하였고 M…
9ba3020 
…apper클래스를 만들어서 적절한 값으로 변경하여 주는 기능 추가 및 지속 작업 중
pknujsp added a commit that referenced this issue Aug 1, 2023
@pknujsp
#187 의약품 상세 정보 응답 시 재심사 날짜로 인해 발생하는 데이터 오류 수정, UI에 표시할 데이터 구조 수정하였고 M…
b97adbd 
…apper클래스를 만들어서 적절한 값으로 변경하여 주는 기능 추가 및 지속 작업 중
pknujsp added a commit that referenced this issue Aug 10, 2023
@pknujsp
#187 Retrofit2에 CustomCallAdapter를 추가하여 오류 처리를 일관적 으로 한곳에서 처리하도록 개선
a4f9e31
@tgyuuAn tgyuuAn removed the fix label Feb 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pknujsp pknujsp

Labels
박준성 버그 Something isn't working 안드로이드 🔨refactor🔨 코드 리팩터링!
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pknujsp @tgyuuAn