c#中的java等效Iterator.remove

Question

代码已从 Java 迁移到 C#。

Java代码：

for (List<ItemNY> level : highU.getLevels())
{
    Iterator<ItemNY> iterItemset = level.iterator();
    while(iterItemset.hasNext())
    {
        ItemNY c = iterItemset.next();

        // Code... 

        if(c.getU() < min)
            {
            iterItemset.remove();
            highU.decreaseCount();
            }
     }
}

C# 代码：（转换后编写的代码）

foreach (List<ItemNY> level in highU.getLevels())
{
    ItemNY c;
    using (IEnumerator<ItemNY> iterItemset = level.GetEnumerator())
    {
        while (iterItemset.MoveNext())
        {
            c= iterItemset.Current;
            //CODE

            foreach (TransactionTP transaction in database.getTransactions())
            {
                int transactionUtility = 0;
                int matchesCount = 0;
                for (int i = 0; i < transaction.size(); i++)
                {
                    if (c.getItems().Contains(transaction.get(i).item))
                    {
                        transactionUtility += transaction.getItemsUtilities()[i].utility;
                        matchesCount++;
                    }
                }
                if (matchesCount == c.size())
                {
                    c.incrementUtility(transactionUtility);
                }
            }

            //END CODE

            if (c.getU() < min)
            {
                iterItemset.remove();  //ERROR
                highU.decreaseCount();
            }
        }
    }
}

我想从 IEnumerator 中删除一个项目，我该怎么做？

注意： C#部分写的代码比较完整，更多的代码写在(CODE)到(END CODE)部分。

Answer 1

我想从 IEnumerator 中删除一个项目，我该怎么做？

很简单：你没有。

我不懂 Java，但这个想法在 .NET 中没有意义。 正如您在文档中所读到的：

枚举器可用于读取集合中的数据，但不能用于修改底层集合。

正如还提到的，应该直接使用foreach而不是IEnumerator<T> ：

foreach (List<ItemNY> level in highU.getLevels())
{
    foreach (var item in level)
    {
        if (item.getU() < min)
        {
            level.Remove(item);  //Error, you are modifying a collection being enumerated
            highU.decreaseCount();
        }
    }
}

您必须将与过滤器匹配的元素放在一个新集合中，以便您可以将它们从level删除。

foreach (List<ItemNY> level in highU.getLevels())
{
    var filteredItems = new List<ItemNY>();

    foreach (var c in level)
    {
        foreach (TransactionTP transaction in database.getTransactions())
        {
            int transactionUtility = 0;
            int matchesCount = 0;
            for (int i = 0; i < transaction.size(); i++)
            {
                if (c.getItems().Contains(transaction.get(i).item))
                {
                    transactionUtility += transaction.getItemsUtilities()[i].utility;
                    matchesCount++;
                }
            }

            if (matchesCount == c.size())
            {
                c.incrementUtility(transactionUtility);
            }
        }

        if (c.getU() < min)
        {
            filteredItems.Add(c);
            highU.decreaseCount();
        }
    }

    foreach (var item in filteredItems)
    {
        level.Remove(item);
    } 
}

Answer 2

如果真正的目标是从列表中删除项目，那么我将使用List.RemoveAll ，并使用条件定义要删除的项目。

如果您需要额外的计数器更新，那么您可以在前后取一个尺寸。 尽管至少在目前的情况下不需要减少计数方法，因为它似乎跟踪列表的大小，所以我会放弃该方法，并依赖列表计数。