Feature/connect quest layer

[jinwoong16]

📕 Issue Number

Close #47

📙 작업 내역

구현 내용 및 작업 했던 내역

• Quest View를 모든 레이어와 연결합니다.
  • DI 생성 후, 주입
  • Presentation, Domain, Data가 연결되어 데이터가 잘 나오네요.

📘 작업 유형

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

📋 체크리스트

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

📝 PR 특이 사항

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

• 현재는 HomeViewController의 create(with:)메서드에서 QuestViewModel을 받아서, QuestView와 이어주고 있습니다. 나중에 다른 ViewModel들도 이곳에 추가될 것 같은데, 다른 분들 의견이 궁금하네요.

static func create(with questViewModel: QuestViewModel, 
                                     followingViewModel: FollowingViewModel,
                                     calendarViewModel: CalendarViewModel) -> HomeViewController {
        let vc = HomeViewController()
        vc.setup(questViewModel: questViewModel,
                      followingViewModel: FollowingViewModel,
                      calendarViewModel: CalendarViewModel) // 각 viewmodel들을 vc내의 view들과 연결해줍니다.
        
        return vc
}

[wickedRun]

음 간단하게 연결이 되네요 확실히. 고생하셨읍니다.
특이 사항에 관해서는 얘기 해봐야 할 것 같습니다.

[Jeonhui]

특이사항에 관해서 제 생각은 가장 주된 ViewModel에 경우에는 init을 오버라이딩하여서 추가하는 방식으로 해도 될 것 같고, 꼭 init을 추가하지 않더라도 create 함수를 그대로 사용해도 될 것 같습니다.
추가적으로 다른 ViewModel을 추가하는 것은 ViewModel Protocol을 만들어 addViewModel<VM: ViewModel>(with: VM)과 같은 형식으로 함수를 만들어도 좋을 것 같습니다.
만약에 이렇게 구현한다고 하면, 아래와 같이 구성이 될 것 같네요.

func makeHomeViewController() -> HomeViewController {
         let vc = HomeViewController.create(with: makeQuestViewModel())
         vc.addViewModel(with: {ViewModel객체})
         return vc
     }

특이사항에 적어주신 내용을 제가 제대로 이해한 건지를 모르겠네요. 코드 작성하시느라 너무 고생 많으셨습니다 🙃

[jinwoong16]

특이사항에 관해서 제 생각은 가장 주된 ViewModel에 경우에는 init을 오버라이딩하여서 추가하는 방식으로 해도 될 것 같고, 꼭 init을 추가하지 않더라도 ViewModel Protocol을 만들어 기존에 만드신 create함수를 addViewModel<VM: ViewModel>(with: VM)과 같은 형식으로 함수를 만들어도 좋을 것 같습니다. 특이사항에 적어주신 내용을 제가 제대로 이해한 건지를 모르겠네요. 코드 작성하시느라 너무 고생 많으셨습니다 🙃

다시 읽어보니 제가 너무 두루뭉실하게 썼네요... 조금더 직관적이게? 변경했습니다. HomeViewController 같은경우에는 안에 여러 뷰들(캘린더, 퀘스트 등)이 있어서 그들의 view model을 이어주는 과정에서 코드가 길어지는 현상이 생길 것 같은데, 코드가 길어지면 늘 불안해지더라고요. 그래서 다른분들 의견이 궁금했습니다. 스크럼 때, 한번 더 이야기할게요!
ps. 제네릭 쓰시는게 되게 자연스럽네요. 부럽습니다...

[Jeonhui]

특이사항에 관해서 제 생각은 가장 주된 ViewModel에 경우에는 init을 오버라이딩하여서 추가하는 방식으로 해도 될 것 같고, 꼭 init을 추가하지 않더라도 ViewModel Protocol을 만들어 기존에 만드신 create함수를 addViewModel<VM: ViewModel>(with: VM)과 같은 형식으로 함수를 만들어도 좋을 것 같습니다. 특이사항에 적어주신 내용을 제가 제대로 이해한 건지를 모르겠네요. 코드 작성하시느라 너무 고생 많으셨습니다 🙃

다시 읽어보니 제가 너무 두루뭉실하게 썼네요... 조금더 직관적이게? 변경했습니다. HomeViewController 같은경우에는 안에 여러 뷰들(캘린더, 퀘스트 등)이 있어서 그들의 view model을 이어주는 과정에서 코드가 길어지는 현상이 생길 것 같은데, 코드가 길어지면 늘 불안해지더라고요. 그래서 다른분들 의견이 궁금했습니다. 스크럼 때, 한번 더 이야기할게요! ps. 제네릭 쓰시는게 되게 자연스럽네요. 부럽습니다...

기존 코드도 되게 깔끔하게 잘 짜여졌다고 생각해요. 다만 제너릭을 사용했을 때에 각각의 View Model 타입들을 구별해줘야될 것같아서 조금 더 같이 생각해보면 좋을 것 같아요.
ps. 이번에 clean architecture을 진행하면서 많이 배웠습니다. 나중에 시간되시면 Data나 Firebase쪽 코드 보시는 것도 추천드립니다👍🏻

[sprituz]

고생하셨습니다..1!!!!!!