在我的 MVC 应用程序中，为什么添加此代码后性能如此缓慢？

Question

有没有什么方法可以产生同样的想法，即在这些值上将日期倒退一天而不会对搜索时间造成巨大影响？ 如果没有这个代码块，它会在一秒钟内加载一个新页面。添加后，页面加载之间的时间为 6-7 秒。

//incidents is an IQueryable

//Set date backwards one day if incident is entered during third shift after midnight
        for (var x = 0; x < incidents.Count(); x++){
            DateTime createdDate = incidents.ToList().ElementAt(x).DateCreated;
            TimeSpan createdTime = createdDate.TimeOfDay;
            if (createdTime > startTime && createdTime < endTime){
                incidents.Skip(x).First().DateCreated = new DateTime(2019,01,01);
            }
        }

Answer 1

由于incidents 是IQueryable，您正在执行数据库调用2x+1 次。一次用于Count，一次用于ToList，一次用于Skip(x).First每个事件。

您可以尝试在循环之前调用ToList()并处理水合列表以查看性能是否有所提高。

Answer 2

这可能与许多.ToList()、.Skip() 和.Count() 通话有关。 请尝试以下方法：

foreach (var incident in incidents.ToList())
{
    var createdDate = incident.DateCreated;
    var createdTime = createdDate.TimeOfDay;
    var index = incidents.IndexOf(indcident);
    if (createdTime > startTime && createdTime < endTime){
        incidents.Skip(index).First().DateCreated = new DateTime(2019,01,01);
    }
}

而且，我对.Skip() 不是很熟悉，但你能不能把代码简化成这样？

foreach (var incident in incidents.ToList())
{
    var createdDate = incident.DateCreated;
    var createdTime = createdDate.TimeOfDay;
    if (createdTime > startTime && createdTime < endTime){
        incident.DateCreated = new DateTime(2019, 01, 01);
    }
}