c# - 我怎样才能使这个 LINQ 查询更清晰？

Question

我最近编写了一个 LINQ 查询来获取包含最近 6 个月的安置金额的 Dictionary。

它返回月份字符串的 Dictionary - Decimal Amount 对。

看起来有点笨拙。有哪位 LINQ 高手能够帮助我重构它以使其更简洁吗？

/// <summary>
/// Gets the last 6 months of Placement History totalled by Month 
/// for all Agencies
/// </summary>
/// <returns></returns>
public Dictionary<string, decimal> getRecentPlacementHistory()
{
    var placementHistoryByMonth = new Dictionary<string, decimal>();

    using (DemoLinqDataContext db = new DemoLinqDataContext())
    {
        for (int i = 0; i < 6; i++)
        {
            Decimal monthTotal = 
              (from a in db.Accounts
               where 
                 (a.Date_Assigned.Value.Month == DateTime.Now.AddMonths(-i).Month &&
                  a.Date_Assigned.Value.Year == DateTime.Now.AddMonths(-i).Month)
               select a.Amount_Assigned).Sum();
            String currentMonth = DateTime.Now.AddMonths(-i).ToString("MMM");

            placementHistoryByMonth.Add(currentMonth, monthTotal);
        }
        return placementHistoryByMonth;
    }
}

Answer 1

第一个问题:

where (a.Date_Assigned.Value.Month == DateTime.Now.AddMonths(-i).Month &&
       a.Date_Assigned.Value.Year == DateTime.Now.AddMonths(-i).Month)

后一个表达式不应该以 .Year 而不是 .Month 结尾吗？当然，您很少会得到一个值为 1-12 的年份...

我会提取“当前月份”的概念，因为您经常使用它。请注意，您还多次使用当前时间，如果它在月底的午夜运行，可能会产生奇怪的结果......

public Dictionary<string, decimal> getRecentPlacementHistory()
{
    var placementHistoryByMonth = new Dictionary<string, decimal>();
    using (DemoLinqDataContext db = new DemoLinqDataContext())
    {
        DateTime now = DateTime.Now;

        for (int i = 0; i < 6; i++)
        {
            DateTime selectedDate = now.AddMonths(-i);

            Decimal monthTotal = 
               (from a in db.Accounts
                where (a.Date_Assigned.Value.Month == selectedDate.Month &&
                       a.Date_Assigned.Value.Year == selectedDate.Year)
                select a.Amount_Assigned).Sum();

            placementHistoryByMonth.Add(selectedDate.ToString("MMM"),
                                        monthTotal);
        }
        return placementHistoryByMonth;
    }
}

我意识到这可能是您试图摆脱的循环。您可以尝试计算出整批日期的上限和下限，然后在相关范围内按 a.Date_Assigned 的年/月进行分组。老实说，它不会更漂亮。请注意，如果您可以成功的话，这只是对数据库的一次查询。