MVC5中如何从Teacher模型为Student创建视图填充下拉列表

你遇到的空引用异常，核心原因是你在视图里把单个Teacher对象当成下拉列表的数据源了，而且控制器也没把教师列表传递到视图里。咱们一步步来解决这个问题：

你的Student模型目前只有Teacher导航属性，但下拉列表需要绑定一个用于存储选中值的标量属性（也就是教师的ID），同时要和导航属性关联起来，这样EF才能正确处理外键关系。修改Student模型：

public class Student {
    // 保留原有所有属性不变...
    
    // 添加这个属性，用于绑定下拉列表的选中值
    [Required]
    [Display(Name = "Teacher")]
    public int TeacherID { get; set; }

    // 给导航属性加ForeignKey特性，关联到TeacherID
    [ForeignKey("TeacherID")]
    public Teacher Teacher { get; set; }

    // 保留原有其他属性...
}

你需要在两个关键位置添加逻辑：

1. GET版本的Create动作：返回创建视图前，获取所有可用的教师（比如只取Active=true的），把列表放到ViewBag里传递给视图。
2. POST版本的Create动作：如果模型验证失败，需要重新加载教师列表，否则返回视图时下拉列表会丢失数据。

修改StudentsController的代码：

public class StudentsController : Controller
{
    private readonly YourDbContext _context; // 替换成你实际的DbContext类名

    public StudentsController(YourDbContext context)
    {
        _context = context;
    }

    // GET: Students/Create
    public IActionResult Create()
    {
        // 准备教师下拉列表数据：取活跃教师，生成SelectListItem集合
        ViewBag.TeacherID = _context.Teachers.Where(t => t.Active)
            .Select(t => new SelectListItem 
            { 
                Value = t.ID.ToString(), 
                Text = $"{t.FirstName} {t.LastName}" // 拼接教师全名显示
            }).ToList();
        
        return View();
    }

    // POST: Students/Create
    [HttpPost]
    [ValidateAntiForgeryToken]
    public async Task<IActionResult> Create([Bind("ID,FirstName,LastName,DateOfBirth,EmailAddress,ContactNumber,ParentGuardianName,TeacherID,MembershipOption,SignedUpDate,Active")] Student student)
    {
        if (ModelState.IsValid)
        {
            _context.Add(student);
            await _context.SaveChangesAsync();
            return RedirectToAction(nameof(Index));
        }
        // 模型验证失败时，重新加载教师列表，同时保留用户之前选中的ID
        ViewBag.TeacherID = _context.Teachers.Where(t => t.Active)
            .Select(t => new SelectListItem 
            { 
                Value = t.ID.ToString(), 
                Text = $"{t.FirstName} {t.LastName}",
                Selected = t.ID == student.TeacherID
            }).ToList();
        
        return View(student);
    }
}

原来的视图代码错误地用Model.Teacher作为数据源（这是单个对象，不是集合），现在改成绑定TeacherID属性，并用ViewBag里的教师列表作为数据源：

<div class="form-group">
    @Html.LabelFor(model => model.TeacherID, htmlAttributes: new { @class = "control-label col-md-2" })
    <div class="col-md-10">
        @Html.DropDownListFor(m => m.TeacherID, 
            (IEnumerable<SelectListItem>)ViewBag.TeacherID, 
            "- Please select a Teacher -", 
            new { @class = "form-control" })
        @Html.ValidationMessageFor(model => model.TeacherID, "", new { @class = "text-danger" })
    </div>
</div>

• 如果你给Teacher模型添加了FullName计算属性（比如public string FullName => $"{FirstName} {LastName}"），可以把SelectListItem的Text直接改成t.FullName，代码会更简洁。
• 用ViewBag是快速实现的方式，如果你项目规模较大，推荐使用专门的ViewModel来封装视图需要的所有数据（包括Student实体和教师列表），这样代码的可维护性更好。

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