c# - 我应该如何在 C# 中重写一个非常大的复合 if 语句？

Question

在我的 C# 代码中，我有一个足够无辜地开始的 if 语句:

if((something == -1) && (somethingelse == -1) && (etc == -1)) {
    // ...
}

它在成长。我想现在里面肯定有20个条款。

我应该如何处理这个？

Answer 1

尽可能使用门。

if 语句

if(bailIfIEqualZero != 0 && 
   !string.IsNullOrEmpty(shouldNeverBeEmpty) &&
   betterNotBeNull != null &&
   !betterNotBeNull.RunAwayIfTrue &&
   //yadda

重构后的版本

if(bailIfIEqualZero == 0)
  return;

if(string.IsNullOrEmpty(shouldNeverBeEmpty))
  return;

if(betterNotBeNull == null || betterNotBeNull.RunAwayIfTrue)
  return;

//yadda