[swift4]Cellを削除してから前の画面に戻ると削除したはずのCellが復活している。

Question

#質問したいこと
Todoリストで追加したものをスワイプで消す→前の画面に戻る→もう一度Todoリストの画面に戻る→消したはずのcellが復活している
#コード

import UIKit

//classの継承を追加
class ViewController: UIViewController{
    
    //テキストフィールドの設定
    @IBOutlet weak var TodoTextField: UITextField!
    
    @IBOutlet weak var tableView: UITableView!
    
    // アイテムの型
    class Item {
        var title : String
        var done: Bool = false
        
        init(title: String) {
            self.title = title
        }
    }
    // この配列に作ったアイテムを追加していく
    var itemArray: [Item] = []
    
    var todoKobetsunonakami = [String]()
    
    //最初からあるコード
    override func viewDidLoad() {
        super.viewDidLoad()
        
        //追加画面で入力した内容を取得する
                if UserDefaults.standard.object(forKey: "TodoList") != nil {
                    todoKobetsunonakami = UserDefaults.standard.object(forKey: "TodoList") as! [String]
                }
        
        
        if let todos = UserDefaults.standard.object(forKey: "TodoList") as? [String] {
            todoKobetsunonakami = todos
        }
    }
    
    //最初からあるコード
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
    }
    
    //追加ボタンの設定
    @IBAction func TodoAddButten(_ sender: Any) {
        //変数に入力内容を入れる
        todoKobetsunonakami.append("Todo(todoKobetsunonakami.count)")
        
        //追加ボタンを押したらフィールドを空にする
        TodoTextField.text = ""
        
        //変数の中身をUDに追加
        UserDefaults.standard.set(todoKobetsunonakami, forKey: "TodoList" )
        
        
        tableView.reloadData()
    }
}

extension ViewController: UITableViewDelegate, UITableViewDataSource {
    
    //UITableView、numberOfRowsInSectionの追加(表示するcell数を決める)
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        //戻り値の設定(表示するcell数)
        return todoKobetsunonakami.count
    }
    
    //UITableView、cellForRowAtの追加(表示するcellの中身を決める)
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        //変数を作る
        let TodoCell : UITableViewCell = tableView.dequeueReusableCell(withIdentifier: "TodoCell", for: indexPath)
        //変数の中身を作る
        TodoCell.textLabel?.text = todoKobetsunonakami[indexPath.row]
        //戻り値の設定（表示する中身)
        return TodoCell
    }
    
    //セルの編集許可
    func tableView(_ tableView: UITableView, canEditRowAt indexPath: IndexPath) -> Bool {
        return true
    }
    
    //スワイプしたセルを削除
    func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCell.EditingStyle, forRowAt indexPath: IndexPath) {
        if editingStyle == UITableViewCell.EditingStyle.delete {
            
            //itemArray.remove(at: indexPath.row)
            todoKobetsunonakami.remove(at: indexPath.row)
            tableView.deleteRows(at: [indexPath as IndexPath], with: UITableView.RowAnimation.automatic)
        }
    }
}

よろしくお願いします。

Answer 1

Todoを消したときも

UserDefaults.standard.set(todoKobetsunonakami, forKey: "TodoList" )

のように保存をしてください。

Answer 2

削除した後の保存がないからですね。
セルの削除の部分に以下を追加してください。
ちゃんと確認していなくて申し訳ありませんが、これでできると思います。

//スワイプしたセルを削除
func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCell.EditingStyle, forRowAt indexPath: IndexPath) {
        if editingStyle == UITableViewCell.EditingStyle.delete {

            //itemArray.remove(at: indexPath.row)
            todoKobetsunonakami.remove(at: indexPath.row)
            
            // todokebetsunonakamiが更新されたので保存
            UserDefaults.standard.set(todoKobetsunonakami, forKey: "TodoList" )


            tableView.deleteRows(at: [indexPath as IndexPath], with: UITableView.RowAnimation.automatic)
        }
    }

あとUserDefaultsから読み込みの処理が二重になっているので片方消してください。

//最初からあるコード
    override func viewDidLoad() {
        super.viewDidLoad()

        //追加画面で入力した内容を取得する
        if let todos = UserDefaults.standard.object(forKey: "TodoList") as? [String] {
            todoKobetsunonakami = todos
        }
    }

Comments

[Kaguya_4869]

ありがとうございます！
できました！