MVC - 在 foreach 循环中计数，最佳实践

Question

我一直致力于在 foreach 循环中获取计数记录。我将需要在一个页面上运行许多这些计数。我正在寻找最有效的方法来做到这一点。

我已经走到了这一步，但我不确定我是否走在正确的道路上。如果是，我如何将这些数据放入我的视图中。

视图模型

public class AgtLeadStatsListVM
{
    public string LoanAgent { get; set; }
    public DateTime LeadDate { get; set; }
    public int LeadDailyCt { get; set; }
    public int LeadWeeklyCt { get; set; }
    public int LeadMTDCt { get; set; }
    public int LeadYTDCt { get; set; }
    public IEnumerable<MWFUser> AgentList { get; set; }
    public virtual WebLead Lead { get; set; }
}

控制器

 var model = new AgtLeadStatsListVM();            
 {
 // Get Selected Agent's Information
 var AgentList = from l in db.MWFUsers
                 where (l.UserTitle == "Banker"
                 select l;

    foreach (var agent in AgentList)
    {

    // Daily Lead Count
    var LeadDailyCt = db.WebLeads.Count(x => (x.LoanAgent == agent.UserEmail)
    && (x.LeadDate >= todayDate && x.LeadDate <= todayEndDay));

    // Weekly Lead Count
    var LeadWeeklyCt = db.WebLeads.Count(x => (x.LoanAgent == agent.UserEmail)
                                         && x.LeadDate >= firstOfWeek
                                         && x.LeadDate <= todayEndDay);

    // Monthly Lead Count
    var LeadMTDCount = db.WebLeads.Count(x => (x.LoanAgent == agent.UserEmail)
                                         && x.LeadDate >= firstOfMonth
                                         && x.LeadDate <= todayEndDay);

   // YTD Lead Count
   var LeadYTDCount = db.WebLeads.Count(x => (x.LoanAgent == agent.UserEmail)
                                         && x.LeadDate >= firstOfMonth
                                         && x.LeadDate <= todayEndDay);
}
}

查看

@model LoanModule.ViewModels.AgtLeadStatsListVM

<div>
    @foreach (var item in Model.AgentList)
    {
    <p>@Model.LoanAgent</p>
    <p>@Model.LeadDailyCt</p>
    <p>@Model.LeadWeeklyCt</p>
    <p>@Model.LeadMTDCt</p>
    <p>@Model.LeadYTDCt</p>
    }

我的视图收到此错误：对象引用未设置为对象的实例。 （在线：@foreach（Model.AgentList 中的变量项））

我错过了什么？

谢谢。

Answer 1

var model = new AgtLeadStatsListVM(); 末尾的分号表示您不再在该行之后的对象初始化程序中。您可能正在尝试使用的语法更接近以下内容：

var agents = 
    from l in db.MWFUsers
    where l.UserTitle == "Banker"
    select l;

var model = new AgtLeadStatsListVM
{
    // Get Selected Agent's Information
    AgentList = agents.ToList(),
    // Daily Lead Count
    LeadDailyCt = agents.Sum(a => db.WebLeads.Count(
        x => (x.LoanAgent == a.UserEmail)
            && (x.LeadDate >= todayDate && x.LeadDate <= todayEndDay)))
    // ...
}

顺便说一句，如果您想在一次往返中获取所有这些信息，您可以使用这个基于group by 的技巧。

var model = 
    (from agent in agents
    let webLeads = db.WebLeads.Where(x => x.LoanAgent == agent.UserEmail)
    group new{agent, webLeads} by 0 into g
    select new AgtLeadStatsListVM
    {
        // Get Selected Agent's Information
        AgentList = g.Select(e => e.agent).ToList(),
        // Daily Lead Count
        LeadDailyCt = g.Sum(e => e.webLeads.Count(x => x.LeadDate >= todayDate && x.LeadDate <= todayEndDay)),
        // ...
    }).FirstOrDefault();

更新

从你的 cmets 看来，这更像是你想要的：

var model = 
    (from agent in agents
    let webLeads = db.WebLeads.Where(x => x.LoanAgent == agent.UserEmail)
    select new AgtLeadStatsListVM
    {
        // Get Selected Agent's Information
        LoanAgent = agent.UserEmail,
        // Daily Lead Count
        LeadDailyCt = webLeads.Count(x => x.LeadDate >= todayDate && x.LeadDate <= todayEndDay),
        // ...
    }).ToList();

还有你的查看代码：

@model IEnumerable<LoanModule.ViewModels.AgtLeadStatsListVM>

<div>
    @foreach (var item in Model)
    {
      <p>@item.LoanAgent</p>
      <p>@item.LeadDailyCt</p>
      <p>@item.LeadWeeklyCt</p>
      <p>@item.LeadMTDCt</p>
      <p>@item.LeadYTDCt</p>
    }

AgentList 属性应该从您的模型中完全删除。

Answer 2

我的视图收到此错误：对象引用未设置为 对象的实例。 （在线：@foreach（var item in 模型.AgentList))

AgentList 为空。

此外，您还没有正确初始化模型。

具体就是这行代码

var model = new AgtLeadStatsListVM();

创建一个AgtLeadStatsListVM 类型的新对象，其中

• LoanAgent 为空
• LeadDate 1/1/0001 12:00:00 AM
• LeadDailyCt 为 0
• LeadWeeklyCt 为 0
• LeadMTDCt 为 0
• LeadYTDCt 为 0
• AgentList 为空
• WebLead 是领导

默认值，因为您没有设置任何值。可能你想使用对象初始化器，你不需要();。我们只是这样写：

var model = new AgtLeadStatsListVM
{
    LoadAgent = "Name of the LoadAgent",
    LeadDate = DateTime.Now.Utc,
    LeadDailyCt = agents.Sum(a => db.WebLeads.Count(
    x => (x.LoanAgent == a.UserEmail)
        && (x.LeadDate >= todayDate && x.LeadDate <= todayEndDay)))
    // Do the same for the rest of the corresponding properties.
}

Answer 3

我将忽略您遇到的错误（请参阅其他答案）并仅参考 最佳实践和最有效的方法来计算问题的一部分。

最有效的方法（至少在我看来）是对结果使用一些缓存技术并每天更新缓存（因为您使用的最大分辨率是每天）。显然，选择合适的缓存机制取决于您的应用程序。它可以从在应用程序启动时将一些数据存储在静态变量中，到运行专用的Redis server（或任何其他快速数据结构存储）。这里的底线是：尽量减少对 DB 的查询次数并缓存任何合适的数据。