不能在 ForEach 中将 HasOne 关系设置为 null

Question

假设有一个模型有条目。每个条目都有一种货币。我想像这样更新货币：

var model = await _dbContext.Models
    .Include(x => x.Entries) 
    .FirstOrDefaultAsync(x => x.Id.ToString().Equals(command.Id), cancellationToken);

model.Name = command.Name;

command.Entries.ForEach(x =>
{
    var entry = _dbContext.Entries
        .Find(Guid.Parse(x.Id));

    var currency = _dbContext.Currencies
        .FirstOrDefault(y => y.Id.Equals(x.CurrencyId));

    entry.Currency = currency;
});

await _dbContext.SaveChangesAsync(cancellationToken);

以下是货币关系的定义方式：

modelBuilder.Entity<Entry>()
    .HasOne<Currency>(x => x.Currency)
    .WithMany();

我正在调试可以为货币分配一个值但不能为空的问题。例如。

command.Entries.ForEach(x =>
{
    var entry = _dbContext.Entries
        .Find(Guid.Parse(x.Id));

    entry.Currency = null;
});

不会将货币设置为 null 但这有效：

command.Entries.ForEach(x =>
{
    var entry = _dbContext.Entries
        .Find(Guid.Parse(x.Id));

    var currency = ...

    entry.Currency = currency;
});

货币已正确更新。现在，当我在 ForEach 之外进行更新时，它会起作用：

var entry = _dbContext.Entries.Find(Guid.Parse(".."));

entry.Currency = null;

await _dbContext.SaveChangesAsync(cancellationToken);

所以 ForEach 似乎是问题所在。任何想法为什么？奇怪的是，货币可以更新为一个值，但不能在 ForEach 内部设置为 null。

Answer 1

ForEach 不是 LINQ 命令，也没有转换为 SQL。这是一个 List 方法，它所做的只是 foreach(var item in thatList) 会做的事情。真正的区别在于 lambda 是一个独立于代码其余部分的作用域。 entry 是那里的孤儿。 _dbContext 本身就是一个捕获变量。

另一个问题是代码非常慢，因为它一个一个地加载实体。 ForEach 应替换为 _dbContext.Entries.Where(e=>listOfIds.Contains(e.Id))。这将在 SQL 中生成 ID in (@id1, @id2, @id3,....) 子句。

问题的代码可以替换为：

// Set up the parameters
var commandId=Guid.Parse(command.Id);
var entryIds=command.Entries.Select(c=>Guid.Parse(c.Id)).ToList();

//Make the changes

var model = await _dbContext.Models
    .FirstOrDefaultAsync(x => x.Id == commandId, cancellationToken);

model.Name = command.Name;


var entries=_dbContext.Entries.Where(e=>entryIds.Contains(e.Id));

foreach(var entry in entries)
{
    entry.Currency = null;
};

//Save the changes
await _dbContext.SaveChangesAsync(cancellationToken);

x => x.Id.ToString().Equals(command.Id) 被替换为 x.Id == commandId，因为转换为字符串会阻止服务器使用任何涵盖 Id 的索引。索引是根据列的值创建的，不能在过滤像cast(id as varchar(16))这样的函数结果时使用

即使该代码也可以替换为：

var model = await _dbContext.Models.Find(commandId);

英孚核心 7

在 EF Core 7 中，Bulk updates 可用于生成最终的 UPDATE 语句，而无需首先加载对象。问题的操作可能只是 SQL 中的两个 UPDATE 语句：

UPDATE Models
SET Name=@name
WHERE ID=@commandId;

UPDATE Entries
SET Currency=NULL
Where ID in (@id1, @id2,....);

在 EF Core 7 中，这可以使用 ExecuteUpdateAsync 方法完成：

await  _dbContext.Models
    .Where(m => x.Id == commandId)
    .ExecuteUpdateAsync(s => s
        .SetProperty(b => b.Name, b => name));

await _dbContext.Entries
    .Where(e=>entryIds.Contains(e.Id))
    .ExecuteUpdateAsync(s => s
        .SetProperty(b => b.Currency, b => null));

缺点是 ExecuteUpdate 立即执行，如果需要回滚多个更改，则需要显式事务