ASP.NET Core 6 MVC商品编辑异常：重复创建与图片强制上传

问题1：编辑商品生成重复项而非覆盖原数据 #

原因分析 #

• 编辑视图未传递商品Id字段，ASP.NET Core MVC无法识别这是对现有实体的编辑请求，会将提交的Product对象视为新实体执行插入操作，导致重复数据。
• 直接使用_context.Update(product)时，若传入的product对象未被EF Core上下文跟踪，可能引发意外的插入行为。

修复方案 #

1. 在编辑视图的表单中添加隐藏字段传递商品Id和原图片路径：

<input type="hidden" asp-for="Id" />
<input type="hidden" asp-for="Image" />

2. 修改Edit方法，先从数据库获取原实体，再映射提交的字段值，避免上下文跟踪问题：

public async Task<IActionResult> Edit(int id, Product product)
{
    if (id != product.Id)
    {
        return NotFound();
    }

    ViewBag.CategoryId = new SelectList(_context.Category.OrderBy(x => x.Sorting), 
                                        "Id", "Name", product.CategoryId);

    var existingProduct = await _context.Product.FindAsync(id);
    if (existingProduct == null)
    {
        return NotFound();
    }

    // 检查Slug唯一性（排除当前商品）
    product.Slug = product.Name.ToLower().Replace(" ", "-");
    var slugExists = await _context.Product
                                   .Where(x => x.Id != id)
                                   .AnyAsync(x => x.Slug == product.Slug);

    if (slugExists)
    {
        ModelState.AddModelError("", "该商品已存在。");
        return View(product);
    }

    // 处理图片上传
    if (product.ImageUpload != null)
    {
        string uploadsDir = Path.Combine(webHost.WebRootPath, "media/products");

        // 删除旧图片（默认图noimage.png不删除）
        if (!string.Equals(existingProduct.Image, "noimage.png", StringComparison.OrdinalIgnoreCase))
        {
            string oldImagePath = Path.Combine(uploadsDir, existingProduct.Image);
            if (System.IO.File.Exists(oldImagePath))
            {
                System.IO.File.Delete(oldImagePath);
            }
        }

        // 上传新图片
        string imageName = Guid.NewGuid().ToString() + "_" + product.ImageUpload.FileName;
        string filePath = Path.Combine(uploadsDir, imageName);

        using (var fs = new FileStream(filePath, FileMode.Create))
        {
            await product.ImageUpload.CopyToAsync(fs);
        }

        existingProduct.Image = imageName;
    }

    // 更新其他字段
    existingProduct.Name = product.Name;
    existingProduct.Slug = product.Slug;
    existingProduct.Price = product.Price;
    existingProduct.CategoryId = product.CategoryId;
    existingProduct.Description = product.Description;

    try
    {
        await _context.SaveChangesAsync();
        TempData["Success"] = "商品编辑成功！";
        return RedirectToAction("Index");
    }
    catch (DbUpdateConcurrencyException)
    {
        if (!ProductExists(product.Id))
        {
            return NotFound();
        }
        else
        {
            throw;
        }
    }

    return View(product);
}

// 辅助方法：检查商品是否存在
private bool ProductExists(int id)
{
    return _context.Product.Any(e => e.Id == id);
}

问题2：编辑时被强制要求上传新图片 #

原因分析 #

• 视图中注释掉了Image的隐藏字段，导致提交表单时product.Image为null，若数据库中Image字段设置为必填，会触发验证错误。
• 自定义验证属性[ImageExtention]可能错误地设置为必填，导致未上传图片时验证失败。

修复方案 #

1. 恢复视图中Image的隐藏字段（已在问题1的修复中包含），确保原图片路径能提交到后台，不上传新图时保留原图片。
2. 调整[ImageExtention]自定义验证属性，确保仅在上传图片时验证格式，而非强制要求上传：

public class ImageExtentionAttribute : ValidationAttribute
{
    protected override ValidationResult IsValid(object value, ValidationContext validationContext)
    {
        var file = value as IFormFile;
        if (file == null)
        {
            // 未上传图片时通过验证
            return ValidationResult.Success;
        }

        // 验证图片格式逻辑
        var allowedExtensions = new[] { ".jpg", ".jpeg", ".png", ".gif" };
        var extension = Path.GetExtension(file.FileName).ToLower();
        if (!allowedExtensions.Contains(extension))
        {
            return new ValidationResult("仅允许上传JPG、JPEG、PNG、GIF格式的图片。");
        }

        return ValidationResult.Success;
    }
}

3. 确保Product类中Image字段允许为空（当前代码public string? Image { get; set; }已正确设置，若数据库对应字段为必填，需修改为允许空值）。

修改后的编辑视图关键部分 #

<form asp-action="Edit" enctype="multipart/form-data">
    <div asp-validation-summary="ModelOnly" class="text-danger"></div>
    <!-- 添加Id和Image的隐藏字段 -->
    <input type="hidden" asp-for="Id" />
    <input type="hidden" asp-for="Image" />

    <!-- 其他表单字段保持不变 -->
    <div class="form-group">
        <label asp-for="Name" class="control-label"></label>
        <input asp-for="Name" class="form-control" />
        <span asp-validation-for="Name" class="text-danger"></span>
    </div>

    <!-- 图片上传部分 -->
    <div class="form-group">
        <label asp-for="Image" class="control-label">当前图片</label>
        <img src="~/media/products/@Model.Image" width="200" alt="" />
    </div>

    <div class="form-group">
        <label class="control-label">新商品图片</label>
        <input asp-for="ImageUpload" class="form-control" />
        <img src="" id="imgpreview" class="pt-2" alt="" />
        <span asp-validation-for="ImageUpload" class="text-danger"></span>
    </div>
</form>

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