假设我有一个带有return的if语句。从效率的角度来看,我应该使用

if(A > B):
    return A+1
return A-1

or

if(A > B):
    return A+1
else:
    return A-1

在使用编译语言(C)或脚本语言(Python)时,我应该选择其中一种还是另一种?


当前回答

来自Chromium的风格指南:

返回后不要使用else:

# Bad
if (foo)
  return 1
else
  return 2

# Good
if (foo)
  return 1
return 2

return 1 if foo else 2

其他回答

关于编码风格:

大多数编码标准,无论哪种语言,都禁止从一个函数中使用多个return语句,这是一种糟糕的做法。

(尽管我个人认为在一些情况下,多个返回语句是有意义的:文本/数据协议解析器,具有广泛错误处理的函数等)

所有这些行业编码标准的共识是,表达式应该写成这样:

int result;

if(A > B)
{
  result = A+1;
}
else
{
  result = A-1;
}
return result;

关于效率:

上面的例子和问题中的两个例子在效率上是完全相同的。在所有这些情况下,机器代码必须比较A > B,然后分支到A+1或A-1计算,然后将结果存储在CPU寄存器或堆栈上。

编辑:

来源:

MISRA-C:2004规则14.7,其中引用了… IEC 61508 - 3。第三部分,表B.9。 IEC 61508 - 7。C.2.9。

如果可能的话,我个人会尽量避免其他阻塞。参见反if运动

而且,他们不收“额外”的费用,你知道:p

“简单胜于复杂”&“可读性为王”

delta = 1 if (A > B) else -1
return A + delta

由于return语句终止了当前函数的执行,因此这两种形式是等效的(尽管可以说第二种形式比第一种形式更易于阅读)。

这两种形式的效率是相当的,如果if条件为假,底层机器码必须执行跳转。

注意,Python支持一种语法,允许你在你的情况下只使用一个return语句:

return A+1 if A > B else A-1

任何明智的编译器,你应该观察到没有区别;它们应该被编译为相同的机器代码,因为它们是等效的。

版本A更简单,这就是我使用它的原因。

如果你在Java中打开了所有的编译器警告,你会在第二个版本中得到一个警告,因为这是不必要的,而且会增加代码的复杂性。