String.IsNullOrEmpty(myString) 与 myString != null

Question

这三者哪一个更好呢？

string myString = ""; 
String.IsNullOrEmpty(myString);

vs

string myString = "";
if(myString.Length > 0 || myString != null)

vs 

string myString = "";
if (m.Length > 0 | m != null)

前者更清楚，但它们之间有性能差异吗？如果字符串永远不为空，就像从文本框中取出的那样，它可能为空但不能为 null ，该怎么办？

Answer 1

好吧，问题中的版本：

if(myString.Length > 0 || myString != null)

肯定会更糟，因为您应该测试 null first （不是第二) - 理想情况下在 null 上短路，这样您就不会尝试调用 .Length。但通常我只会使用 string.IsNullOrEmpty 。如果需要，您始终可以编写一个扩展方法以使其不那么冗长（您可以对 null 值调用扩展方法）。

static bool HasValue(this string s) {
    return !string.IsNullOrEmpty(s);
}

Answer 2

使用string.IsNullOrEmpty(str)。它更清晰、更简洁。它不会成为您应用程序的瓶颈。

如果您只需要检查字符串“空”，那么我会检查 string.Empty 因为它更好地表达了您的意图。

Answer 3

我会使用 IsNullOrEmpty。

以后再看代码的时候会更容易解析。

这是另一个有点奇怪的原因。后来的一些程序员肯定会出现，抓着胡子说“我认为 myString.trim().Length != 0 更好”并更改它。

正如其他人指出的那样：检查空秒是一个潜在的空访问错误正在等待发生 - 库例程保证没问题。

Answer 4

正如其他人所说，出于可维护性的目的，IsNullOrEmpty() 优于手动检查，并且由于 JIT 编译器关于内联的运行时决策，性能不太可能受到影响（请参阅 Eric Gunnerson 的评论）。

如果其他人想知道实际的 .NET 实现是什么样子，请参阅以下 .NET 4 代码：

[TargetedPatchingOptOut("Performance critical to inline across NGen image boundaries")]
public static bool IsNullOrEmpty(string value)
{
    if (value != null)
    {
        return (value.Length == 0);
    }
    return true;
}

该属性指示该方法也将内联到 NGen（即本机）图像中。

Answer 5

如果您对如何测试字符串引用的不同状态不放心（显然您就是这样，因为您弄错了……；），那么 String.IsNullOrEmpty 是更好的选择。

使用 IsNullOrEmpty 方法：

if (String.IsNullOrEmpty(s)) ...

相当于对 null 和零长度使用短路测试：

if (s == null || s.Length == 0) ...

如果您知道引用不能为 null，则可以跳过该检查并仅检查长度

if (s.Length == 0) ...

： IsNullOrEmpty 方法也适用于正常情况，但在出现问题且引用实际上为 null 的情况下，IsNullOrEmpty 方法会默默地接受它，而您通常会这样做希望被告知该错误。

Answer 6

实现为：

if (s == null || s.Length == 0) ... 。

我相信 String.IsNullOrEmpty(String s)在 API 中

Answer 7

我相信 String.IsNullOrEmpty(String s) 的实现如下：
if (s == null || s.Length == 0) ...
在 API 中。


这是错误的。尝试一下，您将得到一个异常，因为将尝试这两个语句。如果 s 为 null，则 s.Length 将抛出异常。