delegate先のクラスで定義したプロパティの値がnilになってしまう

Question

前提・実現したいこと

delegateを使って下記の処理を実現したいです。

「Top画面」という画面の上に、containerViewを配置し、
containerViewの上にViewを配置しています。

Top画面に対して、クラス：ViewControllerを割り当てています。

Top画面の配下のViewの、さらに配下のCotainerAに、
下記の「冷蔵庫画面」を配置していて、クラス:cabinViewControllerを設定しています。

※delegateについての私の理解（間違いありましたらご指摘ください）
delegateとは、自分のクラスの処理内容を他のクラスに実行してもらう（移譲する）こと。
下記の実装が必要。
①protcolの定義
②protcolに準拠した、処理内容を記載するクラス(今回やりたいことだとviewControllerB)
③処理の移譲先を指定するクラス（今回やりたいことだとviewControllerA）

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

下記のとおりコードを記載しているのですが、myTableView.reloaDate()の処理で、
myaTableViewの値がnilになってしまうエラーが発生してしまい、解決方法がわかりません。

該当のソースコード

①protocol

protocol ReloadDataDelegate {
    func reloadTabeleDate()
}

②CabinviewController（冷蔵庫画面）

class CabinViewController:UIViewController, UITableViewDelegate, UITableViewDataSource, UINavigationControllerDelegate,ReloadDataDelegate {

    @IBOutlet weak var myTableView: UITableView!

    override func viewDidLoad() {
        super.viewDidLoad()
        myTableView.delegate = self
        myTableView.dataSource = self
     }

    //MARK:- デリゲート
    func reloadTabeleDate(){
        print("test")
        myTableView.reloadData()
    }
}

③ViewController（TOP画面）

class ViewController: UIViewController,  UINavigationControllerDelegate {

    var delegate:ReloadDataDelegate!

    // MARK: - ContainerView
    var containers:Array<UIView>=[]
    @IBOutlet weak var containerA: UIView!
    @IBOutlet weak var containerB: UIView!
    @IBOutlet weak var containerC: UIView!


    override func viewDidLoad() {
        super.viewDidLoad()

        let cabinVC = CabinViewController()
        self.delegate = cabinVC

        containers=[containerA,containerB,containerC]
        containerView.bringSubviewToFront(containers[containerIndex])
}

~(中略)~
    //MARK:- segmentBarの値に応じて、containerViewの表示を切り替え
    @IBAction func SegmentAction(_ sender: UISegmentedControl) {

        let currentContainerView=containers[sender.selectedSegmentIndex]
        containerView.bringSubviewToFront(currentContainerView)
    }

~(中略)~
    //NavigationBarのボタン
    @IBAction func templateButton(_ sender: Any) {
        reload()
}

    func reload() {
        self.delegate?.reloadTabeleDate()    
    }
}

試したこと

実行すると、reloadTabeleDate()内の、"test"の出力がされることは確認しています。

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

Swift　5.X

Comments

[hoshi-takanori]

画面 A と B はどうやって遷移しますか？ または同時に表示されますか？
A の viewDidLoad で B のインスタンスを生成してますが、これは実際に表示されるやつですか？
また、delegate プロパティの宣言が A ではなく B にあるのは何故でしょうか。

[TsukubaDepot]

ご提示いただいているコードですが、コンパイルできない間違いが多く残っているようです。
たとえば、ViewControllerA で
self.delegate = viewControllerB
というコードを書かれていますが、self（つまり、ViewControllerA）には delegate というプロパティがありません。

中略されて提示されるのは致し方がないと思いますが、間違いがあると何が根本的な理由なのか指摘できないため、動かしているコードをそのままコピーし、不要なところを削除してもらえませんでしょうか。

[TsukubaDepot]

あと、ViewControllerA が親で、ViewControllerB のインスタンスを持っているのであれば、delegate モデルを使う必要はないといもいます。
ViewControllerAは ViewControllerB のインスタンスを知っているので、当然 ViewControllerB が持つメソッドやプロパティは直接見えます。

また、delegate としての作り方も間違っているようです（protocol をクラスのみに準拠させる、！ではなく？で参照させるなど）。

[sta_sato]

@hoshi-takanoriさん
＞＞画面 A と B はどうやって遷移しますか？ または同時に表示されますか？
同時に表示されます。A画面の上にB画面を載っけています。
＞＞A の viewDidLoad で B のインスタンスを生成してますが、これは実際に表示されるやつですか？
すみません、「実際に表示される」とはどういった意味になりますでしょうか？
＞＞また、delegate プロパティの宣言が A ではなく B にあるのは何故でしょうか。
申し訳ありませんが、記載誤りのため、質問を修正いたします。

@TsukubaDepotさん
申し訳ありません、コードを追記します。また、誤って記載している箇所もあるので、追記します
（コンパイルエラーにはなっていません。）

[TsukubaDepot]

細かいところを突くようで申し訳ないのですが、正確に説明していただかないことにはこちらも理解できませんので、正確な記述となるようにご質問を変更していただけますでしょうか。

>（追記）
>A画面の上に、Viewがあり、このViewの下に、ContentViewBがあります。
A画面とは、具体的にどの画面のことでしょうか。
「Top画面 scene」のことでしょうか。
また、この「Top画面 scene」に関連づけられているViewControllerのクラス名は何になりますでしょうか。

加えて、「ContentViewB」というのがありますが、これは「ContainerB」のことでしょうか。

> ContentViewBに対して、ViewControllerBを設定しています。

ViewControllerB というクラスは実際には「CabinViewController」なのでしょうか。
そうすると、ViewControllerA でインスタンスにしている次のコード

> let viewControllerB = ViewControllerB()
> self.delegate = viewControllerB

の　ViewControllerB() は、実際には CabinViewController() のことなのでしょうか。
このあたりを書き換えてしまうと、もはやご質問の意図を掴むことができないので、正確に記述していただけますでしょうか。

> (名前がわかりづらくなってしまっていますが、ContainerA~Cのうち、
> ContainerAに設定しているクラスをviewControllerBとして質問を書いています)

ViewControllerB（実際はCabinViewController() ）は、実際には Container View に入っているということでしょうか。

[sta_sato]

@TsukubaDepot
申し訳ありません、省略化して表現しようと結果、正しくないかつ混乱させてしまう記載になってしまいました。そのままのコードで一度質問内容を更新いたします。

Answer 1

コメントを受けて修正します。

~~とりあえず、ご質問の内容から確実に言えることだけご指摘します。
~~
結論としては、はやりインスタンスの関係を正しく把握しない状態でコーディングされていることが原因です。

        let let cabinVC = CabinViewController()

という行があり、たしかに CabinViewController はインスタンス化されます。

しかし、StoryBoard 状に配置した Container View にはCabinViewController が紐づけられていいるはずで（embeded segueとして関連づけられているはず）で、ここで改めてインスタンス化した CabinViewController とは全く無関係になっています。

コード内でインスタンス化された CabinViewController は単にインスタンスが出来るるだけであり、viewDidLoad()などのライフサイクルに関するメソッドや、

    @IBOutlet weak var myTableView: UITableView!

など、StoryBoard と関連づける UI部品のインスタンス化も行われません。

したがって、直接インスタンス化しただけでは、UITableViewは作成されず、myTableView の値は初期値の nil のままとなってしまいます。

したがって、

    func reloadTabeleDate(){
        print("test")
        myTableView.reloadData()
    }

この行にあるmyTableView.reloadData()のメソッドチェインに失敗し、実行時エラーとなっています。

以上が、今回のエラーの原因です。

###対策

さて、追記していただいたコードや StoryBoard 上の階層から想像すると、ContainverView を使い、そこで TableView を表示を（つまり、CabinViewController の表示を）されています。

そうすると、必要な作業は、Container View に入れられた CabinViewController のインスタンス（より具体的に言うと、メモリ上に展開されたアドレス）を調べ、そこから直接 CabinViewController のメソッドを操作することになると思われます。

Container View に追加された （一般的に言うところの）View Controllerのインスタンスは、children という [UIViewController]の配列に入ますので、そこから CabinViewController のインスタンスを探し出す必要があります。

全てコードベースで Container View に追加するのであれば、それは自明なのですが、StoryBoard で追加したのであれば、自分で探す必要があります（ちなみに、Container View というのは、実際には UIView に過ぎず、StoryBoard で紐づければ一連の厄介な手続きが自動化されるだけの話です。逆にいうと、自動化されるために手間がかかる作業も増えるということです）。

Container View が一つだけであれば、配列のインデックスを決め打ちで調べることも可能ですが、安全性や、あるいは今回みたいに複数の Container View を使うのであれば、なんらかの方法で調べる必要があります。

たとえば、viewDidLoad()で

        if let index = self.children.firstIndex(where: { vc -> Bool in
            return vc is CabinViewController
        }) {
            print("Cabin:", index)
        }

みたいな方法を使い、children に入っているインスタンスとのパターンマッチを行うことでインデックスを調べる必要があります。

正確に言うと、is 演算子は CabinViewController クラスを元にして作られたインスタンスとのマッチングであり、仮に children の内部に複数のCabinViewControllerから作られたインスタンスが存在するのであれば、それはこの方法で調べることはできませんし、全体構成を根本的に見直す必要があると思います

それはさておき、上記の方法で index がわかれば、

   // クラスのプロパティとして宣言
    var cabinVC: CabinViewController?

   //　viewDidLoad() 内部などで
    if let index = self.children.firstIndex(where: { vc -> Bool in
            return vc is CabinViewController
        }) {
　　        cabinVC = self.chidren[index] as! CabinViewController
        }

のような方法で、vc に CabinViewController のインスタンス（具体的なメモリ上のアドレス）を得ることができますので、あとは

   cabinVC?.reloadTabeleDate()  

のような感じで、直接 CabinViewController のメソッドを操作することになります。

delegate の適用

今回のようなパターンであれば、無理に delegate パターンに持ち込む必要はないと思いますし、上記のようになんらかの方法で対象となるインスタンスを探すのであれば、delegate としてインスタンスを保持する理由もあまりないとおもます（結果として、行っていることは同じなので）。

もちろん、CabinViewController を汎用的なクラスとして使いたいということであれば、delegate パターンを適用することになりますが、その場合には CabinViewController のインスタンス化の流れから根本的に見直す必要があると思います。

たとえば、StoryBoard で Container View を使うのではなく、UIView を配置し、instantinateViewController で従属させたいView Controllerを正しく初期化させ、addChild で child として追加し、その view を UIView のサブビューとして登録し、必要に応じて画面全面に持ってくるという流れを全て手動で行えば、また話は違ってくると思います（それでも、delegate パターンを使う必然性はあまりないと思います）。

全体の構成

もう一つ考えていただきたいのは、全体の構成です。

ViewController 内部に多くの Container View を取り込み、複数の View Controller を扱っている時点で、もはや質問者さんご自身が管理不能な状態に陥っているように思えます。

全体像が見えないのでなんとも言えないのですが、一つの画面に対してこれほどたくさんの View Controller を適用する必要があるのか、もう一度設計をよく見直された方がいいかもしれません。

ざっとみた感じであれば、一つの View Controller 内部に Table View を配置し、適切に制約（constraint）などを設定すれば、それで住むだけの話のようにも思えます。

ただ、これは期待される構成としてどのような構成を考えていらっしゃるのかよく聞かない限り判断できませんが、ただこの内容のご質問だと、聞いている方も把握するのは難しいのではないかと感じています。

今後の方針として

それでもやはり Container View を使って作りたい、ということであれば、

ひとつの　View Controller に一つの Container View だけを持つサンプルを作り、そこから Container View 内部に配置した View Controller のインスタンスを特定し、操作する

という、ごく単純で明快なサンプルを作り、そこで納得のゆく結果を得てから拡張されるのが良いかと思います。

別件のご質問も拝見しましたが、そこで指摘されているように View と View Controller の違い、特に役割の違いについて誤解を受けているように私も感じました。

もしかしたら、なんらかのサンプルを元にオリジナルのアプリを作ろうとされているのかもしれません。

でも、結局はごく最小限の構成を理解しないことにはそれより大きい構成を作ることは難しい話なので、まずはごく簡単なサンプル作りから始め、着実に積み重ねられた方が良いのではないでしょうか。