0

设置

我有一段代码使用自定义 IComparer 对 DataGridView 进行排序：

public class CustomComparer: IComparer
{
    public int Compare(object x, object y)
    {
        DataGridViewRow row1 = (DataGridViewRow)x;
        DataGridViewRow row2 = (DataGridViewRow)y;

        if (row1.ReadOnly && row2.ReadOnly)
        {
            return 0;
        }
        else if (row1.ReadOnly && !row2.ReadOnly)
        {
            return 1;
        }
        else
        {
            return -1;
        }
}

问题

奇怪的是，当我执行以下行时（填充行之后）：

grid.Sort(new CustomComparer());

我收到 ArgumentOutOfRangeException 消息“索引超出范围。参数：索引”。

更多事实

进一步调查发现：

  • 我正在排序的 DataGridView上没有BindingSource - 已手动添加行。
  • 错误的堆栈跟踪只有一级深 - 它发生在 mscorlib 的 InternalDictionary 上
  • 奇怪的事实 #1 - 只有在任何时候，我的自定义比较器对其任何比较都返回 -1 时才会发生这种情况
  • 如果我将 Sort 方法更改为不再使用我的 CustomComparer，则不会引发异常。

解决方法

最后一个事实导致我重写了 Compare() 方法以遵循 .NET 的 CompareTo 方法：

DataGridViewRow row1 = (DataGridViewRow)x;
DataGridViewRow row2 = (DataGridViewRow)y;

return row1.ReadOnly.CompareTo(row2.ReadOnly);

这神秘地起作用了。不再抛出异常。

因此，尽管我有一个解决方法，但我想知道是否有人知道为什么这可能是一个修复，以及问题可能是什么。我查看了 CompareTo 的实现，它还返回 -1 ...

4

2 回答 2

4

juharr 是对的，但这就是为什么他是对的：

您的实现Compare不是对称的，这意味着如果row1.ReadOnly == false并且row2.ReadOnly == false您返回，则-1意味着“row1小于”。如果您使用相同的值进行比较，则变为小于。这可能会混淆需要对称的排序算法。 row2row2 row1Compare

正确的比较应该是：

    if (row1.ReadOnly == row2.ReadOnly)  // change && to ==
    {
        return 0;
    }
    else if (row1.ReadOnly && !row2.ReadOnly)
    {
        return 1;
    }
    else
    {
        return -1;
    }

这很可能bool.CompareTo(bool)会返回，这就是为什么您的“解决方法”（我认为这是一个更好的解决方案）有效的原因。

于 2013-01-08T17:02:29.510 回答
2

问题是-1当两者都有时ReadOnly你正在返回false。在这种情况下你应该返回0。

只需将您的第一个 if 语句更改为

 if((row1.ReadOnly && row2.ReadOnly) || !(row1.ReadOnly || row2.ReadOnly))
于 2013-01-08T16:51:42.167 回答