スウィフトのconfigureカスタムセル

Question

私はのUITableViewコントローラで、このコードを持っている：

var results: [Item] = [] 
... 
func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell { 
    let cell = tableView.dequeueReusableCellWithIdentifier("OrderCell") as! OrderTableViewCell! 
    cell.item = results[indexPath.row] 
    return cell 
} 

とOrderTableViewCell.swift：

class OrderTableViewCell: UITableViewCell { 


    var item: Item! { 
     didSet { 
      self.setupCell() 
     } 
    } 

    @IBOutlet var itemNo: UILabel! 
    @IBOutlet var itemPrice: UILabel! 

    override func awakeFromNib() { 
     super.awakeFromNib() 
    } 

    override func setSelected(selected: Bool, animated: Bool) { 
     super.setSelected(selected, animated: animated) 
    } 

    func setupCell() { 
     itemNo.text = order.document_no 
     itemPrice.text = order.sum 
    } 

} 

は今、私は知っている必要がありますがdidSetで（セットアップセルにこの良い方法です）、またはこのフィールドをuitableview controllerからitemNoとitemPriceと入力する方が良いでしょうか？

Answer 1

IMHO、ビュー依存のロジックをUITableViewControllerに追加するよりも優れています。

iOSアプリケーションでは、ここにある「モデルビューコントローラ」パターンがよく使用されます。このパターンの詳細はさまざまですが、一般的に「疎結合」されたコンポーネントを持つことは良いことです。つまり、コントローラーは、どの表のセル（ビュー）ラベルがどのモデル情報を表示しているかに関係するべきではありません。

また、このようなビューのこの内部のようなあなたのロジックをカプセル化すると、きれいに将来的にItem（モデル）への更新を観察するために、あなたが設定されます（完全に追加のコントローラとの対話の必要性を排除します）。この自動更新は、Key-Value Observingなどの方法で実行できます。

MVCのWikipediaページには、コンポーネントが通常どのように対話するかを説明する優れた図があります。

出典：https://en.wikipedia.org/wiki/Model–view–controller#Description

Answer 2

よりよいそうのように行われるだろうと私には思える：

これを使用して

func setupCell(_data:DataObject) { 
    // update layout depending on data 
    itemNo.text = _data.document_no 
    itemPrice.text = _data.sum 
} 

：

func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell { 
// this way avoids forced unwrapping. much safer 
if let cell = tableView.dequeueReusableCellWithIdentifier("OrderCell") as? OrderTableViewCell{ 
     cell.setupCell(results[indexPath.row]) 
     return cell 
    } 
    return UITableViewCell() 
} 

ためdequeueReusableCellWithIdentifierのテーブルビューは、セルのインスタンスを再利用していることに注意してください。セルが表示されるたびに、cellForRowAtIndexPathが再び呼び出され、そのセルを再度レイアウトする必要があります。そのデータは永久的ではありません。