在控制流使其多余的情况下应使用“其他”吗？

有时我会偶然发现类似于以下示例的代码（此函数的确切作用超出了此问题的范围）：

function doSomething(value) {
  if (check1(value)) {
    return -1;
  }
  else if (check2(value)) {
    return value;
  }
  else {
    return false;
  }
}

正如你所看到的，if，else if和else语句与配合使用return的语句。对于一个偶然的观察者来说，这似乎很直观，但是我认为（从软件开发人员的角度来看）删除else-s并简化这样的代码会更加优雅：

function doSomething(value) {
  if (check1(value)) {
    return -1;
  }
  if (check2(value)) {
    return value;
  }
  return false;
}

这是有道理的，因为return语句后的所有内容（在相同范围内）将永远不会执行，从而使上面的代码在语义上与第一个示例相同。

以上哪一项更符合良好的编码习惯？两种方法在代码可读性方面都有缺点吗？

编辑：重复的建议已被提出，该问题仅供参考。我认为我的问题涉及另一个主题，因为我并不是要避免像另一个问题中那样重复陈述。这两个问题都试图减少重复，尽管方式略有不同。

我喜欢一个没有的人else，这是为什么：

function doSomething(value) {
  //if (check1(value)) {
  //  return -1;
  //}
  if (check2(value)) {
    return value;
  }
  return false;
}

因为这样做并没有破坏任何东西，这是不应该的。

讨厌所有形式的相互依赖（包括命名函数check2()）。隔离所有可以隔离的内容。有时您需要，else但我在这里看不到。

我认为这取决于代码的语义。如果您的三种情况相互依赖，请明确说明。这提高了代码的可读性，并使其他任何人都更容易理解。例：

if (x < 0) {
    return false;
} else if (x > 0) {
    return true;
} else {
    throw new IllegalArgumentException();
}

在这里，您显然取决于这x三种情况的价值。如果您省略了last else，那么将不太清楚。

如果您的案例不直接相互依赖，请忽略它：

if (x < 0) {
    return false;
}
if (y < 0) {
    return true;
}
return false;

在没有上下文的简单示例中很难显示这一点，但我希望您能理解我的意思。

我更喜欢第二个选项（分离ifs，没有else if和早期return）BUT即只要代码块是短。

如果代码块很长，最好使用else if，因为否则您可能对代码的几个出口点没有清楚的了解。

例如：

function doSomething(value) {
  if (check1(value)) {
    // ...
    // imagine 200 lines of code
    // ...
    return -1;
  }
  if (check2(value)) {
    // ...
    // imagine 150 lines of code
    // ...
    return value;
  }
  return false;
}

在这种情况下，最好使用else if，将返回码存储在一个变量中，最后只包含一个返回语句。

function doSomething(value) {
  retVal=0;
  if (check1(value)) {
    // ...
    // imagine 200 lines of code
    // ...
    retVal=-1;
  } else if (check2(value)) {
    // ...
    // imagne 150 lines of code
    retVal=value;
  }
  return retVal;
}

但是，由于应该努力使函数简短，因此我想说要使其简短，并在第一个代码段中尽早退出。

我在不同情况下都使用。在验证检查中，我省略了其他内容。在控制流中，我使用else。

bool doWork(string name) {
  if(name.empty()) {
    return false;
  }

  //...do work
  return true;
}

与

bool doWork(int value) {
  if(value % 2 == 0) {
    doWorkWithEvenNumber(value);
  }
  else {
    doWorkWithOddNumber(value);
  }
}

第一种情况读起来更多，就像您先检查所有先决条件，将其排除在外，然后进行到要运行的实际代码一样。因此，您真正想要运行的多汁代码直接属于该函数的范围；这才是真正的功能。
第二种情况读起来更像两个路径都是要运行的有效代码，它们在某种条件下与另一个函数同等重要。因此，它们彼此属于相似的范围级别。

我见过的唯一真正重要的情况是这样的代码：

List<?> toList() {
    if (left != null && right != null) {
        Arrays.asList(merge(left, right));
    }
    if (left != null) {
        return Arrays.asList(left);
    }
    if (right != null) {
        return Arrays.asList(right);
    }
    return Arrays.asList();
}

显然这里有问题。问题是，尚不清楚是否return应添加或Arrays.asList删除。如果不深入检查相关方法，就无法解决此问题。您可以通过始终使用完全穷举的if-else块来避免这种歧义。这个：

List<?> toList() {
    if (left != null && right != null) {
        Arrays.asList(merge(left, right));
    } else if (left != null) {
        return Arrays.asList(left);
    } else if (right != null) {
        return Arrays.asList(right);
    } else {
        return Arrays.asList();
    }
}

除非您在first中添加显式return，否则不会编译（使用静态检查的语言）if。

有些语言甚至不允许第一种样式。我尝试仅在严格命令性上下文中或长方法的前提下使用它：

List<?> toList() {
    if (left == null || right == null) {
        throw new IllegalStateException()
    }
    // ...
    // long method
    // ...
}

如果您试图按照代码进行操作并且是错误的做法，那么从这样的函数中退出三个功能确实会造成混乱。

如果该函数有一个出口，则需要其他出口。

var result;
if(check1(value)
{
    result = -1;
}
else if(check2(value))
{
    result = value;
}
else 
{
    result = 0;
}
return result;

实际上，我们走得更远。

var result = 0;
var c1 = check1(value);
var c2 = check2(value);

if(c1)
{
    result = -1;
}
else if(c2)
{
    result = value;
}

return result;

公平地说，您可以为输入验证争取一个早期回报。只是避免将整个逻辑包装在if中。

我宁愿省略多余的else语句。每当我看到它时（在代码审查或重构中），我想知道作者是否理解控制流以及代码中可能存在错误。

如果作者认为else语句是必需的，因此肯定不能理解return语句对控制流的影响，那么在此函数中以及一般而言，这种缺乏理解是错误的主要来源。