如何优化C#代码跳过多次迭代 高效拼接指定格式过滤条件

核心改进思路 #

• 仅单次遍历data集合，同时收集两类条件片段，完全消除重复遍历的开销
• 用内置的string.Join处理逻辑连接符，从根源上避免手动拼接、截取末尾多余连接符的脆弱操作
• 用元组解构替代原代码中无意义的Item1位置访问，提升代码语义可读性
• 兼容空集合、单元素集合的边界场景，避免原代码中LastIndexOf返回-1引发的运行异常

具体实现代码 #

// 单次遍历收集两类条件片段
var equalityClauses = new List<string>();
var inequalityClauses = new List<string>();
// 解构元组，未使用的第二个值用弃元占位
foreach (var (name1Value, _) in data)
{
    equalityClauses.Add($"(Name1 = '{name1Value}' AND Name2 = '{name1Value}')");
    inequalityClauses.Add($"Name1 != '{name1Value}'");
}

// 各片段用对应逻辑连接符拼接，天然无多余后缀
var equalityPart = string.Join(" OR ", equalityClauses);
var inequalityPart = inequalityClauses.Count > 0 
    ? $"({string.Join(" AND ", inequalityClauses)})" 
    : string.Empty;

// 最终拼接，自动跳过空片段，不会残留多余逻辑连接符
var finalFilter = string.Join(" OR ", new[] { equalityPart, inequalityPart }
    .Where(part => !string.IsNullOrEmpty(part)));

效果验证 #

针对示例测试数据源，最终生成的过滤串与需求格式完全一致：

(Name1 = 'd1' AND Name2 = 'd1') OR (Name1 = 'd3' AND Name2 = 'd3') OR (Name1 != 'd1' AND Name1 != 'd3')

对比原实现的实际收益 #

• 循环开销减少一半：原实现需要完整遍历两次data集合，当列表元素数量较多时，循环带来的开销会明显降低
• 拼接效率更高：string.Join会预计算所有片段的总长度一次性分配内存，比原实现反复触发string +=的内存拷贝操作效率高很多
• 逻辑更稳定：原实现靠硬编码查找最后一个"AND"的位置做截取，一旦条件值本身包含"AND"文本，就会定位错误截坏字符串；空集合场景下还会因为LastIndexOf返回-1直接抛出越界异常，改进后的逻辑完全规避了这些问题
• 可读性更强：元组解构直接给值赋予语义名称，条件片段的收集、拼接流程和最终过滤串的结构一一对应，不需要理解中间的字符串截取操作就能快速理清逻辑

注：实际生产环境拼接动态查询条件时，建议搭配参数化查询处理特殊字符与注入风险，不要直接将用户可控值拼接到查询串中。

内容的提问来源于stack exchange，提问作者user584018