给定代码的优化版本是什么？

Question

我有一段代码，我在其中检查复选框，比较给定代码中数据表的内容：

 foreach (DataRow dr in dtResult.Rows)
 {
    for (var i = 0; i < chkboxListWorkTypes.Items.Count; i++)
    {
      if (chkboxListWorkTypes.Items[i].Value.Equals(dr["WorkTypeID"].ToString()))
      {
         chkboxListWorkTypes.Items[i].Selected = true;
      }
    }
 }

任何 labmda 或 linq 表达式都很棒。

Answer 1

真正的优化将是数据绑定。

Answer 2

我就会这样做，

 foreach (DataRow dr in dtResult.Rows)
 {
    string cWorkTypeID = dr["WorkTypeID"].ToString();
    for (var i = 0; i < chkboxListWorkTypes.Items.Count; i++)
    {
         if(chkboxListWorkTypes.Items[i].Value.Equals(cWorkTypeID))
           chkboxListWorkTypes.Items[i].Selected =true;
    }
 }

如果复选框上的 ID 不同（可能是不同），

 foreach (DataRow dr in dtResult.Rows)
 {
    string cWorkTypeID = dr["WorkTypeID"].ToString();
    for (var i = 0; i < chkboxListWorkTypes.Items.Count; i++)
    {
         if(chkboxListWorkTypes.Items[i].Value.Equals(cWorkTypeID))
         {
            chkboxListWorkTypes.Items[i].Selected = true;
            break;
         }
    }
 }

Answer 3

尽管它在幕后的作用与 @Aristos 的回答几乎相同，但按以下方式执行会减少代码行数：

foreach (DataRow dr in dtResult.Rows)
{
    ListItem item = chkboxListWorkTypes.Items.FindByValue(dr["WorkTypeID"].ToString());
    if (item != null)
    {
         item.Selected = true;
    }
}

Answer 4

您本质上是在执行 JOIN 操作。有 3 种类型常用的连接算法 - 嵌套循环、散列和合并。您的代码正在使用嵌套循环算法 - 这似乎适合 10 和 5 的表大小，除非其中一个或两个已经排序（在这种情况下，合并联接可能更合适）。

我无法想象这会成为任何现实应用程序中的瓶颈 - 但是，我们可以通过一些假设对其进行一些改进。

1. 让我们从提升变量开始，以确保它们不会被重新计算（JIT 优化器可能会自动执行此操作，但这并没有什么坏处）：
   <前><代码>
   int chkBoxCount = chkboxListWorkTypes.Items.Count;
   foreach（dtResult.Rows 中的 DataRow dr）
   {
   字符串 rowValue = dr["WorkTypeID"].ToString();
   for (var i = 0; i < chkBoxCount; i++)
   {
   var chkBox = chkboxListWorkTypes.Items[i];
   if (chkBox.Value.Equals(rowValue))
   {
   chkBox.Selected = true;
   }
   }
   }
2. 假设 rowValue 是一个字符串，让我们使用强制转换：
   <前><代码>
   int chkBoxCount = chkboxListWorkTypes.Items.Count;
   foreach（dtResult.Rows 中的 DataRow dr）
   {
   字符串 rowValue = (字符串)dr["WorkTypeID"];
   for (var i = 0; i < chkBoxCount; i++)
   {
   var chkBox = chkboxListWorkTypes.Items[i];
   if (chkBox.Value.Equals(rowValue))
   {
   chkBox.Selected = true;
   }
   }
   }
3. 由于我们永远不会取消选中复选框，因此布尔比较平均应该比字符串比较更快：
   <前><代码>
   int chkBoxCount = chkboxListWorkTypes.Items.Count;
   foreach（dtResult.Rows 中的 DataRow dr）
   {
   字符串 rowValue = (字符串)dr["WorkTypeID"];
   for (var i = 0; i < chkBoxCount; i++)
   {
   var chkBox = chkboxListWorkTypes.Items[i];
   if (!chkBox.Selected && chkBox.Value.Equals(rowValue))
   {
   chkBox.Selected = true;
   }
   }
   }
4. 假设每个复选框值都是唯一的，我们可以在找到结果时停止迭代：
   <前><代码>
   int chkBoxCount = chkboxListWorkTypes.Items.Count;
   foreach（dtResult.Rows 中的 DataRow dr）
   {
   字符串 rowValue = (字符串)dr["WorkTypeID"];
   for (var i = 0; i < chkBoxCount; i++)
   {
   var chkBox = chkboxListWorkTypes.Items[i];
   if (!chkBox.Selected && chkBox.Value.Equals(rowValue))
   {
   chkBox.Selected = true;
   休息; // 停止寻找
   }
   }
   }
5. 由于您有 10 行和 5 个复选框 - 我们应该反转循环。由于我们在找到值时中断内部循环，因此使内部循环成为 2 中较大的一个会更有效：
   <前><代码>
   int chkBoxCount = chkboxListWorkTypes.Items.Count;
   for (var i = 0; i < chkBoxCount; i++)
   {
   var chkBox = chkboxListWorkTypes.Items[i];
   if (chkBox.Selected) 继续；
   foreach（dtResult.Rows 中的 DataRow dr）
   {
   字符串 rowValue = (字符串)dr["WorkTypeID"];
   if (chkBox.Value.Equals(rowValue))
   {
   chkBox.Selected = true;
   休息;
   }
   }
   }
6. 假设您不关心文化或区分大小写的比较，我们可以使用 OrdinalIgnoreCase：
   <前><代码>
   int chkBoxCount = chkboxListWorkTypes.Items.Count;
   for (var i = 0; i < chkBoxCount; i++)
   {
   var chkBox = chkboxListWorkTypes.Items[i];
   if (chkBox.Selected) 继续；
   foreach（dtResult.Rows 中的 DataRow dr）
   {
   字符串 rowValue = (字符串)dr["WorkTypeID"];
   if (string.Equals(chkBox.Value, rowValue, StringComparison.OrdinalIgnoreCase))
   {
   chkBox.Selected = true;
   休息;
   }
   }
   }

这就是我能做的最好的事情了。如果在如此小的数据集上存在任何可测量的差异，我会感到惊讶。所以，这就是你真正应该做的：

 var workTypeIds = dtResult.Rows.Cast<DataRow>().Select(dr => (string)dr["WorkTypeId"]);
 foreach (var chk in chkBoxListWorkTypes) 
 {
     if (workTypeIds.Contains(chk.Value))
     {
        chk.Selected = true;
     }
 }    

或者，使用 Each 扩展：

 var workTypeIds = dtResult.Rows.Cast<DataRow>().Select(dr => (string)dr["WorkTypeId"]);
 chkBoxListWorkTypes.Where(c => workTypeIds.Contains(c.Value)).Each(c => c.Selected = true);

这可能会稍微慢一些，但在我看来读起来要好得多。