这似乎是最简单的可能，但我不认为这是简单的。

if (xElementAltItem != null)
{
    if (xElementAltItem.Value.Trim() == "null")
    {
        if (generalstatuscode.Contains(currentstatuscode))
        {
            statuscodeToSet = "1";
            if (xElementupdateDate != null)
            {
                if (xElementupdateDate.Value == "01-JAN-2099")
                {
                    statuscodeToSet = "2";
                }
                if (xElementupdateDate.Value != "01-JAN-2099")
                {
                    statuscodeToSet = "3";
                }
            }
        }
        if ((currentstatuscode == "Act-NotOrd"))
        {
            statuscodeToSet = "5";
        }
    }
    else
    {
        if (generalstatuscode.Contains(currentstatuscode))
        {
            statuscodeToSet = "4";
        } 
    }
}

您确实需要通过创建一些有意义的变量名来梳理这里的逻辑，使条件读起来像英语，否则您的代码很容易出现错误。

更新：我似乎已经引入了一个差异wrt在处理更新日期的原始，因为它似乎我折叠成两个三个国家（哎呀！）这出现在步骤#2/#3.在任何情况下，我相信下面仍然是一个适用的模板遵循.

第一步：消除重复的变量条件检查。

这极大地简化了条件句，并使其更容易看到“应该”发生的事情。

var nullAlt = xElementAltItem != null ? xElementAltItem.Value.Trim() == "null" : false;
var hasCurrentStatus = generalstatuscode.Contains(currentstatuscode);
var updateDate = xElementupdateDate != null ? xElementupdateDate.Value : "";

if (nullAlt && hasCurrentStatus)
{
    statuscodeToSet = "1";
}
if (nullAlt && hasCurrentStatus && updateDate == "01-JAN-2099"))
{
    statuscodeToSet = "2";
}
if (nullAlt && hasCurrentStatus && updateDate != "01-JAN-2099"))
{
    statuscodeToSet = "3";
}
if (!nullAlt && hasCurrentStatus)
{
    statuscodeToSet = "4";
} 
if (nullAlt && currentstatuscode == "Act-NotOrd")
{
    statuscodeToSet = "5";
}

第二步：将单独的条件分组，并使用“嵌套”和“else”条件重写代码。

这将消除大部分（如果不是全部）重复检查，并提供贯穿整个条件结构的“一条路径”。分组的顺序取决于每个条件的逻辑重要性以及它消除分支之间重复的程度。在这种情况下，代码按nullAt分组，然后按hasCurrentState分组。

// ..
if (nullAlt) { 
  if (hasCurrentStatus) {
    statuscodeToSet = "1";
    if (updateDate == "01-JAN-2099"))
    {
      statuscodeToSet = "2";
    } else { /* updateDate != "01-JAN-2099" */
    {
      statuscodeToSet = "3";
    }
  } else if (currentstatuscode == "Act-NotOrd")
  {
    statuscodeToSet = "5";
  }
} else { /* !nullAlt */
  if (hasCurrentStatus)
  {
    statuscodeToSet = "4";
  } 
}

第三步：将代码放入一个单独的函数中并返回结果。

也就是说，不要将“状态”分配给变量。除了保持整洁的代码分离和自文档化之外，这很有用，因为这样可以更容易地发现无效的代码路径/逻辑。
例如，return "1"在它所处的位置没有意义（因为这样就不能到达状态“2”或“3”，如果它被删除，那么状态“1”就永远不会返回！），并且有一些“否则”的情况没有得到处理。

string GetStatusCode (..) {
  // ..
  if (nullAlt) { 
    if (hasCurrentStatus) {
      // whoops! What was this doing??
      return "1";
      if (updateDate == "01-JAN-2099"))
      {
        return "2";
      } else { /* updateDate != "01-JAN-2099" */
      {
        return "3";
      }
    } else if (currentstatuscode == "Act-NotOrd")
    {
      return "5";
    } else {
      // return what here?
    }
  } else { /* !nullAlt */
    if (hasCurrentStatus)
    {
      return "4";
    } else {
      // return what here?
    } 
  }
}

在最后一个阶段，中间变量的使用可以恢复，而不会损失太多的可读性，因为重复的表达式已经被消除了。然而，除了严格要求延迟计算的情况外，单独的变量是完全有效的-如果它们使代码更干净，请使用它们。
使用枚举或常量也是一种改进：NeedsUpdate的状态值比"3"提供更多信息。
我建议保持简单，并遵循上面概述的方法。
YMMV。

试试这个：

if (xElementAltItem == null)
    return;
statuscodeToSet = "1";
if (xElementAltItem.Value.Trim() == "null")
    if (generalstatuscode.Contains(currentstatuscode))
        statuscodeToSet = xElementupdateDate != null && xElementupdateDate.Value != "01-JAN-2099" ? "3" : "2";
    statuscodeToSet = currentstatuscode == "Act-NotOrd" ? "5" : null;
else
    statuscodeToSet = generalstatuscode.Contains(currentstatuscode) ? "4" : null;

@Samiey Mehdi走在正确的道路上。
但这与if语句无关，它是关于构建规则引擎，或者将您的代码置于构建规则引擎的位置
反转Samiey代码：

statuscodeToSet = "1";
if (xElementAltItem == null) return;

return xElementAltItem.Value.Trim() != "null"
        ? generalstatuscode.Contains(currentstatuscode) ? "4" : null
        : null;

return !generalstatuscode.Contains(currentstatuscode)
        ? currentstatuscode == "Act-NotOrd" ? "5" : null
        : statuscodeToSet;

return generalstatuscode.Contains(currentstatuscode)
        ? xElementupdateDate != null && xElementupdateDate.Value != "01-JAN-2099" ? "3" : "2"
        : statuscodeToSet;

现在每一个“返回”都是一个规则。
我们可以把每个都放在它自己的接口/类中。
将这些接口/类放入工厂
调用工厂获取规则列表
循环遍历列表以执行规则
该枚举可以是foreach或Parellel。foreach
通过我们的代码

• 干净
• 可测试
• 快