.NET Core中添加任务时如何同时新增关联司机实体

问题描述 #

我正在开发.NET Core项目，当前任务页面关联Drivers和Missions两张数据表。传统流程需要先填充司机表，才能在任务中选择已有司机；但我希望在添加任务时，若输入新司机信息，点击添加任务按钮后，该司机能同时被添加至Drivers表和Missions表。

我尝试了以下代码：

[HttpPost]
public async Task<ActionResult<List<Mission>>> AddMission(MissionDto request)
{
    var driver = _context.Drivers
    .include(p=>p.Missions)
    .SingleOrDefault(p => p.Name == request.name)
    ?? new Driver
    {
        Name = request.Name,
        Age = request.Age
        //...
        //...
    };

    var newMission = new Mission
    {
        TimeCreated = Convert.ToDateTime(DateTime.Now.GetDateTimeFormats()[0]),
        MissionId = request.ShipmentId,
    };

    driver.Missions.Add(newMission);
    _context.SaveChanges();
}

我对新增司机的实现方式存在疑问，请问这种写法是否正确？恳请各位提供建议与帮助。

分析与修正建议 #

你的核心思路是可行的，但代码存在几处需要修正和优化的地方：

1. 异步方法未使用异步操作 #

方法声明为async Task<ActionResult<List<Mission>>>，但调用的是同步方法_context.SaveChanges()，会阻塞线程。应改为异步版本：

await _context.SaveChangesAsync();

2. 司机查询的大小写与笔误问题 #

• 代码中request.name是小写，后续request.Name是大写，属于笔误，需保持一致；
• 直接用==比较名称可能存在大小写敏感问题，建议忽略大小写：

.SingleOrDefaultAsync(p => p.Name.Equals(request.Name, StringComparison.OrdinalIgnoreCase))

3. 新司机实例未显式添加到上下文 #

通过?? new Driver()创建的新实例，EF的关系追踪可能自动识别，但为了避免潜在问题，建议显式添加：

if (driver == null)
{
    driver = new Driver
    {
        Name = request.Name,
        Age = request.Age
        // 其他属性赋值
    };
    _context.Drivers.Add(driver);
}

4. 时间创建逻辑冗余 #

Convert.ToDateTime(DateTime.Now.GetDateTimeFormats()[0])完全没必要，直接使用DateTime.UtcNow（推荐用UTC时间避免时区问题）即可：

TimeCreated = DateTime.UtcNow,

5. 方法未返回符合定义的结果 #

方法声明返回ActionResult<List<Mission>>，但代码无return语句，需补充返回逻辑，例如：

await _context.SaveChangesAsync();
return Ok(await _context.Missions.Include(m => m.Driver).ToListAsync());

修正后的示例代码 #

[HttpPost]
public async Task<ActionResult<List<Mission>>> AddMission(MissionDto request)
{
    // 忽略大小写查询已有司机
    var driver = await _context.Drivers
        .Include(p => p.Missions)
        .SingleOrDefaultAsync(p => p.Name.Equals(request.Name, StringComparison.OrdinalIgnoreCase));

    // 未找到则创建新司机并添加到上下文
    if (driver == null)
    {
        driver = new Driver
        {
            Name = request.Name,
            Age = request.Age
            // 补充其他必填属性
        };
        _context.Drivers.Add(driver);
    }

    var newMission = new Mission
    {
        TimeCreated = DateTime.UtcNow,
        MissionId = request.ShipmentId,
        // 若模型有DriverId外键，需显式赋值：DriverId = driver.Id
    };

    driver.Missions.Add(newMission);
    await _context.SaveChangesAsync();

    // 返回包含司机关联的任务列表
    return Ok(await _context.Missions.Include(m => m.Driver).ToListAsync());
}

额外建议 #

• 唯一性约束：如果司机名称需要唯一，建议在Drivers表的Name字段添加数据库唯一约束，避免并发场景下重复创建；
• DTO验证：在MissionDto中添加数据验证特性（如[Required]），确保前端传入的司机信息完整；
• 事务保障：若后续有更复杂的多表操作，可显式使用EF事务确保数据一致性。

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