c# – 检查OpenFileDialog.OpenFile是否为null

2019年8月5日 190次阅读

我在
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示例有时写得不好,或者更糟糕,所以不要盲目地复制/粘贴他们所有的例子:)

点赞