C# NullReference 异常和 ReSharper 建议

Question

这就是我所写的：

if ((lstProperty[i].PropertyIdentifier as string).CompareTo("Name") == 0)

Resharper 给我一个错误（我是 ReSharper 的新手...我正在尝试），它建议我：

  if (((string) lstProperty[i].PropertyIdentifier).CompareTo("Name") == 0)

为什么第二个是 NullException 安全的？ 对我来说，如果出现空值，两者都会崩溃？

Answer 1

两个示例在相同的情况下都会成功或失败，并且当它们成功时，行为将是相同的。

当它们失败时，结果会略有不同：第二个示例稍早失败（在强制转换时），并出现更具体的异常（InvalidCastException 与 NullReferenceException）。

主要好处是调试：当它们失败时，您可以比第一个示例获得更多有关第二个示例失败原因的信息。 具体来说，如果 PropertyIdentifier 为 null 与非 string，您可以区分第二种情况，但不能区分第一种情况。

此外，如果您处于 try/catch 中，则可以在单独的代码路径中处理非 string 情况，而不是 null 情况。 但是，您可能不应该这样编码：如果您这样做，那么您就做错了其他事情。

如果您在各种情况下逐步执行以下代码，可能有助于阐明情况：

var propertyI = lstProperty[i];
var propertyIdentifier = propertyI.PropertyIdentifier;

// pick one of these:
var propertyIdentifierAsString = propertyIdentifier as string;
var propertyIdentifierAsString = (string)propertyIdentifier;

if (propertyIdentifierAsString.CompareTo("Name") == 0)

Answer 2

如果无法执行强制转换，“as”运算符将返回 null，而 C 风格强制转换如果无法执行，将抛出异常。

我建议将其分解为多个语句：

string propertyIdentifier = lstProperty[u].PropertyIdentifier as string;
if(propertyIdentifier != null && propertyIdentifier.CompareTo("Name") == 0)
{
    ... your if statement ...
}

Resharper 不应抱怨这一点，并且如果 PropertyIdentifier 为 null 或不是字符串，您也不会得到 NullReferenceException。