我的代码有些慢，有什么建议吗？

Question

我有一个大记录列表，我需要遍历每条记录，添加一些过滤器和计算并将其添加到我的另一个列表中。我认为逐条执行会影响性能，因为显示 900 条记录大约需要 12 秒。

我无法确定为什么花费的时间太长。我使用我的 chrome 开发人员工具来确定它在哪里运行缓慢。后来发现loading 0.2s，scripting 3s，rendering 3s，idle 3s，其他2s。

也许我正在使用实体框架，而 DataTables 让它变慢了。或者我的查询可能有问题。以下是我的代码：

public ActionResult Index(int id, string language)
{
 var All_Employees = from employee in db.Employees
                     .Include(x => x.Country).Include(x => x.Status)
                     where enployee.GenderId == id
                     select employee ;

 var List = new List<EmployeeListViewModel>();

 foreach(var Record in All_Employees.ToList()
         .OrderByDescending(x=> ParseDate(x.JoiningDate)))
 {
      EmployeeListViewModel item = new EmployeeListViewModel();
      item.Id = Record.Id;
      item.Code = Record.Code;
      if(Record.CountryId != null)
      {
          if(language == "en")
          {
             item.Country = Record.Country.NameE;
          }
          else
          {
             item.Country = Record.Country.NameA;
          }
      }

      item.Date = Record.JoiningDate;
      int WorkingDays = 0;
      if(Record.JoiningDate != null)
      {
        DateTime Joining= Convert.ToDateTime(ParseDate(Record.Record.JoiningDate));

        TimeSpan t = DateTime.Now.Date - Joining;
        int Days = int.Parse(t.TotalDays.ToString());
        if (Days > 0)
        {
            WorkingDays = Days;
        }
      }
      item.Days = WorkingDays.ToString();

      if (Record.StatusId != null)
      {
         if (language == "en")
         {
            item.Status = Record.Status.NameE;
         }
         else
         {
            item.Status = Record.Status.NameE;
         }
      }

     List.Add(item);
 }
 return View(List);
}

另一个原因可能是我正在转换我的日期：

private static DateTime? ParseDate(string dateString)
{
   if(dateString != null)
   {
     return DateTime.ParseExact(dateString, "dd-MM-yyyy", CultureInfo.GetCultureInfo("en-US"), DateTimeStyles.None);
   }
   else
   {
     return null;
   }           
}

由于某些原因，我不想在我的数据库中将日期归档为 DateTime。

在我目前的情况下，提高性能的最佳方法是什么？

Answer 1

在您的代码中使用了过多的转换，并且列表被创建了两次。一种可以执行如下代码的方法。

 List<EmployeeListViewModel> lstData = EmployeeListViewModel.ToList();
 for(int i = 0; i < lstData.Count; i++)
 {
     //Put logic here for required changes like Language and Date.
     if(lstData[i].CountryId != null)
     {
          if(language == "en")
             lstData[i].Country = lstData[i].Country.NameE;
          else
             lstData[i].Country = lstData[i].Country.NameA;
      }
 }

尝试减少专门针对字符串和日期时间的转换。下面是例子。

int Days = int.Parse(t.TotalDays.ToString());

在上面的行中，t.TotalDays 始终是 int 类型，无需再次转换为 string 和 int。