[Swift]子ビュー（モーダル）から親ビューの表示更新処理とメモリリーク問題の存在有無

Question

質問内容

初投稿です。独学でSwiftの開発をしています。
Storyboardを利用したiOSアプリ開発にて、親ビューからモーダルで子ビューを開き、子ビューのボタンが押されると子ビューを閉じて親ビューの表示を更新する、という処理をSwiftで作成しています。
これ自体は実現できているのですが、書き方がメモリリークの原因となる相互参照になってしまっていないか、もしそうであれば良い書き方があれば教えていただければと思っています。

該当のソースコード

①親ビューから子ビューを開く際に、親ビューの関数を子ビューのプロパティに渡す
②子ビューでボタンを押された時に、親ビューから渡された関数が実行される
③親ビューのラベルが更新される
（簡略化していますが、一応以下の記法で狙い通りの動きになっています。）

import UIKit

//親ビュー
class ParentViewController: UIViewController {
    @IBOutlet weak var label: UILabel!

    //①遷移時に子に親ビューを更新する関数を渡す
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if segue.identifier == "toChild" {
            if let childVc = segue.destination as? ChildViewController {
                childVc.update = self.updateParentView
            }
        }
    }

    //③子から親ビューが更新される
    func updateParentView() {
        self.label.text = "更新されたよ"
    }
}

//子ビュー
class ChildViewController: UIViewController {
    @IBOutlet weak var label: UILabel!
    
    var update = {}

    //②子のボタンを押すと親ビュー更新関数を実行して閉じる
    @IBAction func button_Pushed(_ sender: Any) {
        self.update()

        DispatchQueue.main.asyncAfter(deadline: .now() + 0.3) {
            self.dismiss(animated: true, completion: nil)
        }
    }
}

試したこと

上記ソースコードだと、関数updateと親ビューインスタンスの循環参照になってしまうのではないか？と懸念しています。
（この部分理解が十分でなく用語が正しくなければすみません。）

もしかして、↓の「改」のようにクロージャーとして子ビューに渡せばいいのかもしれませんが、
同じ関数（上記コードだとupdateParentView関数）を親ビューでも利用したいときなど、
できればこの書き方はしたくないな、という気もしており、上記記法で問題がなければこれでいきたいと思っています。

    //①遷移時に子に親ビューを更新する関数を渡す「「改」」
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if segue.identifier == "toChild" {
            if let childVc = segue.destination as? ChildViewController {
                childVc.update = {[weak self] in

                    self?.label.text = "更新されたよ"
                }
            }
        }
    }

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

以下の環境を使用しています。
Swift5.4
Xcode12.4
StoryBoard

どうぞよろしくお願いいたします。

Comments

[TsukubaDepot]

とりあえずコメントだけですが、この状態だと相互参照にはならないと思います。
ただし、たとえばUITabBarController などのように、複数の ViewController を保持するようなインタフェースと併用すると、あまり都合がよくない「かも」しれません。
ただ、通常は delegate パターンを使って実装すると思います。
ほかにもクロージャを渡すとか、メッセージのやりとり（Notification Center とか RxSwift）などを使う方法もありますが、実装しやすいのは Delegate パターンではないでしょうか。

[ys203]

ありがとうございます！
色々みていると、確信はないですがdelegateでやった方が安全なようですね・・・
大変参考になりました。

[mskRR]

deinit書いてみれば？

[ys203]

ありがとうございます。そうですね、おかげさまでdeinitの実用法を理解できました。

Answer 1

メモリーリークを調査するツールがXcodeに内包されています。
他者のブログですが、ここが読みやすい感じでした。
https://kerubito.net/technology/3298

手法としてはUITableViewでも使用されている、delegateを使ってはどうでしょう？

//親
class ParentViewController: UIViewController {

    @IBOutlet weak var label: UILabel!

    //①遷移時に子に親ビューを更新する関数を渡す
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if segue.identifier == "toChild" {
            if let childVc = segue.destination as? ChildViewController {
                childVc.delegate = self
            }
        }
    }
}

extension ParentViewController: ChildViewControllerDelegate {
    func update() {
        self.label.text = "更新されたよ"
    }
}

//子
protocol ChildViewControllerDelegate: class {
    func update()
}

class ChildViewController: UIViewController {

    /// デリゲート
    weak var delegate: ChildViewControllerDelegate?

    //②子のボタンを押すと親ビュー更新関数を実行して閉じる
    @IBAction func button_Pushed(_ sender: Any) {
        self.delegate?.update()

        DispatchQueue.main.asyncAfter(deadline: .now() + 0.3) {
            self.dismiss(animated: true, completion: nil)
        }
    }
}

Comments

[ys203]

ありがとうございます！
ご紹介いただいたページを参照して、問題部分を特定できました。
delegate定義も目から鱗です。大変参考になりました。