UICollectionCellViewをタップ時に色が変わるようにしたい

Question

cellをタップした時にそのcellの色を変わるようにしたんですが、うまく行かないのですがなんででしょうか。また、どう直せばいいのか教えて欲しいです。

...省略...
func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
        let cell : CustomUICollectionViewCell = collectionView.dequeueReusableCell(withReuseIdentifier: "MyCell", for: indexPath) as! CustomUICollectionViewCell
        cell.image.backgroundColor = UIColor.red
        print("Num: (indexPath.row)")
        usePhoto = indexPath.row
        
        
        
        
    }
    
    func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        return icon.count
    }
    
    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        let cell : CustomUICollectionViewCell = collectionView.dequeueReusableCell(withReuseIdentifier: "MyCell", for: indexPath) as! CustomUICollectionViewCell
        cell.image.image = icon[indexPath.row]
        return cell
    }

import UIKit

class CustomUICollectionViewCell: UICollectionViewCell {
    var  image = UIImageView()
    var myview = UIView()
    required init(coder aDecoder: NSCoder) {
        super.init(coder: aDecoder)!
    }
    
    override init(frame: CGRect) {
        super.init(frame: frame)
        
       
        image = UIImageView(frame: CGRect(x:0, y:0, width:frame.width, height:frame.height))
        myview = UIImageView(frame: CGRect(x:0, y:0, width:frame.width, height:frame.height))
        image.backgroundColor = UIColor.clear
        //myview.backgroundColor = UIColor.red
        self.contentView.addSubview(myview)
        self.contentView.addSubview(image)
    }
    

}

Comments

[fuzzball]

「うまく行かない」とは？

[退会済みユーザー]

色が変わらないです。

Answer 1

とりあえず、

cell.contentView.backgroundColor = .red

で色は変わると思います。

あとは、「セルの再利用」について調べて下さい。

セルの取得

セルの取得方法が間違っています。

if let cell = collectionView.cellForItem(at: indexPath) {
    cell.contentView.backgroundColor = .red
}

Comments

[退会済みユーザー]

変わらなかったです。

[fuzzball]

セルの取得方法について追記しました。

[退会済みユーザー]

できました。ありがとうございます。
しかし、再利用の仕組みがよくわからないです。
タップしていないセルが反応したりします。

[fuzzball]

画面外に出たセルは（画面外に出たときの状態のまま）再利用されます。
今回の場合だと、背景色が赤い状態で画面外に出たセルは、dequeueReusableCell(withReuseIdentifier:for:)でセルを取得したとき、すでに背景色が赤い状態、ということです。

これを解決するには、
・全セルの背景色の状態を保持するための変数（配列など）を用意する。
・collectionView(_:didSelectItemAt:)の中では、上記変数を変更し、セルを更新（リロード）する。
・collectionView(_:cellForItemAt:)の中では、上記変数を参照してセルの背景色を設定する。
という流れになります。

UITableViewも同じ仕組みなので、そちらで探した方が多くの記事が見つかるのではないかと思います。

[退会済みユーザー]

できました。ご丁寧にありがとうございます。