デリゲートメソッドが実行されない

Question

DetailViewControllerで合計金額を再計算し、その結果をIndexViewControllerに渡し、sumMoneyLabelに再描画するという機能を実現したいです。
以下のようにプロトコルとデリゲートを実装したのですが、ラベルが際描画されませんでした。
デリゲートメソッドを実装している部分でログを出してみたのですが、ログは出力されなかったので、そもそもデリゲートメソッドが実行されていないのではないかと思っています。
どのように改修するべきでしょうか？ご指導お願いいたします。

IndexViewController

import UIKit
import Firebase
import FirebaseFirestore

class IndexViewController: UIViewController,UITableViewDelegate,UITableViewDataSource,DetailViewControllerDelegate {

    
    let detailView = DetailViewController() //detailViewControllerのインスタンス化
    let uid = Auth.auth().currentUser!.uid
   
    var sumMoneyString = "0"
    var titleString = String()
    var money = Int()
    var descriptionString = String()
    var documentIdString = String()
    var patientsArray = [Patiences]()
    var documentIdArray:[Any] = []
    
    @IBOutlet weak var sumMoneyLabel: UILabel!
    @IBOutlet weak var tableView: UITableView!


    override func viewDidLoad() {
        
        super.viewDidLoad()
        tableView.delegate = self
        tableView.dataSource = self
        detailView.delegate = self    //デリゲートを自分に指定
        sumMoneyLabel.text = sumMoneyString
        
        fetchData(uid: uid)
        
    }
    //セクションの数の指定
    func numberOfSections(in tableView: UITableView) -> Int {
        return 1
    }
　　 //セルの数の指定
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return patientsArray.count
    }
　　 //セルの指定
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "cell")
        let titleCell = cell?.viewWithTag(1) as! UILabel
        let moneyCell = cell?.viewWithTag(2) as! UILabel
        let documentId = documentIdArray[indexPath.row]
        let description = patientsArray[indexPath.row].description
        titleCell.text = patientsArray[indexPath.row].title
        moneyCell.text =  String(patientsArray[indexPath.row].money)
        return cell!
    }
　　 //セルの高さの指定
    func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat {
        return self.view.frame.size.height / 5
    }
    //セルが押された時の挙動　DetailVCへ移動
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        titleString = patientsArray[indexPath.row].title
        money = patientsArray[indexPath.row].money
        documentIdString = documentIdArray[indexPath.row] as! String
        descriptionString = patientsArray[indexPath.row].description
        performSegue(withIdentifier: "toDetail", sender: nil)　//DetailVCへ遷移
    }
    //DetailVCへの値の受け渡し
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        let detailVC = segue.destination as! DetailViewController
        detailVC.titleString = titleString
        detailVC.descriptionString = descriptionString
        detailVC.moneyString = String(money)    }
    //テーブルに表示するデータを取得し、配列patientsArrayに挿入
    func fetchData(uid:String){
        let docRef = db.collection("patients")
        docRef.whereField("uid", isEqualTo: uid).getDocuments { (QuerySnapshot, err) in
            for document in QuerySnapshot!.documents {
                self.documentIdArray.append(document.documentID)
                let title = document.data()["title"]
                let money = document.data()["money"]
                let description = document.data()["description"]
                self.patientsArray.append(Patiences(title: title as! String, money: money as! Int, description: description as! String))
            }
            
            self.tableView.reloadData()
        }

       }
    //DetailVC側から受け取った値をString型に変換し、sumMoneyLabelに代入
    func didAdded(sumMoney: Int) {　　　　　//デリゲートメソッドの実装
        print(sumMoney)
        sumMoneyLabel.text = String(sumMoney)
        print(sumMoneyLabel.text)
        print("合計金額の表示")
    }

}

DetailViewController

import UIKit
protocol DetailViewControllerDelegate{　　//プロトコルの定義
    func didAdded(sumMoney:Int)
}

class DetailViewController: UIViewController {

    var delegate:DetailViewControllerDelegate? = nil
    var titleString = String()
    var moneyString = String()
    var descriptionString = String()

    @IBOutlet weak var titleLabel: UILabel!
    @IBOutlet weak var moneyLabel: UILabel!
    @IBOutlet weak var descriptionLabel: UILabel!
    override func viewDidLoad() {
        super.viewDidLoad()
        
        titleLabel.text = titleString
        moneyLabel.text = moneyString
        descriptionLabel.text = descriptionString
    }
    //ボタンが押されたら、数字をsumMoneyに足し、デリゲートメソッドを実行
    @IBAction func patient(_ sender: Any) {        
        var sumMoney = Int((self.storyboard?.instantiateViewController(identifier: "index") as! IndexViewController).sumMoneyString)      
        var patientMoney = Int(moneyString)       
        sumMoney = sumMoney! + patientMoney!       
        self.delegate?.didAdded(sumMoney: sumMoney!) //デリゲートメソッドの実行
        
    }

}

Answer 1

// コメントアウト
//let detailView = DetailViewController() //detailViewControllerのインスタンス化

// コメントアウト
//detailView.delegate = self    //デリゲートを自分に指定

override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
    let detailVC = segue.destination as! DetailViewController
    detailVC.delegate = self  // デリゲートを設定
}

protocol DetailViewControllerDelegate: AnyObject{　　//プロトコルの定義
    func didAdded(sumMoney:Int)
}

class DetailViewController: UIViewController {
    weak var delegate: DetailViewControllerDelegate?
}

Comments

[oeiqgfodgfhps]

ありがとうございます！無事に動きました！
なぜそのように記述しないと動かないのでしょうか！

[退会済みユーザー]

コメントアウトしたdetailViewは遷移先とは別物なので、遷移先となるdetailVCにデリゲート設定しないとうごかない。
AnyObjectはdelegateをweakで宣言(循環参照を回避)するため。