ASP.NET MVC中更新数据时如何物理删除旧图片？

首先你的现有代码存在几个风险点：比如用户没有上传新图片时jobimage会是null，调用jobimage.FileName或者SaveAs会直接报错；另外直接使用传入的job.JobImage可能不准确（比如模型绑定出错或者被恶意篡改），应该从数据库先获取原始的图片路径。

下面是优化后的完整实现：

[HttpPost]
[ValidateAntiForgeryToken]
public ActionResult Edit(Job job, HttpPostedFileBase jobimage)
{
    if (ModelState.IsValid)
    {
        // 1. 从数据库获取原始的Job记录，拿到真实的旧图片路径
        var existingJob = db.Jobs.Find(job.Id);
        if (existingJob == null)
        {
            return HttpNotFound();
        }

        // 2. 判断是否上传了新图片
        if (jobimage != null && jobimage.ContentLength > 0)
        {
            // 定义上传目录路径
            string uploadDir = Server.MapPath("~/Uploads");
            
            // 先删除旧图片（先判断文件是否存在，避免文件不存在时抛异常）
            string oldImagePath = Path.Combine(uploadDir, existingJob.JobImage);
            if (System.IO.File.Exists(oldImagePath))
            {
                System.IO.File.Delete(oldImagePath);
            }

            // 处理新图片：建议给文件名加前缀或者重命名，避免重名覆盖
            string fileName = Path.GetFileName(jobimage.FileName);
            // 可选：用GUID生成唯一文件名，防止重名
            // string fileName = $"{Guid.NewGuid()}_{Path.GetFileName(jobimage.FileName)}";
            
            string newImagePath = Path.Combine(uploadDir, fileName);
            jobimage.SaveAs(newImagePath);

            // 更新Job对象的图片路径
            existingJob.JobImage = fileName;
        }

        // 3. 更新其他字段（除了图片的其他属性）
        db.Entry(existingJob).CurrentValues.SetValues(job);
        db.SaveChanges();

        return RedirectToAction("Index");
    }

    // 模型验证失败，返回编辑页面（补充你的ViewBag数据）
    ViewBag.CategoryId = new SelectList(db.Categories, "Id", "Name", job.CategoryId);
    // 这里可以继续添加其他需要的ViewBag项...
    return View(job);
}

关键优化点说明： #

• 先获取原始记录：确保拿到的旧图片路径是数据库里存储的真实值，而不是用户提交的可能被篡改的数据
• 判断上传状态：只有当用户确实上传了新图片（jobimage != null且ContentLength > 0）时，才执行删除旧图和保存新图的操作
• 文件存在检查：删除旧图片前先判断文件是否存在，避免因文件已被删除或不存在导致的异常
• 可选的文件名重命名：使用GUID生成唯一文件名，避免不同用户上传同名文件时互相覆盖

额外注意事项： #

• 确保~/Uploads目录有读写权限，否则保存或删除文件会报错
• 如果你的Job实体的JobImage字段允许为空，还要处理旧图片为空的情况（此时无需删除）

内容的提问来源于stack exchange，提问作者El Mehdi El Mellali