二フクラを使ってXcodeでSNSの投稿削除ボタンを作りたい。

Question

問題

二フクラを使ってXcodeでSNSの投稿削除ボタンを作りたい。

カスタムセルを使って、テーブルビューを作っています。
カスタムセルの中に削除ボタンを押すと削除画面に進み、
二フクラのデータベースから指定のデータを削除できる仕様を考えています。

そのためにテーブルビューのコントローラでカスタムセルのコントローラーに

timeLineCell.deleteButton.tag = indexPath.row

でobjectIdを渡して

カスタムセルで

    @IBAction func deleteButton(_ sender: Any) {

                let vc = self.storyboard?.instantiateViewController(withIdentifier: "deleteViewController") as! deleteViewController

                vc.objectId = self.deleteButton.tag

                self.show(vc, sender: nil)

    }

として削除画面に
objectIdを持って遷移しようと考えているのですが

"Value of type 'cusTableViewCell' has no member 'storyboard'"

というエラーが出て遷移できません。
セルにはstorybboardが無いという意味だとは思い、
テーブルのあるコントローラーからの遷移を考えたのですが、
今度はカスタムセルのボタンのアクションの設定の仕方がどうしてもわかりません。

どちらの方法でもいいのでobjectIDの変数を持って画面遷移できたらと考えています。
以下コードです。

import UIKit

class ViewController: UIViewController,UITableViewDelegate,UITableViewDataSource {

    @IBOutlet var timeLineTable: UITableView!
    
    var data = [NCMBObject]()
    
    var imgData = [String:Data]()
    
    var refreshControl = UIRefreshControl()
    
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
        
        self.timeLineTable.delegate = self
        self.timeLineTable.dataSource = self
        
        let xib = UINib(nibName: "cusTableViewCell", bundle: nil)
        self.timeLineTable.register(xib, forCellReuseIdentifier: "cusTableViewCell")
        
        self.refreshControl.addTarget(self, action: #selector(ViewController.refreshData), for: .valueChanged)
        self.timeLineTable.addSubview(self.refreshControl)
        
        
        
    }
    
    override func viewWillAppear(_ animated: Bool) {
        // mBaasからデータを取得してくれる人を呼んでくる
        
        let query = NCMBQuery(className: "timeLine")
        
        query?.order(byDescending: "createDate")
        
        query?.findObjectsInBackground({ (values, error) in
            
            if error == nil {
                
                self.data = values as! [NCMBObject]
                
                self.timeLineTable.reloadData()
            }
        })
    }
    
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        
        return self.data.count
        
    }
    
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        
        
        let timeLineCell = self.timeLineTable.dequeueReusableCell(withIdentifier: "cusTableViewCell", for: indexPath) as! cusTableViewCell
        
        timeLineCell.timeLineText.text = self.data[indexPath.row].object(forKey: "timeLineMessage")as? String
        
        
        let file = NCMBFile.file(withName: "(self.data[indexPath.row].objectId!).jpg", data: nil) as! NCMBFile
                
        timeLineCell.timeLineImageView.image = nil
        
       timeLineCell.deleteButton.tag = indexPath.row

        
        let id = self.data[indexPath.row].objectId!
        if self.imgData[id] != nil {
            timeLineCell.timeLineImageView.image = UIImage(data: self.imgData[id]!)
            return timeLineCell
        }
        
        // データを取得してもらう
        file.getDataInBackground { (data, error) in
            
            if error == nil {
                
                timeLineCell.timeLineImageView.image = UIImage(data: data!)
                
                self.imgData[self.data[indexPath.row].objectId!] = data!
            }
        }
        
        return timeLineCell
    }
    
    
    
    func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat {
        // 300でお願いします
        return 300
    }
    
    @objc func refreshData() {
        
        let query = NCMBQuery(className: "timeLine")
        
        query?.order(byDescending: "createDate")
        
        query?.findObjectsInBackground({ (values, error) in
            
            self.refreshControl.endRefreshing()
            
            self.data = [NCMBObject]()
            self.imgData = [String:Data]()
            
            if error == nil {
                
                self.data = values as! [NCMBObject]
                
                
                self.timeLineTable.reloadData()
            }
        })
    }

    
    

//    @IBAction func deleteButton(sender: AnyObject) {
//        // testクラスへのNCMBObjectを設定
//        let obj6 = NCMBObject(className: "timeLine")
//        // objectIdプロパティを設定
//        obj6?.objectId = self.deleteButton.tag
//        // 設定されたobjectIdを元にデータストアからデータを取得
//        obj6?.fetchInBackground({ (error) in
//            if error != nil {
//                // 取得に失敗した場合の処理
//            }else{
//                // 取得に成功した場合の処理
//                obj6?.deleteInBackground({ (error) in
//                    if error != nil {
//                        // 削除に失敗した場合の処理
//                    }else{
//                        // 削除に成功した場合の処理
//                        let query = NCMBQuery(className: "timeLine")
//                        
//                        query?.order(byDescending: "createDate")
//                        
//                        query?.findObjectsInBackground({ (values, error) in
//                            
//                            self.refreshControl.endRefreshing()
//                            
//                            self.data = [NCMBObject]()
//                            self.imgData = [String:Data]()
//                            
//                            if error == nil {
//                                
//                                self.data = values as! [NCMBObject]
//                                
//                                
//                                self.timeLineTable.reloadData()
//                            }
//                        })
//                        
//                    }
//                })
//            }
//        })
//
//    }
    
    
}

import UIKit

class cusTableViewCell: UITableViewCell {

    @IBOutlet var timeLineImageView: UIImageView!
  
    var openBrowser: ((String)->())!
    
    @IBOutlet var timeLineText: UITextView!
    
    @IBOutlet var deleteButton: UIButton!
    
    override func awakeFromNib() {
        super.awakeFromNib()
        // Initialization code
    }

    override func setSelected(_ selected: Bool, animated: Bool) {
        super.setSelected(selected, animated: animated)

        // Configure the view for the selected state
    }
    
    @IBAction func deleteButton(_ sender: Any) {

                let vc = self.storyboard?.instantiateViewController(withIdentifier: "deleteViewController") as! deleteViewController

                vc.objectId = self.deleteButton.tag

                self.show(vc, sender: nil)

    }
    
}

ご教授いただけらありがたいです。
よろしくお願いいたします。

Answer 1

TableViewのあるViewController側からの遷移という考え方は合ってます。
この場合，セルの中にあるボタンをタップした場合の処理をViewController側に移譲すれば良いです。

// セル側の処理
import UIKit

// プロトコルを実装
protocol DeleteButtonDelegate: class {
    // 削除ボタンをタップした際にdelegateメソッドを実行
    func moveToDeleteScreen(tag: Int)
}


class cusTableViewCell: UITableViewCell {

    weak var deleteButtonDelegate: DeleteButtonDelegate?

    // 略

    @IBAction func deleteButton(_ sender: Any) {
        // 削除ボタンがタップされた際の処理を移譲
        self.deleteButtonDelegate.moveToDeleteScreen(tag: self.deleteButton.tag)
    }
}

// ViewController側の処理
class ViewController: UIViewController,UITableViewDelegate,UITableViewDataSource {

    // 略

    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {

        let timeLineCell = self.timeLineTable.dequeueReusableCell(withIdentifier: "cusTableViewCell", for: indexPath) as! cusTableViewCell

        // ボタンを押した際の処理を移譲されるの私です    
        timeLineCell.deleteButtonDelegate = self

        // 略

        return timeLineCell
    }
}

extension ViewController: DeleteButtonDelegate {

    /// 削除ボタンが押された時の処理を実行
    func moveToDeleteScreen(tag: Int) {
        let vc = self.storyboard?.instantiateViewController(withIdentifier: "deleteViewController") as! deleteViewController
        vc.objectId = tag
        self.show(vc, sender: nil)
    }
}

こんな感じでいかがでしょうか？

Comments

[suma0402]

回答ありがとうございます
以下のエラーが出てしまい動きませんでした。

extension ViewController: DeleteButtonDelegate部分に
Declaration is only valid at file scope

vc.objectId = tag部分に
Value of type 'deleteViewController' has no member 'objectId'

protocol等、
まだ勉強しきれていないところなので、自分でも再度調べさせてもらいます！

[TakuyaAso]

ViewController の extension なので class の中に書いてはいけません。
回答を少し修正しました。class {} の外に出してみてください。

DeleteViewController.swift の方に下記のプロパティを定義すれば値渡しできると思いますー
class DeleteViewController: UIViewController {
var objectId: Int?
// 略
}

[suma0402]

xcodeでエラーが出ずにbuild出来るようになりました！
ただ削除ボタンを押すと
AppDelegate.swiftファイルで
class AppDelegate: UIResponder, UIApplicationDelegate {
Thread 1: signal SIGABRT

とエラーが出るのですが
これはボタンが
うまく変数に設定しきれてないという事なのでしょうか？
今までも何度かこのエラーが出てきていて、
その度にググっては分からず作り直しを繰り返しており悩んでいます。
少し質問内容とずれている気がしますが
もしよろしかったら回答行っただけたら幸いです。

[TakuyaAso]

Xcodeの下のデバッグエリア(ログがみれる)ところのエラー内容がわかればという感じですね。
Reason とかで検索すると引っかかると思うのですがどんなエラーでしょう？

[suma0402]

返信ありがとうございます。
そのような調べ方があるとは知りませんでした。。
下記のようなエラーが出ました。

2019-07-28 19:33:49.800983+0900 pictweet2[35417:2003243] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Storyboard (<UIStoryboard: 0x600003fc4400>) doesn't contain a view controller with identifier 'deleteViewController''
(lldb)

[TakuyaAso]

現在のStoryboardにDeleteViewControllerの画面がないか，Storyboard ID が間違っているかのエラーですね。確認してみてください。参考記事はこちらです(最後のエラーの説明が参考になるかと)。
https://qiita.com/colorrabbit/items/c8cb6f815e6dc0b6beba

[suma0402]

Storyboard ID が間違っているのが原因でした。
初歩的なミスでした。
修正すると無事変数を持って遷移しました。
ありがとうございます！

[TakuyaAso]

おお，解決して良かったです！