Swift 構造体を配列にしたものを保存する方法について・tableview

Question

実現したいこと・やったこと
textfieldから入力した文字をtablviewにTODOリストの様に表示させ、
最終的には投稿時間やタグなど他の情報も扱えるように配列を含んだ構造体にし、
さらにuserDefaultsなどで保存機能を持たせようとコードを書き換えてみたのですが、
保存機能はついたものの一行だけしか保存されなかったりとうまくいかず、
他に先行例を検索してみても見つからなかったのでこちらで質問しました。

質問
以下がuserDefaultsを加える前の状態のコードなのですが、
textfieldから構造体の配列に入れた文字を保存する方法についてアドバイスお待ちしておりますm(_ _)m

import UIKit

struct Plan : Codable{
    let doing: String
    //イニシャライザ
    init(doing: String) {
        self.doing = doing
    }
}

class ViewController: UIViewController, UITableViewDataSource, UITableViewDelegate {
    //tableviewに表示させるスケジュールリスト
    var todo = [Plan]()
    
    @IBOutlet weak var textfield: UITextField!
    @IBOutlet weak var table: UITableView!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
        table.dataSource = self
        table.delegate = self    
    }

    @IBAction func add(_ sender: Any) {
        let mozi = textfield.text!
        todo.append(Plan(doing: mozi))
        table.reloadData()
    }
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return todo.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let plan = todo[indexPath.row]
        let cell = tableView.dequeueReusableCell(withIdentifier: "Cell",for: indexPath)
        cell.textLabel?.text = plan.doing
        return cell
    }
}

8/30　追記
似たような質問の回答を下に自分なりに修正してみたのですが、またしても配列に一行しかappendされないという事象が起き、
保存はうまくいっても、配列として保存ができないという結果です。

import UIKit

struct Plan : Codable{
    let doing: String
    //イニシャライザ
    init(doing: String) {
        self.doing = doing
}
}

class ViewController: UIViewController, UITableViewDataSource, UITableViewDelegate {
    //tableviewに表示させるスケジュールリスト
    var todo = [Plan]()
    
    var readArray = [Plan]()
    
   
    
    @IBOutlet weak var textfield: UITextField!
    @IBOutlet weak var table: UITableView!
    
    
    
    

    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
        table.dataSource = self
        table.delegate = self
        
        readArray = Plan.readGoals()
        

        
    }

    @IBAction func add(_ sender: Any) {
        let mozi = textfield.text!
        //todo.append(Plan(doing: mozi))
        
        var saveArray: [Plan] = []
        saveArray.append(Plan(doing: mozi))

        // goal配列をUserDefaultsに保存
        Plan.saveGoals(Plan: saveArray)
        table.reloadData()
        
        
        
        print(readArray[0].doing) //ここまでは保存されているようです
        print(readArray[1].doing) //わずか２個目からThread 1: Fatal error: Index out of rangeという警告文がつきました。
        print(readArray[2].doing) 
    }
    
    
    
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        let readArray = Plan.readGoals()
        let aaa = readArray
        return aaa.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        
        
        
        
        let plan = readArray[indexPath.row]
        let cell = tableView.dequeueReusableCell(withIdentifier: "Cell",for: indexPath)
        cell.textLabel?.text = plan.doing
        return cell
    }
    
}

extension Plan {
    static let userDefaultGoalKey = "goalSaveKey"

    static func saveGoals(Plan: [Plan]) {
        let defaults = UserDefaults.standard
        defaults.set(Plan.map{ $0.doing }, forKey: userDefaultGoalKey)
        defaults.synchronize()
    }

    static func readGoals() -> [Plan] {
        let defaults = UserDefaults.standard
        if let goalArray = defaults.object(forKey: userDefaultGoalKey) as? [String] {
            return goalArray.map{ Plan(doing: $0) }
        }
        return []
    }
}

Answer 1

UserDefaultsで配列を保存するだけでいいですかね？
投稿時間やタグなども構造体に追加すれば大丈夫だと思います。

private let todoListKey = "TodoListKey"
var todo = [Plan]()

override func viewDidLoad() {
    super.viewDidLoad()

    // 省略

    // 画面表示時にUserDefaultsからTODOリストを取得
    if let data = UserDefaults.standard.array(forKey: todoListKey) as? [Data] {
        self.todo = data.map { try! JSONDecoder().decode(Plan.self, from: $0) }
        }
}

@IBAction func add(_ sender: Any) {
    let mozi = textfield.text!
    todo.append(Plan(doing: mozi))

    // ローカルの配列に追加してUserDefaultsに保存
    // Data型で保存(Codable使っているので下記のようにエンコードできます)
    let data = self.todo.map { try? JSONEncoder().encode($0) }
    UserDefaults.standard.set(data, forKey: todoListKey)

    table.reloadData()
}

Comments

[samson66]

回答ありがとうございます！
上記の方法で修正してみたのですが、テキストフィールドから文字を打ち込むと「[User Defaults] Attempt to set a non-property-list object ("プロジェクト名.Plan(doing: \"打ち込んだ文字\")"」というエラーが発生していましました。

[TakuyaAso]

Data型でCodable使えるのでエンコードして保存，デコードするように変更しました

[samson66]

ありがとうございます！無事に動きました！
まだ保存の書き方に慣れていないので今回のソースをよく振り返ろうと思います。

[TakuyaAso]

おお良かったです！
サンプルとか簡単な検証にはUserDefaultsは便利ですが
実際はRealmなどのDataBaseに保存した方がいいので
お時間あったら挑戦してみてくださいー