Изменение значения одного элемента в коллекции влияет на все повторяющиеся элементы.

Вам нужно что-то вроде метода клонирования или конструктора копирования.

public class Product
{
  public int Id { get; set; }
  public string Name { get; set; }
  public int Quantity { get; set; }

  public Product()
  {
      this.Id = 0;
      this.Name = null;
      this.Quantity = 0;
  }

  public Product(Product product)
  {
      this.Id = product.id;
      this.Name = product.Name;
      this.Quantity = product.Quantity;
  }
}

IList<Product> myProducts = new List<Product>();

Product product1 = new Product() { Id = 0, Name = "Buzz Cola" };
Product product2 = new Product() { Id = 1, Name = "Choco Bites" };
Product product3 = new Product(product1); // Use copy-constructor.

myProducts.Add(product1);
myProducts.Add(product2);
myProducts.Add(product3);

myProducts[0].Quantity = 1;

И теперь все должно быть хорошо. Вы можете использовать это вместе с вашим методом клонирования для одновременного создания большого количества клонов.

Просто отметим, что этот код по-прежнему имеет очень неприятный вкус — вы создаете разные экземпляры продукта с одинаковыми идентификаторами. Я могу только догадываться, но хотите ли вы создать что-то вроде корзины для покупок, в которой элементы корзины имеют количество и продукт? Если да, вам действительно следует подумать о разделении класса продуктов на два класса. И снова подумайте о доступности вашей недвижимости.

public class Product
{
  public Int32 Id { get; private set; }
  public String Name { get; private set; }
}

public class ShoppingCartItem
{
  public Product Product { get; private set; }
  public Int32 Quantity { get; set; }
}

public class ShoppingCart
{
  public IList<ShoppingCartItem> Items { get; private set; }
}

Это решает ваши текущие проблемы, потому что больше нет необходимости в клонировании продуктов.

Product — это ссылочный тип, и ваш метод GetProducts просто дает несколько ссылок на один и тот же объект Product.

Вот почему обновление одного экземпляра приводит к обновлению всех остальных — все они являются ссылками на один и тот же объект.

Я предполагаю, что он дважды ссылается на ваш первый экземпляр с ID = 0 вместо двух отдельных экземпляров, как вы ожидаете.

Попробуйте изменить идентификатор третьего экземпляра с 0 -> 2 и посмотрите, «исправляет» ли это его.

и myProducts.ElementAt(0), и myProducts.ElementAt(1) будут ссылаться на один и тот же объект.

Не уверен, что это лучший способ исправить это: может быть, прежде чем добавлять в список, проверьте, есть ли у вас уже ссылка на объект? если вы это сделаете, глубоко клонируйте объект и вставьте его...

После исправления нескольких опечаток в цикле WriteLine (скопируйте/вставьте рабочий код) ошибка не воспроизводится, мой вывод:

Id: 0  Quantity: 1
Id: 1  Quantity: 0
Id: 0  Quantity: 0

После изменений:

Вы создаете только 2 экземпляра, и поэтому вы страдаете от очень простого факта, что у вас есть 3 ссылки, но только 2 экземпляра. И вывод такой, какой должен быть. Будет немного понятнее, если вы также напечатаете свойство Name.

Но то, что вы, по-видимому, хотите, где-то в этой очень сложной истории Enumerator/Concat, — это клонировать ваши продукты.

Чарли Солтс может восстановить свой ответ, он был прав.

Вместо этого используйте индексатор.

myProducts[0].Quantity = 1