swift tableViewのセル選択時の動作について

Question

セルにはあらかじめ配列partに入れたstring型の文字を表示しています。
この時セルを選択した時にそのセルの文字を配列passの文字に変えたいのですがこのプログラムではできませんでした。
また再度セルを選択した時にその文字がpassならpartをpartならpassを表示するようにコードを記述しました

func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
let cell:UITableViewCell=tableView.dequeueReusableCell(withIdentifier: "MyCell", for: indexPath)
if cell.textLabel!.text==part[indexPath.row]{
cell.textLabel!.text=pass[indexPath.row]
}else{
cell.textLabel!.text=part[indexPath.row]
}
}

セル選択時に何も起こらずただセルがグレーになるだけでした
解決方法を教えてくださいませんか

Answer 1

dequeueReusableCell()で返ってくるセルは表示されているセルではありません。表示を更新するために画面に表示中のセルを取得するにはcellForRow(at:)メソッドを使用します。

override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
    let cell = tableView.cellForRow(at: indexPath)
    cell?.textLabel?.text = ...
}

ただし、ここで変更した内容は、このセルが画面外に出てセルが再利用されるときに失われますので、その場合でも状態を保持したい場合は、選択したという状態を保持して、func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCellメソッドで返すセルもその状態を反映する必要があります。

Comments

[comet0310]

解決しました！
ありがとうございます！

Answer 2

選択した時の状態を保持したかったので、Structの配列でモデルを作成しました。
セルを選択した時に呼ばれるデリゲートメソッドdidSelectRowAtでモデルの選択状態を変更して リロードすると文字が変わります。

import UIKit

class ViewController: UIViewController, UITableViewDelegate, UITableViewDataSource {
    
    @IBOutlet weak var tableView: UITableView!
    // Data Array
    var dataArray: Array<DataModel> = []
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        // データを作成
        createData()
        
        tableView.estimatedRowHeight = 44
        tableView.rowHeight = UITableViewAutomaticDimension
    }
    
    func createData() {
        for i in 0...10 {
            dataArray.append(DataModel(part: "part -- \(String(i))", pass: "pass ** \(String(i))"))
        }
    }
    
    // MARK: - TableView Delegate & DataSource
    
    // Row Count
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return dataArray.count
    }
    
    // Generate Cell
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath)
        let data = dataArray[indexPath.row]
        
        // データの選択状態でpartかpassの文字列を出し分ける
        cell.textLabel?.text = data.isSelectPass ? data.pass : data.part
        return cell
    }
    
    // Select Cell
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        tableView.deselectRow(at: indexPath as IndexPath, animated: true)
        
        // データの選択状態を変更してテーブルをリロード
        dataArray[indexPath.row].selectChange()
        tableView.reloadData()
    }
}


// partとpassを持つデータ
struct DataModel {
    
    var part: String!
    var pass: String!
    var isSelectPass = false
    
    init(part: String = "", pass: String = "", isSelectPass: Bool = false) {
        self.part = part
        self.pass = pass
        self.isSelectPass = isSelectPass
    }
    
    mutating func selectChange() {
        self.isSelectPass = !self.isSelectPass
    }
}

Comments

[comet0310]

すごい丁寧にありがとうございます。
構造体を使ったやり方はとても参考になります。
ちなみに私の書いたコードでは何がいけなかったかを教えてくださいませんか?
テーブルをリロードしないと変化しないということでしょうか？

[_Kentarou]

セルは使いまわされているということはご存知ですか？下にスクロールした時に上に消えたセルが下から使いまわされて出てきます。
cellForRowAtのメソッドの中で以下の文でセルのラベルのテキストを表示してみてください。
print(cell.textLabel?.text)

そうするとセルが表示された時に前の文字と違う文字のセルが渡されてくることが分かると思います。

なので記載されているコードでは正確な判定ができないということです。

[comet0310]

なるほど、、、
参考になります！
詳しい解説ありがとうございます