如何从列表中删除内容

Question

我对编码很陌生。有没有其他方法可以编写此代码以从 c# 中的列表中删除内容？

public void RemoveItem(string itemDescription)
{
    MenuItem found = new MenuItem(false, 0, "", ""); // must be a better way - to fix

    foreach (MenuItem item in MenuItems)
    {
        if (item.Description == itemDescription)
        {
            found = item;
        }
    }

    MenuItems.Remove(found);
}

Answer 1

有几种方法可以改进此代码。其中一些已经被提及。

---

正如@tim-schmelter 评论的那样

// removes the last matching occurrence from the menu and avoids the "dummy" object
public void RemoveItem(string itemDescription)
{
    MenuItem found = null; // this is better

    foreach (MenuItem item in MenuItems)
    {
        if (item.Description == itemDescription)
        {
            found = item;
        }
    }

    if (found != null)
        MenuItems.Remove(found);
}

---

最初由@SomeBody 回答

// removes the first matching occurrence from the menu
public void RemoveItem(string itemDescription)
{
    foreach (MenuItem item in MenuItems)
    {
        if (item.Description == itemDescription)
        {
            MenuItems.Remove(found);
            break; // or return;
        }
    }
}

---

// same but replacing the loop with a LINQ approach
public void RemoveItem(string itemDescription)
{
    MenuItem found = MenuItems.FirstOrDefault(item => item.Description == itemDescription);

    if (found != null)
        MenuItems.Remove(found);
}

---

// removing all occurences (assumes that MenuItems is from type `List<MenuItem>`
public void RemoveItem(string itemDescription)
{
    MenuItems.RemoveAll(item => item.Description == itemDescription);
}

---

// assuming List<MenuItem>, removing the first occurrence only
public void RemoveItem(string itemDescription)
{
    int index = MenuItems.Find(item => item.Description == itemDescription);

    if (index >= 0)
        MenuItems.RemoveAt(index);
}

Answer 2

我假设MenuItems 是List<MenuItem>。

在这种情况下，正如 Johnathan 在 cmets 中所说，MenuItems.RemoveAll(item => item.Description == itemDescription); 将删除列表中符合您的条件的所有元素，这可能是最有效的选择：

public void RemoveItem(string itemDescription)
{
    MenuItems.RemoveAll(item => item.Description == itemDescription);
}

不过，理想情况下，我会使用 ID 字段来确保您确实删除了正确的元素。

Answer 3

从性能的角度来看，如果列表变得非常大或经常被遍历（希望菜单列表不是这种情况），则另一种解决方案特别有意义。

到目前为止，大多数给出的答案都需要两次查找（一次用于查找项目，一次用于删除项目），或者通过调用谓词函数需要一些开销。

以下可能是最快的方法，并且适用于每个实现 IList<T> 的集合：

// remove last matching item
public void RemoveItem(string itemDescription)
{
    for (int i = MenuItems.Count - 1; i >= 0; i--) // reverse for-loop, to avoid Count property call on each iteration
    {
        if (MenuItems[i].Description == itemDescription)
        {
            MenuItems.RemoveAt(i);
            return;
        }
    }

Answer 4

@geraldmayr 的回答很棒。

我唯一要改变的是 LINQ 中原始答案的方法，使其成为一个干净的内衬。

    public static void RemoveItem(string itemDescription)
    {
        menuItems.Remove(menuItems.First(menuItem => menuItem.ItemDescription == itemDescription));
    }