0

Code

var stepRatingView: AXRatingView = AXRatingView(frame: CGRectZero)

stepRatingView.sizeToFit()
stepRatingView.stepInterval = 1.0
stepRatingView.addTarget(self, action: Selector(ratingChanged(stepRatingView)), forControlEvents: UIControlEvents.ValueChanged)
self.secondView.addSubview(stepRatingView)

func ratingChanged(sender: AXRatingView){
    print(stepRatingView.value)
    self.label.text =  String(sender.value)
}

どうしたいのか

Amazonなどで見られる「★★★★★」の評価を、スライドで行おうと思い、
コードをSwiftに読み替えています。

Objective-Cのコードではラベルの値が変化するのですが、
Swiftに変換するとラベルの値が変化しません。

うまくaddTargetのactionが接続できていないのでしょうか。

1

1 件の回答 1

3

Selectorインスタンスの作り方が間違っています。

Selector(ratingChanged(stepRatingView))と言うのは、Selectorのコンストラクターに、引数としてratingChanged(stepRatingView)と言う メソッドを呼び出した結果 を渡しています。(従ってこの行が実行された時にratingChanged(_:)が実行されているはずです。)ratingChanged(_:)の戻り値はVoidですので、結局Selector()と同じ結果になります。

Selectorインスタンスをコンストラクターで作成する場合には、Selector("Objective-C形式のメソッド名")の形を使います。あなたのコードだとこんな感じです。

    stepRatingView.addTarget(self, action: Selector("ratingChanged:"), forControlEvents: .ValueChanged)

ただし、Xcode7.3以降をお使いでしたら、(Swift3への準備として)#selector構文を使われた方が良いでしょう。

短い書き方(同じクラス内のメソッドで、オーバーロードされていない場合)

    stepRatingView.addTarget(self, action: #selector(ratingChanged), forControlEvents: .ValueChanged)

長い書き方(できるだけ曖昧になりにくい書き方)

    stepRatingView.addTarget(self, action: #selector(ViewController.ratingChanged(_:)), forControlEvents: .ValueChanged)

(ViewControllerは、ご自分のview controllerクラス名に置き換えてください。)

#selector構文を使うとコンパイル時に指定されたメソッドが存在するかどうかのチェックをしてくれますので、「addTarget(_:action:forControlEvent:)は確かに実行されているのにactionメソッドが呼ばれない」と言ったミスを減らすことができます。お試しください。

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

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