移除Optional与nil：优化UITableView的cellForRow方法代码

咱们先梳理下原代码里的几个核心问题：直接访问数组元素存在越界崩溃风险，Optional 类型的进球数用 String(describing:) 会输出 "nil" 这种不友好文本，还有 Cell 复用的兜底逻辑可以更严谨。下面是一步步的优化方案：

1. 先解决数组越界问题 #

直接通过 indexPath.row 取 events 元素，一旦数据源长度和 tableView 行数不匹配，直接就会崩溃。咱们先加一层安全校验：

func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    // 确保索引在数组合法范围内
    guard indexPath.row < events.count else {
        print("⚠️ 索引超出数据源范围：\(indexPath.row)")
        return UITableViewCell()
    }
    let event = events[indexPath.row]
    
    // ... 后续逻辑
}

2. 优化 Cell 复用逻辑 #

原代码的 guard 已经做了类型转换校验，但建议提前在 viewDidLoad 里注册 Cell，这样 dequeueReusableCell(withIdentifier:) 基本不会返回 nil，兜底逻辑也能更清晰：

// 在 viewDidLoad 里提前注册 Cell
override func viewDidLoad() {
    super.viewDidLoad()
    tableView.register(EventsCell.self, forCellReuseIdentifier: "EventsCell")
}

// 然后在 cellForRow 里
guard let cell = tableView.dequeueReusableCell(withIdentifier: "EventsCell", for: indexPath) as? EventsCell else {
    fatalError("❌ 无法转换为 EventsCell，请检查 Identifier 或 Cell 类型")
}

这里用带 for: indexPath 的方法，系统会保证返回非 nil 的 Cell，兜底用 fatalError 可以快速发现开发阶段的错误，比返回空 Cell 更直观。

3. 优雅处理 Optional 类型的进球数 #

原代码里 result.goalsHomeTeam 如果是 Optional，用 String(describing:) 会显示 "nil"，咱们可以用 nil 合并运算符给个友好的默认值，比如 "0" 或者空字符串：

// 假设 goalsHomeTeam 是 Int? 类型
cell.homeGoalLbl.text = event.result.goalsHomeTeam.map(String.init) ?? "0"
cell.awayGoalLbl.text = event.result.goalsAwayTeam.map(String.init) ?? "0"

如果允许显示空文本，就把 "0" 换成 ""。用 map 可以安全地把 Optional 值转换成字符串，避免直接解包的风险。

4. 完整优化后的代码 #

把上面的点整合起来，最终代码如下：

override func viewDidLoad() {
    super.viewDidLoad()
    // 提前注册 Cell，避免复用失败
    tableView.register(EventsCell.self, forCellReuseIdentifier: "EventsCell")
}

func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    // 安全获取数据源元素
    guard indexPath.row < events.count else {
        print("⚠️ 警告：索引 \(indexPath.row) 超出数据源范围")
        return UITableViewCell()
    }
    let event = events[indexPath.row]
    
    // 安全获取复用 Cell
    guard let cell = tableView.dequeueReusableCell(withIdentifier: "EventsCell", for: indexPath) as? EventsCell else {
        fatalError("❌ 错误：无法将复用 Cell 转换为 EventsCell，请检查 Identifier 或 Cell 类定义")
    }
    
    // 绑定数据，处理 Optional 值
    cell.homeLabel.text = event.homeTeamName ?? "未知主队"
    cell.awayLabel.text = event.awayTeamName ?? "未知客队"
    cell.homeGoalLbl.text = event.result.goalsHomeTeam.map(String.init) ?? "0"
    cell.awayGoalLbl.text = event.result.goalsAwayTeam.map(String.init) ?? "0"
    
    return cell
}

额外建议 #

• 如果 homeTeamName 和 awayTeamName 也是 Optional 类型，同样用 nil 合并运算符给默认值，避免 Label 显示空白。
• 可以把 Cell 的数据绑定逻辑封装到 EventsCell 内部，比如加一个 configure(with event: Event) 方法，让 cellForRow 更简洁：

// 在 EventsCell 里
func configure(with event: Event) {
    homeLabel.text = event.homeTeamName ?? "未知主队"
    awayLabel.text = event.awayTeamName ?? "未知客队"
    homeGoalLbl.text = event.result.goalsHomeTeam.map(String.init) ?? "0"
    awayGoalLbl.text = event.result.goalsAwayTeam.map(String.init) ?? "0"
}

// 然后在 cellForRow 里
cell.configure(with: event)

这样代码的职责更清晰，也方便后续维护。

内容的提问来源于stack exchange，提问作者Nenad Savic