减少数据库调用

Question

在1个DB调用中执行此操作的最佳方法是什么？

if (dbContext.Owners.Any(i => i.Value == dog.OwnerID)) //Check if owner exists in db 
{ 
    //Assign the owner's contact number to the dog entry 
    dbDog.OwnerContactNumber = dbContext.Owners.First(i => i.Value == dog.OwnerID).ContactNumber; 
} 

我的想法：

Owner owner = dbContext.Owners.FirstOrDefault(i => i.Value == dog.OwnerID); //Get the owner 
if (owner) 
{ 
    dbDog.OwnerContactNumber = owner.ContactNumber; //Assign the contact number 
} 

但不得不宣布额外的所有者变量，感觉不好。而且我有一堆这些if语句，所以我将不得不创建一堆额外的不需要的所有者变量。

我该如何做得更好？

Answer 1

您只需检查数据库中是否有所有者的ContactNumber，而不是整个所有者，因为您没有更新它。你甚至可以做到这一点，而不使用“不必要”的变量：

dbDog.OwnerContactNumber = 
    dbContext.Owners 
       .Where(i => i.Value == dog.OwnerID) 
       .Select(o => o.ContactNumber) 
       .FirstOrDefault() ?? dbDog.OwnerContactNumber; 

所以，如果没有业主号被发现，不会发生变化。