我在
the msdn doc中看过这段代码:
Stream myStream = null;
OpenFileDialog openFileDialog1 = new OpenFileDialog();
[...] // Some init
try
{
if ((myStream = openFileDialog1.OpenFile()) != null)
{
using (myStream)
{
// Insert code to read the stream here.
}
}
}
但是Resharper轻轻地告诉我,检查null是没用的:
我应该相信Resharper还是微软?
最佳答案 R#是正确的,如果你反编译该类(参见下面的代码),它的实现方式永远不会返回null(它总是返回一个流或抛出异常):
public Stream OpenFile()
{
IntSecurity.FileDialogOpenFile.Demand();
string str = this.FileNamesInternal[0];
if (str == null || str.Length == 0)
throw new ArgumentNullException("FileName");
new FileIOPermission(FileIOPermissionAccess.Read, IntSecurity.UnsafeGetFullPath(str)).Assert();
try
{
return (Stream) new FileStream(str, FileMode.Open, FileAccess.Read, FileShare.Read);
}
finally
{
CodeAccessPermission.RevertAssert();
}
}
那说:
>添加空检查不会改变该特定用例的性能,因此您可以保留它.作为一般规则,在使用之前检查不属于您的函数的结果为null并不是一种不好的做法
>就像每个软件一样,R#有时会出现错误(例如:参见:Why does ReSharper think that “thread.Name == null” is always false?),所以不要盲目遵循它的所有建议:)
> MSDN示例有时写得不好,或者更糟糕,所以不要盲目地复制/粘贴他们所有的例子:)