SwiftでViewController間のdelegateをする方法

Question

###前提・実現したいこと
SwiftでViewController間でdelegateをしようと思ってるいるのですが、うまく行きません。

コードでは、SecondViewController上のボタンを押すと、FirstViewControllerのtest()が実行されるようにしようとしています。

###発生している問題・エラーメッセージ
SecondViewControllerでのdelegateがnilになってしまい、FirstViewControllerにあるtest()を呼んでくれません。

###該当のソースコード

//FirstViewController
import UIKit

class FirstViewController: UIViewController, TestDelegate {
        
    func test() {
        print("called delegate")
    }

    override func viewDidLoad() {
        super.viewDidLoad()
        
        let storyboard = UIStoryboard(name: "Main", bundle: nil)
        let secondVC = storyboard.instantiateViewController(withIdentifier: "Second") as! SecondViewController
        
        secondVC.delegate = self
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
    }
}

//SecondViewController
import UIKit

protocol TestDelegate {
    func test()
}

class SecondViewController: UIViewController {
    
    var delegate:TestDelegate?
    
    @IBAction func pushButton(_ sender: UIButton) {
        delegate?.test()
    }
    
    override func viewDidLoad() {
        super.viewDidLoad()
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
    }
}

###試したこと
昨日、ViewからViewControllerへのdelegateを質問した際に、ViewControllerで指定したViewのインスタンスが正しくstoryboard上のViewをさしていないという指摘をいただきました。

そのため、FirstViewControllerからStoryboard上のSecondViewControllerを示すようにコードを書いてみたつもりなのですが、やはりインスタンスを正しく指定できていないのか、SecondViewControllerのdelegateがnilとなっています。

###補足情報(言語/FW/ツール等のバージョンなど)
StoryBoardはMain.storyboardです。
SecondViewControllerのstoryboard IDは"Second"としています。

Answer 1

FirstViewからSecondViewへの遷移方法がコードからは読み取れないので予想になってしまいますが、
おそらくSecondViewのインスタンスが都度作成されちゃってるんだと思います。
あと、viewDidLoad()内でViewをインスタンスするようなコードは書かない方が無難です。
参考にセグエを使ったFirstViewController.swiftを載せときます。

import UIKit

class FirstViewController: UIViewController, TestDelegate {
    
    func test() {
        print("called delegate")
    }
    
    override func viewDidLoad() {
        super.viewDidLoad()
    }
    
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
    }
    
    override func viewDidAppear(_ animated: Bool) {
        performSegue(withIdentifier: "next", sender: nil)
    }

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

}

2017.12.7 追記

TabBarのインスタンス経由でFirstViewControllerのインスタンスを取得
（FirstViewControllerがイニシャルビューと仮定してます）

// FirstViewController.swift
import UIKit

class FirstViewController: UIViewController, TestDelegate {
    
    func test() {
        print("called delegate")
    }
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
    }
    
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
    }
}

// SecondViewController.swift
import UIKit

protocol TestDelegate {
    func test()
}

class SecondViewController: UIViewController {
    
    var delegate:TestDelegate?
    
    @IBAction func pushButton(_ sender: UIButton) {
        delegate?.test()
        
    }
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
           // TabBarのインスタンス経由でFirstViewControllerのインスタンスを取得
           if let tabvc = UIApplication.shared.keyWindow?.rootViewController as? UITabBarController  {
                // FirstViewControllerのインスタンスをdelegateにセット
                self.delegate = tabvc.viewControllers?.first as? TestDelegate
            }

    }
    
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
    }
}

Comments

[pascal1623]

遷移の仕方も問題になるのですね。説明不足で申し訳ありません。

FirstViewControllerとSecondViewControllerはTabBarで切り替えるように作っています。
そもそも作りたいアプリの中でつまづいているのがTabBarで切り替えるViewController同士でdelegateをしたいという部分です。

まだまだインスタンスの生成についてよく理解していないと思うのですが、TabBarでは生成されたインスタンスの表示を切り替えているだけだと思っておりました。

[fromageblanc]

tabbarで追記しました

[pascal1623]

ありがとうございます。
この方法でうまく行くことを確認させていただきました。

もしよろしけばもう少し教えていただきたいのですが、
tabbarで管理しているどのViewController同士でもdelegateできるようにしたいと思っています。
SecondViewContorollerに
if let tabvc = UIApplication.shared.keyWindow?.rootViewController as? UITabBarController {
let firstVC = tabvc.viewControllers?[0]
self.delegate = firstVC as? TestDelegate
と書いたところ、やはりdelegateが呼ばれます。

しかし、FirstViewControllerに
if let tabvc = UIApplication.shared.keyWindow?.rootViewController as? UITabBarController {
secondVC = tabvc.viewControllers?[1] as? SecondViewController
secondVC?.delegate = self
と書いてもdelegateが設定されません。

また元々の記載では、なぜダメなのかもよくわからずなんともすっきりしません。
大変申し訳ありませんが教えていただけましたら助かります。

[fromageblanc]

viewDidLoad内に書いちゃうと一度しか実行されないので、viewDidAppearに移してください

override func viewDidAppear(_ animated: Bool) {
if let tabvc = UIApplication.shared.keyWindow?.rootViewController as? UITabBarController {
let secondVC = tabvc.viewControllers?[1] as? SecondViewController
secondVC?.delegate = self
}
}

[pascal1623]

本当ですね！！
これでdelegateが呼ばれました！
正直、まだ理解できていないことも多いのですが、今回教えてもらったことを参考に精進させていただきます。
本当にありがとうございました！