UICollectionViewでスクロールして選択するときの奇妙な動作

Question

UICollectionViewのカスタムセルにチェックマークを表示する際に問題があります。最初のいくつかのタップでは、すべてが期待通りに機能しますが、スクロールやタップを繰り返したり、選択したセルをクリックしたりすると、gifのように動作が奇妙になります。おそらく私はこれについて間違った方法で行くつもりですか？ .addCheck（）と.removeCheck（）は、私が作成したカスタムUICollectionViewCellクラス内のメソッドであり、チェックマークイメージを追加するか、セルビューから削除するだけです。以下はThe odd behavior shown here

override func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell { 

    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "cell", for: indexPath) as! ColorUICollectionViewCell 

    // Configure the cell 
    let color = colorList[(indexPath as NSIndexPath).row] 
    cell.delegate = self 
    cell.textLabel.text = color.name 
    cell.backgroundColor = color.color 

    if color.selected { 
     cell.addCheck() 
    } 
    else { 
     cell.removeCheck() 
    } 

    return cell 
} 

// user selects item 
override func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) { 

    // set colors to false for selection 
    for color in colorList { 
     color.selected = false 
    } 

    // set selected color to true for selection 
    let color = colorList[indexPath.row] 
    color.selected = true 
    settings.backgroundColor = color.color 
    //userDefaults.set(selectedIndex, forKey: "selectedIndex") 
    collectionView.reloadData() 
} 

は私のカスタムセル内addCheck（）とremoveCheck（）関数がどのように見えるかです。最初

func addCheck() { 
    // create check image 
    let checkImage = UIImage(named: "checkmark") 
    checkImageView = UIImageView(frame: CGRect(x: 0, y: 0, width: bounds.size.height/4, height: bounds.size.height/4)) 
    checkImageView.image = checkImage!.withRenderingMode(UIImageRenderingMode.alwaysTemplate) 
    checkImageView.tintColor = UIColor.white() 

    // add the views 
    addSubview(checkImageView) 
} 

func removeCheck() { 
    if checkImageView != nil { 
     checkImageView.removeFromSuperview() 
    } 
} 

Answer 1

オフ、あなたは少しあなたのdidSelectを簡素化することができます：あなたのcellForItemAt方法で言語に基づいて

override func collectionView(collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) { 

    // set colors to false for selection 
    for (index, color) in colorList.enumerate() { 
     if index == indexPath.row { 
      color.selected = false 
      settings.backgroundColor = color.color 
     } 
     else { 
      color.selected = false 
     } 
    } 

    collectionView.reloadData() 
} 

、私はあなたがタップしたときに第2のチェックマーク画像を追加している推測しています同じセルに2回、正しくトラッキングされていないので、セルがちょうどコレクションビューがリロードされたあと、周りを回転し続けるようになる

あなたのセルクラス、または少なくともaddCheckとremoveCheckのロジックが投稿されています。

私が推奨することは、選択に基づいて単純に表示/非表示にするときに、セル上にチェックマークが付いた画像を永久に保持することです。これにより、collectionViewも高速化するはずです。