C#在datagridview中搜索重复项

Question

使用 ms visual studio 和 csharp .net4。

这是我必须检查重复的代码

    public void CheckForDuplicate()
    {
        DataGridViewRowCollection coll = ParetoGrid.Rows;
        DataGridViewRowCollection colls = ParetoGrid.Rows;
        List<string> listParts = new List<string>();
        int count = 0;
        foreach (DataGridViewRow item in coll)//379
        {
            foreach (DataGridViewRow items in colls)//143641
            {
                if (items.Cells[5].Value == item.Cells[5].Value)  
                {
                    if (items.Cells[2].Value != item.Cells[2].Value)
                    {
                        listParts.Add(items.Cells["Keycode"].Value.ToString());
                        count++;
                        dupi = true;

                        //txtDupe.Text = items.Cells["Keycode"].Value.ToString();
                        //this.Refresh();
                    }
                }
            }
        }
        MyErrorGrid.DataSource = listParts;
    }

这是在允许用户保存之前的检查。

private void butSave_Click(object sender, EventArgs e)
    {
        CheckForDuplicate();
        if (dupi == true)
        {
            txtDupe.Clear();

            dupi = false;
        }
        else
        {
            SaveMyWorkI();
            dupi = false;
        }
    }

这是它正在查看的数据：

现在，我知道逻辑一定是有缺陷的，因为无论如何它都会保存。 我基本上是在 pareto1 上的每个单元格中搜索，以查看用户是否进行了任何 重复，如果是这样，它将不会保存，而是在另一个 datagridview 中显示零件编号等....这就是计划。

那么有人可以看看这个并告诉我

1)在我的逻辑中，这失败在哪里，如果检查是正确的呢？

2) 列表是否可以添加信息，如果可以，那么简单的绑定到数据网格 视图是否足以显示结果？

3)如果这只是一种非常糟糕的搜索方式，是否有人可以提供反映我想要实现的目标的代码。

非常感谢您未来的 cmets。

更新:: 好的，感谢您的帮助，我的算法现在可以工作了，但我的最后一个问题是显示在帕累托列上重复的零件编号，而不是显示长度。

public void CheckForDuplicate()
    {
        DataGridViewRowCollection coll = ParetoGrid.Rows;
        DataGridViewRowCollection colls = ParetoGrid.Rows;
        List<string> listParts = new List<string>();
        int count = 0;
        foreach (DataGridViewRow item in coll)//379
        { 
            foreach (DataGridViewRow items in colls)//143641
            {
                count++;
                if ((items.Cells[5].Value != null))
                {
                    if ((items.Cells[5].Value != null) && (items.Cells[5].Value.Equals(item.Cells[5].Value)))
                    {
                        if ((items.Cells[2].Value != null) && !(items.Cells[2].Value.Equals(item.Cells[2].Value)))
                        {
                            listParts.Add(items.Cells["Keycode"].Value.ToString());

                            dupi = true;
                        }
                    }
                }
            }
        }
        MyErrorGrid.DataSource = listParts;
        var message = string.Join(Environment.NewLine, listParts);
        //MyErrorGrid.DataSource = message;
        MessageBox.Show(message);

    }

即使消息框正确显示结果？绑定到我的数据网格时我错过了什么吗？

Answer 1

这是一个简单的示例，展示了如何在数据输入期间执行验证。您可以通过多种方式自定义错误的显示方式（包括某种自定义对话框来解决错误），从而为您提供更好的解决方案。

public partial class Form1 : Form
{
    BindingSource bs;
    DataTable dt;    public Form1()
    {
        InitializeComponent();

        BindingList<BindingClass> data = new BindingList<BindingClass>
            { 
                new BindingClass{ Name = "one" }, 
                new BindingClass { Name = "two"} 
            };

        dataGridView1.DataSource = data;
        dataGridView1.CellValidating += new DataGridViewCellValidatingEventHandler(dataGridView1_CellValidating);

    }

    void dataGridView1_CellValidating(object sender, DataGridViewCellValidatingEventArgs e)
    {
        foreach (DataGridViewRow row in dataGridView1.Rows)
        {
            if (row.Index != e.RowIndex & !row.IsNewRow)
            {
                if (row.Cells[0].Value.ToString() == e.FormattedValue.ToString())
                {
                    dataGridView1.Rows[e.RowIndex].ErrorText =
                        "Duplicate value not allowed";                    

                    e.Cancel = true;
                    return;
                }
            }
        }
        dataGridView1.Rows[e.RowIndex].ErrorText = string.Empty;
    }

} 

    public class BindingClass
    {
        public string Name { get; set; }
    }

}

当然，这并不总是符合您对用户喜欢使用的内容的要求，但我认为看到另一个选项会有所帮助。

Answer 2

您正在与 == 和 != 进行比较。

items.Cells[5].Value 公开一个对象。

在您的情况下，这很可能是基于引用相等性进行相等性检查，这可能不是您想要的。尝试使用类似items.Cells[5].Value.Equals(item.Cells[5].Value)

还请考虑在可用的最简单抽象上解决此类问题。例如。如果您将网格绑定到对象集合，那么您可以针对该对象集合执行清理操作，而忽略您在其上固定的任何 UI。

您还可以考虑使用 LINQ 命名空间中的 Distinct 扩展方法并为其提供 IEqualityComparer^* 以确保最有效的删除重复项的代码在.NET Framework 由您使用。

---

^*)IEqualityComparer 是一种抽象，当您认为两个对象相等时，它允许您在一个地方进行定义。 Distinct 提供了一个重载，您可以在其中指定这样的比较器。

Answer 3

看看这是否适合你

 var dup = dataGridView1.Rows.Cast<DataGridViewRow>().Distinct().Where(g => g.Index != 0);

排除索引为0的行。它是标题行。