点击TableView按钮触发Segue跳转并传递数据的技术疑问

我来给你捋捋这个实现的优化方向和正确步骤哈，你当前的写法把每个数据字段单独存到ViewController的属性里，不仅代码冗余，还容易因为cell复用导致数据错乱（比如滚动后cell被复用，闭包里的cell引用可能指向新的cell）。推荐用数据模型+Segue sender传参的方式，更清晰可靠：

1. 先定义一个数据模型类 #

把需要传递的所有字段封装成一个结构体，管理起来更方便：

struct ServiceData {
    let servname: String
    let region: String
    let contPer2: String
    let shift: String
    let date: String
    let nationality: String
    let price2: String
    // 其他需要传递的字段都可以加在这里
}

然后把你的数据源改成[ServiceData]类型，初始化时把零散数据转成模型对象存进去，比如：

var services: [ServiceData] = []

// 假设你之前的数据源是零散数组，初始化时可以这么转
for i in 0..<servnames.count {
    let data = ServiceData(
        servname: servnames[i],
        region: regions[i],
        contPer2: contper[i],
        shift: shifttype[i],
        date: dates[i],
        nationality: nationalities[i],
        price2: totalPrices[i]
    )
    services.append(data)
}

2. 配置TableView Cell时传递数据模型 #

在cellForRowAt里，给cell的点击闭包直接传递当前行的模型，同时注意避免循环引用：

override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    let cell = tableView.dequeueReusableCell(withIdentifier: "YourCellID", for: indexPath) as! YourCustomCell
    let serviceData = services[indexPath.row]
    
    // 给cell设置显示内容
    cell.service.text = serviceData.servname
    cell.date.text = serviceData.date
    cell.totalprice.text = serviceData.price2
    // 其他字段的赋值...
    
    // 设置按钮点击闭包，用[weak self]避免循环引用
    cell.onSummaryTapped = { [weak self] in
        guard let self = self else { return }
        // 直接通过sender传递模型，触发Segue
        self.performSegue(withIdentifier: "YourSegueID", sender: serviceData)
    }
    
    return cell
}

3. 在Segue准备方法中传递数据 #

重写prepare(for:sender:)方法，拿到传递的模型，赋值给目标控制器：

override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
    if segue.identifier == "YourSegueID",
       let targetVC = segue.destination as? YourTargetViewController,
       let serviceData = sender as? ServiceData {
        // 把模型数据传给目标控制器的对应属性
        targetVC.servname = serviceData.servname
        targetVC.region = serviceData.region
        targetVC.contPer2 = serviceData.contPer2
        // 其他字段依次传递...
    }
}

为什么要这么做？ #

• 避免cell复用坑：直接捕获当前行的serviceData，不依赖cell的属性，就算cell被复用，闭包里的数据也是当前行的正确数据。
• 代码更整洁：用模型统一管理所有字段，不用维护一堆零散的ViewController属性。
• 降低出错概率：统一的模型传递，减少字段遗漏或者赋值错误的情况。

如果暂时不想建模型类，也可以把当前行的indexPath通过sender传递，然后在prepare方法里根据indexPath去数据源数组拿数据，但还是推荐用模型的方式，更符合MVC的设计思想。

内容的提问来源于stack exchange，提问作者mrs.bassim