tableviewの削除ボタンが反応しない

Question

前提・実現したいこと

tableviewの編集モードで削除ボタン（マイナスボタン）を押すと横からdeleteが表示されて、セルを削除できるようにしたいです

発生している問題・エラーメッセージ

削除ボタン（マイナスボタン）は表示されましたが、押しても反応しません
並び替えはできるようになりました

該当のソースコード

import UIKit

class ViewController: UIViewController {

    @IBOutlet weak var tableView: UITableView!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        tableView.dataSource = self
        tableView.isEditing = true
    }
    
}

extension TabManagementViewController: UITableViewDataSource {
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return 3
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = UITableView
        cell.textLabel?.text = "サンプル"
        return cell
    }
    
// セルの削除 //
    func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCell.EditingStyle, forRowAt indexPath: IndexPath) {
        print("削除①") // 押しても表示されませんでした
        if editingStyle == .delete {
            print("削除②") // こちらも同様に押しても表示されませんでした
            tableView.deleteRows(at: [indexPath], with: .automatic)
        }
    }
    
    func tableView(_ tableView: UITableView, moveRowAt sourceIndexPath: IndexPath, to destinationIndexPath: IndexPath) {
        // 並び替え
    }
    
}

試したこと

tableView.allowsSelection = true

や

tableView.allowsSelectionDuringEditing = true

をviewDidLoad()の中に追加してみましたが、何も変わりませんでした

func tableView(_ tableView: UITableView,canEditRowAt indexPath: IndexPath) -> Bool{
        return true
}

を// セルの削除 //の前に追加しましたが、同じく何も変わりませんでした

補足情報（FW/ツールのバージョンなど）

最近、swiftの勉強を始めたばかりで質問するのも今回が初めてなので至らない点があると思いますが、どうぞよろしくお願いします

Answer 1

tableView の Delegate や DataSource の処理先が ViewController （つまりself）になっていますが、実際にプロトコル準拠しているクラスや、求められている関数を定義しているクラスが別のクラス(TabManagementViewController)になっているので意図した動きになっていないのではないでしょうか。

つまり、

class ViewController: UIViewController {

    @IBOutlet weak var tableView: UITableView!

    override func viewDidLoad() {
        super.viewDidLoad()
        // ここの設定が自身のクラスになっている
        tableView.dataSource = self
        tableView.delegate = self
　　
    }

}

としているにもかかわらず、実際にdelegate などを宣言しているのは、別のクラス（の拡張）である

extension TabManagementViewController: UITableViewDataSource {

    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return 3
    }
    /// 以下略
}

としている問題のような気がします。

selfじゃなくて、TabManagementViewController クラスのインスタンスを指定しなければいけないのではないでしょうか。

あるいは、delegate先を自分自身にするのであれば、ViewControllerクラスをUITableViewDataSourceとUITableViewDelegateプロトコルに準拠させて、次のような感じに書けばいいような気がします。

（TableViewを表示させて削除させるための簡単なサンプルです）

import UIKit

class ViewController: UIViewController, UITableViewDelegate, UITableViewDataSource {
    
    @IBOutlet weak var tableView: UITableView!
    
    var cellData = ["カクレクマノミ", "ナンヨウハギ", "キイロハギ", "ロイヤルグラマ"]
    
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
        
        tableView.delegate = self
        tableView.dataSource = self
        // 編集可能にする -> 削除アイコンを表示する
        // tableView.isEditing = false の場合はスワイプで削除
        //tableView.isEditing = true
    }
    
    // セルの個数を返す datasource
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return cellData.count
    }

    // セルにデータを表示する datasource
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "tableViewCell", for: indexPath)
        cell.textLabel!.text = cellData[indexPath.row]
        
        return cell
    }
    
    // セルの編集許可
    // この関数がない場合はデフォルトで削除可能
    // false を返すと削除できない
//    func tableView(_ tableView: UITableView, canEditRowAt indexPath: IndexPath) -> Bool {
//        return false
//    }

    // スワイプしたセルを削除
    func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCell.EditingStyle, forRowAt indexPath: IndexPath) {
        if editingStyle == .delete {
            // 実データの削除
            cellData.remove(at: indexPath.row)
            
            // 表示するデータの削除
            tableView.deleteRows(at: [indexPath], with: .automatic)
        }
    }
}

Answer 2

よく読んでませんでした。ごめんなさい。

cellForRowAtの中身ができてませんので、削除とか並び替えよりも先にきちんと表示できるようにしてみてください。
tableViewを表示するveiwcontrollerもどれなのかわかりません。

このへんでも参考にしてみてください

UITableViewDelegateにも

extension TabManagementViewController: UITableViewDataSource, UITableViewDelegate {

}

Comments

[退会済みユーザー]

回答ありがとうございます
UITableViewDelegateを加えてみましたが、同じく反応しませんでした
viewDidLoad()の中にtableView.delegate = selfは入れました

[h_crane]

問題点が２つ

①
表面的にセルだけ消しても消えないかと思います。
裏側のデータを消してからでないと更新してもセルは元のままかと思います。
（裏側消さなくても消そうと思えば消せますが）

②
今回の場合は``numberOfRowsInSection ``で3を返してしまっているので、ずっとかわらないかと、、。

適当に
``var model: [String] = ["a", "b", "c"]``

と定義して、numberOfRowsInSectionでは``model.count``を返しましょう。
その上で①を解決しましょう。

①は以下で解決。
少し古いですが、簡潔なので以下を添付しておきます。
https://qiita.com/Lulu34/items/b0c88d1e1163d50f743b

[退会済みユーザー]

いろいろあると思いますが、前提として、はられたコードはコンパイルとおんないです。
並び替えまでできましたとありますが、コードのコピペ違いでしょうかね。

[h_crane]

> 少し古いですが

と記載したように、古いコードなので新しいswiftへの対応が必要です。

> はられたコードはコンパイルとおんないです。

マウントを取るわけではないですが、エラーコードを読んで検索すれば解決できることが多いので行いましょう。今後勉強する上で大切です！

それは置いといて、解決策としては、新しくコメントされている方のコードを参照すると良いと思います。

[退会済みユーザー]

h_craneさんへ、
はられたコードってのは、質問文のコードの方ですよ〜

コメント追加したのは、h_craneさんの貼ったリンク先の内容を参考にして該当箇所をなおしたとしても、
ビルドさえできないからです。
cellForRowAt内のlet cell = UITableVeiw(これ動くの？)とか、拡張してるviewControllerが違うとか、もろもろ

解決作としてtsukubadepotさんの回答を参考にしてってのは同意です。

[退会済みユーザー]

すみません
cellForRowAt内のlet cell = UITableVeiwと拡張しているviewControllerが違うのは、自分のコピーミスです