そもそもの元のコードですが、tableView(_:cellForRowAt:)
の中で、let cell = UITableViewCell()
のようにセルクラス(今の場合は、UITableViewCell
クラス)のイニシャライザを使ってセルのインスタンスを生成するのは、「誤り」と言ってしまって良いでしょう。
コンパイルエラーも実行時エラーも出ないのでやってしまいがちなのですが、システムに大きな負荷をかけ、スクロールにもやたらと時間がかかったりするようになります。
tableView(_:cellForRowAt:)
の中でセルのインスタンスを取得する場合、必ずdequeueReusableCell(withIdentifier:for:)
、またはdequeueReusableCell(withIdentifier:)
のどちらかのメソッドを使用して下さい。
CustomCell.swiftにCustomCell
と言うクラスが定義してあるとすると、例えば次のようになります。
override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
let cell = tableView.dequeueReusableCell(withIdentifier: "CustomCell", for: indexPath) as! CustomCell
let rp = repo[indexPath.row]
cell.textLabel?.text = rp["full_name"] as? String ?? ""
cell.detailTextLabel?.text = rp["language"] as? String ?? ""
cell.tag = indexPath.row
return cell
}
(実際のカスタムセルの場合、textLabel
やらdetailTextLabel
やらは、適切なプロパティ名に変えてやる必要があるでしょう。)
ちなみに危険な強制キャストas!
を使っていますが、これは「もし設定が間違えていたらアプリをクラッシュさせてくれ」と言う意味でわざとやっています。一度実行すれば確実に検出できるような場所では、このようにわざとクラッシュしてしまう書き方をする場合があります。
さて、上記のコードがクラッシュしないような設定ですが、UITableViewController
を使うなら、storyboard上で設定した方が楽かもしれませんが、ここではコードで設定する方法を書いておきます。
override func viewDidLoad() {
super.viewDidLoad()
// Do any additional setup after loading the view.
self.tableView.register(CustomCell.self, forCellReuseIdentifier: "CustomCell")
//...
}
最初のコードでwithIdentifier:
に"CustomCell"
と言う文字列を指定していますが、その文字列はCustomCell
と言うクラスを表しているんだよ、と言うのを登録することになります。
(本当は"CustomCell"
と言う文字列は定数宣言しておいた方が良いところです。)
実際に上記の修正をやってみて、どのように表示されるか試してみて下さい。うまくいかない点やよくわからない点があれば、またコメントなどでお知らせください。