0

以下のコードのCellの部分を別のCustomCell.swiftなどで実装したいのですが、どうすればよいのでしょうか?

override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = UITableViewCell()
        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
}

2 件の回答 2

0

そもそもの元のコードですが、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"と言う文字列は定数宣言しておいた方が良いところです。)


実際に上記の修正をやってみて、どのように表示されるか試してみて下さい。うまくいかない点やよくわからない点があれば、またコメントなどでお知らせください。

2
  • 実装自体はうまくいったのですが、CustomCell.swiftが空になっています。うまく活用するためにはなにをいれたほうがいいのでしょうか?
    – zunda
    Commented 2020年7月2日 11:52
  • @zunda さん、「うまくいった」と言うのが、ちゃんと意図したとおりの表示になっていると言うのであれば、CustomCell.swiftには、(ほとんど空の)CustomCellクラスの定義があればそれで構いません。(その場合、クラス定義も必要なかったかもしれないんですが、それはまた別の問題。)「実装自体はうまくいったけれど、他にうまくいっていない点がある」と言うのであれば、具体的にその内容をお知らせください。
    – OOPer
    Commented 2020年7月2日 12:28
0

viewDidLoad等、UITableViewのインスタンスをメンバーに持つクラスで、UITableViewを使い始める前に対象のUITableViewのインスタンスに
register(_:forCellReuseIdentifier:)か、register(_ nib: UINib?, forCellReuseIdentifier identifier: String)で、このidentifierにはこのカスタムセル(のクラス)を使ってねと登録する必要があります。

ただし、上のソースで

let cell = UITableViewCell()

と毎回新規にcellを作成していますが、
dequeueReusableCell(withIdentifier identifier: String) -> UITableViewCell?や、dequeueReusableCell(withIdentifier identifier: String, for indexPath: IndexPath) -> UITableViewCellで、再利用可能なセルがないか確認した方がセルを毎回新規に作るよりパフォーマンスが向上すると思います。

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

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