クロージャーを用いた値渡しがうまくいかない

Question

Closureを理解するため、簡単なサンプルを作成中です。

FirstViewControllerのnextButtonでSecondViewControllerへ画面遷移
↓
SecondViewControllerのcountButtunを押すと数字が足されてcountLabelに反映
↓
SecondViewControllerのaddButtonでFirstViewControllerに戻る

この挙動でcountLabel（SecondViewController）の値をクロージャーを用いて
label（FirstViewController）に渡したいのですがうまくいきません。

import UIKit

class FirstViewController: UIViewController {

    let second = SecondViewController()
    
    @IBOutlet weak var label: UILabel!{
        didSet{
            label.text = "0"
            
        }
    }
   
    
    @IBAction func nextButton(_ sender: UIButton) {
    
            let secondStoryboard = UIStoryboard(name: "Second", bundle: nil)
            let secondVC = secondStoryboard.instantiateInitialViewController() as!SecondViewController
        
            let nav = self.navigationController!
            nav.pushViewController(secondVC, animated: true)
            second.tap { CountModel in
            print("ーーーー渡されたよーーーーーーー")
            self.label.text = CountModel.count.description
    }
        
        
    }
    
    override func viewDidLoad() {
        super.viewDidLoad()
  
 }

}

import UIKit

class SecondViewController: UIViewController{
   
    
   private var countmodel = CountModel.init(count: 0)
    
    @IBOutlet weak var countLabel: UILabel!{
        didSet{
            countLabel.text = countmodel.count.description
        }
    }

    @IBAction func countButtun(_ sender: UIButton) {
        countmodel.count = countmodel.count + 1
        countLabel.text = countmodel.count.description
        print(countmodel)
    }
    
    
    @IBAction func addButton(_ sender: Any) {
        tap()
        self.navigationController?.popViewController(animated: true)
        
    }
    
    //クロージャー関数
    func tap(completion:((CountModel)->Void)? = nil){
        print("tapしたよ")
        print(countmodel)
        completion?(CountModel(count: countmodel.count))
    }

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

import Foundation

struct CountModel {
    var count:Int
}

以上がコードです。

playground上での簡単なクロージャーのサンプルは理解できるのですが
実際に非同期処理でなぜ値が渡らないかが分かりません。

どなたかよろしくお願いいたします。

Comments

[tomato879241]

>非同期処理でなぜ値が渡らないかが分かりません
「非同期処理」とはどういう意味でしょうか？
ただSecondViewControllerにprotocolをつけて、値をdelegateを通してFirstViewControllerに渡せばいいのでは？

Answer 1

やりたいことはたぶんこういうことかと。なお、これは「非同期処理」ではなく、「コールバック」ですね。

FirstViewController.swift

• SecondViewController は storyboard から生成するので、直接生成しても意味はありません。
• second.tap をここで呼ぶと、画面遷移する直前にタップした時の動作を行うことになります。
  代わりに、secondVC の callback を設定すると良いのでは。
• nav.pushViewController は secondVC の準備ができてからの方が良い気がします。

 class FirstViewController: UIViewController {
 
-    let second = SecondViewController()
 
     // 略
 
     @IBAction func nextButton(_ sender: UIButton) {
         let secondStoryboard = UIStoryboard(name: "Second", bundle: nil)
         let secondVC = secondStoryboard.instantiateInitialViewController() as!SecondViewController
 
-        let nav = self.navigationController!
-        nav.pushViewController(secondVC, animated: true)
-        second.tap { CountModel in
+        secondVC.callback = { CountModel in
             print("ーーーー渡されたよーーーーーーー")
             self.label.text = CountModel.count.description
         }
 
+        let nav = self.navigationController!
+        nav.pushViewController(secondVC, animated: true)
     }

SecondViewController.swift

• コールバック関数 callback をプロパティに持つ必要があります。
• tap メソッドでは、コールバック関数を引数に取るのではなく、事前に設定された callback を呼びましょう。

 class SecondViewController: UIViewController{
 
     private var countmodel = CountModel.init(count: 0)
 
+    var callback: ((CountModel) -> Void)?
 
     // 略
 
-    //クロージャー関数
-    func tap(completion:((CountModel)->Void)? = nil){
+    func tap() {
         print("tapしたよ")
         print(countmodel)
-        completion?(CountModel(count: countmodel.count))
+        callback?(countmodel)
     }

Comments

[KCROW]

ご回答ありがとうございます！やりたかったことができました！！