My foreach не работает

Question

В этом коде с использованием foreach, даже когда я печатаю идентификатор, которого нет в списке, он все еще продолжается. Почему это происходит, и как я могу это исправить?

foreach (Product item in listOfProducts) 
{ 
    if (item.Id == id) 
    { 
     item.UnitInStocks = unitsInStocks; 
     item.Price = price; 
     Console.WriteLine("the product details has been changed"); 
     Console.ReadLine(); 
    } 
    else 
     Console.WriteLine("this product id does not exist"); 
    } 
    break; 
} 

Answer 1

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

Если вы положили отсутствующую фигурную скобку, то вы набираете из цикла foreach на первой итерации, имеет ли первый продукт соответствующий идентификатор или нет.

Если вы настроены на использование foreach, то вы, вероятно, хотите что-то вроде:

var productFound = false; 
foreach (Product item in listOfProducts) 
{ 
    if (item.Id == id) 
    { 
     productFound = true; 
     item.UnitInStocks = unitsInStocks; 
     item.Price = price; 
     Console.WriteLine("the product details has been changed"); 
     break; 
    }  
} 

if (!productFound) 
{ 
    Console.WriteLine("this product id does not exist"); 
} 

Я предпочел бы избежать foreach все вместе:

var product = listOfProducts.FirstOrDefault(prod => prod.Id == id); 
if (product != null) 
{ 
    product.UnitInStocks = unitsInStocks; 
    product.Price = price; 
    Console.WriteLine("the product details has been changed"); 
} 
else 
{ 
    Console.WriteLine("this product id does not exist"); 
} 

Вам нужно using System.Linq; на В верхней части файла используется FirstOrDefault.