XCode、Swift：ButtonとPickerViewの選択に合わせて画像を表示したい

Question

前提・実現したいこと

XCode、Swiftの質問です。Storyboardを使っています。

ButtonとPickerViewの選択に合わせた画像を表示したいです。

Buttonを３つ【イルカ】【犬】【猫】を設置し、PickerViewを１つ：データリスト【【青、赤、黄色】、【丸、四角】】を設置しました。
Buttonで最後に押された項目（例えば：イルカ）とPickerViewの１つ目のコンポーネントで選択されたデータ（例えば：青）を組み合わせた画像（例えば：青いイルカ）を表示するアプリを作成したいです。

用意している画像（と画像名、すべてpng file）は次の９つです。
青いイルカ(iruka-1)、青い犬(inu-1)、青い猫(neko-1)、
赤いイルカ(iruka-2)、赤い犬(inu-2)、赤い猫(neko-2)、
黄色いイルカ(iruka-3)、黄色い犬(inu-3)、黄色い猫(neko-3)

発生している問題・エラーメッセージ

Buttonを押したときに画像が表示されますが、PickerViewで選択が起こったときには表示されません。
PickerViewで選択した際にも該当の画像が表示されるようにしたいです。

該当のソースコード

import UIKit


class ViewController: UIViewController,UIPickerViewDelegate,UIPickerViewDataSource {
    //Storyboardとの設定・紐付け

    @IBOutlet weak var ImageView: UIImageView!

    @IBOutlet var pickerview: UIPickerView!
    
    @IBOutlet var Button01: UIButton!
    @IBOutlet var Button02: UIButton!
    @IBOutlet var Button03: UIButton!
    
    //imageList(画像名の割当)
    let imageList = [["1","2","3"],["maru","shikaku"]]
    
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
    }
    //UIPickerViewの列の数を返す
    func numberOfComponents(in pickerView: UIPickerView) -> Int {
        return imageList.count
     }
    
    //UIPickerViewの行数、リストの数を返す
    func pickerView(_ pickerView: UIPickerView, numberOfRowsInComponent component: Int) -> Int{
        return imageList[component].count
    }
    
    //UIPickerViewの画面での表示を指定する
    func pickerView(_ pickerView: UIPickerView, titleForRow row: Int, forComponent component: Int) -> String?{
        return imageList[component][row]
    }  
    
    //画像を表示するfunc
    func view_image(file_image: String){
        let  file_image2 = "(imageList[pickerview.selectedRow(inComponent:0)]).png"
        ImageView.image = UIImage(named: "(file_image)(file_image2)")
    }

    //Buttonが押されたときのAction
    
    @IBAction func tapButton01(_ sender: Any){
        let file_image = "iruka-"
        view_image(file_image: file_image)
    }
    @IBAction func tapButton02(_ sender: Any){
        let file_image = "inu-"
        view_image(file_image: file_image)
    }
    
    @IBAction func tapButton03(_ sender: Any) {
        let file_image = "neko-"
        view_image(file_image: file_image)
    }
         
    //UIPickerViewが選択されたときの動作
    func pickerView(_ pickerView: UIPickerView, didSelectRow row: Int, inComponent component: Int){

        view_image(file_image: file_image)
    }
    

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


}
 

試したこと

pickerView内に引数を受け渡すことができない＋Buttonで設定したfile_imageを@IBAction外で引き継げないという２つの問題がありそうです。

補足情報

Storyboard(UIkit)を使っています。
よろしくお願いいたします。

Comments

[tomato879241]

func pickerView(_ pickerView: UIPickerView, didSelectRow row: Int, inComponent component: Int)
のdelegate methodの使い方が間違っています。そのdelegate methodではそれぞれのcomponentの選択されたものが返ります。

pickerView.selectedRow(inComponent: 0)

は最初のcomponentで選択されたindexが返るので、imageList[pickerView.selectedRow(inComponent: 0)]が選択されたarrayの要素になります。

[yohoho]

ご指摘ありがとうございます。事情により元コードを載せることが出来ず、簡略化させて書こうとしたら色々と間違えていました。修正いたしましたので、よろしくお願いいたします。

[hoshi-takanori]

file_image を ViewController のプロパティにすれば良いのでは…。
あと、let file_image2 = imageList[0][pickerview.selectedRow(inComponent: 0)] のような。

[yohoho]

ありがとうございます。
>>file_imageをViewControllerのプロパティにすれば良いのでは

すみません、こちら具体的にどういう意味かご教示いただけますと幸いです…

>>let file_image2 = imageList[0][pickerview.selectedRow(inComponent: 0)] のような。
おっしゃる通りです。ご指摘ありがとうございます。

Answer 1

とりあえずこんな感じでいかがでしょうか。

 class ViewController: UIViewController,UIPickerViewDelegate,UIPickerViewDataSource {
     // 略
 
+    var file_image = "iruka-" // file_image を ViewController のプロパティにする
 
     override func viewDidLoad() {
         super.viewDidLoad()
         // Do any additional setup after loading the view.
 
+        view_image() // 最初に画像を表示したかったらここで。
     }
 
     // 略
 
     //画像を表示するfunc
-    func view_image(file_image: String){
-        let  file_image2 = "(imageList[pickerview.selectedRow(inComponent:0)]).png"
+    func view_image() { // file_image をプロパティにしたので、引数は不要。
+        let file_image2 = imageList[0][pickerview.selectedRow(inComponent: 0)]
         ImageView.image = UIImage(named: "(file_image)(file_image2)")
     }
 
     //Buttonが押されたときのAction
 
     @IBAction func tapButton01(_ sender: Any){
-        let file_image = "iruka-"
-        view_image(file_image: file_image)
+        file_image = "iruka-" // ローカル変数ではなくプロパティを更新。
+        view_image()
     }
 
     // 略 (tapButton02, tapButton03 も tapButton01 と同様)
 
     //UIPickerViewが選択されたときの動作
     func pickerView(_ pickerView: UIPickerView, didSelectRow row: Int, inComponent component: Int){
 
-        view_image(file_image: file_image)
+        view_image()
     }

Comments

[yohoho]

ありがとうございます！！できました！！
本当に助かりました。
ありがとうございます！！