一对一关系Controller编写：创建Review时ApplicationUser字段必填报错

问题根源 #

你的Review模型中ApplicationUser导航属性被EF验证逻辑判定为必填，但表单提交时并未传递该对象；即使手动关联用户，也未处理ModelState中已触发的验证错误。另外，若业务场景是当前登录用户创建自身评论，下拉选择用户的设计其实可以省略，直接关联当前用户更合理。

修正步骤 #

1. 优化模型（可选但推荐） #

由于ReviewId已标记为[ForeignKey("ApplicationUser")]且[Required]，导航属性ApplicationUser无需额外添加[Required]（外键已保证关联关系），可避免双重验证冲突：

using System.ComponentModel.DataAnnotations;
using System.ComponentModel.DataAnnotations.Schema;

namespace ZaryadApp.Models
{
    public class Review
    {
        [ForeignKey("ApplicationUser")]
        [Required]
        public string ReviewId { get; set; }
        [Required]
        public string Text { get; set; }
        public DateTime CreatedAt { get; set; }
        // 移除[Required]，由外键保证与用户的关联
        public virtual ApplicationUser ApplicationUser { get; set; }
    }
}

2. 修复POST Create方法 #

场景1：当前登录用户创建评论（无需选择用户）

直接获取当前登录用户ID并关联，同时清除ModelState中关于ApplicationUser的错误：

using System.Security.Claims;
// ...其他引用

[HttpPost]
[ValidateAntiForgeryToken]
public async Task<IActionResult> Create([Bind("Text,CreatedAt")] Review review)
{
    // 获取当前登录用户ID
    var currentUserId = User.FindFirstValue(ClaimTypes.NameIdentifier);
    
    // 给Review设置主键（同时是关联用户的外键）
    review.ReviewId = currentUserId;
    // 关联用户实体
    review.ApplicationUser = await _context.Users.FindAsync(currentUserId);

    // 清除ModelState中关于ApplicationUser的验证错误（表单未传递该对象）
    ModelState.Remove(nameof(Review.ApplicationUser));
    
    if (ModelState.IsValid)
    {
        _context.Add(review);
        await _context.SaveChangesAsync();
        return RedirectToAction(nameof(Index));
    }
    return View(review);
}

场景2：允许选择用户（如后台管理）

保留下拉选择逻辑，确保ReviewId正确传递后关联用户，再清除对应验证错误：

[HttpPost]
[ValidateAntiForgeryToken]
public async Task<IActionResult> Create([Bind("ReviewId,Text,CreatedAt")] Review review)
{
    // 通过ReviewId关联用户实体
    review.ApplicationUser = await _context.Users.FindAsync(review.ReviewId);
    
    // 清除ApplicationUser的验证错误
    ModelState.Remove(nameof(Review.ApplicationUser));
    
    if (ModelState.IsValid)
    {
        _context.Add(review);
        await _context.SaveChangesAsync();
        return RedirectToAction(nameof(Index));
    }
    ViewData["ReviewId"] = new SelectList(_context.Users, "Id", "Id", review.ReviewId);
    return View(review);
}

3. 调整视图（匹配业务场景） #

场景1：当前用户创建评论

移除下拉选择ReviewId的部分：

<form asp-action="Create">
    <div asp-validation-summary="ModelOnly" class="text-danger"></div>
    <div class="form-group">
        <label asp-for="Text" class="control-label"></label>
        <input asp-for="Text" class="form-control" />
        <span asp-validation-for="Text" class="text-danger"></span>
    </div>
    <div class="form-group">
        <label asp-for="CreatedAt" class="control-label"></label>
        <input asp-for="CreatedAt" class="form-control" />
        <span asp-validation-for="CreatedAt" class="text-danger"></span>
    </div>
    <div class="form-group">
        <input type="submit" value="Create" class="btn btn-primary" />
    </div>
</form>

场景2：保留用户选择

保留原下拉代码，确保asp-for="ReviewId"与后台绑定字段一致即可。

关键说明 #

• 一对一关系中，Review的主键ReviewId同时作为外键关联ApplicationUser的Id，创建时必须保证ReviewId与目标用户Id完全一致。
• 手动关联ApplicationUser后，必须清除ModelState中该属性的验证错误——因为表单未传递这个对象，默认验证会触发不必要的报错。
• 使用User.FindFirstValue(ClaimTypes.NameIdentifier)可直接获取当前登录用户Id，无需额外查询（除非需要完整用户实体）。

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