如何从 Foreach 循环中的列表中动态删除项目 [重复]

Question

如我们所见，代码抛出错误是有原因的。

Error:{"Collection was modified; enumeration operation may not execute."}

但是必须有一种方法可以做到这一点，使用 foreach（而不是 for），使用 List。请不要告诉我Deep Copy之类的制作方法

var cloneList = NumberList.ToList()

并从中删除项目。因为如果我们有 100 万个元素，我们将使其成为 200 万个用于删除等。我不需要这个。

代码sn-p在这里；

public class HomeController : Controller
{
    // GET: Home
    public ActionResult Index()
    {
        var numberList = new List<int>();
        numberList.AddRange(new List<int>() { 1, 2, 3, 4, 5, 6, 7, 8, 9 });
        var totalNumber = 0;
        foreach (var item in numberList)
        {
            totalNumber += item;
            if (item == 5)
            {
                numberList.Remove(item);
            }
        }
        return View(totalNumber);
    }
}

请帮我了解一下这个烧脑代码 sn-p..

谢谢。

更新：问题不在于在最后一点获得列表元素的Sum()。问题的核心是，如何在 foreach 循环时从列表中删除项目。

Answer 1

您正在 foreach 循环中修改您的集合，因此在下一次迭代中集合元素编号发生变化并且 foreach 无法迭代，试试这个

public ActionResult Index()
{
    var numberList = new List<int>();
    numberList.AddRange(new List<int>() { 1, 2, 3, 4, 5, 6, 7, 8, 9 });
    var totalNumber = 0;
    var numberListTemp = numberList;
    foreach (var item in umberListTemp)
    {
        totalNumber += item;
        if (item == 5)
        {
            numberList.Remove(item);
        }
    }
    return View(totalNumber);
}

Answer 2

先计算总数，然后删除项目：

var totals = numberList.Sum();  
numberList.RemoveAll(i=>i==5);

Answer 3

如果双重迭代和深拷贝是我的方法：

        ...
        var numberList = new List<int>();
        numberList.AddRange(new List<int>() { 1, 2, 3, 4, 5, 6, 7, 8, 9 });
        var totalNumber = 0;
        foreach (var item in YieldingNotFive(numberList))
        {
            totalNumber += item;
        }
        ...

    private IEnumerable<int> YieldingNotFive(List<int> numberList)
    {
        foreach (int item in numberList)
        {
            if (item != 5)
            {
                yield return item;
            }
        }
        yield break;
    }

或

    var numberList = new List<int>();
    numberList.AddRange(new List<int>() { 1, 2, 3, 4, 5, 6, 7, 8, 9 });
    var totalNumber = 0;
    foreach (var item in numberList.Where(x => x != 5))
    {
        totalNumber += item;
    }

Answer 4

尝试为循环添加一个计数器，并在达到您的条件时删除该元素。

public class HomeController : Controller
{
    // GET: Home
    public ActionResult Index()
    {
        var numberList = new List<int>();
        numberList.AddRange(new List<int>() { 1, 2, 3, 4, 5, 6, 7, 8, 9 });
        var totalNumber = 0;
        var count = 0;
        foreach (var item in numberList)
        {
            totalNumber += item;
            if (item == 5)
            {
                numberList.Remove(count);
            }
            count = count + 1;
        }
        return View(totalNumber);
    }
}

但我不知道从索引中删除的函数的名称。

Answer 5

我找到了答案。从下面您可以看到传统 foreach 和 Parallel.ForEach 之间的性能细节。

我希望这会有用。

class Program
{
    static void Main(string[] args)
    {           
        List<string> colorList = new List<string>();
        colorList.AddRange(new List<string>() {
                                  "1. Red",
                                  "2. Green",
                                  "3. Blue",
                                  "4. Yellow",
                                  "5. White",
                                  "6. Black",
                                  "7. Violet",
                                  "8. Brown",
                                  "9. Orange",
                                  "10. Pink"
        });
        Console.WriteLine("Traditional foreach loop\n");
        //start the stopwatch for "for" loop
        var sw = Stopwatch.StartNew();
        foreach (string color in colorList)
        {
            Console.WriteLine("{0}, Thread Id= {1}", color, Thread.CurrentThread.ManagedThreadId);
            Thread.Sleep(10);
        }
        Console.WriteLine("Foreach loop execution time = {0} seconds\n", sw.Elapsed.TotalSeconds);

        //-------------------------------------------------------------------------------------------------------------------------------------------------------

        Console.WriteLine("Using Parallel.ForEach");
        //start the stopwatch for "Parallel.ForEach"
        sw = Stopwatch.StartNew();
        int index = 0;
        Parallel.ForEach(colorList, (color, state) =>
        {

            ++index;
            if (index > colorList.Count - 1)
            {
                state.Stop();
            }

            Console.WriteLine("{0}, Thread Id= {1}", color, Thread.CurrentThread.ManagedThreadId);
            if (color == "2. Green")
            {
                colorList.RemoveRange(3, 2);
            }
            Thread.Sleep(10);
        }
        );
        Console.WriteLine("Parallel.ForEach() execution time = {0} seconds", sw.Elapsed.TotalSeconds);
        Console.Read();
    }
}