带唯一Item ID的动态表单字段如何在控制器中取值更新

针对你遇到的动态生成购物车表单，在控制器中无法正确提取对应商品ID和数量的问题，结合你最终调试成功的方案，整理如下：

问题回顾 #

你有一个动态生成的表单，每个数量输入框的ID为Item_@item.Id，但最初统一设置了name="qty"，导致后端无法区分每个数量对应的商品。服务器端生成字段的代码如下：

<input type="text" name="qty" id="Item_@item.Id" value="@item.Qty">

你在控制器的Update方法中，不知道如何从IFormCollection里提取商品ID和对应的数量值。

解决方案 #

1. 调整前端输入字段的name属性 #

将每个输入框的name改为和ID一致的格式，让提交的表单键直接包含商品ID：

<input asp-for="@item.Quantity" name="Item_@item.ProductId" id="Item_@item.ProductId" />

2. 控制器中处理表单数据 #

通过筛选以Item_开头的表单键，提取商品ID，再获取对应的数量值，执行购物车条目更新：

public async Task<IActionResult> Update(IFormCollection form)
{
    ShoppingCartItem updateItem;
    int qty = 0;
    int pId = 0;
    
    // 只处理我们需要的商品数量字段
    foreach (string item in form.Keys.Where(k => k.StartsWith("Item_")))
    {
        // 从键名中提取商品ID
        pId = int.Parse(item.Replace("Item_", ""));
        
        // 查询对应的购物车条目
        updateItem = (from c in _context.ShoppingCartItems
                      where (c.ShoppingCartId == ShoppingCartItem.ShoppingCartId && c.ProductId == pId)
                      select c).FirstOrDefault();
        
        // 更新数量
        qty = int.Parse(form[item]);
        updateItem.Quantity = qty;
    }
    
    await _context.SaveChangesAsync();
    return RedirectToAction("Index", "Cart");
}

额外优化建议 #

为了让代码更健壮，建议添加空引用判断和安全的数值解析，避免非法输入或不存在的条目导致报错：

public async Task<IActionResult> Update(IFormCollection form)
{
    ShoppingCartItem updateItem;
    int qty;
    int productId;
    
    foreach (var itemKey in form.Keys.Where(k => k.StartsWith("Item_")))
    {
        // 安全解析商品ID
        if (!int.TryParse(itemKey.Replace("Item_", ""), out productId))
            continue;
        
        // 异步查询条目，符合异步编程最佳实践
        updateItem = await _context.ShoppingCartItems
            .FirstOrDefaultAsync(c => c.ShoppingCartId == ShoppingCartItem.ShoppingCartId && c.ProductId == productId);
        
        // 确保条目存在且数量解析成功
        if (updateItem != null && int.TryParse(form[itemKey], out qty))
        {
            updateItem.Quantity = qty;
        }
    }
    
    await _context.SaveChangesAsync();
    return RedirectToAction("Index", "Cart");
}

这个方案已经可以正常工作，优化后的代码会更稳定哦！

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