TableViewCellにAccessoryViewで追加したUISwitchの動作をセルごとに記述したい。

Question

前提・実現したいこと

swiftでトレーニングメニューを表示するアプリを作っています。
ViewTableCellにUISwitchをAccessoryViewで追加しています。
Switchをonにすると該当の行の種目名と紐づいたfavos[i].favに1が代入されます。offにすると0です。変数はjsonファイルを読み書きし保存しています。
画像のページから前に戻り,またこのページを生成した時その変数(favos[i].fav)が0か1かによって各セルのUISwitchのon,offを変えて表示したいです。

発生している問題・エラーメッセージ

どのSwitchが何行目のセルに属しているのか判別できず,すべてのSwitchがonになってしまいます。
Switchのon,offを個別に設定する方法を教えて欲しいです。

該当のソースコード

/*TableViewCellの描画部分*/
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell: UITableViewCell! = tableView.dequeueReusableCell(withIdentifier: "events")
        let item = items.filter{$0.part == tappedPart}[indexPath.row]//押されたボタンの部位の配列の中身をitemに代入する。
        cell.textLabel?.text=item.name
        let Switch = UISwitch()
        cell.accessoryView = Switch
        Switch.tag = indexPath.row
        Switch.addTarget(self, action: #selector(pushedSwitch(_:)), for: UIControl.Event.valueChanged)
        
        for i in 0..<favos.count{
            if(favos[i].fav == 1){
                    Switch.isOn = true
            }
        }
        return cell
    }
/*Switchが押された時の処理*/
    @objc func pushedSwitch(_ sender: UISwitch){
        //print("indexPathRow")
        //print(sender.isOn)
        //print(sender.tag)
        let item = items.filter{$0.part == tappedPart}[sender.tag]
        if (sender.isOn == true){
            print(item.name)
            for i in 0..<favos.count{
                print(favos[i].name)
                if(favos[i].name == item.name){
                    favos[i].fav = 1
                    print(favos[i])
                    break
                }
                if(i == favos.count - 1){
                    let name = item.name
                    favos.append(Favorite(name: name ,fav: 1))
                    print(favos)
                }
            }
        }else{
            for i in 0..<favos.count{
                if(favos[i].name == item.name){
                    favos[i].fav = 0
                    print(favos[i])
                }
            }
        }
        
    }
/*ファイルの保存部分*/
    override func viewWillDisappear(_ animated: Bool) {
        super.viewWillDisappear(animated)
        guard let url = Bundle.main.url(forResource: "favorite", withExtension: "json") else { fatalError("ファイルが見つかりません") }
        guard let data = try? JSONEncoder().encode(favos) else{
            fatalError("ファイル読み込みエラー")
        }
        print(data)
        do{
            try data.write(to: url)
        }catch{
            fatalError("ファイル書き込みエラー")
        }
    }

補足情報（FW/ツールのバージョンなど）

Xcode ver.12.3
iOS 13.5
iPhone 12 Pro Max
ここにより詳細な情報を記載してください。

Answer 1

全体のデータ構造が見えないのですが、「スイッチの状態 = 実施したトレーニングの種類」だと思いますので、item に相当するクラスでトレーニングの有無を管理させれば良いのではないでしょうか。

下記の例だと、AccessoryView に追加する UISwitch の tag に indexPath.row を入れてますが、Cell そのもののカスタムクラスを作り、その中で個別にクラスを保持し、そこでスイッチの状態を監視させるという方法もありますし、その場合は indexPath.row に依存しなくなるのでもっと良いかもしれません。

ここでは UserDefault でデータの永続化を行っていますし、その都合上構造体を使っていますが、その部分はご自身の設計に合わせて変更していただければと思います。

あくまでも一例ということで。

import UIKit

// スイッチの状態もプロパティとして持たせる
struct Item: Codable {
    var name: String
    var isOn = false
    
    init(name: String) {
        self.name = name
    }
}

class ViewController: UIViewController {
    // ここでは UserDefaults に逐次保存しているだけ。
    var items: [Item] {
        set {
            UserDefaults.standard.setEncoded(newValue, forKey: "array")
        }
        
        get {
            UserDefaults.standard.decodedObject([Item].self, forKey: "array") ?? []
        }
    }
        
    @IBOutlet weak var tableView: UITableView! {
        didSet {
            tableView.dataSource = self
        }
    }
    
    override func viewDidLoad() {
        super.viewDidLoad()
                
        // items が空であれば新規作成
        if items.isEmpty {
            items = (1...30).map { Item(name: String($0)) }
        }
    }
}

extension ViewController: UITableViewDataSource {
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return items.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath)
        
        cell.textLabel?.text = items[indexPath.row].name
        
        let sw = UISwitch()
        sw.isOn = items[indexPath.row].isOn
        sw.addTarget(self, action: #selector(switchChanged(_:)), for: .valueChanged)
        sw.tag = indexPath.row
        
        cell.accessoryView = sw
        
        return cell
    }
    
    // Switch につけた tag で管理する
    @objc func switchChanged(_ sender: UISwitch) {
        items[sender.tag].isOn = sender.isOn
    }
}

//https://qiita.com/masakihori/items/7a0ed9d109800d714c0b
extension UserDefaults {
  func setEncoded<T: Encodable>(_ value: T, forKey key: String) {
    guard let data = try? JSONEncoder().encode(value) else {
       print("Can not Encode to JSON.")
       return
    }

    set(data, forKey: key)
  }

  func decodedObject<T: Decodable>(_ type: T.Type, forKey key: String) -> T? {
    guard let data = data(forKey: key) else {
      return nil
    }

    return try? JSONDecoder().decode(type, from: data)
  }
}

Comments

[zepi1395]

返信が遅くなり大変申し訳ありません。
ご教授いただいたコードを参考に試行してみたところ、
前のページに戻っても正常に描画されるようになりました。
ありがとうございました。