swift4 delegateメソッド が呼ばれない件について

Question

こんばんは。
swiftにてios開発を始めたばかりの者です。
自作でdelegateの処理を書いたのですが、メソッドが実行されません。

具体的には、
ボタンを押した際に、通知をして通知先(BBBController)に
AAAControllerで持っている値をBBBControllerに送り、その後navigationbarを一つ前(BBBControllerの画面)に戻す
というような処理を行いたいと思っています。

以下の2つのコードを書いて
コンソールを確認しながら動かしているのですが,registerTapButton()のメソッド が動いていないようです。

書いたコードは以下の通りです。

//AAAController.swift

protocol RegisterButtonDelegate {
    //buttonを押した際のdelegateメソッド の記述
    func registerTapButton()
  
}

class AAAController: UIViewController {


     weak var delegate: RegisterButtonDelegate?
     
     override func viewDidLoad(){
      略
     }     

〜略〜

     @IBAction func registerButton(_ sender: Any) {

       let nav = self.navigationController!                
       let bbbController = nav.viewControllers[nav.viewControllers.count - 2] as! BBBViewController

       //このmessegeをbbbControllerに送りたい。
       bbbController.sendMessage = "aaaa"
       
       //delegateメソッド の実行(brakepointで確認したらnilになっていました)     
       self.delegate?.registerTapButton()

       //navigationbarのときの画面戻る動作
       self.navigationController?.popViewController(animated: true)
     }
}

//BBBController.swift

class BBBController: UIViewController, RegisterButtonDelegate {
     var sendMessage: String?

     over func viewDidLoad() {
         super.viewDidLoad()

         let aaaController: AAAController? = AAAController()
         aaaController?.delegate = self

     }

　　　〜略〜
　　　
    func registerTapButton() {
      print("動作確認")
      print(self.sendMessage)
    }
}

アドバイスどうぞよろしくお願いします。

Comments

[fuzzball]

BBB → AAAはどうやって遷移しているのでしょうか？あと、delegate変数をどのように定義しているか書いて下さい。

[waya]

delegate変数を追加しました。BBB -> AAAはtableviewのセルをタップした時にsegueで繋いで表示しています。

Answer 1

Delegateの書き方が正しくないですね、以下の様にすると動くと思います。
参考URL: プロトコルとデリゲートのとても簡単なサンプルについて

protocol RegisterButtonDelegate: class {
    //buttonを押した際のdelegateメソッド の記述
    func registerTapButton(message: String)
}

//AAAController.swift

class AAAController: UIViewController {
    
    weak var delegate: RegisterButtonDelegate?
    
    @IBAction func registerButton(_ sender: Any) {
        
        delegate?.registerTapButton(message: "Message from AAA")
        navigationController?.popViewController(animated: true)
    }
}

//BBBController.swift

class BBBController: UIViewController, RegisterButtonDelegate {
    
    var sendMessage: String?
    
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if segue.identifier == "Storyboardで設定したIdentifier",
            let aaa = segue.destination as? AAAController {
            aaa.delegate = self
        }
    }
    
    // MARK: - RegisterButtonDelegate
    
    func registerTapButton(message: String) {
        
        self.sendMessage = message
        print(self.sendMessage ?? "")
    }
}

Comments

[waya]

ご回答ありがとうございます。
つまり、私の記法との違いはAAAの呼び出しをViewDidAppear内にて行うという認識でよろしいでしょうか?

[_Kentarou]

いや、それ以外も結構変えてますよ、、、ViewDidAppearに変えたのは画面表示前に遷移していたので移しただけです。

[waya]

失礼なコメントをしてしまい、すみません。
まずはじめにdelegate変数の定義を質問に掲載し忘れていたようです。
ViewDidAppear内にてAAAControllerを呼び出すとAAAにて表示予定の値がnilになり動きが停止してしまいます。(AAAControllerを呼び出した際に(BBB -> AAAの遷移がセルの状態を受け取ってAAAにて画面を表示しているため)
そのため_Kentarouさんの書いたコードをviewDidLoadにて書き換え、delegateメソッド の呼び出し部分を
self.delegate?.registerTapButton(message: "Message from AAA")に変更したのですがやはり動作しません。

また、ViewDidAppearを使うのであれば、Delegateを用いずにAAAから送った値をその場で表示すればよいと思ったのですがどうなんでしょうか。

ちなみにDelegateはこちらのページを参考に書いています。
https://qiita.com/sachiko-kame/items/d8a16148e9ed70074c78

[_Kentarou]

参考にしているページはUIViewをViewControllerに乗せてボタンタップ時にDelegateメソッドを呼んでいるので、今回の画面遷移するパターンとは少し違います。

Segueを使っているのでしたらprepare(for segue: UIStoryboardSegue, sender: Any?)でdelegateを設定してください。

回答を編集しましたのでそちらを見てみてください。

[waya]

なるほど・・・・非常に納得しました。
詳しいご回答ありがとうございました。ありがとうございます！

Answer 2

BBB→AAAへの遷移はsegueを利用しているとのことですが、その場合delegateのセットの仕方に問題があるようです。

BBBControllerのviewDidLoad内で以下の様にされていますが、
この場合、遷移先のAAAControllerとは別のAAAControllerのインスタンスを生成してdelegateをセットすることになっています。

         let aaaController: AAAController? = AAAController() //遷移先のAAAとは別モノ
         aaaController?.delegate = self

以下の様にprepareForSegue内で遷移先のAAAControllerを取得してdelegateをセットすれば遷移先のAAAからdelegateが呼び出せるかと思います。

class BBBController: UIViewController, RegisterButtonDelegate {
     var sendMessage: String?

    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if let vc = segue.destination as? AAAController {
            vc.delegate = self
        }
    }

　　　〜略〜

Comments

[waya]

詳しい解説ありがとうございます。
納得しました。参考にして開発していきたいと思います。