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

기존 MembersAPIWorker Network Module 코드를 리팩토링해요 #711

Merged
merged 9 commits into from
Nov 26, 2024

Conversation

Do-hyun-Kim
Copy link
Collaborator

@Do-hyun-Kim Do-hyun-Kim commented Nov 22, 2024

🔵PR을 올리기 전 아래 사항을 확인해주세요.

  • 구현한 로직과 기능이 올바르게 작동되는지 충분히 테스트해주세요.
  • 코드의 성능이나 메모리 효율성이 적절하게 고려되었는지, 불필요한 코드가 없는지 검토해주세요.
  • 이번 PR에서 구현된 주요 기능이나 해결된 문제에 대해 자세히 서술해주세요.
    (위 내용은 지워주세요)

😽개요

  • 카메라 관련 비즈니스 로직 간소화
  • 카메라 API 통신 실패 시 예외 처리 로직 추가
  • MembersAPI, MembersAPIWorker 리팩토링
  • MembersRepository 및 DTO, Entity 코드 리팩토링
  • AccountResignViewController UI 코드 리팩토링

🛠️ 카메라 관련 비즈니스 로직 간소화 하였습니다.

  • 비즈니스 로직 코드를 UseCase 내부에 구현하여 최대한 Reactor 코드를 간소화 하다록 하였습니다.
  • 카메라 통신 실패 시 catch 연산자를 사용하여 해당 오류에 맞게 Toast 및 오류 문구를 띄우도록 하였습니다.

Member 관련 MembersAPIWorker 리팩토링

  • Swagger 기준으로 Member API EndPoint를 통합하였습니다.
  • 추가로 기존에 사용되었던 DTO, Entity 네이밍이 코드 컨벤션에 맞지 않아서 제거하고 RequestDTO, ResponseDTO로 변경하였습니다.

AccountResingViewController UI 코드 리팩토링

  • AccountResingViewController 내부 UI 코드를 일부분만 수정 하였습니다.
  • AccountResignViewController 에서 화면전환 코드를 Navigator를 사용하도록 리팩토링 하였습니다.
업로드 예외 처리 로직

🟡차후 계획 (Optional)

  • RealEmoji, Mission 관련 코드도 Swagger 기준으로 통합 하면 Camera 코드 일부분 조금 리팩토링 하겠습니다.

✅테스트 케이스

  • 게시물 업로드 API 호출이 잘 되는지 확인해요
  • 프로필 이미지 업로드 API 호출이 잘 되는지 확인해요

🙏🏻아래와 같이 PR을 리뷰해주세요.

  • PR 내용이 부족하다면 보충 요청해주세요.
  • 코드 스타일이 팀의 규칙에 맞게 작성되었는지, 일관성을 유지하고 있는지 확인해주세요.
  • 코드에 대한 문서화나 주석이 필요한 부분에 적절하게 작성되어 있는지 확인해주세요.
  • 구현된 로직이 효율적이고 올바르게 작성되었는지, 아키텍처를 잘 준수하고 있는지 검토해주세요.
  • 네이밍, 포매팅, 주석 등 코드의 일관성이 유지되고 있는지 확인해주세요.

- PickDIContainer, ResignDIContainer 파일 제거
- MembersAPI, APIWorker 코드 리펙토링
- Members 관련 UseCase 생성
- 사용하지 않는 PickUseCase 제거
- Members 관련 DTO, Entity 코드 리팩토링 및 코드 컨벤션에 맞게 파일 생성
@Do-hyun-Kim Do-hyun-Kim requested a review from akrudal November 22, 2024 05:53
@Do-hyun-Kim Do-hyun-Kim self-assigned this Nov 22, 2024
@Do-hyun-Kim Do-hyun-Kim marked this pull request as draft November 22, 2024 05:53
- Camera 업로드 관련 비즈니스 로직 코드 수정
- Members, Camera 관련 Navigator 코드 추가
- CreateMembersPreSignedURLUseCase 비즈니스 로직 코드 추가
- MembersRepository, UseCase 회원 탈퇴 API Body 값 제거, createMemberImagePreSignedURL memberId Parameters 제거
- ProfileViewReactor 회원 프로필 이미지 변경시 API 예외 처리 로직 추가
- CameraViewReactor 비즈니스 로직 코드 리팩토링 및 예외 처리 로직 추가
- ProfileDIContainer UseCase 의존성 주입 코드 추가
- FetchDailyMissionContentUseCase 추가
- MissionContentEntity, MissionTodayContentResponseDTO 추가
- CameraViewReactor Mission관련 비즈니스 로직 수정
-  MissonDIContainer FetchMissionContentUseCase 의존성 추가
@Do-hyun-Kim Do-hyun-Kim marked this pull request as ready for review November 26, 2024 13:45
@Do-hyun-Kim Do-hyun-Kim merged commit 30f5581 into develop Nov 26, 2024
@Do-hyun-Kim Do-hyun-Kim deleted the refactor/#710-member-apiworker-modify branch November 27, 2024 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant