ios-project-manager: A Swift repository from kdaramz

팀원

팀원	사용 언어
Joons
Glenn
꼬말

사용한 기술

UICollectionView
UICollectionViewDiffableDataSource
UICollectionViewDragDelegate, UICollectionViewDropDelegate

Refactor

기존에 PopOverViewController의 View들을 Private으로 설정해줄 수 없었다. -> ViewController에서 접근하고 있었기때문이다. 그래서 ViewController에서는 PopOverViewController에 사용할 Thing만 넘겨주고, 그 Thing의 유무에 따라 Add를 위한 PopOverViewController인지 Edit를 위한 PopOverViewController인지 결정해주었습니다.
- ```
    init(collectionView: ListCollectionView, thing: Thing?) {
        super.init(nibName: nil, bundle: nil)
        self.collectionView = collectionView
        
        if let thing = thing {
            configureEdit(with: thing)
        } else {
            configureAdd()
        }
    }
```
  그래서 기존에 ViewController에서 접근하여 설정했던 값들을 PopOverViewController에서 설정하여 View들에게 접근제한자를 설정해줄 수 있게 해주었습니다.

기존에 PopOverViewController가 화면에 보여지지 않을 CollectionView를 소유하고 있었는데 이를 PopOverViewController에 Protocol을 생성해주면서 Delegate 패턴을 사용하여 해결해 주었습니다.

//PopOverViewController.swift
protocol PopOverViewDelegate: AnyObject {
    func addThingToDataSource(_ popOverViewController: PopOverViewController, thing: Thing)
    func editThingToDataSource(_ popOverViewController: PopOverViewController, thing: Thing)
}

//ViewController.swift
extension ViewController: PopOverViewDelegate {
    func addThingToDataSource(_ popOverViewController: PopOverViewController, thing: Thing) {
        todoCollectionView.insertDataSource(thing: thing, state: .todo)
    }
    
    func editThingToDataSource(_ popOverViewController: PopOverViewController, thing: Thing) {
        guard let state = thing.state else { return }
        switch state {
        case .todo:
            todoCollectionView.updateThing(thing: thing)
        case .doing:
            doingCollectionView.updateThing(thing: thing)
        case .done:
            doneCollectionView.updateThing(thing: thing)
        }
    }
}

그래서 더이상 PopOverViewController가 불필요하게 CollectionView를 들고다니지 않을 수 있게 리팩토링 해주었습니다.

문제점

CollectionView Drag and Drop 기능 구현 중 하나의 CollectionView에 있는 cell을 다른 CollectionView로 이동하려고 할 때 이전의 cell은 지워지지 않고 그대로 남아있는 현상.
- 원인 : Diffable DataSource에서 Delete를 진행하는 과정에서 DataSource에서 해당 데이터를 찾지 못함.
- 해결 방법 : 기존의 struct로 생성되어있던 Thing 모델을 class로 바꾸는 과정에서 Diffable DataSource에서 필요로 하는 Hashable 과 Equatable에 대한 정의를 해주지 않았음.
- ```
    //Thing.swift
        override var hash: Int {
        var hasher = Hasher()
        hasher.combine(id)
        return hasher.finalize()
    }
    
    override func isEqual(_ object: Any?) -> Bool {
        guard let object = object as? Thing else { return false }
        return self.id == object.id
    }
```
  object에 대한 hash값과 isEqual을 정의해주며 id값을 비교하여 Diffable DataSource에서 데이터를 찾을 수 있게 해주었다.

    init(frame: CGRect, collectionViewLayout layout: UICollectionViewLayout, collectionType: State) {
        super.init(frame: frame, collectionViewLayout: layout)
        self.collectionType = collectionType
        things = DataSource.shared.getDataByState(state: collectionType)
        configureCollectionView()
        configureDataSource()
        configureSnapshot()
    }
    
    required init?(coder: NSCoder) {
        super.init(coder: coder)
        preconditionFailure("모르겠어요.")
        self.collectionType = collectionType	//Error!
        things = DataSource.shared.getDataByState(state: collectionType)
        configureCollectionView()
        configureDataSource()
        configureSnapshot()
    }

Required init? 과 init의 구현을 똑같이 해주려고 하는데 기존의 init은 collectionType이라는 파라미터를 하나 더 받아온다. 하지만 required init?에 collectionType이라는 파라미터를 넣으면 required init?을 또 생성하라는 에러메시지가 나오는 문제

kdaramz/ios-project-manager

팀원

사용한 기술

Refactor

문제점