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

DetailVC에서도 HeaderDelegate를 사용하도록 수정 #43

Open
jinios opened this issue Sep 30, 2018 · 0 comments
Open

DetailVC에서도 HeaderDelegate를 사용하도록 수정 #43

jinios opened this issue Sep 30, 2018 · 0 comments
Assignees
Labels
refactor need to refactor

Comments

@jinios
Copy link
Owner

jinios commented Sep 30, 2018

  • expande/collapse 기능이 있는 테이블뷰의 델리게이트를 담당하는 뷰컨트롤러는 HeaderDelegate(or FooterDelegate)를 사용하도록 변경
  • HeaderDelegate에는 [Branch]타입을 커스텀하여 expandable하게 데이터를 관리할 수 있는 커스텀 모델 객체 ExpandCollapseTogglable를 사용하게되어있음
  • 이 커스텀타입이나 프로토콜을 사용하지 않으면 뷰컨트롤러가 테이블뷰가 expand상태인지 아닌지를 판별하는 isExpanded속성을 직접 가지고 관리할 필요가 없어짐
  • 또한 Expand이거나 Collapse상태일때 표시해야하는 데이터도 뷰컨트롤러가 아닌 모델객체가 관리하게된다
  • 따라서 DetailVC가 테이블뷰의 데이터로 사용하고있는 [Branch]를 [ExpandCollapseTogglable]타입으로 변경하고, HeaderDelegate 프로토콜을 채택하여 추상화하여 뷰컨트롤러가 직접 데이터를 관리/사용하지 않는 구조로 개선한다.
@jinios jinios added the refactor need to refactor label Sep 30, 2018
@jinios jinios self-assigned this Sep 30, 2018
jinios added a commit that referenced this issue Oct 5, 2018
- Modify protocol implement to abstract expand/collapse holiday list feature for MainVC, DetailVC
- Delete DetailHeaderDelegate
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor need to refactor
Projects
None yet
Development

No branches or pull requests

1 participant