关于c#：ifs，else ifs和多次返回

Ifs, else ifs and multiple returns

这不是一个关于被链接的国际单项体育联合会或国际单项体育联合会以及其他国际单项体育联合会的问题。我已经看到很多这样的问题了。

我的问题也不在于性能，而在于编码标准和可读性。

考虑一下在我工作的项目中我经常看到的以下简单的伪代码：

1
2
3
4
5
6
7
8
9
10
11
12

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

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

1
2
3
4
5
6
7
8
9
10
11

if (expression > 0)
{
return 1;
}

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

return 0;

当然，还有第三种选择，即没有一个方法应该有一个以上的返回语句，当方法的复杂性较低时，我觉得这太限制和麻烦了：

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16

int retVal;

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

return retVal;

在编写这些类型的构造时，上面列出的选项之一是否更正确？从性能上讲，我知道选择完全不相关，但从可读性的角度来看，我更喜欢避免if-else if语句。也就是说，很多同事不同意我的观点，尽管他们不能给我任何令人信服的论据。

相关讨论

这是一种偏好。
一切都是关于你喜欢什么。我认为第一个选项看起来最好，因为它以一种合乎逻辑的方式将相关的if组合在一起。如果你能跳过背带，它看起来会更整洁。
投票人想发表评论吗？
@中间人：当然。你的问题是没有好的或坏的答案。StackOverflow不适合讨论这类事情。
对于程序员来说，这可能是更好的要求。
我更喜欢选项3。由于超过10年，我们避免使用"break"(开关语句中除外)和"return"(函数结束时除外)。经验表明，该编码标准具有一定的可读性，在调试模式下，在过程结束时设置单个断点具有一定的优势。
@Alexandrevin&231；然后投票将其锁定。投反对票时，国际海事组织应该针对以下不恰当的问题："为我做作业"、"C坏了，我的代码不起作用(后面跟着1000行代码)"或"我想用C写一个3D引擎，请给我样本代码"…
@涂鸦，如果你仍在使用10年前的编码标准，你可能会想检查它们，例如如果你使用lambdas字典或模式匹配，那么if/else if和switches可以被弃用和避免；同样，使用linq在很大程度上消除了对循环的需要，因此break成为一个没有问题的问题。最后，如果将所有方法都保持在eg<25行，那么无论返回多少，可读性都很容易。
@大卫阿诺：我们仍在维护和升级超过10年的项目。在查看发布到本论坛的C代码时，您将看到代码中有大量"if"语句(以及一些"switch")。你有没有提到过禁止"if"语句的编码标准？
@涂鸦，谁说要禁止江户十一〔0〕的？那太傻了。但是，通过采用更好的编码实践(如读取干净的代码、可靠的、功能性的技术等)，您就不再依赖于循环复杂的方法，如switch和chained/nested if/else if。
@大卫·阿诺：我最初的评论建议在条件指令中使用"return"，例如"if"(嵌套与否)。
我这边投赞成票。虽然这是一种偏好，但讨论起来很有趣。

(P)有很多方法。你可以从你表演的两种方式中选一种You could do:(p)字母名称(P)Or fully use the third operator:(p)字母名称(P)5.Yet another option is to use a functional approach，such as offered by my own succint library and do：(p)字母名称(P)It's all a matter of preference and there are really isn't a right answer.(p)

(P)考虑(p)字母名称(P)总的来说，它对优先项目和周刊的依赖程度很高。(p)(P)如果所有三个案例(EDOCX1，英文本0，EDOCX1，英文本1，英文本DOCX1，英文本2)都有一些对白，它们是你的第一个选择，看的对我很好。(p)(P)如果Default Path应该返回EDOCX1的英文第3个字母，或是字母名称0，或是字母名称1，或是字母名称1的话，我会为第二个选项而去。(p)

相关讨论

(P)I think for(p)字母名称(P)This does not look performance wise correct and reason being every time it is doing condition matching.(p)(P)Rather in following case this will match till it finds correct condition and ignore rest(p)字母名称(P)I think if/else if/else is better over performanceFor readability perspective you can also prefer using案例I think performance wise if else is better choice than multiple IFS(p)

相关讨论

布尔奇1(P)你可以使用第三个运营者为简单的情况下和他们的案件，但如果你检查多个条件使用第三个运营者，这将是不允许的。(p)字母名称(P)For conditions having multiple checks use if and else-if as you have provided in your question.(p)字母名称(P)This answer has been updated after get remarks from@chris and@inbetween(p)

相关讨论