具有默认值的对象初始值设定项内的集合初始值设定项-解网

问：

我只是偶然发现了以下问题：

class Settings
{
    // Let's set some default value: { 1 }
    public ICollection<int> AllowedIds = new List<int>() { 1 };
}

static void Main(string[] args)
{
    var s = new Settings
    {
        AllowedIds = { 1, 2 }
    };

    Console.WriteLine(string.Join(", ", s.AllowedIds)); // prints 1, 1, 2
}

我理解为什么会发生这种情况：不是赋值，而是对象初始值设定项中的集合初始值设定项，即它是的隐式调用。AllowedIds = { 1, 2 }AllowedIds.Add(1); AllowedIds.Add(2)

不过，对我来说，这是一个陷阱，因为它看起来像一个作业（因为它使用）.=

作为一个 API/库开发人员（假设我是开发类的人），想要坚持最小意外原则，我能做些什么来防止我的库的使用者落入这个陷阱吗？Settings

脚注：

在这种特殊情况下，我可以使用代替（因为重复项对没有意义），这将产生的预期结果。尽管如此，初始化仍会产生违反直觉的结果。ISet/HashSet<int>ICollection/ListAllowedIds1, 2AllowedIds = { 2 }1, 2
我在 C# github 存储库上找到了一个相关的讨论，它基本上得出的结论是，是的，这个语法令人困惑，但它是一个旧功能（于 2006 年引入），如果不破坏向后兼容性，我们就无法更改它。

C# API 设计集合初始值设定项最小惊奇

class Settings
{
    // Let's set some default value: { 1 }
    public IEnumerable<int> AllowedIds { get; set; } = new List<int> { 1 };
}

static void Main(string[] args)
{
    var s = new Settings
    {
        AllowedIds = new List<int> { 1, 2 }
    };

    Console.WriteLine(string.Join(", ", s.AllowedIds)); // prints 1, 2
}

这样，您仍然允许调用方使用 setter 设置新的集合，同时防止您提到的陷阱。

此解决方案的问题在于 Settings 类不再真正很好地控制 AllowedIds 属性。例如，Settings 可能想要实现一个方法。初始值是一个列表，但在中，有人可以将其设置为 int 数组（该数组也实现了，但不允许添加。我认为 Settings 需要有一个 IEnumerable 属性 getter，然后是一些代码来允许添加 IDAddAllowedIdMainIEnumerable <int>

0赞 rexcfnghk 6/3/2020

@Flydog57是的。你所描述的是使类不可变或有一个受控突变的方法。但是，由于 OP 的代码一开始就已经公开了，即他们应该很清楚类是可变的，所以我的回答是防止调用者落入 OP 所描述的陷阱的最简单方法。SettingsAddAllowedIdAllowedIdsICollection<int>

上一个：使用用户定义的前缀和后缀递增运算符对类型进行 ++o++ 抱怨

下一个：“Least Astonishment”和可变的默认参数

具有默认值的对象初始值设定项内的集合初始值设定项

Collection initializers inside object initializers with default values

评论

评论