ASP.NET MVC5文件上传下载问题：文件未保存至服务器文件夹

我来帮你搞定这个文件上传下载的问题，看了你的代码后，发现核心问题出在上传时的文件夹判断逻辑和下载时的路径匹配上，咱们一步步修复：

一、上传逻辑的问题与修复 #

你的Create Action里的文件夹判断完全写反了，而且没有把正确的文件名绑定到实体，导致文件没保存到服务器。

原代码的问题： #

1. 错误的文件夹判断：Directory.Exists(Server.MapPath("~/uploads/" + file.FileName)) —— 你这里是在检查是否存在一个和文件名同名的文件夹，而不是检查~/uploads主文件夹是否存在。这会导致只有当存在这个同名文件夹时才不保存文件，但实际上咱们需要先确保~/uploads文件夹存在，再把文件存进去。
2. 未将文件名赋值给实体：你没有把上传后的文件名赋值给operation.attached，虽然你说数据库保存了文件名，但可能是前台表单绑定的默认值，逻辑上应该从上传的文件对象里获取。
3. 未处理文件重名：如果用户上传同名文件，会直接覆盖旧文件，建议加个时间戳或者GUID来避免重名。

修复后的Create Action代码： #

public ActionResult Create([Bind(Include = "op_id,op_name,op_typeid,op_date,budget,op_statusid,sup_id,order_no,order_typeid,attached,notes")] operation operation, HttpPostedFileBase file)
{
    if (ModelState.IsValid)
    {
        if (Request.Files.Count > 0)
        {
            file = Request.Files[0];
            if (file.ContentLength > 0)
            {
                // 1. 定义上传根路径
                var uploadPath = Server.MapPath("~/uploads/");
                
                // 2. 确保uploads文件夹存在，不存在则创建
                if (!Directory.Exists(uploadPath))
                {
                    Directory.CreateDirectory(uploadPath);
                }
                
                // 3. 处理文件名，避免重名（可选但推荐）
                var fileName = $"{Guid.NewGuid()}_{Path.GetFileName(file.FileName)}";
                // 如果不需要处理重名，直接用原文件名：var fileName = Path.GetFileName(file.FileName);
                
                // 4. 保存文件到服务器
                var filePath = Path.Combine(uploadPath, fileName);
                file.SaveAs(filePath);
                
                // 5. 将文件名赋值给实体，保存到数据库
                operation.attached = fileName;
            }
        }
        
        db.operations.Add(operation);
        db.SaveChanges();
        return RedirectToAction("Index");
    }
    
    // 如果模型验证失败，返回视图
    return View(operation);
}

二、下载逻辑的问题与修复 #

你的下载Action里硬编码了路径c:/uploads/，但上传时文件是存在网站根目录的~/uploads下，两个路径大概率不匹配，导致找不到文件。

原代码的问题： #

• 路径不匹配：var FileVirtualPath = "c:/uploads/" + name; —— 应该用Server.MapPath获取网站根目录下的uploads路径，而不是硬编码本地磁盘路径。
• 未检查文件是否存在：如果文件不存在，直接返回File会报错，建议加判断。

修复后的GetFile Action代码： #

[HttpGet]
public ActionResult GetFile(string name)
{
    if (string.IsNullOrEmpty(name))
    {
        return HttpNotFound("文件名不能为空");
    }
    
    // 获取正确的服务器文件路径
    var filePath = Path.Combine(Server.MapPath("~/uploads/"), name);
    
    if (!System.IO.File.Exists(filePath))
    {
        return HttpNotFound("文件不存在");
    }
    
    // 返回文件供下载
    return File(filePath, System.Net.Mime.MediaTypeNames.Application.Octet, Path.GetFileName(filePath));
}

三、前台代码的小优化 #

上传的前台代码可以保持不变，但下载的ActionLink文本可以更友好一点，比如改成“下载附件”：

@Html.ActionLink("下载附件", "GetFile", new { name = item.attached })

这样修改后，文件就能正常保存到服务器的~/uploads文件夹，数据库里也会存正确的文件名，下载时也能找到对应的文件了。

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