강의 관련 질문이 있습니다.

[kickbell]

안녕하세요. 수진님,

덕분에 감사히 잘 배우고 있습니다 ^^

강의 관련 질문이 있습니다.

복잡한 플로우 만들기 - 2, 22:22분 지점인데요.

뷰없는 리블렛인 Topup리블렛에서 헬퍼 메소드를 사용해서 present/dismiss 를 해주셨잖아요.

근데 그거 이미 있는 유틸리티에 present/dismiss 관련 로직이 있길래 그걸로 대체해도 잘 돌아가더라구요.

그래서 그냥 무시할까 했는데,

뷰없는 리블렛 같은 경우는 func cleanupViews() 에서 이미 띄운 뷰가 있으면 책임지고 dismiss를 해줘야 하잖아요?

근데 이것 때문에 헬퍼메소드를 쓰신건가..싶어서요.

지금은 아래처럼 해놓았는데 이렇게 하면 안되는 걸까요? navigationControllable을 지워버리고 헬퍼메소드는 없앴어요.

final class TopupRouter {
    func cleanupViews() {
      if viewController.uiviewController.presentationController != nil {
        viewController.dismiss(completion: nil)
      }
    }

    // MARK: - Private

    private let viewController: ViewControllable
  
    //헬퍼메소드 필요없는 것 같아서 삭제.18분쯤. 필요하면 다시 적도록.
    func attachAddPaymentMethod() {
      if addPaymentMethodRouting != nil {
        return
      }
      
      let router = addPaymentMethodBuildable.build(withListener: interactor)
      let navigation = NavigationControllerable(root: router.viewControllable)
      navigation.navigationController.presentationController?.delegate = interactor.presentationDelegateProxy
      viewController.present(navigation, animated: true, completion: nil)
      
      self.addPaymentMethodRouting = router
      attachChild(router)
    }
    
    func detachAddPaymentMethod() {
      guard let router = addPaymentMethodRouting else {
        return
      }
      
      viewController.dismiss(completion: nil)
      detachChild(router)
      addPaymentMethodRouting = nil
    }
}

[kickbell]

아 이어지는 강의에서 navigationControllable를 사용하고 있군요.. 다행히 완성본 코드가 별차이가 없어서 잘 가져다가 썼습니다. 😀