ASP.NET MVC下拉列表选中值无法正确返回，导致FileType表新增数据的问题

看起来你遇到了两个核心问题：下拉列表的选中值没有被正确获取，同时Entity Framework意外在FileType表中新增了记录。这两个问题其实是相关的，让我一步步帮你解决：

问题根源 #

1. 视图绑定与Post参数不匹配：你的视图里下拉列表绑定的是FileT属性，但Post动作方法接收的是Ftype参数，名称不匹配导致模型绑定无法正确获取选中的ID。
2. EF对未跟踪实体的处理：即使参数名匹配，直接接收FileType对象时，模型绑定只会填充Id属性，但EF会把这个未被上下文跟踪的对象当成新实体，从而执行插入操作。

解决方案 #

步骤1：修正视图的下拉列表绑定字段 #

下拉列表应该绑定到外键字段FileTypeId，而不是整个FileT对象，这是MVC中处理下拉列表选择的常规做法：

<div class="form-group row"> 
    <label asp-for="FileTypeId" class="col-sm-2 col-form-label">文件类型</label> 
    <div class="col-sm-10"> 
        <select asp-for="FileTypeId" class="custom-select mr-sm-2" asp-items="@Model.FtypeList"></select> 
    </div> 
</div>

步骤2：优化Post方法的参数接收 #

建议直接使用你的FileUploadViewModel来接收表单数据，而不是零散的参数，这样更清晰且不易出错：

[HttpPost] 
public async Task<IActionResult> Index(FileUploadViewModel vm, List<IFormFile> files) { 
    // 先验证模型有效性
    if (!ModelState.IsValid) {
        // 重新加载下拉列表数据并返回视图
        vm.FtypeList = context.FileTypes 
            .Select(a => new SelectListItem() { Value = a.Id.ToString(), Text=a.FileTypeName }).ToList();
        return View(vm);
    }

    foreach (var file in files) { 
        var basePath = Path.Combine(Directory.GetCurrentDirectory() + "\\Files\\"); 
        bool basePathExists = System.IO.Directory.Exists(basePath); 
        if (!basePathExists) Directory.CreateDirectory(basePath); 
        var fileName = Path.GetFileNameWithoutExtension(file.FileName); 
        var filePath = Path.Combine(basePath, file.FileName); 
        var extension = Path.GetExtension(file.FileName); 
        DateTime dateTime = DateTime.UtcNow; 
        var createdOn = dateTime.ToShortDateString(); 

        if (!System.IO.File.Exists(filePath)) { 
            using (var stream = new FileStream(filePath, FileMode.Create)) { 
                await file.CopyToAsync(stream); 
            } 

            // 获取已存在的FileType实体
            var existingFileType = context.FileTypes.Find(vm.FileTypeId);
            if (existingFileType == null) {
                ModelState.AddModelError("FileTypeId", "选择的文件类型不存在");
                vm.FtypeList = context.FileTypes 
                    .Select(a => new SelectListItem() { Value = a.Id.ToString(), Text=a.FileTypeName }).ToList();
                return View(vm);
            }

            var fileModel = new FileModel { 
                Year = createdOn, 
                PublishedOn = vm.PublishedOn, // 建议FileModel中用DateTime类型存储，避免字符串转换
                FileType = file.ContentType, 
                Extension = extension, 
                Name = vm.Name, 
                Description = vm.Description, 
                FilePath = filePath, 
                Author = vm.Author, 
                Ftype = existingFileType, // 关联已存在的实体
                FileTypeId = vm.FileTypeId // 同时设置外键字段
            }; 
            context.Files.Add(fileModel); 
            await context.SaveChangesAsync(); // 用异步版本更高效
        } 
    } 

    TempData["Message"] = "File successfully uploaded to File System."; 
    return RedirectToAction("Index","Home"); 
}

步骤3：确保FileModel的导航属性配置正确 #

检查你的FileModel类，确保外键和导航属性的关系配置正确（如果没有用Fluent API，用数据注解标注）：

public class FileModel {
    // 其他属性...
    public int FileTypeId { get; set; } // 外键字段

    [ForeignKey(nameof(FileTypeId))] // 明确指定外键对应关系
    public FileType Ftype { get; set; } // 导航属性
}

可选优化：避免查询数据库的Attach方式 #

如果你不想每次都查询FileType表，可以通过Attach方法告诉EF这个实体是已存在的（前提是你确定FileTypeId是有效的）：

var fileType = new FileType { Id = vm.FileTypeId };
context.Attach(fileType); // 标记为已跟踪的现有实体，避免EF插入新记录

var fileModel = new FileModel { 
    // 其他属性...
    Ftype = fileType,
    FileTypeId = vm.FileTypeId
};

关键总结 #

• 下拉列表始终绑定到外键字段，而不是整个实体对象，这能避免模型绑定的歧义。
• 在关联现有实体时，要么从数据库查询获取跟踪的实体，要么用Attach方法标记实体为已存在，防止EF误插入新记录。
• 尽量使用ViewModel接收表单数据，让代码更整洁且易于维护。

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