ios 8 UIButton

Question

質問です。

今までsimulatorのバージョン9.3.1でビルドしてたのですが、
8.0もビルドしたくて、インストールしたのですが、

import UIKit

class CustomView: UIView {

    override init(frame: CGRect) {
        super.init(frame: frame)
        backgroundColor = UIColor.whiteColor()
        let button = UIButton()
        button.frame.size = CGSize(width: 100, height: 100)
        button.frame.origin = CGPoint(x: 100, y: 100)
        button.backgroundColor = UIColor.blueColor()
        button.addTarget(ViewController(), action: #selector(ViewController.tapButton), forControlEvents: .TouchUpInside)
        self.addSubview(button)
    }
    
    required init?(coder aDecoder: NSCoder) {
        fatalError("init(coder:) has not been implemented")
    }
}

class ViewController: UIViewController {
    override func loadView() {
        super.loadView()
        let customView = CustomView(frame: view.frame)
        self.view = customView
    }

   func tapButton() {
       print("call")
   }
}

このコードがiOS9.3だと大丈夫なのですが、ios8.1のシュミレーターだと落ちてしまいます。
原因がわかる方いらっしゃいますか？

Comments

[fuzzball]

customViewとViewControllerの関係を教えて下さい。また、iOS9.3だと落ちないとのことですが、動作も正常なのでしょうか？

[_Kentarou]

こちらのコードではいろいろ間違えている部分があり、ビルドすらできません。
最低限自身が確認しているような動くコードを載せてください。

[tosi_1211]

コードを修正しました。
CustomViewはViewControllerのBaseViewです。
iOS9.3だと正常に動きます。

[fuzzball]

まだエラー出てますが？質問する気あるんでしょうか？あと、コードは ``` で囲って下さい。

[fuzzball]

Storyboard上の質問です。Viewのカスタムクラスはどうなっていますか？buttonはどこに配置していますか？

[tosi_1211]

すみませんstoryboardの画面をどうやって投稿していいかわからないので、
コードのみで書くと、上のコードです。

[tosi_1211]

どこでエラーが発生しているでしょうか？
自分の手元ではエラーがありません

[fuzzball]

まだエラー出てますが。新規にプロジェクトを作って試してみてはどうですか？

[tosi_1211]

新規でプロジェクト作って動作確認できました

Answer 1

fuzzballさんが解説している通りかと思います。

記載されているコードを変更する形で修正するとButtonのイベントを受けるViewControllerのインスタンスを初期化時に渡してそれに対してActionを設定するといいと思います。

class ViewController: UIViewController {
    override func loadView() {
        super.loadView()
        let customView = CustomView(vc: self, frame: view.frame)
        self.view = customView
        print(UIDevice.currentDevice().systemVersion)
    }
    
    func tapButton() {
        print("call")
    }
}

import UIKit
class CustomView: UIView {
    
    init(vc: ViewController, frame: CGRect) {
        super.init(frame: frame)
        backgroundColor = UIColor.whiteColor()
        let button = UIButton()
        button.frame.size = CGSize(width: 100, height: 100)
        button.frame.origin = CGPoint(x: 100, y: 100)
        button.backgroundColor = UIColor.blueColor()
        button.addTarget(vc, action: #selector(vc.tapButton), forControlEvents: .TouchUpInside)
        self.addSubview(button)
    }
    
    required init?(coder aDecoder: NSCoder) {
        fatalError("init(coder:) has not been implemented")
    }
}

Comments

[tosi_1211]

初期化の仕方とても参考になりました。ありがとうございます！

Answer 2

CustomViewのこの部分、

button.addTarget(ViewController(), action: ... )

これだと、新しいViewControllerを生成し、それに対してイベントを割り当てていますが、この新しいViewControllerはどこにも使われていないので、直後に解放されてしまっているのではないかと思います。
タップされたときにイベントの主（ぬし）がいないので落ちてしまうんじゃないかと。（想像）

iOS9で落ちないのは、イベントの飛び先が無かったときに、同名の関数（tapButton）を勝手に探しているんじゃないかと。（これも想像です）

で、直し方ですが、

1. CustomViewからViewControllerのインスタンスを拾えるようにして、addTarget時にそれを使う。
2. ViewControllerの中でtapButton()をbuttonにaddTargetする。
3. tapButton()をCustomViewの中に持ってくる。

などなど。

とりあえず3番が手っ取り早いですが。

Comments

[tosi_1211]

複数の解決方法のご提案ありがとうございます。