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

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

or

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

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


当前回答

关于编码风格:

大多数编码标准,无论哪种语言,都禁止从一个函数中使用多个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。

其他回答

来自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。

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

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

从性能的角度来看,这在Python中并不重要,我认为对所有现代语言都是如此。

归根结底是风格和可读性。第二个选项(if-else块)比第一个更具可读性,而且比一行三元操作可读性强得多。