.NET 通用列表问题，按设计工作，需要解决。 增加价值而不是参考

Question

我有一个 List 并且我需要避免我即将概述的行为：

    // assume cls and numberToAdd are parameters passed in.
    int pos = numberToAdd;
    List<MyClass> objs = new List<MyClass>(numberToAdd);

    for(int i = 0; i < numberToAdd; i++)
    {
        objs.Add(cls);
        objs[i].X = -((pos * cls.Width) + cls.Width / 2);
        pos--;
    }

    Console.WriteLine(objs[0].X + "\r\n" + objs[1].X);

这会导致此写入行打印相同的值。

基本上我需要的是改变“Add”方法的行为。 我想添加具有相同值的对象的新实例，而不仅仅是对同一对象的引用。 我知道这会使用更多内存。

Answer 1

看起来像同一个对象，cls 正在向列表中添加 numberToAdd 次。

您需要在循环内创建对象的新实例。

for(int i = 0; i < numberToAdd; i++)
{
    MyClass c=new MyClass(...);
    c.X = -((pos * c.Width) + c.Width / 2);
    objs.Add(c);
    pos--;
}

Answer 2

让 MyClass 实现 ICloneable 然后简单地使用

objs.Add((MyClass)cls.Clone());

Answer 3

可能想要扩展 cls 以包含克隆方法。 如果您无法修改 cls 类，则扩展方法将起作用。

然后改变
objs.Add(cls);
到
objs.Add(cls.Clone());

Answer 4

啊。 您需要将 cls 实现为结构体，以便它展现按值传递语义。

或者您需要在 cls 对象上实现一个克隆方法，以便您可以创建一个方法来克隆该对象，从而创建对其的新引用，以便您可以执行此操作。

即使您无权访问对象本身，您也可以创建一个扩展方法来执行此操作。

public cls Clone(this cls)
{
    //initialise the object to clone it.
    return new cls(this.param1, this.param2,...) 
}

Answer 5

“cls”变量是什么？ 只需在每个循环内创建一个新循环即可。 你想要做的就是克隆它； 但老实说这会令人困惑。 我建议每个循环创建一个新的。

-- 编辑

注意到您会评论“cls”。

我建议您然后克隆它（假设您无法更改呼叫并执行我的上述建议）。 只需创建一个新属性并复制所有属性即可完成此操作。

通常，如果您可以控制“cls”的类型，则只需创建一个“复制”构造函数即可。 就像这样：

class Foo {
    private int x;

    public Foo (Foo toCopy) {
       this.x = toCopy.x;
    }

    ...
}