使用 ListView c# 时减少循环次数

Question

我有一个 listView 看起来像这样：

我想在数量列中迭代每个项目。我已经写了一个代码来做到这一点：

            foreach (ListViewItem item in listView1.Items)
        {

            for (int quantity = 1; quantity < (Convert.ToInt32(item.SubItems[1].Text) + 1); quantity++)
            {
                 //do something
            }
        }

我想知道我是否可以只用一个循环而不是两个循环来做到这一点。

谢谢

Answer 1

我不确定你为什么要这样做，但如果你认为有必要并且有帮助，你可以做一些这样的 linq：

var items = listView1.Items.OfType<ListViewItem>();
var query = items.SelectMany(item => 
               Enumerable.Range(1, Convert.ToInt32(item.SubItems[1].Text))
                         .Select(i => new { Item = item, Index = i});
foreach(var element in query)
{
    Console.WriteLine("Book: {0}, i: {1}", element.Item.Text, element.Index);
}

结果：

Book: book1, i: 1
Book: book1, i: 2
Book: book1, i: 3
Book: book1, i: 4
Book: book2, i: 1
...
Book: book2, i: 11
Book: book3, i: 1
...
Book: book3, i: 13

但是正如你在评论中提到的可读性，我真的认为你的两个嵌套循环足够可读，或者至少比这个链接更具可读性。
还有一点要明确：这不会减少迭代次数，只会减少循环的嵌套。

---

一种更具可读性的方法是将一些功能封装在额外的方法中：

foreach(ListViewItem item in ListViewItems)
    HandleItem(item);

private void HandleItem(ListViewItem item)
{
    for (int i=1; i<Convert.ToInt32(item.SubItems[1].Text)+1; i++)
        HandleItemWithIndex(item, i);
}
private void HandleItemWithIndex(ListViewItem item, int index)
{
    // Do something
}

这样（如果您适当地命名方法）很容易了解发生的情况以及每个方法的用途。