PickerViewの使い方

Question

PickerViewを用いた実装をしたいのですが、
想定通りに動かず困っております。

下記コードでpickerViewを2つ表示することができたのですが、
ドラムロールの中身が両方一緒になってしまいます。
しかし、実際に選択してみるとtextLabelには互いに違う選択内容が反映されております。
どこを訂正すれば良いのでしょうか。

import UIKit

class ViewController: UIViewController {
    
    @IBOutlet weak var textField: UITextField!
    @IBOutlet weak var textField2: UITextField!
    
    var pickerView: UIPickerView = UIPickerView()
    var pickerView2: UIPickerView = UIPickerView()
    
    
    let list: [String] = ["メインクーン", "シャム", "ロシアンブルー", "アメリカンショートヘア", "ネベロング", "ビクシーボブ", "ラガマフィン", "ラパーマ"]
    let tono: [String] = ["tomtooof", "シャム", "dvecs v", "アedfvdvcvdvdev", "ネevdfvdsvグ", "ビクdvsdvsdv", "ラガマdvdsン", "ラパーsdvs sd!"]
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        // ピッカー設定
        pickerView.delegate = self
        pickerView.dataSource = self
        pickerView.showsSelectionIndicator = true
        
        pickerView2.delegate = self
        pickerView2.dataSource = self
        pickerView2.showsSelectionIndicator = true
        // 決定バーの生成
        let toolbar = UIToolbar(frame: CGRect(x: 0, y: 0, width: view.frame.size.width, height: 35))
        let spacelItem = UIBarButtonItem(barButtonSystemItem: .flexibleSpace, target: self, action: nil)
        let doneItem = UIBarButtonItem(barButtonSystemItem: .done, target: self, action: #selector(done))
        toolbar.setItems([spacelItem, doneItem], animated: true)
        // インプットビュー設定
        textField.inputView = pickerView
        textField.inputAccessoryView = toolbar
        
        textField2.inputView = pickerView2
        textField2.inputAccessoryView = toolbar
    }
    // 決定ボタン押下
    @objc func done() {
        textField.endEditing(true)
        textField.text = "(list[pickerView.selectedRow(inComponent: 0)])"
        
        textField2.endEditing(true)
        textField2.text = "(tono[pickerView2.selectedRow(inComponent: 0)])"
    }
}

extension ViewController : UIPickerViewDelegate, UIPickerViewDataSource {
    
    // ドラムロールの列数
    func numberOfComponents(in pickerView: UIPickerView) -> Int {
        return 1
    }
    // ドラムロールの行数
    func pickerView(_ pickerView: UIPickerView, numberOfRowsInComponent component: Int) -> Int {

        return list.count
    }
    
    // ドラムロールの各タイトル
    func pickerView(_ pickerView: UIPickerView, titleForRow row: Int, forComponent component: Int) -> String? {
        return list[row]
    }
}

Answer 1

そもそもpickerviewが2つも必要でしょうか？
同時に2個表示する(できるのかな？？？)つもりがないなら、
2個もいらないのではないでしょうか？

通常は1個用意し、
選んだtextFieldにより表示内容を切り替えるのが普通の使い方だと思いますよ。
textFieldにtagをつけて、選択されたtextFieldのtagによって、
読み込むリストを切り替えるようにコードを書けば解決すると思います。

thyda.eiquaさんが投稿しているようにtagをtextFieldにつけて
pickerView.tagではなく、textField.tagで条件分岐をしてくださいね。
あと、textField didselectにpickerView.reloadInputViews()を忘れないように。

Comments

[tonosam]

なるほど参考になります！
詳細説明感謝いたします！！！！

[thyda.eiqau]

ああ、そうですね、確かに……何も考えずに言われてみたままのコードを提示していました。追記修正しておきます。

[tonosam]

お願いいたします！！！

Answer 2

// ピッカー設定
pickerView.delegate = self
pickerView.dataSource = self
pickerView.showsSelectionIndicator = true
pickerView.tag = 1;

pickerView2.delegate = self
pickerView2.dataSource = self
pickerView2.showsSelectionIndicator = true
pickerView2.tag = 2;

// ドラムロールの行数
func pickerView(_ pickerView: UIPickerView, numberOfRowsInComponent component: Int) -> Int {
    switch(pickerView.tag) {
    case 1: return list.count;
    case 2: return tono.count;
    default: return 0;
    }
}

// ドラムロールの各タイトル
func pickerView(_ pickerView: UIPickerView, titleForRow row: Int, forComponent component: Int) -> String? {
    switch(pickerView.tag) {
    case 1: return list[row];
    case 2: return tono[row];
    default: return nil;
    }
}

追記修正

hamejiさんの回答をうけて修正します。上記のコードでも動きますが、現実的にpickerViewが2つ必要になるシーンは無いと思いますので……
また、同様の質問が過去にあったようです。あわせてご確認ください。

import UIKit

class ViewController: UIViewController, UITextFieldDelegate, UIPickerViewDelegate, UIPickerViewDataSource {
    // MARK: - Properies Define
    @IBOutlet weak var textField: UITextField!
    @IBOutlet weak var textField2: UITextField!
    var pickerView: UIPickerView = UIPickerView()
    let list: [String] = ["メインクーン", "シャム", "ロシアンブルー", "アメリカンショートヘア", "ネベロング", "ビクシーボブ", "ラガマフィン", "ラパーマ"]
    let tono: [String] = ["tomtooof", "シャム", "dvecs v", "アedfvdvcvdvdev", "ネevdfvdsvグ", "ビクdvsdvsdv", "ラガマdvdsン", "ラパーsdvs sd!"]

    // いま編集しようとしている textField の tag
    var inEditingTextFieldTag: Int = 0;

    // MARK: - View Lyfecycle
    override func viewDidLoad() {
        super.viewDidLoad()

        // ピッカー設定
        pickerView.delegate = self
        pickerView.dataSource = self
        pickerView.showsSelectionIndicator = true

        // 決定バーの生成
        let toolbar = UIToolbar(frame: CGRect(x: 0, y: 0, width: view.frame.size.width, height: 35))
        let spacelItem = UIBarButtonItem(barButtonSystemItem: .flexibleSpace, target: self, action: nil)
        let doneItem = UIBarButtonItem(barButtonSystemItem: .done, target: self, action: #selector(done))
        toolbar.setItems([spacelItem, doneItem], animated: true)

        // インプットビュー設定
        textField.inputView = pickerView
        textField.inputAccessoryView = toolbar
        textField.tag = 1
        textField.delegate = self
        textField2.inputView = pickerView
        textField2.inputAccessoryView = toolbar
        textField2.tag = 2
        textField2.delegate = self
    }
    
    // !!!: 中略

    // MARK: - UITextFieldDelegate
    /// いま編集しようとしている textField の tag を Property に保持しておく
    func textFieldDidBeginEditing(_ textField: UITextField) {
        inEditingTextFieldTag = textField.tag
        pickerView.reloadInputViews()
    }

    // MARK: - UIPickerViewDataSource
    // ドラムロールの列数
    func numberOfComponents(in pickerView: UIPickerView) -> Int {
        return 1
    }

    // ドラムロールの行数
    func pickerView(_ pickerView: UIPickerView, numberOfRowsInComponent component: Int) -> Int {
        // 将来的に textField の数が増えたらcase文を増やしていく
        switch(inEditingTextFieldTag) {
        case 0: return list.count;
        case 1: return tono.count;
        default: return 0;
        }
    }

    // MARK: - UIPickerViewDataSource
    // ドラムロールの各タイトル
    func pickerView(_ pickerView: UIPickerView, titleForRow row: Int, forComponent component: Int) -> String? {
        // 将来的に textField の数が増えたらcase文を増やしていく
        switch(inEditingTextFieldTag) {
        case 0: return list[row];
        case 1: return tono[row];
        default: return nil;
        }
    }
}

Comments

[tonosam]

解決いたしました！！！
本当にありがとうございます！

[hameji]

上記でもできますが、正確には無駄なpickerView生成をしており誤りだと思います。

[tonosam]

今後の実装内容としましては、同じようなpickerViewを５つ配置する予定なので、よろしければ無駄なPickerViewに関して教えていただければ幸いです。、、

[tonosam]

なるほど。。。
ここまでかなり明確に書いて頂き、これ以上にくらい嬉しいです。。

本当にありがとうございました！！！！！！！