Question

我正在进行代码审查，并找到一条线，其中一位开发人员使用这样的空对象初始值设定项创建对象：

List<Floorplan> floorplans = new List<Floorplan> { };

我不知道为什么他会以这种方式实例化而不是：

List<Floorplan> floorplans = new List<Floorplan>();

使用空对象初始化器对象实例化有任何缺点吗？除了在风格上不一致之外，我想知道是否有理由避免这种方法。有些东西告诉我这种味道，但我想在我说什么之前确定。

Answer 1

编译结果相同。

以下C＃：

static void Main()
{
    var x = new List<int>();
    var y = new List<int> { };
}

编译成以下IL：

.method private hidebysig static 
    void Main () cil managed 
{
    // Method begins at RVA 0x2050
    // Code size 14 (0xe)
    .maxstack 1
    .entrypoint
    .locals init (
        [0] class [mscorlib]System.Collections.Generic.List`1<int32> x,
        [1] class [mscorlib]System.Collections.Generic.List`1<int32> y
    )

    IL_0000: nop
    IL_0001: newobj instance void class [mscorlib]System.Collections.Generic.List`1<int32>::.ctor()
    IL_0006: stloc.0
    IL_0007: newobj instance void class [mscorlib]System.Collections.Generic.List`1<int32>::.ctor()
    IL_000c: stloc.1
    IL_000d: ret
} // end of method Program::Main

如果您向集合添加值：

static void Main()
{
    var x = new List<int>();
    x.Add(1);
    var y = new List<int> { 1 };
}

这是由此产生的IL：

.method private hidebysig static 
    void Main () cil managed 
{
    // Method begins at RVA 0x2050
    // Code size 32 (0x20)
    .maxstack 2
    .entrypoint
    .locals init (
        [0] class [mscorlib]System.Collections.Generic.List`1<int32> x,
        [1] class [mscorlib]System.Collections.Generic.List`1<int32> y,
        [2] class [mscorlib]System.Collections.Generic.List`1<int32> '<>g__initLocal0'
    )

    IL_0000: nop
    IL_0001: newobj instance void class [mscorlib]System.Collections.Generic.List`1<int32>::.ctor()
    IL_0006: stloc.0
    IL_0007: ldloc.0
    IL_0008: ldc.i4.1
    IL_0009: callvirt instance void class [mscorlib]System.Collections.Generic.List`1<int32>::Add(!0)
    IL_000e: nop
    IL_000f: newobj instance void class [mscorlib]System.Collections.Generic.List`1<int32>::.ctor()
    IL_0014: stloc.2
    IL_0015: ldloc.2
    IL_0016: ldc.i4.1
    IL_0017: callvirt instance void class [mscorlib]System.Collections.Generic.List`1<int32>::Add(!0)
    IL_001c: nop
    IL_001d: ldloc.2
    IL_001e: stloc.1
    IL_001f: ret
} // end of method Program::Main

这显示了集合初始值设定项仅语法糖的方式。因为集合初始化器最初不是C＃的一部分，所以我认为人们更习惯于构造函数语法。如果我遇到一些使用空集合初始化程序的代码，我会想知道为什么，但它肯定没有任何严重的可读性问题。如果一个人足够聪明地理解代码，那么{} vs ()就不应该如此沮丧，以至于它破坏了一个人理解代码正在做什么的能力。这归结为意见问题。做你的团队同意的事情，如果只是你，那就把它用在心里。

Answer 2

在这种情况下，没有缺点，但有利于可读性。如果您在实例化时不需要额外的括号，那么请不要简单明了地使用它们。

我甚至会说赞成生产力。这里的很多答案都在试图回答这个问题。即使一种技术比另一种技术更好，我们也在谈论什么是微观优化。

Answer 3

没有理由在那里使用{ }初始化，所以我会说删除它。开发人员正在使用静态集合初始化程序。你唯一的理由是你可以用一些物品开始收藏，因为没有它真的没有意义。话虽如此，结果将是一个空集合，就像他正常调用构造函数一样。

Answer 4

它们基本相同，但使用默认构造函数更具可读性。

Answer 5

有很多评论说“默认构造函数更具可读性”。没有充分理由认为()比{}更具（或更少）可读性，因此您在这里有效地进行民意调查。

如果您担心正确性，您的同事的代码就可以了。如果您担心编码标准，则应确保更改代码以符合您的标准（无论该标准是什么）。

使用空对象初始化器有什么缺点吗？

5 个答案: