Swiftのメソッドとクロージャについて

Question

Swiftで以下のようにViewからイベントを親に通知したい時にクロージャのハンドラを作ろと思うのですが、VC側でこれをメソッドで作っておきたいのですがうまく行きません。
以下のようなエラーが出てしまいます。

エラー

Cannot assign value of type '()' to type '() -> Void'

コード

class hogeView: UIView
    var tapEbentHandler:(()->Void)?

    func hogeTap(){
    //Viewのタップを検知した時に送る
    tapEbentHandler?()
    }
}

class HogeVC: UIViewController {
    func viewDidLoad() {
        これができない
        hogeView.tapEventHandler = handleEvent()
　　　　　
　　　　　こっちだったら可能だが処理が長くなると読みづらくなるので避けたい
        hogeView.tapEventHandler = {
            //ホゲホゲ
        }
    }
   
    func handleEvent() -> Void  {
        //ここのメソッドの中に処理を書きたい
    }
}

メソッドで書きたい時どうしたら良いのでしょうか？

Answer 1

かっこ付きの handleEvent() だと、メソッドを呼び出して、その結果を代入することになります。
メソッドそのものを代入するには、かっこなしで handleEvent と書くとできます。

-        hogeView.tapEventHandler = handleEvent()
+        hogeView.tapEventHandler = handleEvent

(追記) ただし、handleEvent がメソッドの場合、self が強参照されるので、これを view に設定すると循環参照になってメモリーリークの原因になりますので、クロージャにして weak self するか、delegate にすべきですね。

        hogeView.tapEventHandler = { [weak self] in
            self?.handleEvent()
        }

Comments

[hodoru3sei]

ありがとうございます！循環参照になってしまうのは盲点でした

[hodoru3sei]

追加で質問させてください！
循環参照になってしまう問題は以下のようにweakで定義すれば解決すると考えているんですがこちら認識あっていますでしょうか？
weak var tapEbentHandler:(()->Void)?

[hoshi-takanori]

そうすると weak は class 型にしか使えないというエラーになりますね…。

[hodoru3sei]

あーなるほど、、

Answer 2

こっちだったら可能だが処理が長くなると読みづらくなるので避けたいのであれば、以下のようにして、タップしたことを通知するためのdelegateを使うのはどうでしょうか？

protocol hogeDelegate {
    func tapEventHandler()
}

class hogeView: UIView {

    var delegate: hogeDelegate?

    func hogeTap(){
    //Viewのタップを検知した時に送る
        delegate?.tapEventHandler()
    }
}

class HogeVC: UIViewController, hogeDelegate {
    
    override func viewDidLoad() {
        super.viewDidLoad()

        hogeView.delegate = self

    }
    
    func tapEventHandler() {
        //ここのメソッドの中に処理を書きたい
        print("tapしました。")
    }
}