[Swift4]cellとcellの間に空白のcellが入ってしまう。

Question

#質問したいこと
うまく伝えられないのですが…
Todoリストを作っていて、保存して表示したcellとcellの間にいらない空白のcellが出てきてしまいます。画像を見て頂くと分かるかと思います。
#コード

import UIKit
var TodoKobetsunonakami = [String]()

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

#やってみたこと
ホームページなどで検索してみて、そのコードを書いてみました。
なぜか空白ができてしまうので、その解決法を教えて下さい！！

Comments

[rcftdbeu]

空文字で登録したデータが残っていませんか？
作成したアプリを再インストールした場合も同じように表示されますか？

[Kaguya_4869]

作成したアプリを再インストールした場合も同じように表示されてしまいます。

[rcftdbeu]

念のための確認ですが，シミュレータ上でアプリを削除してからインストールしているでしょうか？
(UserDefaultを初期化しても再現するでしょうか?)

[Kaguya_4869]

アプリを削除してからやっています。

[退会済みユーザー]

func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
//戻り値の設定(表示するcell数)
print("TodoKobetsunonakami.count:", TodoKobetsunonakami.count) // <= これ
return TodoKobetsunonakami.count
}

「これ」の部分を足して実行するとコンソールにどう表示されますか？

[fuzzball]

空白のセルは全て「いらない空白」ってことでしょうか？（「いる」「いらない」は、こちらでは判断できませんので‥）

[fuzzball]

>>2019/07/21 18:31
>>作成したアプリを再インストールした場合も同じように表示されてしまいます。
再インストールしたら、何も表示されなくなるはずですが？

>>2019/07/21 20:45
>>アプリを削除してからやっています
「削除」ってアンインストールってことでいいですか？
アプリを終了させてるだけじゃないですよね？

[fuzzball]

（上のコメントに統合）

[Kaguya_4869]

アプリをシュミレーター上から消してもう一度動かしています。

[Kaguya_4869]

空白のcellは全ていらないです

[fuzzball]

私のコメントの
>>再インストールしたら、何も表示されなくなるはずですが？
これにも返信お願いします。

>>アプリをシュミレーター上から消してもう一度動かしています。
だから「消して」っていうのはアンインストールのことですか？（アイコン長押しの後✕ボタン）
https://support.apple.com/ja-jp/HT207618

[Kaguya_4869]

>>再インストールしたら何も表示されなくなるはずですが？
何も表示されません。
>>「消して」というのはアンインストールのことですか？
そうです。アンインストールのことです。

[fuzzball]

>>何も表示されません。
じゃあ何が問題なのでしょうか？

[Kaguya_4869]

どんどん追加していくと、いらない空白のcellが増えていきます。

[rcftdbeu]

実際にこちらの環境で動作させたわけではありませんが，コードを見る限り，自分で空白のToDoを登録する操作以外に空白のCellが増えるような箇所は見つけられません．
以下の2点を確認してみてください．
・func TodoAddButtenにブレークポイントを張って，意図したタイミングで呼ばれているか．
・他のコードでUserDefaults内のキーTodoListに変更を加える箇所はないか．

Answer 1

貼っていただいたコードをそのままコピペして動かしてみたところ、
いくつかエラーが出たので修正しました。

空白のセルが入るということは確認できなかったため、
その原因が何なのかを説明することはできませんが、期待している動作にはなっていると思います。

TableView関連の設定はコードでやっていないみたいだったのでStoryboard上で行っています。

またTodoTextFieldが質問に貼っていただいたスクショには写っていなかったので、
とりあえずコメントアウトしています。

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 の意味がなかったので修正
        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のリロードがなかった
        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が使われていないので Index out of range になってクラッシュ
            //itemArray.remove(at: indexPath.row)
            todoKobetsunonakami.remove(at: indexPath.row)
            tableView.deleteRows(at: [indexPath as IndexPath], with: UITableView.RowAnimation.automatic)
        }
    }
}

Comments

[Kaguya_4869]

ありがとうございます。
extension ViewController: UITableViewDelegate, UITableViewDataSource
のところでエラーが２つ出てしまっていて、
１．Redundant conformance of 'ViewController' to protocol 'UITableViewDataSource'
２．Redundant conformance of 'ViewController' to protocol 'UITableViewDelegate'
というエラーなんですがどのように解消すれば良いでしょうか？

[hayabusabusash]

UITableViewDataSourceとUITableViewDelegateの継承が二箇所で書かれているかもしれません。
ViewControllerの継承が以下のようになっているかもう一度確認していただけないでしょうか？
class ViewController: UIViewController {
// ....
}

[Kaguya_4869]

すごいですね！！
UITableViewDataSourceとUITableViewDelegateの継承が二箇所で書かれていたので、class ViewController: UIViewControllerに直したら、エラーが消えました！！
本当にありがとうございます！！

[hayabusabusash]

????