c# - 优化 TreeView 方法
问题描述
我有一个非常大的插入子级的方法TreeView，基本上我做了两个不同的查询并设置了一些小的更改 IE fir foreach 插入父级的第一个子级和之前创建的第二个 foreach 插入子级的子级。
Foreach 查询只是更改了条件AND [ParentDesignKey] IS NULL，但其中的变量太相似了。
所以在这里我只有两个不同的foreach，但我需要再添加4个第一个foreach（第二个foreach）的孩子，我不想要非常大的方法。有没有办法优化这个？
完整方法：
private void AssignDesign(DataTable designItemList, DataTable designFolioList, TreeNode parent, string designName, string legacyKey, int indexProject)
{
var iDesign = 0;
foreach (var currentDesignItem in designItemList.Select($"[Design Name] = '{designName}' AND [Project Number] = {legacyKey} AND [ParentDesignKey] IS NULL"))
{
var designItemName = currentDesignItem["DesignItemName"] as string;
var designItemKey = currentDesignItem["DesignKey"] as int?;
var designItemStatus = currentDesignItem["Design Status"] as string;
var designItemIsLocked = currentDesignItem["IsLocked"] as bool?;
var designItemStatusKey = currentDesignItem["StatusKey"] as int?;
var designItemCreated = currentDesignItem["Design Created"] as DateTime?;
var designItemRunCount = currentDesignItem["RunCount"] as int?;
var designItemRunUnConCount = currentDesignItem["RunUnconCnt"] as int?;
var designItemShopsStatus = currentDesignItem["ShopsStatus"] as int?;
var designItemShopStatusExtended = currentDesignItem["ShopStatusExtended"] as string;
var designItemFolio = designFolioList.Select($"[DesignKey] ={designItemKey}").FirstOrDefault()["Folio"].ToString();
if (designItemName is null || designItemShopsStatus is null || designItemShopStatusExtended is null || designItemKey is null || designItemStatus is null || designItemStatusKey is null || designItemCreated is null || designItemRunCount is null || designItemRunUnConCount is null || designItemIsLocked is null) continue;
var projectIndex = parent.Index;
parent.Nodes[indexProject].Nodes.Add(designItemName);
parent.Nodes[indexProject].Nodes[iDesign].Tag = $"{designItemKey.Value}|{projectIndex}|BDCD";
parent.Nodes[indexProject].Nodes[iDesign].Name = "Design";
parent.Nodes[indexProject].Nodes[iDesign].ToolTipText = $"Design: {designItemName} - Folio: {designItemFolio}";
parent.Nodes[indexProject].Nodes[iDesign].ContextMenuStrip = contextMenuDesign;
//load icon for design item
this.LoadDesignStatusNodeIcon(parent.Nodes[indexProject].Nodes, iDesign, designItemStatusKey.Value, designItemIsLocked.Value);
var parentNodes = parent.Nodes[indexProject].Nodes[iDesign].Nodes;
parentNodes.Add(new TreeNode($"{designItemCreated.Value.ToLongDateString()} ({designItemCreated.Value.ToString("yyMMdd")})", 8, 8)
{
Name = "Design Created",
ToolTipText = "Created Date"
});
parentNodes.Add(new TreeNode(designItemStatus, 18, 18)
{
Name = "Design Status",
ToolTipText = "Design Status",
Tag = $"{designItemStatusKey.Value}|{designItemIsLocked}"
});
var imgKey = LoadShopsNodeIcon(designItemShopsStatus.Value);
parentNodes.Add(new TreeNode(designItemShopStatusExtended, imgKey, imgKey)
{
Name = "Shops Drawing Status",
ToolTipText = "Shops Drawing Status",
Tag = designItemShopsStatus.Value
});
if (designItemRunCount.Value == 0 && designItemRunUnConCount.Value > 0)
{
parentNodes.Add(new TreeNode("Unconsolidated Runs: " + designItemRunUnConCount.Value)
{
Name = "Unconsolidated Runs",
ToolTipText = "Unconsolidated Runs",
Tag = designItemRunUnConCount.Value
});
}
else
{
parentNodes.Add(new TreeNode("Run Count: " + designItemRunCount.Value)
{
Name = "Run Count",
ToolTipText = "Run Count",
Tag = designItemRunCount.Value
});
}
var iChangeOrderDesign = 0;
foreach (var item in designItemList.Select($"[Design Name] = '{designName}' AND [Project Number] = {legacyKey} AND [ParentDesignKey] = {designItemKey}"))
{
var changeOrderDesignItemName = item["DesignItemName"] as string;
var changeOrderDesignItemKey = item["DesignKey"] as int?;
var changeOrderDesignItemStatus = item["Design Status"] as string;
var changeOrderDesignItemIsLocked = item["IsLocked"] as bool?;
var changeOrderDesignItemStatusKey = item["StatusKey"] as int?;
var changeOrderDesignItemCreated = item["Design Created"] as DateTime?;
var changeOrderDesignItemRunCount = item["RunCount"] as int?;
var changeOrderDesignItemRunUnConCount = item["RunUnconCnt"] as int?;
var changeOrderDesignItemShopsStatus = item["ShopsStatus"] as int?;
var changeOrderDesignItemShopStatusExtended = item["ShopStatusExtended"] as string;
var changeOrderDesignItemFolio = designFolioList.Select($"[DesignKey] ={changeOrderDesignItemKey}").FirstOrDefault()["Folio"].ToString();
var changeOrderParentDesignKey = item["ParentDesignKey"] as int?;
if (changeOrderDesignItemName is null || changeOrderDesignItemShopsStatus is null || changeOrderDesignItemShopStatusExtended is null || changeOrderDesignItemKey is null || changeOrderDesignItemStatus is null || changeOrderDesignItemStatusKey is null || changeOrderDesignItemCreated is null || changeOrderDesignItemRunCount is null || changeOrderDesignItemRunUnConCount is null || changeOrderDesignItemIsLocked is null) continue;
//Create lowest level and assign their tree nodes
projectIndex = parent.Index;
TreeNode tn = parent.Nodes[indexProject].Nodes[iDesign].Nodes.Add(changeOrderDesignItemName);
parent.Nodes[indexProject].Nodes[iDesign].Nodes[tn.Index].Tag = $"{changeOrderDesignItemKey.Value}|{projectIndex}|CO";
parent.Nodes[indexProject].Nodes[iDesign].Nodes[tn.Index].Name = "Design";
parent.Nodes[indexProject].Nodes[iDesign].Nodes[tn.Index].ToolTipText = $"Change Order: {changeOrderDesignItemName} - Folio: {changeOrderDesignItemFolio}";
parent.Nodes[indexProject].Nodes[iDesign].Nodes[tn.Index].ContextMenuStrip = contextMenuDesign;
//load icon for changeOrderDesign item
var changeOrderparentNodes = parent.Nodes[indexProject].Nodes[iDesign].LastNode.Nodes;
changeOrderparentNodes.Add(new TreeNode($"{changeOrderDesignItemCreated.Value.ToLongDateString()} ({changeOrderDesignItemCreated.Value.ToString("yyMMdd")})", 8, 8)
{
Name = "Change Order Design Created",
ToolTipText = "Created Date"
});
changeOrderparentNodes.Add(new TreeNode(changeOrderDesignItemStatus, 18, 18)
{
Name = "Design Status",
ToolTipText = "Design Status",
Tag = $"{changeOrderDesignItemStatusKey.Value}|{changeOrderDesignItemIsLocked}"
});
var changeOrderimgKey = LoadShopsNodeIcon(changeOrderDesignItemShopsStatus.Value);
changeOrderparentNodes.Add(new TreeNode(changeOrderDesignItemShopStatusExtended, changeOrderimgKey, changeOrderimgKey)
{
Name = "Shops Drawing Status",
ToolTipText = "Shops Drawing Status",
Tag = changeOrderDesignItemShopsStatus.Value
});
if (changeOrderDesignItemRunCount.Value == 0 && changeOrderDesignItemRunUnConCount.Value > 0)
{
changeOrderparentNodes.Add(new TreeNode("Unconsolidated Runs: " + changeOrderDesignItemRunUnConCount.Value)
{
Name = "Unconsolidated Runs",
ToolTipText = "Unconsolidated Runs",
Tag = changeOrderDesignItemRunUnConCount.Value
});
}
else
{
changeOrderparentNodes.Add(new TreeNode("Run Count: " + changeOrderDesignItemRunCount.Value)
{
Name = "Run Count",
ToolTipText = "Run Count",
Tag = changeOrderDesignItemRunCount.Value
});
}
this.LoadChangeOrderNodeIcon(parent.Nodes[indexProject].Nodes[iDesign].Nodes[tn.Index], iChangeOrderDesign, changeOrderDesignItemStatusKey.Value, changeOrderDesignItemIsLocked.Value);
}
iDesign++;
}
}
更新
为了做一个更简单的方法，我做了一个递归并根据收到的项目设置变量：
private void AssignDesign(DataTable designItemList, DataTable designFolioList, TreeNode parent, string designName, string legacyKey, int indexProject, string DesignType, string Condition, int currDesign)
{
var iDesign = 0;
foreach (var currentDesignItem in designItemList.Select($"[Design Name] = '{designName}' AND [Project Number] = {legacyKey} AND [ParentDesignKey] {Condition}"))
{
var designItemName = currentDesignItem["DesignItemName"] as string;
var designItemKey = currentDesignItem["DesignKey"] as int?;
var designItemStatus = currentDesignItem["Design Status"] as string;
var designItemIsLocked = currentDesignItem["IsLocked"] as bool?;
var designItemStatusKey = currentDesignItem["StatusKey"] as int?;
var designItemCreated = currentDesignItem["Design Created"] as DateTime?;
var designItemRunCount = currentDesignItem["RunCount"] as int?;
var designItemRunUnConCount = currentDesignItem["RunUnconCnt"] as int?;
var designItemShopsStatus = currentDesignItem["ShopsStatus"] as int?;
var designItemShopStatusExtended = currentDesignItem["ShopStatusExtended"] as string;
var designItemFolio = designFolioList.Select($"[DesignKey] ={designItemKey}").FirstOrDefault()["Folio"].ToString();
var changeOrderParentDesignKey = currentDesignItem["ParentDesignKey"] as int?;
//var designConfirmDeliveryDate = currentDesignItem["ConfirmDeliveryDate"] as DateTime?;
if (designItemName is null || designItemShopsStatus is null || designItemShopStatusExtended is null || designItemKey is null || designItemStatus is null || designItemStatusKey is null || designItemCreated is null || designItemRunCount is null || designItemRunUnConCount is null || designItemIsLocked is null) continue;
//Create lowest level and assign their tree nodes
var projectIndex = parent.Index;
TreeNodeCollection parentNodes;
if (DesignType == "BDCD")
{
parent.Nodes[indexProject].Nodes.Add(designItemName);
parent.Nodes[indexProject].Nodes[iDesign].Tag = $"{designItemKey.Value}|{projectIndex}|BDCD";
parent.Nodes[indexProject].Nodes[iDesign].Name = "Design";
parent.Nodes[indexProject].Nodes[iDesign].ToolTipText = $"Design: {designItemName} - Folio: {designItemFolio}";
parent.Nodes[indexProject].Nodes[iDesign].ContextMenuStrip = contextMenuDesign;
this.LoadDesignStatusNodeIcon(parent.Nodes[indexProject].Nodes, iDesign, designItemStatusKey.Value, designItemIsLocked.Value);
parentNodes = parent.Nodes[indexProject].Nodes[iDesign].Nodes;
}
else
{
TreeNode tn = parent.Nodes[indexProject].Nodes[currDesign].Nodes.Add(designItemName);
parent.Nodes[indexProject].Nodes[currDesign].Nodes[tn.Index].Tag = $"{designItemKey.Value}|{projectIndex}|CO";
parent.Nodes[indexProject].Nodes[currDesign].Nodes[tn.Index].Name = "Design";
parent.Nodes[indexProject].Nodes[currDesign].Nodes[tn.Index].ToolTipText = $"Change Order: {designItemName} - Folio: {designItemFolio}";
parent.Nodes[indexProject].Nodes[currDesign].Nodes[tn.Index].ContextMenuStrip = contextMenuDesign;
this.LoadChangeOrderNodeIcon(parent.Nodes[indexProject].Nodes[currDesign].Nodes[tn.Index], currDesign, designItemStatusKey.Value, designItemIsLocked.Value);
parentNodes = parent.Nodes[indexProject].Nodes[currDesign].LastNode.Nodes;
}
//load icon for design item
var dateName = string.Empty;
switch (DesignType)
{
case "BDCD":
dateName = "Design Created";
break;
case "CO":
dateName = "Change Order Design Created";
break;
}
parentNodes.Add(new TreeNode($"{designItemCreated.Value.ToLongDateString()} ({designItemCreated.Value.ToString("yyMMdd")})", 8, 8)
{
Name = dateName,
ToolTipText = "Created Date"
});
parentNodes.Add(new TreeNode(designItemStatus, 18, 18)
{
Name = "Design Status",
ToolTipText = "Design Status",
Tag = $"{designItemStatusKey.Value}|{designItemIsLocked}"
});
var imgKey = LoadShopsNodeIcon(designItemShopsStatus.Value);
parentNodes.Add(new TreeNode(designItemShopStatusExtended, imgKey, imgKey)
{
Name = "Shops Drawing Status",
ToolTipText = "Shops Drawing Status",
Tag = designItemShopsStatus.Value
});
if (designItemRunCount.Value == 0 && designItemRunUnConCount.Value > 0)
{
parentNodes.Add(new TreeNode("Unconsolidated Runs: " + designItemRunUnConCount.Value)
{
Name = "Unconsolidated Runs",
ToolTipText = "Unconsolidated Runs",
Tag = designItemRunUnConCount.Value
});
}
else
{
parentNodes.Add(new TreeNode("Run Count: " + designItemRunCount.Value)
{
Name = "Run Count",
ToolTipText = "Run Count",
Tag = designItemRunCount.Value
});
}
var lastItem = string.Empty;
foreach (var item in designItemList.Select($"[Design Name] = '{designName}' AND [Project Number] = {legacyKey} AND [ParentDesignKey] = {designItemKey}"))
{
if (lastItem != parent.Text)
{
AssignDesign(designItemList, designFolioList, parent, designName, legacyKey, indexProject, "CO", $"= {designItemKey}", iDesign);
}
lastItem = parent.Text;
}
iDesign++;
}
}
但正如您所见，我在方法递归中发送查询过滤器：
第一个foreach：
AssignDesign(tvTable, designFolioList, parent, designName, currentProjectNumber, iDesign, "BDCD","IS NULL", 0);
第二个foreach（递归）
AssignDesign(designItemList, designFolioList, parent, designName, legacyKey, indexProject, "CO", $"= {designItemKey}", iDesign);
在第一个我发送"IS NULL"
和第二个$"= {designItemKey}"
有没有办法只查询一次而不是使用不同的过滤器两次？
解决方案
推荐阅读
- html - 关于 HTML 中相对文件路径的问题
- python - 在列表中搜索以按共享内容对元素进行分组
- mongodb - Mongodb 忽略 $exists: true
- ios - Xcode 分发错误 - 无效签名
- json - JSON.stringify() 函数将文件对象替换为空对象
- javascript - 从用户输入查询 MySQL 数据库
- php - 如何为 DateTime 设置 00:00:00
- laravel-5 - 如何使用 Voyager Laravel Admin 在 BREAD 浏览模式下显示缩略图
- ruby-on-rails - 使用 hidden_field 标记时未定义的局部变量或方法“f”
- sql - 在 SQL Server 的打开查询语句中使用参数