entity-framework - 我的 Linq 查询在 EF Core 3.1 中是最优且高性能的吗?
问题描述
我可以删除 vg.First().Voucher 吗?并替换更好的代码?什么是最佳和最佳实践?可以将此代码转换为另一种方法吗?像链式方法?
var query = from v in _journalLineRepository.TableNoTracking
.Include(j => j.Voucher).AsEnumerable()
group v by v.AccountId into vg
select new // <-- temporary projection with group by fields needed
{
AccountId = vg.Key,
Credit = vg.Sum(v => v.Credit),
Debit = vg.Sum(v => v.Debit),
Voucher = vg.First().Voucher
} into vg
join p in _partyRepository.TableNoTracking.Include(p => p.PartyPhones).AsEnumerable() on vg.AccountId equals p.AccountId // <-- additional join(s)
select new PartyDeptorAndCreditorViewModel
{
PartyId = p.Id,
FullName = p.FullName,
PhoneNo = p.PartyPhones.FirstOrDefault(p => p.IsActive)?.Phone,
ProjectId = vg.Voucher.ProjectId,
AccountId = vg.AccountId.Value,
Creditor = vg.Credit,
Deptor = vg.Debit,
Balance = vg.Credit - vg.Debit,
VoucherDate = vg.Voucher.VoucherDate,
VoucherRegisterDate = vg.Voucher.VoucherDate,
BalanceType =
vg.Debit > vg.Credit ? AccountingComplexEnum.ShowPartyBalanceParamSearch.Deptor.ToDisplay(DisplayProperty.Name) :
vg.Debit < vg.Credit ? AccountingComplexEnum.ShowPartyBalanceParamSearch.Creditor.ToDisplay(DisplayProperty.Name) :
AccountingComplexEnum.ShowPartyBalanceParamSearch.ZeroBalance.ToDisplay(DisplayProperty.Name),
};
解决方案
我当然会查看生成的 SQL 查询。从表面上看,我看到一些警告标志,它可能不是在编写查询,而是可能预先执行到内存中的处理,这将是低效的。它首先取决于这些.TableNoTracking
方法/属性返回什么,以及.AsEnumerable
在急切加载连接上的使用。
首先,使用 进行投影时,不需要Select
急切加载连接 ( )。.Include
投影将为您处理连接,前提是它向下投影到 SQL。如果您.Include().AsEnumerable()
拨打电话并且您的查询仍然有效,那么它可能会投射到 SQL。如果它不再工作,那么它正在内存中处理并且效率不高。
编辑:不,内部投影无法解析:关于.Voucher
,您的最终投影使用该实体的 2 个字段,因此您可以在初始投影中替换它:
select new // <-- temporary projection with group by fields needed
{
AccountId = vg.Key,
Credit = vg.Sum(v => v.Credit),
Debit = vg.Sum(v => v.Debit),
Voucher = vg.Select(v => new { v.ProjectId, v.VoucherDate }).First()
} into vg
当涉及到这样的转换时:
BalanceType = vg.Debit > vg.Credit
? AccountingComplexEnum.ShowPartyBalanceParamSearch.Deptor.ToDisplay(DisplayProperty.Name)
: vg.Debit < vg.Credit
? AccountingComplexEnum.ShowPartyBalanceParamSearch.Creditor.ToDisplay(DisplayProperty.Name)
: AccountingComplexEnum.ShowPartyBalanceParamSearch.ZeroBalance.ToDisplay(DisplayProperty.Name),
...在投影中,这会发出警告标志,因为 Linq2EF 需要将投影组合成 SQL,因此ToDisplay
不会理解诸如此类的方法/扩展。相反,由于这仅基于贷方/借方金额,因此我将其移动到由视图模型中的属性计算:
select new PartyDeptorAndCreditorViewModel
{
PartyId = p.Id,
FullName = p.FullName,
PhoneNo = p.PartyPhones
.Where(p => p.IsActive)
.Select(p => p.Phone)
.FirstOrDefault(),
ProjectId = vg.Voucher.ProjectId,
AccountId = vg.AccountId.Value,
Creditor = vg.Credit,
Deptor = vg.Debit,
Balance = vg.Credit - vg.Debit,
VoucherDate = vg.Voucher.VoucherDate,
VoucherRegisterDate = vg.Voucher.VoucherDate
};
然后在视图模型中:
[Serializable]
public class PartyDebtorAndCreditorViewModel
{
// ...
public decimal Balance { get; set; }
public string BalanceType
{
get
{
return Balance < 0
? AccountingComplexEnum.ShowPartyBalanceParamSearch.Deptor.ToDisplay(DisplayProperty.Name)
: Balance > 0
? AccountingComplexEnum.ShowPartyBalanceParamSearch.Creditor.ToDisplay(DisplayProperty.Name)
: AccountingComplexEnum.ShowPartyBalanceParamSearch.ZeroBalance.ToDisplay(DisplayProperty.Name);
}
}
}
推荐阅读
- python - 匹配 DataFrame 中的相似值
- java - 删除不推荐使用的代码是坏主意还是好主意?
- c# - C#中的数值数据类型比较
- css - 将鼠标悬停在整个正面卡片上,而不仅仅是按钮上
- ftp - 如何使用 rclone 制作 ftp 远程
- grails - 如何使用spring-security通过用户名和密码登录?
- node.js - Angular 8 更新期间的 npm 注册表问题
- ruby-on-rails - 如何将 Rails 项目从数字海洋水滴转移到域
- windows - 我的 Anaconda Prompt 标题栏中的数字是多少?
- mongodb - MongoDB 在集合之间搜索一个键