1

カメラロールから選んだ画像をimageViewに描写している状態で、ボタンをタップするとセピア色や白黒にフィルターをかける機能を実装しています。
さらに、ユーザがUISliderを左右にドラッグしてセピア色や白黒の濃淡具合を選択できるような機能を実装したいと思い
独学でコーディングしたのですが、ここから先がわかりません。

import UIKit
class ViewController: UIViewController, UINavigationControllerDelegate, UIImagePickerControllerDelegate {
    @IBOutlet var mainImageView: UIImageView!
    @IBAction func pickImageFromLibrary() {
        if UIImagePickerController.isSourceTypeAvailable(.PhotoLibrary) {
            let controller = UIImagePickerController()
            controller.delegate = self
            controller.sourceType = UIImagePickerControllerSourceType.PhotoLibrary
            self.presentViewController(controller, animated: true, completion: nil)
        }
    }
    override func viewDidLoad() {
        super.viewDidLoad()
        let slider = UISlider()
        slider.frame = CGRect(x: 80, y: 380, width: 200, height: 40)
        slider.minimumValue = 0.0
        slider.maximumValue = 1.0
        slider.addTarget(self, action: "sepiaFilter:", forControlEvents: UIControlEvents.ValueChanged)
        slider.addTarget(self, action: "monoFilter:", forControlEvents: UIControlEvents.ValueChanged)
        self.view.addSubview(slider)
    }
    func imagePickerController(picker: UIImagePickerController,
    didFinishPickingImage image: UIImage, editingInfo: [String : AnyObject]?) {
        self.mainImageView.image = image
        (self.view.viewWithTag(1) as! UIButton).setImage(image, forState: .Normal)
        (self.view.viewWithTag(2) as! UIButton).setImage(sepiaFilter(image, slider: UISlider()), forState: .Normal)
        (self.view.viewWithTag(3) as! UIButton).setImage(monoFilter(image, slider: UISlider()), forState: .Normal)
        picker.dismissViewControllerAnimated(true, completion: nil)
    }
    func sepiaFilter(image: UIImage, slider: UISlider) -> UIImage {
        let sepiaFilter = CIFilter(name: "CISepiaTone")
        sepiaFilter!.setDefaults()
        let ciImage: CIImage = CIImage(image: image)!
        sepiaFilter!.setValue(ciImage, forKey: kCIInputImageKey)
        sepiaFilter!.setValue(slider.value, forKey: "inputIntensity")
        if let outputImage = sepiaFilter!.valueForKey("outputImage") as? CIImage {
            let manipulatedImage = UIImage(CIImage: outputImage)
            self.mainImageView.image = manipulatedImage
       }
        return mainImageView.image!
    }
    func monoFilter(image: UIImage, slider: UISlider) -> UIImage {
        let monoFilter = CIFilter(name: "CIColorControls")
        monoFilter!.setDefaults()
        let ciImage = CIImage(image: image)!
        monoFilter!.setValue(ciImage, forKey: kCIInputImageKey)
        monoFilter!.setValue(slider.value, forKey: "inputSaturation")
        if let outputImage = monoFilter!.valueForKey("outputImage") as? CIImage {
            let manipulatedImage = UIImage(CIImage: outputImage)
            self.mainImageView.image = manipulatedImage
        }
        return mainImageView.image!
    }
    @IBAction func filterTapped(sender:UIButton) {
        switch sender.tag {
        case 1:
            mainImageView.image = (self.view.viewWithTag(1) as! UIButton).imageView?.image
        case 2:
            mainImageView.image = (self.view.viewWithTag(2) as! UIButton).imageView?.image
        default:
            break
        }
    }
}

上記のコードで実機テストをすると、UISliderをドラッグしたときにエラーとなります。
(ビルド前の、エラーマークは無い状態です)

UISlider作成するときのaddTarget()の括弧内で、action: "sepiaFilter:"と"monoFilter:"
を混在させているため、エラーになっていると思うのですが、
文法的にどういった記述をして修正すべきでしょうか。

※追記
説明不足ですみません。
デバッグエリアの表示内容など、下記の通りです。
画像の説明をここに入力

画像の説明をここに入力

画像の説明をここに入力

2
  • ドラッグしたときにエラーとなります についてですが、どのようなエラーが出たのか、デバッグコンソールの出力やスタックトレースの状態、デバッガーがどの行を指しているかなどの状況を可能な限り詳しく質問に含めた方が、早く解決に近づく可能性が高まります。
    – OOPer
    Commented 2016年7月31日 6:09
  • エラー情報の追記ありがとうございました。ちなみにテキストでのエラーメッセージなど削除してしまわれたようですが、回答に引用したい場合もありますので、画像を添付された場合でもテキストベースの情報も残していただいた方が良いでしょう。とりあえずご掲載のエラーの原因と対処方法について回答しましたので、そちらをご確認ください。
    – OOPer
    Commented 2016年7月31日 7:50

1 件の回答 1

1

(編集メモ)sepiaFilter, monoFilterがフィルター用の関数とactionメソッドの両方に使用されていたのに気づかないまま不適切な方法を勧めることになっていたので、修正しました。


あなたが引用されたエラーメッセージで一番肝心なのはこちらになります。

[_picFilter_2016_0725.ViewController sepiaFilter:]: unrecognized selector sent to instance 0x1553f930'

これは「(あなたのアプリの)ViewControllerクラスのインスタンスにsepiaFilter:と言う認識されていないselectorが送られた」と言っています。selectorと言うのは、Objective-C用語ですが、iOSプログラミングでは必須の概念ですのでうっすらくらいには理解しておられると思いますが、要は「ViewControllerクラスにはsepiaFilter:に対応するメソッドは定義されていないよ」と言っていることになります。

あなたのクラスには、

func sepiaFilter(image: UIImage, slider: UISlider) -> UIImage {
    //...
    return mainImageView.image!
}

というメソッドがあるわけですが、このメソッドの(Objective-C表現での)selectorはsepiaFilter:slider:と言うものになります。(Swift2までの話でSwift3では変わります。)

単にsepiaFilter:と言うセレクタを記述すると、それは引数が1個しかない、

func sepiaFilter(param: SomeParamType) -> SomeReturnType {
    //...
    return returnTypeObject
}

と言う型式のメソッドを指定したことになります。
(Objective-Cのselectorでは、戻り値があるかどうか、パラメータや戻り値の型は関係ありませんが、後で述べるtarget-action用のメソッドでは戻り値を返してはいけません。)

「イベントが発生したらsepiaFilter:ってメソッドを呼べって言われたのに、そんなメソッドないじゃんかよぉ!!!」と怒っているので、対処方法はどれかになります。

(1) action:引数に既に存在するメソッドの正しいselectorを指定する
(2) action:引数で指定したメソッドを定義してやる
(3) action:引数で指定した形に合うように既存のメソッドを修正してやる
(4) action:引数の指定を修正した上で、そのメソッドを定義してやる

iOSのUIControl(UISliderもその一種)では、target-actionで指定できるメソッドは、次のどれかの形でなければいけません。

UIControl Class Reference

Listing 1 Swift

@IBAction func doSomething()
@IBAction func doSomething(sender: UIButton)
@IBAction func doSomething(sender: UIButton, forEvent event: UIEvent)

(戻り値は省略していないのではなく、戻り値なしというのが指定です。)

この中の2番目の形式が、あなたの指定したselectorsepiaFilter:に合う引数1つのメソッドですが、あなたのsepiaFilter(_:slider:)メソッドはフィルター関数としても使われているので、そのまま修正してやっても「戻り値なし」の上記の形に合わせることができません。
ですので、ここではactionメソッドをフィルター関数とは別に追加する形の上記(2)の対応が必要でしょう。

@IBAction func sepiaFilter(sender: UISlider) {
    guard let image = self.mainImageView.image else {
        fatalError("something is wrong with mainImageView...")
    }
    self.mainImageView.image = sepiaFilter(image, slider: sender)
}
func sepiaFilter(image: UIImage, slider: UISlider) -> UIImage {
    guard let sepiaFilter = CIFilter(name: "CISepiaTone") else {
        fatalError("something is wrong with CIFilter...")
    }
    sepiaFilter.setDefaults()
    guard let ciImage: CIImage = CIImage(image: image) else {
        print("cannot create CIImage...")
        return image
    }
    sepiaFilter.setValue(ciImage, forKey: kCIInputImageKey)
    sepiaFilter.setValue(slider.value, forKey: "inputIntensity")
    if let outputImage = sepiaFilter.valueForKey("outputImage") as? CIImage {
        let manipulatedImage = UIImage(CIImage: outputImage)
        return manipulatedImage //フィルター関数ではmainImageViewは変更しないで修正後のイメージを戻り値として返す
    }
    return image
}

※「私のアプリをクラッシュさせて演算子」("Crash Me operator"!)は出来るだけ使わない方が良いのとボタンイメージ用のフィルター処理でmainImageViewの方まで変更する必要はないはずなのでフィルター関数の方も若干修正させていただきました。またプログラム中でaddTargetする場合には@IBActionは要らないのですが、フィルター関数側と明示的に区別できるので、そのまま付けておきます。

当然ながら、こちらを修正したら、ほぼ同じ状態になっていたmonoFilter:の方にも同様の修正をしてやらないと、今度はそちらが実行時エラーになってしまいます。

かなり長文になってしまいましたので、こちらのご質問の方の主題は追記にお示しいただいたエラーを解消することとして、一旦ここで置かせていただきます。

1
  • 回答頂き有難うございます。現在最新のXcodeにアップデートして、#selector構文を使ってコーディングをしているのですが、新たなエラーが発生しまして、またあらためて質問させて頂きます。
    – jack_three
    Commented 2016年8月1日 7:25

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

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