多层嵌套三元运算符是否为良好编码实践？有什么可替换方案？

多层嵌套三元运算符是否属于良好编码实践 #

多层（超过2层）嵌套的三元运算符不属于良好编码实践，主要问题包括：

• 可读性差：嵌套结构不符合线性阅读逻辑，层级越深理解成本越高，超过3层的嵌套即便原开发者间隔一段时间后也很难快速梳理清楚分支逻辑
• 易出BUG：修改或新增判断条件时，很难快速确认分支的优先级边界，极易写出不符合预期的逻辑
• 调试困难：断点调试时无法针对单个嵌套分支做断点拦截，问题排查效率极低

你提供的代码中RequestSent、IsUrgent字段赋值以及排序逻辑部分都出现了2~3层的三元嵌套，已经明显降低了代码的可维护性。

推荐替代方案 #

1. 用语法糖简化单层级空判断 #

C# 提供的空值合并运算符??、空条件运算符?.可以替代绝大多数单层级的空判断三元写法，比三元更简洁，比如代码中的：

RestoreNumber = row.Field<string>("RestoreNumber") ? row.Field<string>("RestoreNumber") : row.Field<string>("RestoreNetNumber")

可以直接简化为：

RestoreNumber = row.Field<string>("RestoreNumber") ?? row.Field<string>("RestoreNetNumber")

2. 复杂判断逻辑提取为独立方法 #

2层及以上的嵌套三元逻辑，建议封装为命名清晰的独立方法，调用时只需传参即可，逻辑一目了然，比如你代码中RequestSent的赋值逻辑，可以提取为：

private string GetRequestSentStatus(IEnumerable<Request> requests)
{
    // 先过滤无效请求
    var validReqs = requests.Where(req => req.Status != "Deleted" && req.Status != "Cancelled").ToList();
    if (!validReqs.Any()) return "";
    // 判断全部符合条件
    if (validReqs.All(req => req.Status is "Versont" or "Ontvogen")) return "Ja";
    // 判断部分符合条件
    if (validReqs.Any(req => req.Status is "Versont" or "Ontvogen")) return "Wat";
    return "";
}

调用时直接写RequestSent = GetRequestSentStatus(requests)即可。

3. 扁平if-else分支替代嵌套 #

对于查询构造这类分支逻辑，可以拆分为扁平的if-else结构，避免嵌套三元，比如你代码中的排序逻辑：

qry = sortedDescending
    ? qry.OrderByDescending(b => sortedField == "fDeadline" ? b.fDeadlineForSort : b.RestoreDate)
    : qry.OrderBy(b => sortedField == "fDeadline" ? b.fDeadlineForSort : b.RestoreDate);

可以改写为更易读的形式：

if (sortedField == "fDeadline")
{
    qry = sortedDescending ? qry.OrderByDescending(b => b.fDeadlineForSort) : qry.OrderBy(b => b.fDeadlineForSort);
}
else
{
    qry = sortedDescending ? qry.OrderByDescending(b => b.RestoreDate) : qry.OrderBy(b => b.RestoreDate);
}

4. 枚举替代字符串状态值 #

如果判断返回的是固定的有限状态，建议定义为枚举类型，避免硬编码字符串的拼写错误，同时进一步提升代码可读性。

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