Состояние UISwitch в Tableviewcell сбрасывается при прокрутке пользователя - Swift

Question

Я искал решения по этому вопросу, но ни один из них не подходит для моего варианта использования.

У меня есть таблица внутри viewcontroller, и проблема, с которой я сталкиваюсь, заключается в том, что при прокрутке состояние UISwitch сбрасывается в OFF.Я понимаю, что ячейки таблицы используются повторно, но как мне реализовать решение, которое восстановит состояние UISwitch, когда пользователь прокручивает на основе моего кода ниже

import UIKit

class StirrViewController: UIViewController, UITableViewDelegate, UITableViewDataSource{

@IBOutlet weak var mylabel: UILabel!
var myString = String()

@IBAction func stirrBtn(_ sender: AnyObject) {
}

var timeSelected = String()
var selectedTimeArr = [String]()

override func viewDidLoad() {
    super.viewDidLoad()
    mylabel.text = myString
    self.timeSelected = myString
}

func switchChanged(_ sender : UISwitch!){
    print("table row switch Changed \(sender.tag)")
    print("The switch is \(sender.isOn ? "ON" : "OFF")")
    let kValue = (sender.tag + 1)
    let keyValue = String(kValue)
    if sender.isOn {
        recipeSettings.boolStirrSwitch[keyValue] = true
        recipeSettings.switchedOnArr.append(keyValue)

    } else {
        recipeSettings.boolStirrSwitch[keyValue] = false
    }
}

public func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int
{
    let stringNum = Int(self.timeSelected)
    recipeSettings.recipeTimeSet2 = stringNum!
    return(stringNum)!
}


public func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell
{
    var cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath) as! UITableViewCell
    //here is programatically switch make to the table view
    let switchView = UISwitch(frame: .zero)
    switchView.setOn(false, animated: true)
    switchView.tag = indexPath.row // for detect which row switch Changed
    switchView.addTarget(self, action: #selector(self.switchChanged(_:)), for: .valueChanged)
    cell.accessoryView = switchView

    // Process data displayed in rows(minutes)
    let endTime = Int(self.timeSelected)
    let startTime = Int(1)

    // Recipe time array
    let timeArray: [Int]  = Array(startTime...endTime!)
    let stringTimeArr = timeArray.map{String($0)}

    // Save time array to global variable
    recipeSettings.recipeTimeSetArr = stringTimeArr

    // Create a boolean Array to hold all default false booleans
    let defBool: Bool = false
    var defBoolArr: [Bool] = []

    // Fill the array with the defaults boolean
    for _ in 0..<stringTimeArr.count{defBoolArr.append(defBool)}

    // Map the array to global dictionary containing the Time in an array and default "false" value

    for i in 0..<stringTimeArr.count {
        recipeSettings.boolStirrSwitch[stringTimeArr[i]] = defBoolArr[i]
    }

    // Add the minutes to cell table
    cell.textLabel?.text = stringTimeArr[indexPath.row]
    return cell
}

func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
}

override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
}
}

Как вы можете видеть в моем коде, я сохраняюсостояние каждого переключателя пользовательского интерфейса в словаре глобальных переменных.Как я могу решить проблему изменения состояний UISwitch на основе этого кода?Вся помощь приветствуется.Заранее спасибо

Answer 1

var switchState = [String : Bool]()

• ваш recipeSettings.boolStirrSwitch должен быть объявлен следующим образом.
• Поскольку вы используете timeSelected в качестве numberOfRowsInSection для отображения вашего cell.textLabel из этого, так что вам не нужнодополнительно stringTimeArr для этого.
• Вся обработка, которую вы выполняете в cellForRowAt, произойдет снова и снова, ячейки таблицы будут повторно использованы, поэтому для настройки данных сделайте это в другой function, а затем reload TableView.

Решение вашей проблемы должно выглядеть следующим образом.

import UIKit

class StirrViewController: UIViewController, UITableViewDelegate, UITableViewDataSource{

//make tableView IBOutlet for reloading data
@IBOutlet weak var tableView: UITableView!

@IBOutlet weak var mylabel: UILabel!
var myString = String()

@IBAction func stirrBtn(_ sender: AnyObject) {
}

var timeSelected = String()
var selectedTimeArr = [String]()

override func viewDidLoad() {
    super.viewDidLoad()
    mylabel.text = myString
    self.timeSelected = myString
    self.setdefaultSwitchState()
}

//recipeSettings.boolStirrSwitch should be decleard like that
var switchState = [String : Bool]()

//setDeaultSwitchState
func setdefaultSwitchState(){
    if let timeSelected = Int(self.timeSelected){
        for value in 0..<timeSelected{
            switchState["\(value)"] = false
            //or
            //recipeSettings.boolStirrSwitch["\(value)"] = false
        }
    }
    self.tableView.reloadData()
}


@objc func switchChanged(_ sender : UISwitch!){
    print("table row switch Changed \(sender.tag)")
    print("The switch is \(sender.isOn ? "ON" : "OFF")")
    let kValue = (sender.tag + 1)
    let keyValue = String(kValue)
    if sender.isOn {
        switchState[keyValue] = true

    } else {
        switchState[keyValue] = false
    }
}

public func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int
{
    let stringNum = Int(self.timeSelected)
    recipeSettings.recipeTimeSet2 = stringNum!
    return(stringNum)!
}


public func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell
{
    var cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath) as! UITableViewCell
    //here is programatically switch make to the table view
    let switchView = UISwitch(frame: .zero)
    switchView.setOn(false, animated: true)
    switchView.tag = indexPath.row // for detect which row switch Changed
    switchView.addTarget(self, action: #selector(self.switchChanged(_:)), for: .valueChanged)
    cell.accessoryView = switchView


    cell.textLabel?.text = "\(indexPath.row + 1)"
    if let switchState = switchState["\(indexPath.row)"] {
        if switchState{
            switchView.isOn = true
        }else{
            switchView.isOn = false
        }
    }else{
        switchView.isOn = false
    }

    return cell
}


func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
}

override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
}
}