0

iOS(Swift)でアプリを開発しています。
カスタムViewを作成し、あるViewControllerにaddSubviewし、そのViewControllerにカスタムViewから呼び出されるデリゲートメソッドを実装しています。
このとき、以下のようにCustomView内のUIButtonから直接メソッドの呼び出しを行うと、"unrecognized selector sent to instance xxxx"と、メモリに関するエラーでアプリが落ちてしまいます。

SampleViewController

class SampleViewController: UIViewController {

    override func viewDidLoad() {
        super.viewDidLoad()

        let customView = CustomView(frame: CGRectMake(0, 0, self.view.frame.width, self.view.frame.height))
        self.view.addSubview(customView) 
    }

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

    func sampleFunc() {
        print("sampleFunc")
    }
}

CustomView

class CustomView: UIView {
    let button = UIButton(frame: CGRectZero)

    func setup() {
        button.addTarget(self, action: #selector(SampleViewController.sampleFunc(_:)), forControlEvents: .TouchUpInside)
        self.addSubview(button)
    }

    override init(frame: CGRect) {
        super.init(frame: frame)
        setup()
    }

    required init?(coder aDecoder: NSCoder) {
        fatalError("init(coder:) has not been implemented")
    }

    override func layoutSubviews() {
        // 略
    }
}

一方で、CustomView内に別のメソッドを用意し、そのメソッドから間接的にViewController内のデリゲートメソッドを呼び出した場合は正常に動作しました。

CustomView(修正後)

class CustomView: UIView {
    let button = UIButton(frame: CGRectZero)

    func setup() {
        button.addTarget(self, action: #selector(CustomView.buttonAction(_:)), forControlEvents: .TouchUpInside)
        self.addSubview(button)
    }

    override init(frame: CGRect) {
        super.init(frame: frame)
        setup()
    }

    required init?(coder aDecoder: NSCoder) {
        fatalError("init(coder:) has not been implemented")
    }

    override func layoutSubviews() {
        // 略
    }

    @objc private func buttonAction(sender: AnyObject) {
        UIApplication.sharedApplication().sendAction(#selector(SampleViewController.sampleFunc(_:)), to: nil, from: self, forEvent: nil)
    }

}

これがなぜ間接的に呼び出した場合は成功するのかがわからず、お分かりになる方がいれば教えていただきたいです。宜しくお願いします。

1 件の回答 1

0

メモリ管理は関係ないですね。解放済みのオブジェクトにアクセスした場合も同様の例外が起きることがありますが、今回のはオブジェクトが解放されてるのではなく、文字通り間違ったオブジェクトに対してメソッドを呼んでいます。

addTarget()メソッドの最初の引数targetはどのオブジェクトがselectorで指定したメソッドを呼ぶかという意味ですので、書かれたコードですとCustomView のselfを指定しているので、CustomViewのsampleFunc()を呼ぶ、ということが起こります。

後者の場合に、なぜ同じ例外が起きないかというと、後者もほぼ同様のことをしていますが、sendAction()メソッドの同じ意味の引数であるto引数にnilを指定しているので、メソッドを呼び出すオブジェクトを指定しないことになります。
指定しなかった場合、ボタンからビューとビューコントローラの親子関係を遡って、同じ名前のメソッド(someFunc)を持つオブジェクトを探して、見つかった時点でそのオブジェクトに対して呼びだす、という動きをします。
(レスポンダチェーンといいます。厳密には遡るのはレスポンダチェーンを逆に辿る、ということになります。)

先のaddTarget()メソッドでも、target引数を指定しなければ同じ挙動になります。ですので

`button.addTarget(nil, action: #selector(CustomView.buttonAction(_:)), forControlEvents: .TouchUpInside)

上記のように、最初の引数targetnilにするとクラッシュせず、同じ結果になります。

本来はレスポンダチェーンに任せるのではなく、正しいオブジェクトを指定するべきですので、きちんと修正するならCustomViewにビューコントローラのオブジェクトを渡し、それをボタンのアクションaddTarget()メソッドの引数に指定する、などになります。

2
  • @kishikawaさん詳しく解説いただきありがとうございます。ご回答の最後の「本来はレスポンダチェーンに任せるのではなく、正しいオブジェクトを指定するべきですので、きちんと修正するならCustomViewにビューコントローラのオブジェクトを渡し、それをボタンのアクションaddTarget()メソッドの引数に指定する、などになります。」というのをやるとするならば、下記のような書き方で問題ないのでしょうか。 let vc = SampleViewController() button.addTarget(vc, action: #selector(vc.buttonAction(_:)), forControlEvents: .TouchUpInside)
    – Dave
    Commented 2016年4月15日 10:25
  • @Dave selfではなく、ビューコントローラのインスタンスを渡すのは合ってますが、直前に新しく作ったインスタンスを渡しても期待通りの動作(ボタンを押すと表示中の画面のビューコントローラのメソッドが呼ばれる)にはならないと思います。おそらくやりたいことを実現するにはCustomViewにSampleViewController型のプロパティを作り、それにSampleViewController型のインスタンスをセットし、それをaddTarget()に渡す、となるでしょう。よくわからなければ、別の質問を作ったほうがいいです。 Commented 2016年4月15日 13:34

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。