Ifs，else ifs和多个返回[关闭]

Question

这不是关于链式 ifs 或 ifs 和 else ifs 的问题。我已经在 SO 中看到了很多这样的问题。

我的问题也不是关于性能，更多的是关于编码标准和可读性。

考虑一下我在一个项目中经常看到的以下琐碎的伪代码：

if (expression > 0)
{
    return 1;
}
else if (expression < 0)
{
    return -1;
}
else
{
    return 0;
}

我通常以稍微不同的方式编写这种构造：

if (expression > 0)
{
    return 1;
}

if (expression < 0)
{
    return -1;
}

return 0;

当然还有第三种选择，即任何方法都不应有多个 return 语句，当方法的复杂性较低时，我发现这过于限制和繁琐：

int retVal;

if (expression > 0)
{
    retVal = 1;
}
else if (expression < 0)
{
    retVal = -1;
}
else
{
    retVal = 0;
}

return retVal;

在编写这些类型的结构时，上面列出的选项之一是否更正确？性能方面，我知道选择完全无关紧要，但从可读性的角度来看，我更喜欢避免 if - else if 语句。话虽如此，很多同事都不同意我的观点，即使他们不能给我任何令人信服的论据。

Answer 1

有很多方法。您可以采用您展示的两种方法之一。你可以这样做：

if (expression == 0)
{
    return 0;
}
return expression < 0 ? -1 : 1;

或者完全使用三元运算符：

return expression == 0
    ? 0
    : expression < 0 ? -1 : 1;

另一种选择是使用函数式方法，例如 offered by my own SuccincT library 并执行以下操作：

return expression.Match().To<int>()
                 .Where(e => e < 0).Do(-1)
                 .Where(e => e > 0).Do(1)
                 .Else(0).Result();

这完全是偏好问题，确实没有正确的答案。

Answer 2

考虑

return expression < 0? -1:
       expression > 0? +1:
       0;

一般来说，它在很大程度上取决于偏好和语义。

如果所有三种情况（>0、<0、==0）都有些对称，那么您的第一个选项对我来说看起来不错。

如果默认路径应该返回0，并且>0、<0 是特殊/例外情况，我会选择第二个选项。

Answer 3

我认为

int retVal = 0;

if (expression > 0)
{
    retVal = 1;
}

if (expression < 0)
{
    retVal = -1;
}

return retVal;

这在性能方面看起来并不正确，并且每次进行条件匹配时都是原因。

在以下情况下，这将匹配，直到找到正确的条件并忽略休息

if (expression > 0)
{
    return 1;
}
else if (expression < 0)
{
    return -1;
}
else
{
    return 0;
}

我认为 if/else if/else 优于性能 从可读性的角度来看，您也可以更喜欢使用 案子 我认为 if else if 是比多个 if 更好的选择

Answer 4

正如问题中提到的 “我的问题也不是关于性能，而是关于编码 标准和可读性。”

您可以将三元运算符用于简单的 if 和 then else 情况，但如果您使用三元运算符检查多个条件，它将无法保持可读性。

return expression > 0 ? -1 : 0; 

对于具有多个检查的条件，请使用您在问题中提供的 if 和 else-if。

if (expression > 0)
{
    return 1;
}
else if (expression < 0)
{
    return -1;
}
else
{
    return 0;
}

在收到@Chris 和@InBetween 的评论后，此答案已更新