SwiftでtodoListアプリ

Question

前提・実現したいこと

Swiftでtodoリストのアプリ開発をしています。
＋ボタンをタップしたらtodoを入力します。OKボタンでTable View Cellに追加して表示するところまではできました。
そこに、todoにチェックマークをつけることと、UserDefaultを使用したデータ保存機能を実装しようとしています。
独自クラスMyToDoを作成し、viewDidRoadで保存しているtodoのデータを読み取るようにしました。
アプリ自体は起動するのですが、todoを保存することができず、アプリが落ちてしまいます。

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

MyToDoList[28985:2219996] [User Defaults] Attempt to set a non-property-list object (
"<MyToDoList.MyToDo: 0x6000027f4080>"
) as an NSUserDefaults/CFPreferences value for key todoList

ソースコード

import UIKit

class ViewController: UIViewController, UITableViewDataSource, UITableViewDelegate {
    
    var todoList = [MyToDo]()
    // UITableViewに対して操作を行うため
    @IBOutlet weak var tableView: UITableView!
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return self.todoList.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "todoCell", for: indexPath)
        let myTodo = todoList[indexPath.row]
        cell.textLabel?.text = myTodo.todoTitle
        if myTodo.todoDone {
            cell.accessoryType = UITableViewCell.AccessoryType.checkmark
        } else {
            cell.accessoryType = UITableViewCell.AccessoryType.none
        }
        return cell
    }
    

    override func viewDidLoad() {
        super.viewDidLoad()
        
        let userDefaults = UserDefaults.standard
        if let storedToDoList = userDefaults.object(forKey: "todoList") as? Data {
            if let unarchiveToDoList = try! NSKeyedUnarchiver.unarchiveTopLevelObjectWithData(storedToDoList) as? MyToDo {
                todoList.append(unarchiveToDoList)
            }
        }
    }

    @IBAction func tapAddButton(_ sender: Any) {
        // アラートダイアログを生成
        let alertControllor = UIAlertController(title: "ToDo追加", message: "ToDoを追加してください", preferredStyle: UIAlertController.Style.alert)
        // テキストエリアを追加
        alertControllor.addTextField(configurationHandler: nil)
        // OKボタンを生成
        let okAction = UIAlertAction(title: "OK", style: UIAlertAction.Style.default) { (action: UIAlertAction) in
           // OKボタンがタップされた時の処理
            if let textField = alertControllor.textFields?.first {
                // ToDoの配列に入力値を先頭に格納
                let myTodo = MyToDo()
                myTodo.todoTitle = textField.text!
                self.todoList.insert(myTodo, at: 0)
                
                // テーブルに行が追加されたことをテーブルに通知
                self.tableView.insertRows(at: [IndexPath(row: 0, section: 0)], with: UITableView.RowAnimation.right)
                
                // ToDoの保存処理
                let userDefaults = UserDefaults.standard
                userDefaults.set(self.todoList, forKey: "todoList")
                userDefaults.synchronize()
            }
        }
        // OKボタンを追加
        alertControllor.addAction(okAction)
        
        // CANSELボタンがタップされた時の処理
        let cancelButton = UIAlertAction(title: "CANCEL", style: UIAlertAction.Style.cancel, handler: nil)
        // CANSELボタンを追加
        alertControllor.addAction(cancelButton)
        
        // アラートダイアログを表示
        present(alertControllor, animated: true, completion: nil)
    }

}


class MyToDo: NSObject, NSCoding {
    
    // ToDoのタイトル
    var todoTitle: String?
    // ToDoを完了したかを表すフラグ
    var todoDone: Bool = false
    override init(){
       
    }
    
    // NSCodingプロトコルに宣言されているデシリアライズ処理
    // decode
    required init?(coder aDecoder: NSCoder) {
        todoTitle = (aDecoder.decodeObject(forKey: "todoTitle") as? String)!
        todoDone = aDecoder.decodeBool(forKey: "todoDone")
    }
    
    //encode
    func encode(with aCoder: NSCoder) {
        aCoder.encode(todoTitle, forKey: "todoTitle")
        aCoder.encode(todoDone, forKey: "todoDone")
    }
}

Comments

[hameji]

コードはスコープ（{ } のこと）ごとにインデントの位置を揃え、
「```」を先頭と末尾に行を変えてつけていただけるとコードがみやすくなります。
https://teratail.com/help/question-tips#questionTips3-3-1 の アドバイスのとこ見てください

[o2ka_cz]

アドバイスありがとうございます。
変更してみたので確認よろしくお願いします。

[hameji]

どの段階で落ちるのでしょうか？
tapAddButtonでAlertのOKを押した時ですか？

[o2ka_cz]

はい、そのときに落ちてしまいます。

Answer 1

追記・修正欄でも聞きましたが、
AlertのOKの時に落ちるとの事でしたね。
3つのことを行っている(①新規MyToDoの作成、②tableViewの表示更新、③データuserdefaultsに保存)ので、
それぞれを順にコメントアウト (該当部分を選択状態にし「command」+「/」) し、実行してどの部分が間違っているか確認してください。

おそらく、というかuserdefaultsの保存の仕方だと思います。
https://hitoto28.hatenablog.com/entry/2018/07/09/020000
を参考にされた？かもしれませんが、保存のところで、

// シリアライズ処理
let archiveData = NSKeyedArchiver.archivedData(withRootObject: sampleData)

ということが行われています。
それが抜けているのではないでしょうか？

Comments

[o2ka_cz]

シリアライズ処理を追加したらアプリが落ちなくなりました！
しかし、アプリを再起動したらデータが全て消えていました。
override func viewDidLoad() {
super.viewDidLoad()

let userDefaults = UserDefaults.standard
if let storedToDoList = userDefaults.object(forKey: "todoList") as? Data {
if let unarchiveToDoList = try! NSKeyedUnarchiver.unarchiveTopLevelObjectWithData(storedToDoList) as? MyToDo {
self.todoList.append(unarchiveToDoList)
}
}
}

データを保存できているのかがわかりません。

[hameji]

違う質問になっていますが、、、、
別に立て直してください