프로젝트 매니저 [STEP 2] Hemg #313
Open
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
…into step2 # Conflicts: # ProjectManager/ProjectManager.xcodeproj/project.pbxproj # ProjectManager/ProjectManager/Controller/MainView/MainViewController.swift # ProjectManager/ProjectManager/UseCase/MainViewControllerUseCase.swift
July911
requested changes
Oct 5, 2023
| isa = PBXGroup; | ||
| children = ( | ||
| 63CC815F2AC6AA5B00E6355F /* ReuseIdentifier.swift */, | ||
| 63CC816D2AC6D8CA00E6355F /* CellTitleNamespace.swift */, |
|
|
||
| import UIKit | ||
|
|
||
| protocol AddTodoDelegate: AnyObject { |
| }() | ||
|
|
||
| weak var delegate: AddTodoDelegate? | ||
| private var todoItems: ProjectManager? |
There was a problem hiding this comment.
todoItems: ProjectManager?
조금 어색하게 읽히는거같은데요. 어떤 의도인지 말씀부탁드립니다.
| let doneButton = UIBarButtonItem(title: "Done", style: .done, target: self, action: #selector(doneButton)) | ||
| navigationItem.rightBarButtonItem = doneButton | ||
|
|
||
| if isNew == true { |
There was a problem hiding this comment.
optional도 아닌데 == 이 필요할까요 ?
이름이 충분이 bool 을 커버할거 같습니다.
| dismiss(animated: true) | ||
| } | ||
|
|
||
| @objc private func editButton() { |
| } | ||
|
|
||
| func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell { | ||
| guard let listCell = tableView.dequeueReusableCell(withIdentifier: ReuseIdentifier.listTitleCell, for: indexPath) as? ListTitleCell else { return UITableViewCell() } |
|
|
||
| import UIKit | ||
|
|
||
| final class MainViewModel { |
There was a problem hiding this comment.
지금 필요할까요 ... ? 있어서 얻은 장점은 무엇인가요 ?
UI부분을 제외하면 Test코드를 작성할수 있나요 ?
| func didAddTodoItem(title: String, body: String, date: Date) { | ||
| dataManager.addTodoItem(title: title, body: body, date: date) | ||
| let newTodoItem = ProjectManager(title: title, body: body, date: date) | ||
| useCase.todoItems.append(newTodoItem) |
There was a problem hiding this comment.
여기서
DataManager가 TodoItem을 Add하고,
UseCase도 append를 해주네요. 조금 구조가 이상한 것 같습니다 !
| } | ||
|
|
||
| final class DataManager: DataManagerProtocol { | ||
| private var todoItems = [ProjectManager]() |
There was a problem hiding this comment.
DataManager안에 todoItems 가 들어있네요.
이런 경우라면 TodoItemManager가 맞지 않나요 ?
| var doneItems: [ProjectManager] { get set } | ||
| func moveToDoing(_ item: ProjectManager) | ||
| func moveToDone(_ item: ProjectManager) | ||
| func moveToTodo(_ item: ProjectManager) |
There was a problem hiding this comment.
Enum으로
enum TitleItem {
case todo
case doing
case done
}
이렇게 만드신걸 봤는데, 다르게 설계할 수는 없을까요 ? enum을 활용해서요.
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.
STEP2
안녕하세요. @July911(줄라이)~
개인사정도 생기면서, 연휴도 겹치고해서 정말 너무 늦게 PR을 보내게 되었습니다.
잘한지도 의문이 들면서도 우선을 보내고 피드백을 받고싶어서 진행하게 되었습니다.
확인 부탁드립니다.
감사합니다!
고민 했던점
UseCase와ViewModel분리에 대해서제가 테이블뷰 3개를 통하여 프로젝트를 진행하고있어서
VC에서 반복되는것들이 많이 있었습니다.
그래서 이부분에 대한 로직을 UseCase로 옮겨서 진행하게 되었습니다.
그렇게 진행을 하다가 이제
ViewModel을 만들어 로직을 구현해야하는 과정에서UseCase와ViewModel차이가 뭐지 라는 의문을 가지게 되었고 데이터 바인딩? 굳이? 이것이 정답일까? 여기까지 생각하게 되었습니다.그래서 저는
item관련 로직을UseCase,View관련 로직을ViewModel로 이동하여 분리를 진행했습니다.이렇게 진행하다보니 지금 제가 로직을 구현한것들을 의문을 가지게 되면서 잘 되었는지도 의문을 갖게 되었습니다. 이부분 피드백 부탁드립니다!
해결하지못한점
현시점에서 델리겟 패턴 -> 컴바인으로 변경하고싶었는데 진행하지를 못했습니다. 컴바인에 대한 공부가 부족한거 같습니다ㅠㅠ