点击UITableViewCell按钮生成同键字典数组遇多菜品失效问题求助

嘿，我一眼就看出你代码里的问题啦！现在的逻辑在多菜品操作时会数据混乱，核心原因有两个：一是复用了同一个字典实例，二是判断条目是否存在的逻辑不对。咱们一步步来修正：

问题分析 #

1. 复用全局字典实例：你代码里的dict应该是类成员变量吧？每次点击都修改这个字典再添加到数组，虽然Swift字典是值类型，append时会拷贝，但你用items.contains { $0 == dict }判断时，修改后的字典和数组里旧条目内容已经不一样了，会导致重复添加同一个菜品的多条记录。
2. 错误的更新逻辑：else分支里的dict.updateValue(dishId!, forKey: "id")完全没用，你该更新的是quantity，不是重复设置id。

修正后的代码 #

加按钮点击方法 #

func didTapIncrease(_ sender: PayForOrderCell) {
    guard let tappedIndexPath = tableView.indexPath(for: sender) else { return }
    let dish = dishes[tappedIndexPath.row]
    guard let dishId = dish.id else { return } // 安全解包，避免强制解包崩溃
    
    // 更新cell的数量显示
    sender.quantityValue += 1
    let newQuantity = sender.quantityValue
    
    // 更新价格标签
    if let price = dish.price {
        sender.priceLabel.text = "\(price * newQuantity) $"
    }
    
    // 处理菜品条目数组
    if let existingIndex = items.firstIndex(where: { $0["id"] as? Int == dishId }) {
        // 已有该菜品的条目，更新数量
        items[existingIndex]["quantity"] = newQuantity
    } else {
        // 没有该菜品的条目，创建新字典添加
        let newItem = ["id": dishId, "quantity": newQuantity]
        items.append(newItem)
    }
}

补充减按钮的逻辑（完整起见） #

func didTapDecrease(_ sender: PayForOrderCell) {
    guard let tappedIndexPath = tableView.indexPath(for: sender) else { return }
    let dish = dishes[tappedIndexPath.row]
    guard let dishId = dish.id else { return }
    
    // 防止数量变成负数
    guard sender.quantityValue > 1 else { return }
    
    sender.quantityValue -= 1
    let newQuantity = sender.quantityValue
    
    // 更新价格标签
    if let price = dish.price {
        sender.priceLabel.text = "\(price * newQuantity) $"
    }
    
    // 处理菜品条目数组
    if let existingIndex = items.firstIndex(where: { $0["id"] as? Int == dishId }) {
        if newQuantity == 0 {
            // 如果数量减到0，可以选择从数组中移除该条目（根据你的业务需求决定）
            items.remove(at: existingIndex)
        } else {
            items[existingIndex]["quantity"] = newQuantity
        }
    }
}

关键改进点 #

• 不再复用字典实例：需要新增条目时直接创建新的newItem字典，避免全局变量带来的混乱。
• 用菜品ID作为唯一标识：通过firstIndex(where:)根据id查找已有条目，这才是判断同一个菜品的正确方式，而非比较整个字典内容。
• 安全解包可选值：用guard let替代强制解包!，减少运行时崩溃风险。
• 完善减按钮逻辑：处理数量边界（不能小于1），可选移除数量为0的条目。

内容的提问来源于stack exchange，提问作者Vitalik Kizlov