[IDLE-000] 요양보호사 공고지원하기 #51
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# Conflicts: # project/Projects/Data/DataSource/DTO/RecruitmentPost/RecuritmentPostListForWorkerDTO.swift
present한 ViewController를 곧바로 dismiss할경우 올바르게 dismiss되지 못한다, 등장/퇴장 애니메이션이 충돌할 수 있기 때문이다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
변경된점
요양보호사는 공고에 지원할 수 있다.
기존에 Cell마다 ViewModel을 보유하는 시스템을 개선했습니다.
Cell은 테이블 뷰의 ViewModel에 기능을 사용합니다. 각가의 셀은 postId를 사용하여 호출의 주체를 인식할 수 있도록 변경하였습니다.
테이블 뷰모델이 규모가 커지는 단점이 있지만, Cell마다 존재하는 ViewModel을 1개로 압축함으로써, 매모리를 아낄 수 있습니다.
그리고 Cell에서 발생하는 이벤트를 테이블 뷰에서 직접 수신할 수 있다는 점에서, 상호작용의 깊이가 줄어들게 됩니다.
코드가독성이 올라갔음으로 적절한 트레이드 오프라고 생각합니다.
테이블 뷰모델에 구현한 매서드입니다.
셀전용 ViewModel프로토콜을 테이블 뷰가 추가로 구현하는 방식으로 구현하여, Cell의 재사용성은 떨어지지 않도록 했습니다.
Cell의 바인드 함수
