在foreach循环中重新分配object c#

Question

Not sure I understand why I can do this with a for loop and not a foreach loop?

This is the code that works. Looping through a BindingList Products, finding a match and then assigning that product at index i to the new product that's passed in.

 public static void UpdateProduct(int productToUpdateID, Product productToUpdate)
        {
            

            for (int i = 0; i < Products.Count; i++)
            {
                if (Products[i].ProductID == productToUpdateID)
                {
                    Products[i] = productToUpdate;
                }
            }
        }
            

If I try to do this with a foreach loop I get an error that I cannot assign to the iterator variable. What is the reasoning for this and is there a way to get around it or is using a for loop for this kind of problem the best solution?

This is essentially what I'm trying to do.

          public static void UpdateProduct(int productToUpdateID, Product productToUpdate)
        {

            foreach(Product product in Products)
            {
                if (product.ProductID == productToUpdateID)
                {
                    product = productToUpdate;
                }
            }
        }
            

I can do something like this and reassign all the properties explicitly but want to see if there is another way to do it.

            foreach(Product product in Products)
            {
                if (product.ProductID == productToUpdateID)
                {
                    product.Name = productToUpdate.Name;
                    
                }
            }

Thanks!

Answer 1

The foreach construct is for when you want to do something with each item in the list. That does not seem to be what you are doing. You are modifying the list itself, by removing an item and replacing it.

Personally I would not use a loop at all, I'd just remove the old item and add the new one.

public static void UpdateProduct(int productToUpdateID, Product productToUpdate)
{
    Products.RemoveAll( x => x.ProductID == productToUpdateID );
    Products.Add( productToUpdate );
}

Or if you wish to preserve order:

public static void UpdateProduct(int productToUpdateID, Product productToUpdate)
{
    var index = Products.FindIndex( x => x.ProductID == productToUpdateID );
    Products[index] = productToUpdate;
}

Answer 2

The foreach loop iterates over the elements of a collection, and the iteration variable is simply a reference to the current element in the collection.

The reason you cannot modify the iteration variable itself is that it is a read-only reference to the element in the collection. Modifying the iteration variable would not change the element in the collection; it would only change the reference.

Alternative ways are already mentioned in the above answers.

Answer 3

Just for the record. The best way is to use for loop (not linq, and not foreach). But if you want to use linq you can do it in one line)

products = products.Select(x => x = x.ProductID == productToUpdate.ProductID?productToUpdate:x).ToList();

Answer 4

The reasons have already been given, but as a minor detail: this is sometimes possible; there is an alternative syntax in recent C# that uses a ref -local for the iterator value:

foreach (ref [readonly] SomeType value in source)

which is only available for some scenarios - naked arrays, spans, or custom iterator types with a ref -return Current - and as long as the optional readonly modifier is not used, you can assign directly via the value variable, since this is a direct reference to the underlying source. The uses for this are rare and niche. If Products is a List<T> , you could combine this with CollectionMarshal.AsSpan(...) to achieve what you want, but frankly I'd consider that hacky (apart from other things, it would bypass the list's internal change protections). Basically: don't do this, but: it isn't entirely impossible.