ToDoアプリのチェックマークが外れてしまいます。

Question

ToDoアプリを作っていてCellをタップするとチェックマークが付くという仕組みにしたく、TaskクラスのisCompletedでチェックマークが付いているか付いていないかを確認し、それをUserDefaultで保存しているのですが、チェックをつけて起動し直すとチェックマークが外れてしまいます。全てでは無く残っているものもあるのですが何故このようなことが起きるのでしょうか？

import UIKit

class ViewController: UIViewController, UITableViewDataSource, UITableViewDelegate {

    @IBOutlet weak var tableView: UITableView!
    
    // taskの配列
    var tasks =  [Task]()
    
    override func viewDidLoad() {
          super.viewDidLoad()
          tableView.delegate = self
          tableView.dataSource = self
        if let str = UserDefaults.standard.data(forKey: "タスク") {
            let tasks = try! JSONDecoder().decode([Task].self, from: str)
            self.tasks = tasks
        }
      }
    
    @IBAction func addAction(_ sender: UIBarButtonItem) {
        let alertController = UIAlertController(title: "タスクを追加してください", message: "", preferredStyle: .alert)
        let okAlert = UIAlertAction(title: "追加", style: .default) { (action) in
            let textFiels = alertController.textFields![0]
            let tasktitle = textFiels.text
            let task = Task()
            task.title = tasktitle!
            self.tasks.append(task)
            self.tableView.reloadData()
            self.save()
        }
        let cancelAlert = UIAlertAction(title: "キャンセル", style: .cancel, handler: nil)
        alertController.addTextField { (textField: UITextField) -> Void in
            textField.placeholder = "テキスト"
        }
        alertController.addAction(okAlert)
        alertController.addAction(cancelAlert)
        present(alertController, animated: true, completion: nil)
    }
    
    func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCell.EditingStyle, forRowAt indexPath: IndexPath) {
        if editingStyle == .delete {
            tasks.remove(at: indexPath.row)
            tableView.deleteRows(at: [indexPath], with: .fade)
            save()
        }
    }
    
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        let Select = tableView.cellForRow(at: indexPath)
        if tasks[indexPath.row].isCompleted == false {
            Select?.accessoryType = .checkmark
            tasks[indexPath.row].isCompleted = true
        } else {
            Select?.accessoryType = .none
            tasks[indexPath.row].isCompleted = false
        }
        save()
    }
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return tasks.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
      let cell = tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath)
      cell.textLabel?.text = tasks[indexPath.row].title
        print(tasks[indexPath.row])
      
      if tasks[indexPath.row].isCompleted == true{
          cell.accessoryType = .checkmark
    
      } else {
        cell.accessoryType = .none
      }
        return cell
        
    }
    
    
    func save() {
        let userDefaults = UserDefaults.standard
        let data  = try! JSONEncoder().encode(self.tasks)
        userDefaults.set(data,forKey: "タスク")
        userDefaults.synchronize()
       }

}

Taskクラスのコードです。

import Foundation

class Task:Codable {
    // タスクのタイトル
    var title: String = ""
    // 完了したかどうか
    var isCompleted: Bool = false
}

Comments

[退会済みユーザー]

解決済みのようですが、
チェックマークがずれると気分が悪いでしょうから、面倒くさいですが正常な手順でタスクキルをしてからRunするようにしてみてください。

[taiseiswift]

追記ありがとうございます。
試してみます。

Answer 1

推測ですが、userDefaults.set(:forKey:)を呼んで保存する処理と、その内容を plist として実際にディスクに保存する処理のタイムラグだと思います。

通常の操作であれば、ディスクに保存するタイミングまで意識する必要はないと思います。

一方、Xcode から Run でアプリを走らせた場合、以前走らせていたアプリの UserDefaults はディスク上には反映されないことがあるようで、特にこの場合は以前のデータが保存されていないことは確認しました。

ちなみに、シミュレータであれば/Users/[Username]/Library/Developer/CoreSimulator/Devices/以下の該当するファイル内に plist として UserDefaults の内容が保存されているみたいですが、やはりアプリで set() するタイミングと実ファイルに反映されるタイミングは異なっていました。

ところで、

        userDefaults.synchronize()

上記のメソッドの利用は非推奨となっています。Deprecatedではありませんが、マニュアルを見ると

this method is unnecessary and shouldn't be used.

となっていますので、明示的に記述する必要はないと思います。

• Instance Method synchronize()
• UserDefaultsで保存したTextFieldの入力内容が消えてしまう

以前はこのメソッドを使って書き出しを強制できていたのだと思いますが、いまは完全に OS に任せることになっていると推測されます。

Comments

[taiseiswift]

勉強になりました。
ありがとうございました！