質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.58%

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

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 943

score 100

質問したいこと

うまく伝えられないのですが…
Todoリストを作っていて、保存して表示したcellと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" )
        }
}

やってみたこと

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • fuzzball

    2019/07/22 15:45

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

    キャンセル

  • Kaguya_4869

    2019/07/22 20:09

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

    キャンセル

  • rcftdbeu

    2019/07/22 20:21

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

    キャンセル

回答 1

checkベストアンサー

+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)
        }
    }
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/07/25 21:36

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

    キャンセル

  • 2019/07/25 21:41

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

    キャンセル

  • 2019/07/25 21:45

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

    キャンセル

  • 2019/07/25 21:50

    👍

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.58%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る