CellをSwapしたい

やりたいことはシンプルで、あるセルとあるセルの位置を交換したく、
下記のようにDataSourceのデータをSwapしてreloadDataを行います。

ちなみに、UICollectionViewのmoveItem(at indexPath: IndexPath, to newIndexPath: IndexPath)を使用するとこの現象は起きませんが、今回は２つのセルを入れ替えたいという目的があり、
上下のセルを入れ替えると他のセルも動いてしまうため、使いませんでした。


@objc func handlePanGesture(_ gesture: UIPanGestureRecognizer) {


    // movingCellは移動中のセルの状態を保持しているクラス
    switch gesture.state {

    case .began:
        movingCell = MovingCell(cell: cell.0, originalLocation: cell.0?.center, indexPath: cell.1)

    case .changed:
        movingCell?.cell.layer.zPosition = 100
        movingCell?.cell.center = gesture.location(in: gesture.view!)
    case .ended:
        swapMovingCellWith(cell: cell.0, at: cell.1)
    default:
        movingCell?.reset()
        movingCell = nil
    }
}

private func swapMovingCellWith(cell: UICollectionViewCell?, at indexPath: IndexPath?) {
    guard let cell = cell, let indexPath = indexPath, let moving = movingCell else {
        movingCell?.reset()
        return
    }

    viewModel.swap(from: moving.indexPath.item, to: indexPath.item)
    animate(moving: moving, to: cell)
}

private func animate(moving movingCell: MovingCell, to cell: UICollectionViewCell) {
    panGesture.isEnabled = false

    UIView.animate(withDuration: 0.4, delay: 0, usingSpringWithDamping: 0.1, initialSpringVelocity: 0.7, options: .allowUserInteraction, animations: {

        movingCell.cell.center = cell.center
        cell.center = movingCell.cell.center

    }, completion: { _ in

        // ここが問題
        self.collectionView.reloadData()
        self.panGesture.isEnabled = true
        self.movingCell = nil
    })
}

一見問題なく動きますが、

一番左上のセルを移動させようとすると移動位置がおかしくなります。

画面上では消えたように見えますが、
実際には同じ位置の下にいます。

この現象が起きるのは、一番左上のセルから他のセルに移動された場合のみで、
逆に他のセルから左上のセルに移動させる場合には問題はありませんでした。

色々原因を探ってみましたが、

Storyboardでcellを作成しているから? →コードから生成しても変わらず。

MainThreadで処理していない？→あえてDispatchQueue.main.async内でやってみたが変わらず。

回避策

結局この条件の時だけ、最後に移動させるようにしました。
reloadDataのあとに確実に処理を行うために、animationのcompletionで処理を行なっています。

UIView.animate(withDuration: 0, animations: {
    self.collectionView.reloadData()
}, completion: { _ in
    if movingCell.indexPath.item == 0 {
        cell.frame.origin.x = 0
    }
    self.panGesture.isEnabled = true
    self.movingCell = nil
    completion()
})

まとめ

なぜこの現象が起きるのかがわかっていないのですごい気持ち悪いです
IndexPath(item: 0, section: 0)であることが何か関係しているのか。

もうちょっと調べてみます。

何か他の解決法をご存知の方がいらっしゃいましたら教えてください

【Swift】UICollectionView reloadDataの不思議な挙動？

CellをSwapしたい

回避策

まとめ