是否正在使用短路评估来避免指数超出范围异常的坏习惯？

Question

这个问题适用于每个具有短路AND运算符的编程语言,而不仅仅是C#.

问题很简单 - 使用短路评估来避免超出范围的索引异常,例如:

if ((x > 0) && (bar[x] == foo))

要么

if (((x > 0) && (x < bar.Length)) && (bar[x] == foo))

糟糕的编码风格？我知道我可以像这样嵌套循环:

if (x > 0)
{
    if (bar[x] == foo)
    {
    }
}

但我发现这是非常难以理解的.

Answer 1

我会这样说

if ((x > 0) && (bar[x] == foo))

编码风格也不错.它甚至可能都很好.我肯定更喜欢它if像你描述的嵌套结构.

顺便说一句,我会减少你使用的括号数量.这些都是同样正确的,至少在C#和大多数其他C语言中是这样的:

if (x > 0 && bar[x] == foo)
if (x > 0 && x < bar.Length && bar[x] == foo)

知道该语言的读者(你必须在某种程度上假设这一点)将能够很容易地理解上述短路表达式.那些读者可能会反对嵌套if样式,因为它占用了比获得正确行为所需的更多空间.