C＃NullReference Exception和ReSharper建议

Question

这就是我写的：

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

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

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

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

Answer 1

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

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

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

Resharper不应该抱怨这个，如果PropertyIdentifier为null或者不是字符串，你也不会得到NullReferenceException。

Answer 2

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

当它们失败时，结果会略有不同：第二个示例稍早失败（在转换时），并且具有更具体的异常（ 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)