Resharper：可能对标有 notnull 属性的实体进行 null 赋值

Question

我在 response.GetResponseStream() 上收到此警告 我应该如何处理这个问题？

// Get response  
using (var response = request.GetResponse() as HttpWebResponse)
{
    // Get the response stream  
    if (response != null)
    {
        var reader = new StreamReader(response.GetResponseStream());
        var responseString = reader.ReadToEnd();
        return responseString;
    }
}

为了清楚起见，基于一些误解的答案：

此行不是发生警告的位置：

using (var response = request.GetResponse() as HttpWebResponse)

此行是发生警告的位置：

var reader = new StreamReader(response.GetResponseStream());

Answer 1

var reader = new StreamReader(response.GetResponseStream());

我怀疑 StreamReader 构造函数的参数具有 notnull 属性。请尝试以下操作：

var stream = response.GetResponseStream();
if (stream == null)
  // throw an exception
var reader = new StreamReader(stream);

Answer 2

尝试缩短代码并将一次性资源包装在 using 语句中：

using (var response = request.GetResponse())
using (var reader = new StreamReader(response.GetResponseStream()))
{
    return reader.ReadToEnd();
}

甚至更进一步：

using (var client = new WebClient())
{
    return client.DownloadString("http://foo.bar.com/")
}

Answer 3

如果 response 对象的类型为 HttpWebRequest，则响应将始终为 HttpWebResponse 类型。如果不是，那就永远不会。

要么你在错误的地方测试它（如果你可以传递由 WebRequest 派生的另一个类只是为了丢弃结果，为什么要调用 .GetResponse() ）或测试不必要且没有效果。

我猜 resharper 正在担心这一点，尽管在它下面进行了 null 测试。我会选择直接强制转换：

using (var response = (HttpWebResponse)request.GetResponse())
using(var reader = new StreamReader(response.GetResponseStream()))
  return reader.ReadToEnd();

或者，考虑到您没有使用任何不是从 WebResponse 派生的 HttpWebResponse 成员，根本不进行强制转换：

using (var response = (HttpWebResponse)request.GetResponse())
using(var reader = new StreamReader(response.GetResponseStream()))
  return reader.ReadToEnd();

Answer 4

您可以使用 C# 的 空条件运算符来消除警告：

var reader = new StreamReader(response?.GetResponseStream());

它是一种语法糖，可以避免编写在运行时检查引用的空值的代码。在内部，这段代码相当于：

var reader = new StreamReader(response != null ? response.GetResponseStream() : null);