C# 这个初始化器真的是多余的吗？

Question

我有以下代码行：

var dmrReceived = new DownloadMessagesReport();

StyleCop 和 ReSharper 建议我删除冗余的初始值设定项。但是，如果我将其替换为

DownloadMessagesReport dmrReceived;

“肯定”，这将生成一个未设置为对象实例的对象引用？我正在使用.NET 3.5。您不再需要手动实例化对象吗？

接下来的下一行是：

dmrReceived = dc.DownloadNewMessages(param, param2, param3);

值得注意的是，dc 是从 WCF 服务生成的类。因此 DownloadNewMessages 是一个 WCF Web 服务方法。

Answer 1

如果它是一个字段，它将自动初始化为其默认值 - 对于引用类型，null。然而，鉴于 var ，我猜它不是，而且您实际上是在代码中进一步实例化它，从而丢弃您在此处实例化的值。您不需要在声明变量的地方对其进行初始化。如果您想使用 var ，您可以这样做，但我建议您在实际第一次使用它的地方声明它。

Answer 2

所以您的代码是

var dmrReceived = new DownloadMessagesReport();
dmrReceived = dc.DownloadNewMessages(param, param2, param3);

第二行不会填充您在第一行中创建的对象，但它完全替换该对象。因此不需要第一个赋值（因为第一个对象从未被使用过），这正是 R# 所警告的。

Answer 3

如果dmrReceived在分配之前被访问，那么只会生成对象引用错误。很多时候，resharper 说初始化器是多余的原因是变量总是会在每个可能的执行路径中被分配另一个值。

即

DownloadMessagesReport dmrReceived;

...

if(condition) {
    dmrReceived = new DownloadMessagesReport();
} else {
    throw new Exception("oh no");
}

return dmrReceived.SomeProperty;

访问 SomeProperty 是代码中 dmrReceived 实际上需要有值的第一个位置。如下代码的其余部分所示，如果不为其分配值，则无法访问该行代码，因此，可能已分配的初始值不会在任何执行路径中使用，因此是多余的。

Answer 4

“您是否不再需要手动
实例化对象？”

当然，你需要“手动”实例化对象，否则编译器如何知道何时何地实例化它？

一个简单的场景是这样的：

MyType x;

if ( EverythingWorkedOut )
    x = new MyType(params);
else
    x = null;

如果编译器第一次实例化它，它将是多余的，并且开销更大在所有代码中，

不要相信 ReSharper 或任何其他计算机智能的东西而不是你自己的本能！你知道它们并不总是正确的

，你真的不需要这样做 。 ; 因为它应该是非实例化对象的默认值。

Answer 5

假设这是您的代码：

var dmrReceived = new DownloadMessagesReport();
dmrReceived = dc.DownloadNewMessages(param, param2, param3);

您正在第一行中创建 DownloadMessagesReport 的实例。然后，您可以通过为 dmrReceived 变量分配另一个从 DownloadNewMessages 方法返回的值来丢弃该对象。第一个 new DownloadMessagesReport() 对象是多余的。您实际上创建了垃圾收集器必须在某个时候清理的垃圾。

这就是 ReSharper 和 StyleCop 向您显示警告的原因。

如果您可以在声明变量的同一行中用实际值初始化变量，那么就这样做。

Answer 6

当然这已经足够了吗？

DownloadMessagesReport dmrReceived = dc.DownloadNewMessages(param, param2, param3);