我不喜欢这个……这是在欺骗语言吗？

Question

我已经见过几次类似下面的东西了……我讨厌它。 这基本上是在“欺骗”语言吗？ 或者..您是否认为这是“可以”，因为 IsNullOrEmpty 始终首先被评估？

（我们可能会争论一个字符串从函数中出来时是否应该为 NULL，但这并不是真正的问题。）

string someString;
someString = MagicFunction();

if (!string.IsNullOrEmpty(someString) && someString.Length > 3)
{
    // normal string, do whatever
}
else
{
   // On a NULL string, it drops to here, because first evaluation of IsNullOrEmpty fails
   // However, the Length function, if used by itself, would throw an exception.
}

编辑： 再次感谢大家提醒我这门语言基础知识。 虽然我知道它“为什么”有效，但我不敢相信我不知道/记得这个概念的名称。

（如果有人想要任何背景..我在对代码的不同位置由 NULL 字符串和 .Length > x 异常生成的异常进行故障排除时遇到了这一点。所以当我看到上面的代码时，除了其他所有内容之外，我的挫败感从那里开始。）

Answer 1

您正在利用称为短路的语言功能。 这并不是欺骗语言，而是实际上完全按照其设计用途使用了某个功能。

Answer 2

如果您问是否可以依赖“短路”关系运算符 && 和 ||，那么是的，完全没问题。

Answer 3

这没有什么问题，因为您只是想确保不会出现空指针异常。

我认为这样做是合理的。

通过扩展，您可以使其更清晰，但基本概念仍然有效。

Answer 4

这段代码完全有效，但我喜欢使用 Null Coalesce Operator 来避免 null 类型检查。

string someString = MagicFunction() ?? string.Empty;
if (someString.Length > 3)
{
    // normal string, do whatever
}
else
{
   // NULL strings will be converted to Length = 0 and will end up here.
}

Answer 5

这没有什么问题。

if(条件是从左到右评估的，所以像这样堆叠它们是完全可以的。

Answer 6

在我看来，这是有效的代码（尽管声明一个变量并在下一行分配它非常烦人），但您可能应该意识到，您也可以在字符串长度为 < 的情况下输入 else 块; 3.

Answer 7

在我看来，这就像是对逻辑短路的完全合理的使用——如果有的话，那就是对语言的欺骗。 我最近才使用 VB6，它从来没有短路过，这确实让我很恼火。

需要注意的一个问题是，您可能需要在 else 子句中再次测试 Null，因为正如所写的那样，您最终会得到 Null 字符串和长度小于 3 的字符串。

Answer 8

这是完全有效的，并且以这种方式使用它没有任何问题。 如果您遵循该语言记录的行为，那么一切都很好。 在 C# 中，您使用的语法是条件逻辑运算符，可以在 MSDN

对我来说，这与在同一语句中进行乘法和加法时不使用括号相同，因为语言记录了将首先执行乘法运算。

Answer 9

在大多数情况下，依靠短路是“正确的做法”。 它可以生成简洁的代码，并减少移动部件。 这通常意味着更容易维护。 在 C 和 C++ 中尤其如此。

我会认真重新考虑雇用不熟悉（并且不知道如何使用）短路操作的人。

Answer 10

我觉得没问题:) 你只是确保不访问 NULL 变量。
实际上，我总是在对变量进行任何操作之前进行此类检查（此外，在索引集合等时） - 它更安全，是最佳实践，仅此而已。

Answer 11

这是有道理的，因为 C# 默认情况下会短路条件，所以我认为利用它来发挥你的优势是很好的。 在 VB 中，如果开发人员使用 AND 而不是 ANDALSO，可能会出现一些问题。

Answer 12

我认为这与这样的事情没有什么不同：

INT* pNumber = GetAddressOfNumber();

if ((pNUmber != NULL) && (*pNumber > 0))
{
  // valid number, do whatever
}
else
{
  // On a null pointer, it drops to here, because (pNumber != NULL) fails
  // However, (*pNumber > 0), if used by itself, would throw and exception when dereferencing NULL
}

它只是利用了语言中的一个功能。 我认为，自从 C 开始以这种方式执行布尔表达式（或任何首先执行此操作的语言）以来，这种习惯用法已经很常用了。）

Answer 13

如果您将 c 代码编译成汇编语言，那么短路不仅是正确的行为，而且更快。 在机器语言中，if 语句的各部分被依次求值。 不短路速度较慢。

Answer 14

编写代码对公司来说要花费很多美元。 但维护成本更高！

所以，我同意你的观点：这行代码很可能不会被那些必须在两年内阅读并纠正它的人立即理解。

当然，他会被要求纠正一个关键的生产错误。 他会到处寻找，但可能没有注意到这一点。

我们应该总是为下一个人编码，而他可能不如我们聪明。 对我来说，这是唯一值得记住的事情。

这意味着我们使用明显的语言特征并避免其他特征。

祝一切顺利，西尔万。

Answer 15

有点偏离主题，但如果你像这样在 vb.net 中运行相同的示例，

dim someString as string
someString = MagicFunction()
if not string.IsNullOrEmpty(someString) and someString.Length > 3 then
    ' normal string, do whatever
else
    ' do someting else
end if

这会导致空（无）字符串，但在 VB.Net 中，你可以按如下方式对其进行编码，请在 C# 中执行相同的操作，

dim someString as string
someString = MagicFunction()
if not string.IsNullOrEmpty(someString) andalso someString.Length > 3 then
    ' normal string, do whatever
else
    ' do someting else
end if

添加并使其表现良好同样的方式，读起来也更好。 作为同时进行 vb 和 c' 开发的人，第二个 vb 显示登录略有不同，因此更容易向某人解释存在差异等。

Drux