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

refactor: CallAdapter 적용 #558

Open
2 tasks done
hxeyexn opened this issue Nov 25, 2024 · 1 comment · May be fixed by #574
Open
2 tasks done

refactor: CallAdapter 적용 #558

hxeyexn opened this issue Nov 25, 2024 · 1 comment · May be fixed by #574
Assignees
@hxeyexn
Labels
android We are android>< confirm need confirmation! refactor 리팩토링 (변수 및 메서드 네이밍 변경) ✅ test 테스트 (테스트 코드 추가, 수정, 삭제: 비즈니스 로직에 변경 없음)
Milestone
sprint-7

Comments

@hxeyexn
Copy link
Contributor

hxeyexn commented Nov 25, 2024 •
edited
Loading

🤮 As Is

리팩터링하고자 하는 파트와 이유를 구체적으로 설명해주세요.

리팩터링하고자 하는 파트

  • Network 응답 처리 방식

리팩터링하고자 하는 이유

  • DataSource 내의 네트워크 통신 메서드 모두 handleApiResponse 로 감싸줘야하는 불편함이 있습니다.
  • 데이터 레이어가 handleApi에 대한 의존성을 가질 뿐만 아니라, API 응답 처리에 대한 책임도 가지고 있습니다.

🤩 To Be

리팩터링 방향을 구체적으로 공유해주세요.

  • API 응답 처리에 대한 책임을 CallAdapter에게 위임하여 데이터 레이어와 handleApi 사이의 의존성을 제거하려고 합니다.
    • CallAdapter는 네트워크 응답을 원하는 객체로 변경시켜줍니다.
  • Repository의 네트워크 응답 중복 로직을 최소화하려고 합니다.

😇 이때까지 끝낼게요!

리팩터링 완료 예상 날짜를 작성해주세요. 신중하게 생각해요!

  • 11월 29일 ➡️ 12월 10일 (테스트가 필요해서 시간이 좀 더 걸릴 것 같습니다!)

😵 참고 자료(선택)


🙇‍♀️이슈 확인했어요:)

팀원에게 이슈 확인을 부탁해요!
@hxeyexn hxeyexn added android We are android>< confirm need confirmation! refactor 리팩토링 (변수 및 메서드 네이밍 변경) labels Nov 25, 2024
@hxeyexn hxeyexn added this to the sprint-7 milestone Nov 25, 2024
@hxeyexn hxeyexn self-assigned this Nov 25, 2024
@hxeyexn
Copy link
Contributor Author

hxeyexn commented Nov 28, 2024

@Junyoung-WON @s6m1n

Retrofit에 CallAdapter를 적용하는 법

참고자료 하나 더 추가해뒀습니다!
같이 읽어보면 좋을 것 같아요~

@hxeyexn hxeyexn added the ✅ test 테스트 (테스트 코드 추가, 수정, 삭제: 비즈니스 로직에 변경 없음) label Dec 2, 2024
@hxeyexn hxeyexn mentioned this issue Dec 18, 2024
Closed
@hxeyexn hxeyexn linked a pull request Dec 18, 2024 that will close this issue
refactor: feature/#558-apply-calldapter 테스트용 PR #573
Closed
@hxeyexn hxeyexn linked a pull request Dec 20, 2024 that will close this issue
Open
1 task
@hxeyexn hxeyexn linked a pull request Dec 20, 2024 that will close this issue
refactor: CallAdapter 적용 및 검증 테스트 작성 #558 #574
Open
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hxeyexn hxeyexn

Labels
android We are android>< confirm need confirmation! refactor 리팩토링 (변수 및 메서드 네이밍 변경) ✅ test 테스트 (테스트 코드 추가, 수정, 삭제: 비즈니스 로직에 변경 없음)
Projects
2024-staccato
Status: Todo
Milestone
sprint-7
1 participant
@hxeyexn