ios - 如何使具有多个 UITableViewCell 的 tableView(cellForRowAtIndexPath) 可读？

Question

我有 UITableView 和多个 UITableViewCell，每个单元格都有不同的设计和高度，我现在遇到的问题是方法 tableView(cellForRowAtIndexPath) 看起来很奇怪且不可读，我不知道这是否是该案例的真实和良好实践实现。

我的方法:

func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
        if(indexPath.section == 0){ // linked news item
            let cellIdentifier = "linkedNewsTableViewCell";
            let cell = tableView.dequeueReusableCellWithIdentifier(cellIdentifier, forIndexPath: indexPath) as! LinkedNewsTableViewCell;
            let linked_news = LinkedNews[indexPath.row];
            cell.newTitle.text = linked_news.news_title;
            return cell;
        }else if(indexPath.section > 1 && indexPath.row == 0 && indexPath.section != sections.count+2){ // section header item
            let cellIdentifier = "sectionHeaderTableViewCell";
            let cell = tableView.dequeueReusableCellWithIdentifier(cellIdentifier, forIndexPath: indexPath) as! SectionHeaderTableViewCell;
            let sec = sections[indexPath.section-2];
            cell.lblSectionTitle.text = sec.section_name;
            cell.backgroundColor = Constants.Colors.lightGray;
            return cell;
        }else if(indexPath.section == 2+sections.count){ // all rights reserved item
            let cellIdentifier = "allRightsReservedTableViewCell";
            let cell = tableView.dequeueReusableCellWithIdentifier(cellIdentifier, forIndexPath: indexPath) as! AllRightsReservedTableViewCell;
            cell.backgroundColor = Constants.Colors.lightGray;
            return cell;
        }else if(indexPath.section == 1){ // slider news item
            let cellIdentifier = "newsTableViewCell";
            let cell = tableView.dequeueReusableCellWithIdentifier(cellIdentifier, forIndexPath: indexPath) as! NewsTableViewCell;
            cell.imgVideo.hidden = true;
            let newsItem = SliderNews[indexPath.row];
            cell.txtNews.text = newsItem.news_title;
            cell.lblTime.text = Utilities.timeAgoSinceDate(NSDate(timeIntervalSince1970:newsItem.createdstamp!), numericDates: false);
            do{
            let isSaved = try DBManger.sharedInstance.isSaved(String(newsItem.news_id!));
            cell.isSaved = isSaved;
            cell.news = newsItem;
                if(isSaved == true){
                    cell.btnSave.setImage(UIImage(named: "ic_bookmark_blue"), forState: .Normal);
                }else{
                    cell.btnSave.setImage(UIImage(named: "ic_bookmark"), forState: .Normal);
                }
            }catch{

            }

            if(SliderNews.count-1 == indexPath.row){
                cell.buttomLine.hidden = true;
            }else{
                cell.buttomLine.hidden = false;
            }

            let image = cell.imgNews.getImage(newsItem.image!, timestamp: String(newsItem.createdstamp!), size: "228", qualty: "70");

            cell.imgNews.loadImage(image,contentMode: .ScaleToFill)

            cell.lblType.text = newsItem.section_name;
            cell.backgroundColor = Constants.Colors.lightGray;
            return cell;
        }else{ // section news item
            let sec = sections[indexPath.section-2];
            if(indexPath.row == sec.news.count+1){
                let cellIdentifier = "moreNewsTableViewCell";
                let cell = tableView.dequeueReusableCellWithIdentifier(cellIdentifier, forIndexPath: indexPath) as! MoreNewsTableViewCell;
                cell.lblSectionName.text = "المزيد من \(sec.section_name!)";
                cell.backgroundColor = Constants.Colors.lightGray;
                return cell;
            }
            let cellIdentifier = "newsTableViewCell";
            let cell = tableView.dequeueReusableCellWithIdentifier(cellIdentifier, forIndexPath: indexPath) as! NewsTableViewCell;
            cell.imgVideo.hidden = true;
            let newsItem = sec.news[indexPath.row-1];
            cell.txtNews.text = newsItem.news_title;
            cell.lblTime.text = Utilities.timeAgoSinceDate(NSDate(timeIntervalSince1970:newsItem.createdstamp!), numericDates: false);
            cell.lblType.text = sec.section_name;

            if(sec.news.count == indexPath.row){
                cell.buttomLine.hidden = true;
            }else{
                cell.buttomLine.hidden = false;
            }

            let image = cell.imgNews.getImage(newsItem.image!, timestamp: String(newsItem.createdstamp!), size: "228", qualty: "70");

            cell.imgNews.loadImage(image,contentMode: .ScaleToFill)

            cell.backgroundColor = Constants.Colors.lightGray;

            do{
                let isSaved = try DBManger.sharedInstance.isSaved(String(newsItem.news_id!));
                cell.isSaved = isSaved;
                cell.news = newsItem;
                if(isSaved == true){
                    cell.btnSave.setImage(UIImage(named: "ic_bookmark_blue"), forState: .Normal);
                }else{
                    cell.btnSave.setImage(UIImage(named: "ic_bookmark"), forState: .Normal);
                }
            }catch{

            }

            return cell;
        }

    }

Answer 1

在尝试你的代码之后，你有一些问题需要解决才能让代码看起来更好:

1.您需要一个反射(reflect) tableView 数据源的模型，它应该类似于:

    let currentSectionModel = self.sections[indexPath.section] 
    let currentRowModel = currentSectionModel[]

然后您可以对您的单元格使用更通用的东西来设置模型对象:

cell.setRowModel(currentRowModel)

2. 您的 if 语句决定要呈现的女巫部分非常复杂，

for example this line:

    if indexPath.section > 1 && indexPath.row == 0 && indexPath.section != sections.count+2 {

你必须为此找到更好的逻辑。一旦你像我在第 1 部分中说的那样组织了模型，这应该看起来更清晰，你可以将它更改为一个 switch 语句，它会质疑一个枚举。

3. 所有单元逻辑，我更喜欢在单元内部执行，为了更清晰的 viewController，我在最后的示例代码中这样做了。

4. 不要使用字符串作为标识符，它会导致错误。这就是我更喜欢在 UITableViewCell 上使用扩展以返回类名作为标识符的原因。

5.不使用半列是快速的。

6.您所有的单元格都应该有一个基类，这样您就可以在返回单元格时使用多态性。

7.一旦您有了表示数据源的模型，您就可以使用 Switch 语句而不是 if 语句。

这是我编写的示例代码，您必须对代码进行一些修改才能编译。我只是一个更好的实践例子。 (我没有使用 switch 语句只是因为你的情况太复杂而不能使用简单的枚举。就像我伤心的那样，我不是你必须做的事情，并且让它更简单)

class BaseCellType: UITableViewCell {
}

class AllRightsReservedTableViewCell: BaseCellType {
    // Your implementation
}

class LinkedNewsTableViewCell: BaseCellType {
    func setLinkedNews(linedNews: LinkedNews) {
        // Your implementation
    }
}

class SectionHeaderTableViewCell: BaseCellType {
    func setSectionModel(sectionModel: SectionModel) {
     // Your implementation
    }
}

class MoreNewsTableViewCell: BaseCellType {
    func setSection(section: SectionModel) {
     // Your implementation
    }
}

class NewsTableViewCell: BaseCellType {
    // Your implementation
}

class SectionsModel {
    let rows: [RowModel]
}

extension UITableViewCell {
    static var cellIdentifer: String {
        get {
            return String(self.dynamicType).componentsSeparatedByString("__").last!
        }
    }
}

enum SectionType: Int {
    case AllRightsReserevedSection = 1, LinkedNewItem = 0
}

class ViewController: UIViewController {
    var sections: [SectionsModel]!

    func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
        let cell: BaseCellType

        switch (section: indexPath.section,row: indexPath.row) {
        case (SectionType.AllRightsReserevedSection.rawValue, _):
            cell = tableView.dequeueReusableCellWithIdentifier(AllRightsReservedTableViewCell.cellIdentifer, forIndexPath: indexPath) as! AllRightsReservedTableViewCell;
        case (SectionType.LinkedNewItem.rawValue, _):
            cell = tableView.dequeueReusableCellWithIdentifier(LinkedNewsTableViewCell.cellIdentifer, forIndexPath: indexPath) as! LinkedNewsTableViewCell;
            cell.setLinkedNews(LinkedNews[indexPath.row])
        case let index where index.section > 1 , index.row == 0, index.section != secrion+2:
            cell = tableView.dequeueReusableCellWithIdentifier(SectionHeaderTableViewCell.cellIdentifer, forIndexPath: indexPath) as! SectionHeaderTableViewCell
            cell.setSectionModel(sections[indexPath.section-2])
        case let index where index.section == section.count + 2:
            cell = tableView.dequeueReusableCellWithIdentifier(AllRightsReservedTableViewCell.cellIdentifer, forIndexPath: indexPath) as! AllRightsReservedTableViewCell;
        case let index where index.row == (sec.news.count + 1) :
            cell = tableView.dequeueReusableCellWithIdentifier(MoreNewsTableViewCell.cellIdentifer, forIndexPath: indexPath) as! MoreNewsTableViewCell;
            cell.setSection(sections[indexPath.section-2])
        default:
                cell = tableView.dequeueReusableCellWithIdentifier(NewsTableViewCell.cellIdentifer, forIndexPath: indexPath) as! NewsTableViewCell;
                cell.setNewsItem(sec.news[indexPath.row-1])
            }
        }
    return cell
}