How to save checkMark status for specific UITableViewCell

Question

I have a UITableView in one UIViewController, and I try to store the status of the accessoryType so that when the user reload the app, the cells user selected beforehand using NSUserDefault should display with a checkmark. But the problem I am facing is that when I check if the cell's data is equal to the data in the user default, some cells that weren't selected are displaying with a checkmark too, but it shouldn't do that.

here is my codes:

func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    let cell = newCategoriesTableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath)
    cell.backgroundColor = colorArray[indexPath.row]
    let entry: Categories
    if isFiltering() {
        entry = filteredCategories[indexPath.row]
    } else {
        entry = categoryTitles[indexPath.row]
    }

    cell.selectionStyle = .none

    cell.textLabel?.text = entry.name
    cell.detailTextLabel?.text = entry.description

    let defaults = UserDefaults.standard
    if let userCategortList = defaults.object(forKey: "userCategoryList") as? [String]{
            for category in userCategortList{
                if(cell.textLabel?.text == category){
                    cell.accessoryType = .checkmark
                    break
                }
            }
        }

    return cell
}

score 0 · Answer 1 · answered Nov 07 '17 at 23:29

0

This is likely a cell reuse issue since you're dequeuing cells. You need to make sure to set cell.accessoryType = .none for the cases that aren't being set to the checkmark.

answered Nov 07 '17 at 23:29

raidfive

6,603
1
35
32

1

Might be good to mention that this would be often done in a `prepareForReuse` method on the custom cell (if one exists) – JMFR Nov 07 '17 at 23:32

score 0 · Answer 2 · answered Nov 08 '17 at 05:53

try with below snippet.

    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
            let cell = newCategoriesTableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath)
            cell.backgroundColor = colorArray[indexPath.row]
            let entry: Categories
            if isFiltering() {
                entry = filteredCategories[indexPath.row]
            } else {
                entry = categoryTitles[indexPath.row]
            }

            cell.selectionStyle = .none

            cell.textLabel?.text = entry.name
            cell.detailTextLabel?.text = entry.description

            let defaults = UserDefaults.standard
            if let entry.name ==  (defaults.object(forKey: "userCategoryList") as? [String]){
                  cell.accessoryType = .checkmark                    
                }
            else{
                  cell.accessoryType = .none
                }

            return cell
        }

Save entry.name in user defaults.

How to save checkMark status for specific UITableViewCell

2 Answers2