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

BrowseView 구성 #23

Merged
merged 7 commits into from
Nov 17, 2022
Merged

BrowseView 구성 #23

merged 7 commits into from
Nov 17, 2022

Conversation

jinwoong16
Copy link
Collaborator

📕 Issue Number

Close #18

📙 작업 내역

구현 내용 및 작업 했던 내역

  • Browse Cell 구성
  • Browse View 구성
  • 가짜 view model을 통하여 값을 주입하여 확인

  • cell 클릭시, 해당 유저의 정보를 출력: 나중에 이것을 통해, coordinator에게 일을 시킬 예정

Screenshot 2022-11-16 at 4 08 31 PM

📘 작업 유형

  • 신규 기능 추가
  • 버그 수정
  • 리펙토링
  • 문서 업데이트

📋 체크리스트

  • Merge 하는 브랜치가 올바른가?
  • 코딩컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 없는가?
  • 내 코드에 대한 자기 검토가 되었는가?
  • 변경사항이 효과적이거나 동작이 작동한다는 것을 보증하는 테스트를 추가하였는가?
  • 새로운 테스트와 기존의 테스트가 변경사항에 대해 만족하는가?

📝 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • 다행히 BrowseView는 구성이 어려웠지, 비즈니스 로직은 꽤 단순해 보입니다. UseCase와 Repository Interface 구성이 빠르게 될 것 같아요 :)
tableView.rx.itemSelected
  .subscribe(onNext: { indexPath in
    print(self.viewModel.users[indexPath.row])
  })
  .disposed(by: disposableBag)
  • 현재, BrowseViewController에서 Cell을 클릭시, view model에 저장해둔 users 배열에서 값을 가져와 출력해주는 부분입니다. 나중에 이곳에서 저 정보를 기반으로 coordinator에게 일을 시키면 될 것 같은데, 그것을 클로저 기반으로 할지, rx로 처리할지 생각해봐야 할 것 같아요. 저는 후자가 좋아보입니다. 저번 과제에서 뷰컨과 코디네이터를 combine으로 연결했었는데, 비동기 액션을 combine으로 통일을 해서 괜찮았다고 생각헤요. 다른분들 의견도 궁금하네요 :)



- 유즈케이스에서 user와 그 user의 qeust 배열을 받고, 해당 유저를 users 배열에 append 한다.
- 나중에 이것은, cell을 클릭했을 때, 해당 user의 정보를 detail하게 보여주는 view로 이동시, 사용할 예정!
- 나중에 이곳을 통해서 coordinator에게 이 user의 detail을 보여주는 view를 띄우게 하면 될듯.
@jinwoong16 jinwoong16 linked an issue Nov 16, 2022 that may be closed by this pull request
2 tasks
@jinwoong16 jinwoong16 merged commit c302b7e into develop Nov 17, 2022
@jinwoong16 jinwoong16 deleted the feature/BrowseView branch November 17, 2022 03:00
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.

Browse View
1 participant