Swiftでデータ保存機能(ToDoList)

前提・実現したいこと

SwiftでToDoリストのアプリを開発しています。
画面の＋ボタンをタップすると、ToDoを追加するアラートが表示されます。
OKボタンで保存し、アプリを再起動すると保存したデータが表示されるようにしたいです。

発生している問題

データを保存して、アプリ再起動時に保存してあるデータを読み取る処理を書いたのですが、データが消えてしまいます。
データが保存されていないのか、読み取りができていないのかがわかりません。

該当のソースコード

//
//  ViewController.swift
//  MyToDoList
//

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 {
                self.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
                let archiveData = try! NSKeyedArchiver.archivedData(withRootObject: self.todoList, requiringSecureCoding: false)
                // userDefaults.set(self.todoList, forKey: "todoList")
                userDefaults.set(archiveData, 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")
    }
}

行動規範の内容に同意します

回答1件

ベストアンサー

printなどを用いて検証するのが、バグの修正では常套手段です。
今回の場合は2点あり、
①保存されてるかどうか、②表示ように変換ができているかが検証点ですね。

コードの意味を理解しているなら、どこに書けばいいか考えれば、
すぐわかると思います。

検証方法の答えを示してしまうと、

Swift
1override func viewDidLoad() {
2    super.viewDidLoad()
3
4    let userDefaults = UserDefaults.standard
5    if let storedToDoList = userDefaults.object(forKey: "todoList") as? Data { // userDefaultsにtodoListキーで保存されているかどうかなので、
6        if let unarchiveToDoList = try! NSKeyedUnarchiver.unarchiveTopLevelObjectWithData(storedToDoList) as? MyToDo { // 同様に、DataからMyToDoに変換し、nil(失敗)しないでデータを得た場合
7            self.todoList.append(unarchiveToDoList)
8        } else {
9            print("Dataの変換に失敗しています。")
10        }
11    } else {
12        print("todoListキーでdataが保存されていません。")
13    }
14}

また、変換前のデータを直接print()するのもヒントになるかもしれません。

Swift
1print(storedToDoList)
2
3// as? MyToDoと型変換せずに、printしてみる
4let unarchiveToDoList = try! NSKeyedUnarchiver.unarchiveTopLevelObjectWithData(storedToDoList)
5print(unarchiveToDoList)

よくコードの中身を検証していませんが、
根本的な問題点としては、
MyToDoに変換しているのがおかしい気がします。

保存する時は self.todoList を保存しているので、
var todoList = MyToDoだから、、つまり、MyToDo 1個ではなく、
MyToDoの配列に変換すべきなんじゃないですか？？？
なので、appendも不要で、直接MyToDoの配列に変換し、
代入するすべきだと思います。

よくコードの中身を追って、順に考え、整合性を考えてみてください。

投稿2019/12/10 06:32