Picker viewの実装、表示の？

Question

前提・実現したいこと

Pickerviewを１つの画面に２つ作りたいのですが、その前段階で行き詰まっています。

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

pickerの表示が「？」になっています。

Simulatorでは

Storyboard上

該当のソースコード

class ViewController: UIViewController,UIPickerViewDelegate,UIPickerViewDataSource {
  
    @IBOutlet weak var label1: UILabel!
    
    @IBOutlet weak var label2: UILabel!
    
    @IBOutlet weak var picker1: UIPickerView!
    
    @IBOutlet weak var picker2: UIPickerView!
    
    var data = ["1","2","3"]
    
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view, typically from a nib.
        //これら2つを追加
        picker1.delegate = self
        picker1.dataSource = self
        
        label1.text = "2"
     self.view.addSubview(picker1)
        self.view.addSubview(picker2)
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }

    //表示する列の数
    func numberOfComponents(in pickerView: UIPickerView) -> Int {
        return 1
    }
    //pickerの行数はリスト内容の数
    func pickerView(_ pickerView: UIPickerView, numberOfRowsInComponent component: Int) -> Int {
        return data.count
    }
    //
    func pickerView(namePickerview: UIPickerView, titleForRow row: Int, forComponent component: Int)-> String! {
        
       return data[row]
    }
    //pickerが選択されたときラベルに表示する
    func pickerView(_ pickerView: UIPickerView,
                    didSelectRow row: Int,
                    inComponent component: Int) {
        
        
        label1.text = data[row]
        
    }
}

試したこと

こちらのサイトをみて記述してみました
https://qiita.com/hijion/items/a93971d8bd945e771be7

補足情報（FW/ツールのバージョンなど）

macOS High Sierra10.18
Xcode9.4

Answer 1

#1

picker2のdelegateとdataSourceが設定されていません。

#2

質問には直接関係はありませんが、Storyboard上で配置しているのであれば下記のコードは不要です。

self.view.addSubview(picker1)
self.view.addSubview(picker2)

#3

pickerView(_:titleForRow:forComponent:)が微妙に間違っているので呼び出されていません。
_と?です。

//現在のコード
func pickerView(namePickerview: UIPickerView, titleForRow row: Int, forComponent component: Int)-> String!

//正しいコード
func pickerView(_ namePickerview: UIPickerView, titleForRow row: Int, forComponent component: Int)-> String?

Comments

[LFOHP]

やってみましたが画面の下方(picker2)に、同様に「？」がでます。何かの設定が足りないか、間違えているのでしょうか？

[fuzzball]

何をやったのでしょうか？

[LFOHP]

picker1.dataSource = selfの下に、picker2.delegate = self、picker2.dataSource = selfを追加しました。

[daisuke7]

#3 が結構重要です。
参考URLのコードは Swift 2系のため、第一パラメータの扱いが違います。
Swift 3以降では func pickerView(_ namePickerview: … のように "_" を付けないと、
iOSから呼ばれません。
もし混乱するようであれば、Swift2系のサンプルコードは参考にすべきではありません。

[fuzzball]

>>daisuke7さん
UIPickerViewDataSourceの2つは修正できてるんですけどね‥と思ったら、こっちはrequiredだから無いとエラーになりますね。（なぜ pickerView(_:titleForRow:forComponent:) はoptionalなんだろう？）

[daisuke7]

https://developer.apple.com/documentation/uikit/uipickerviewdelegate

The methods in this group are marked @optional.
However, to use a picker view, you must implement
either the pickerView(_:titleForRow:forComponent:) or
the pickerView(_:viewForRow:forComponent:reusing:) method
to provide the content of component rows.

pickerView(_:viewForRow:forComponent:reusing:) でViewを作って返したいひともいるから、
ってことですね。

[fuzzball]

これは失礼しました。3種類ありますね。
「どれか必須」でエラーにしてくれるといいんですけどね。

[LFOHP]

早速試して、動作しました。swiftの理解ができていないので勉強します。これから２つめのpickerを実装することに挑戦してみます。ありがとうございました