DataGridView仅能执行一次排序的问题排查求助

问题场景 #

加载文件后生成带DataGridView的Tab页，点击Name列头实现升降序排序，首次点击正常，再次点击完全没反应。

涉及代码 #

创建TabControl的代码 #

private void createTabControl()
{
  tabControl = new TabControl();
  tabControl.Dock = DockStyle.Fill;
  foreach (File file in loadedFiles)
  {
    List<String> animals = new List<String>();
    foreach (Animal animal in file.Animals.Values)
    {
      animals.Add(animal);
    }
    TabPage tabPage = new TabPage(Path.GetFileName(file));
    tabControl.TabPages.Add(tabPage);

    DataGridView dataGridView = new DataGridView();
    dataGridView.Dock = DockStyle.Fill;

    dataGridView.DataSource = animals;
    dataGridView.ReadOnly = true;

    tabPage.Controls.Add(dataGridView);
    dataGridView.ColumnHeaderMouseClick += (sender, e) => DataGridView_ColumnHeaderMouseClick(sender, e, dataGridView);
  }
}

排序事件处理代码 #

private void DataGridView_ColumnHeaderMouseClick(object sender, DataGridViewCellMouseEventArgs e, DataGridView dataGridView)
{
  if (dataGridView.DataSource is List<String> animals)
  {
    string columnName = dataGridView.Columns[e.ColumnIndex].Name;
    if (columnName == "Name")
    {
      if (dataGridView.SortOrder == SortOrder.None || dataGridView.SortOrder == SortOrder.Descending)
      {
        animals.sort((x, y) => string.Compare(x.Name, y.Name));
      }
      else
      {
        animals.sort((x, y) => string.Compare(y.Name, x.Name));
      }
    }
  }
  dataGridView.Refresh();
}

问题根源 #

1. 数据源类型完全错配：你声明的是List<String>，但实际塞进去的是Animal对象，导致dataGridView.DataSource is List<String>判断不成立，你的排序逻辑根本没执行——首次点击生效其实是DataGridView自带的默认排序，不是你的代码在起作用。
2. 未更新排序状态标记：就算类型对了，手动排序后没更新DataGridView的SortOrder和SortedColumn属性，下次点击时控件以为当前排序状态没变，触发不了反向排序逻辑。
3. List无法自动通知界面更新：List<T>不支持数据变更通知，排序后调用Refresh()根本没法让DataGridView正确刷新数据。

修复方案 #

第一步：修正数据源类型 #

把List<String>改成List<Animal>，和实际添加的对象类型匹配：

// 创建TabControl时修正数据源
List<Animal> animals = new List<Animal>();
foreach (Animal animal in file.Animals.Values)
{
  animals.Add(animal);
}

// 排序事件里的类型判断同步修正
if (dataGridView.DataSource is List<Animal> animalsList)

第二步：手动维护排序状态 #

排序完成后必须更新DataGridView的排序标记，让它记住当前状态：

private void DataGridView_ColumnHeaderMouseClick(object sender, DataGridViewCellMouseEventArgs e, DataGridView dataGridView)
{
  if (dataGridView.DataSource is List<Animal> animalsList)
  {
    DataGridViewColumn targetColumn = dataGridView.Columns[e.ColumnIndex];
    if (targetColumn.Name == "Name")
    {
      SortOrder newSort;
      // 确定本次排序方向
      if (dataGridView.SortOrder == SortOrder.None || dataGridView.SortOrder == SortOrder.Descending)
      {
        animalsList.Sort((x, y) => string.Compare(x.Name, y.Name));
        newSort = SortOrder.Ascending;
      }
      else
      {
        animalsList.Sort((x, y) => string.Compare(y.Name, x.Name));
        newSort = SortOrder.Descending;
      }

      // 更新DataGridView的排序状态
      dataGridView.SortedColumn = targetColumn;
      dataGridView.SortOrder = newSort;
      
      // 重置数据源触发界面更新（List不支持自动通知）
      dataGridView.DataSource = null;
      dataGridView.DataSource = animalsList;
    }
  }
}

第三步：用BindingList优化通知逻辑 #

如果想更省心，换成BindingList<Animal>，它自带数据变更通知，不用手动重置数据源：

// 创建TabControl时改用BindingList
BindingList<Animal> animals = new BindingList<Animal>();
foreach (Animal animal in file.Animals.Values)
{
  animals.Add(animal);
}
dataGridView.DataSource = animals;

// 排序事件简化版
private void DataGridView_ColumnHeaderMouseClick(object sender, DataGridViewCellMouseEventArgs e, DataGridView dataGridView)
{
  if (dataGridView.DataSource is BindingList<Animal> animalsList)
  {
    DataGridViewColumn targetColumn = dataGridView.Columns[e.ColumnIndex];
    if (targetColumn.Name == "Name")
    {
      SortOrder newSort;
      if (dataGridView.SortOrder == SortOrder.None || dataGridView.SortOrder == SortOrder.Descending)
      {
        ((List<Animal>)animalsList).Sort((x, y) => string.Compare(x.Name, y.Name));
        newSort = SortOrder.Ascending;
      }
      else
      {
        ((List<Animal>)animalsList).Sort((x, y) => string.Compare(y.Name, x.Name));
        newSort = SortOrder.Descending;
      }
      dataGridView.SortedColumn = targetColumn;
      dataGridView.SortOrder = newSort;
      // 通知BindingList数据已变更
      animalsList.ResetBindings();
    }
  }
}

内容的提问来源于stack exchange，提问作者Quack_boi