CollectionViewでの選択されていないセルについて

Question

現在もSwiftを勉強しているものです。

早速ですが表題の件について質問があります。

CollectionViewを使用して、
セルを選択した場合に色を変えるという事を勉強していました。

こちらは分かったのですが、
逆に選択されていないセルの色を変更など出来ないものかと考えております。

どうしても方法を思いつきませんでしたので、
こちらに投稿させて頂きました。

こちらのサイト様を参考に勉強して
タップしたら色を変えるだけを実装しております。

Docs様 CollectionViewの実装

// ViewController

import UIKit

class ViewController: UIViewController, UICollectionViewDelegate, UICollectionViewDataSource {

    var myCollectionView : UICollectionView!

    override func viewDidLoad() {
        super.viewDidLoad()

        // CollectionViewのレイアウトを生成.
        let layout = UICollectionViewFlowLayout()


        // Cell一つ一つの大きさ.
        layout.itemSize = CGSize(width:50, height:50)

        // Cellのマージン.
        layout.sectionInset = UIEdgeInsetsMake(16, 16, 32, 16)

        // セクション毎のヘッダーサイズ.
        layout.headerReferenceSize = CGSize(width:100,height:30)

        // CollectionViewを生成.
        myCollectionView = UICollectionView(frame: self.view.frame, collectionViewLayout: layout)

        // Cellに使われるクラスを登録.
        myCollectionView.register(CustomUICollectionViewCell.self, forCellWithReuseIdentifier: "MyCell")

        myCollectionView.delegate = self
        myCollectionView.dataSource = self

        self.view.addSubview(myCollectionView)

    }

    /*
     Cellが選択された際に呼び出される
     */
    func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {

        print("Num: \(indexPath.row)")

        let cell = collectionView.cellForItem(at: indexPath) as! CustomUICollectionViewCell
        // 選択されたセルを赤にする
        cell.imageView.backgroundColor = .red
    }

    /*
     Cellの総数を返す
     */
    func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        return 8
    }

    /*
     Cellに値を設定する
     */
    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {

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

        return cell
    }

}

// CustomCell

import UIKit

class CustomUICollectionViewCell : UICollectionViewCell{

    var imageView: UIImageView!

    required init(coder aDecoder: NSCoder) {
        super.init(coder: aDecoder)!
    }

    override init(frame: CGRect) {
        super.init(frame: frame)

        // UILabelを生成.
        imageView = UIImageView(frame: CGRect(x:0, y:0, width:frame.width, height:frame.height))
        imageView.backgroundColor = UIColor.white

        // Cellに追加.
        self.contentView.addSubview(imageView)
    }

}

実装したい内容を詳しく説明すると、
セル０をタップして赤に変更する
→セル１をタップしてセル１を赤に変更、セル０を白に戻す
→セル２をタップしてセル２を赤に変更、セル１を白に戻す
→・・・・

というように常に１つのセルの色だけを変更したいです。

どうかご教授お願い致します。

Answer 1

今の実装だと、選択してセルを赤くした後にスクロールさせると、別のセルが赤くなりませんか？
まずはそこから修正しないといけません。（というか、そこを修正すれば質問は解決します）

選択したセルに直接アクセスするのではなく、生成するときに色を変更するようにします。

1. 選択したセル番号を保存する変数を追加する。

型はIndexPathか、セクション未使用ならIndexPath.rowだけOKです。

2. cellForItemAtの中

セルを生成した後で、選択中のセルであれば背景を赤く、そうでなければ白くします。

3. didSelectItemAtの中

選択したセル番号を保存し、tableView.reloadData()でテーブルを更新します。
更新することで、「セルが再生成される = cellForItemAtが呼ばれる」ので、新たに選択したセルの背景は赤く、それ以外のセルは白くなります。

Comments

[rookie]

確かにスクロールすると赤に変わったセルの場所が移動してしまいました。

1. 選択したセル番号を保存する変数を追加する。
var myIndexPath: IndexPath! // 宣言

2. cellForItemAtの中
if indexArray != nil {
if indexArray == indexPath {
cell.imageView.backgroundColor = .red
}
}

3. didSelectItemAtの中
myIndexPath = indexPath
collectionView.reloadData()

を追記しましたが、
タップしていくたびにタップした場所と違う場所まで赤になってしまいます。

まだ不足している部分があると思うのですが、
残念ながら思い付きません。

もう一度ご教授頂けないでしょうか。

よろしくお願いします。

[fuzzball]

「選択しているセルを赤」にするだけではなく、「選択していないセルを白」にしないといけません。そうしないと、一度赤くなったセルは永遠に赤いままです。（セルは使い回されるということを忘れずに）

[rookie]

ご返答ありがとうございます。

なるほど！
適切なアドバイスありがとうございました！！

他にしたい処理に関してもこちらで応用出来そうです。

本当にありがとうございました。
また質問する事があると思いますので、
その時は是非よろしくお願い致します。

Answer 2

非選択時のデリゲートを実装すればできますよ

    func collectionView(_ collectionView: UICollectionView, didDeselectItemAt indexPath: IndexPath) {
        let cell = collectionView.cellForItem(at: indexPath) as! CustomUICollectionViewCell
        cell.imageView.backgroundColor = .white
    }

Comments

[rookie]

ご回答ありがとうございます。

確かにこちらで選択したセルは赤、
他は白になりました。

ただ他の処理等の応用を考えて、
申し訳ありませんが今回はfuzzball様のご回答をベントアンサーとさせて頂きます。

本当にありがとうございました。