一行列表视图正在更新，但也插入了重复的行

Question

大家好，我正在尝试更新正在订购的产品的数量。如果产品已存在于用户 'basket' 中而不是插入新行，则应更新现有项目行中的数量。它正在更新，但也插入了一个新行，其中包含应该添加到原始行的数量，如下所示：

我猜我的代码在逻辑上有问题，但我无法发现它。

 private void btn_add_Click(object sender, EventArgs e)
    {
        try
        {
            ListViewItem item = new ListViewItem(list_Select_Product.SelectedItems[0].Text);
            item.SubItems.Add(list_Select_Product.SelectedItems[0].SubItems[1].Text);
            item.SubItems.Add(txt_quantity.Text);

            bool ok = true;
            if (!validNumbers(txt_quantity))
                ok = false;
            if (!validLength(txt_quantity, 1, 2))
                ok = false;

            if (ok == true)
            {
                foreach (ListViewItem lvi in list_view_orderitems.Items)
                {

                    if(lvi.SubItems[0].Text == list_Select_Product.SelectedItems[0].Text)
                    {
                        int UpdateQunat = Convert.ToInt32(lvi.SubItems[2].Text);
                        int AddMe = Convert.ToInt32(txt_quantity.Text);
                        UpdateQunat = UpdateQunat + AddMe;
                        lvi.SubItems[2].Text = Convert.ToString(UpdateQunat);
                        list_view_orderitems.Items.Add(item);

                    }
                    else if (lvi.SubItems[0].Text != list_Select_Product.SelectedItems[0].Text)
                    {
                        list_view_orderitems.Items.Add(item);

                    }
                }
                if(list_view_orderitems.Items.Count == 0)
                {
                    list_view_orderitems.Items.Add(item);


                }

            }


        }
        catch
        {

            MessageBox.Show("A product must be selected");
        }
    }

Answer 1

仔细观察一下 foreach 循环。您实际上也将其添加到那里。（所以你应该删除它）

foreach (ListViewItem lvi in list_view_orderitems.Items)
{
    if(lvi.SubItems[0].Text == list_Select_Product.SelectedItems[0].Text)
    {
        int UpdateQunat = Convert.ToInt32(lvi.SubItems[2].Text);
        int AddMe = Convert.ToInt32(txt_quantity.Text);
        UpdateQunat = UpdateQunat + AddMe;
        lvi.SubItems[2].Text = Convert.ToString(UpdateQunat);
        // adding it again. This line is not needed.
        list_view_orderitems.Items.Add(item);
     }
     else if (lvi.SubItems[0].Text != list_Select_Product.SelectedItems[0].Text)
     {
         list_view_orderitems.Items.Add(item);
     }
}

Answer 2

我认为这是 if 你需要去掉 item.Add（我标记了它：

  if(lvi.SubItems[0].Text == list_Select_Product.SelectedItems[0].Text)
                    {
                        int UpdateQunat = Convert.ToInt32(lvi.SubItems[2].Text);
                        int AddMe = Convert.ToInt32(txt_quantity.Text);
                        UpdateQunat = UpdateQunat + AddMe;
                        lvi.SubItems[2].Text = Convert.ToString(UpdateQunat);
                      //  list_view_orderitems.Items.Add(item);

                    }

一行列表视图正在更新，但也插入了重复的行

One row of list view is updating, but duplicate row is also inserted

c#

forms

windows

list

view