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

refactor: Calendar 관련 뷰 컨트롤러, 리액터 등 부가 코드 리팩토링 #682

Merged
merged 7 commits into from
Oct 21, 2024

Conversation

rlarjsdn3
Copy link
Collaborator

@rlarjsdn3 rlarjsdn3 commented Oct 16, 2024

😽개요

  • 과거 구 코드를 새로운 코드로 바꾸고, 로직을 가능한 한 단순화시키는 데 중점을 두었습니다.
    **** PR이 너무 커서 죄송합니다.. 어쩔 수가 없었어여..

🛠️작업 내용

캘린더 관련 APIs, Repository 코드 리팩토링

  • 캘린더 관련 API 및 Repository 코드를 BBNetwork 코드로 마이그레이션하였습니다. 다만, 네트워크 실패 등 별도 예외처리는 하지 않았습니다. (추후 구현 예정)

캘린더 관련 뷰 컨트롤러, 셀 및 Reactor 코드 리팩토링

  • 일부 뷰 컨트롤러와 셀의 이름을 변경하였습니다. 일부 지나치게 큰 뷰 컨트롤러와 셀을 여러 뷰로 쪼갰습니다.

    • CalendarPageHeaderView, CalendarPostTitleView, CalendarPostImageView
  • Reactor에서 Navigator로 화면을 전환하고, ServiceProvider로 토스트 및 알림을 띄우두록 코드를 수정하였습니다.

  • 필요없는 일부 변수 및 스트림을 삭제하고, 로직을 간소화하였습니다. 방법을 찾지 못해 코드가 어설픈 부분이 있습니다. 추후 더 나은 방법을 찾는다면 개선할 예정입니다.

새로운 RxSwift 연산자 구현

  • map(with:) 연산자를 구현하였습니다. 사용법은 flatMap(with:)와 같습니다.

🟡차후 계획

  • 아직 보완해야 할 부분이 많지만, 조금씩 다시 리팩토링해나갈 예정입니다. 다시 리팩토링하기 주석을 붙인 건 추후 더 나은 방법을 찾는다면 개선할 부분입니다.

✅테스트 케이스

  • 월 캘린더가 제대로 표시되는지, 셀을 클릭하면 주간 화면으로 이동하는지
  • 주간 캘린더 화면에서 일일 포스트가 제대로 표시되는지
  • 댓글 작성 후, 포스트에 댓글 개수가 제대로 증감되는지
  • 리얼 이모지 시트에서 카메라 화면으로 제대로 이동되는지
  • 주간 캘린더에서 프로필 이미지를 클릭하면 제대로 이동되는지

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

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

close #644

@rlarjsdn3 rlarjsdn3 self-assigned this Oct 17, 2024
@rlarjsdn3 rlarjsdn3 changed the title Refactor/#644 calendar리팩토링 refactor: Calendar 관련 뷰 컨트롤러, 리액터 등 부가 코드 리팩토링 Oct 17, 2024
@rlarjsdn3 rlarjsdn3 marked this pull request as ready for review October 18, 2024 09:48
@rlarjsdn3 rlarjsdn3 merged commit d9d2176 into develop Oct 21, 2024
1 check passed
@rlarjsdn3 rlarjsdn3 deleted the refactor/#644-Calendar리팩토링 branch October 21, 2024 03:42
akrudal pushed a commit that referenced this pull request Nov 1, 2024
* refactor: CommentAPIs 코드 마이그레이션

* refacto: MemoriesCalendarCell 리팩토링

* refactor: MemoriesCalendarCell 코드 리팩토링

* refactor: DailyCalendarViewReactor 리팩토링

* refactor: MemoriesCalendarPostCell 리팩토링

* refactor: MemoriesCalendarPostCellReactor 리팩토링

* refactor: 일부 변수 및 메서드 이름 수정
Do-hyun-Kim added a commit that referenced this pull request Nov 12, 2024
* origin/develop:
  fix: SplashViewReactor 화면 전환 issue 수정 (#702)
  fix: MyUserDefaults memberId, userName Type Annotation String으로 수정 (#700)
  fix: 로그인 버튼 클릭시 touch event cancel 현상 수정해요 (#695)
  fix: QA issue를 수정합니다(#696)
  feat: AlertService 구현 (#692)
  refactor: Calendar 관련 뷰 컨트롤러, 리액터 등 부가 코드 리팩토링 (#682)
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.

refactor: Calendar 뷰 컨트롤러 및 부가 코드 리팩토링
1 participant