在C＃中的列表内修改列表

Question

我试图通过迭代列表来修改内部列表中的值。 但是，无论外部列表如何，我对于内部列表都会得到相同的结果。 我预计maxSpeed的结果对于不同的车辆会有所不同。

希望有人可以对此提供帮助。

请注意，这不是随机数生成器问题。 这只是我产生的示例代码，而在不使用随机数生成器的情况下，此问题确实存在于我的项目代码中。

在这里运行C＃代码

List<Vehicle> mainList = new List<Vehicle>();
List<Properties> defaultPropertyList = new List<Properties>{
    new Properties() { maxSpeed = 0, isTwoDoor = true },
    new Properties() { maxSpeed = 0, isTwoDoor = true },
};


mainList.Add(
    new Vehicle() {
        number = 1,
        property = new List<Properties>(defaultPropertyList)
    }
);
mainList.Add(
    new Vehicle() {
        number = 2,
        property = new List<Properties>(defaultPropertyList)
    }
);           

foreach(Vehicle vehicle in mainList) {
    Random rnd = new Random();
    vehicle.property.ForEach(x => x.maxSpeed =  rnd.Next(1, 100));
}

Answer 1

问题在于，当您为每辆车初始化property字段时，您将添加对相同的两个Properties对象（在顶部定义）的引用。 因此它们存在于所有车辆中，并且当您修改一个时，在两个地方都对其进行了修改（因为它是相同的参考）。

初始化新载具时，您将需要在defaultPropertyList对象的副本。 然后它们将独立存在。

一种方法：为Vehicle构造一个新的构造函数，该构造函数采用默认属性，然后将其复制到那里。

public Vehicle(List<Properties> defaultProps) 
{
  property = new List<Properties>();
  foreach (var p in defaultProps)
  {
    var newProp = new Properties {
                                    maxSpeed = p.maxSpeed,
                                    isTwoDoor = p.isTwoDoor
                                    // add any other properties here
                                 };
    property.Add(newProp);
  }
}

Answer 2

defaultPropertyList只是一个包含一组项目的列表。 每个Vehicle具有相同的List<Properties>实例。 有没有每Vehicle 。 只有一个，他们都在共享它。 这就是为什么无论您如何更改属性，它们都具有相同的属性。

要解决此问题，请不要创建一个共享。 只需创建所需的数量即可。 也许不是你，但是当我刚开始的时候，我害怕创建很多对象，并认为我可以通过创建尽可能少的对象来进行优化。

的确，我们不想不必要地创建许多昂贵的对象，但不必为此小气。 根据需要创建多个。 例如，在Web应用程序中，仅跟踪单个请求就不可能跟踪创建了多少个对象。

您可以这样做：

Random rnd = new Random(); // Despite what I just said,
                           // you only need one of these this time.
foreach(Vehicle vehicle in mainList) {        
    vehicle.property = new List<Properties>{
        new Properties() { maxSpeed = rnd.Next(1, 100), isTwoDoor = true },
        new Properties() { maxSpeed = rnd.Next(1, 100), isTwoDoor = true },
}