1

我的表单上有两个控件：一个带有工作人员列表的列表框和一个面板，该面板用作显示有关其工作的详细信息（卡片）的容器。当用户点击工人的名字时，我会在面板上显示卡片。卡片是一个用户控件，具有一些相当简单的 UI（2 个组框、3 个文本框和几个标签）和简单的逻辑（设置标签的前景色）。

卡片是在运行时创建的。以前的卡片从面板中删除，新的卡片被添加 - 每个工人的卡片数量是 1 到 4。这里很有趣。一切正常，直到大约。第五次点击工人。似乎 GC 开始了，大约需要两秒钟（0.3sx 数量的先前移除的卡）来处理旧卡（之前移除）并显示新卡。如果以前在工人之间移动效果很好，那么它会变得非常缓慢。经过一番探索，我发现问题在于Dispose我使用的控件的方法。通话base.Dispose()大约需要 0.3 秒。

这是我的代码：

private void ShowCards(List<Work> workItems) {
  var y = 5;
  panelControl1.SuspendLayout();
  panelControl1.Controls.Clear();

  foreach (var work in workItems) {
    var card = new Components.WorkDisplayControl(work);
    card.Top = y;
    card.Left = 10;

    y += card.Height + 5;

    panelControl1.Controls.Add(card);
  }

  panelControl1.ResumeLayout(true);
  Application.DoEvents();
}

到目前为止我已经尝试过：

  • 隐藏卡片而不是丢弃 - 在工人之间移动时它工作得更快，但在关闭表格时会支付罚款
  • 隐藏卡片并有一个单独的线程来处理它们 - 没有变化
  • 添加 10 张卡片并立即处理它们进行测试 - 慢
  • 添加 10 张卡片并立即在构造函数中处理它们进行测试- 快！
  • 用“正常”替换了 DevExpress 控件——没有变化
  • 更换工人时手动处理旧卡而不是移除它们 - 工人之间的每一次移动都会变慢：for (var ix = panelControl1.Controls.Count - 1; ix >= 0; --ix) { panelControl1.Controls[ix].Dispose();}
  • 分析它 - 这就是我在Dispose. 我可以追溯到Control.DestroyHandle
  • 调用我控制Controls.Clear()的Dispose方法 - 超级奇怪的行为和异常
  • 从我的用户控件中删除了所有控件 - 快一点，但仍然很慢
  • panelControl1移除和添加卡片时隐藏- 没有变化
  • 关闭后台 GC - 没有变化
  • 添加卡片AddRange

由于从构造函数调用相同的功能时运行速度很快，我相信原因必须在（控制）句柄中的某个地方。

我只是找不到这种奇怪行为的原因。我会很感激任何想法....

更新：在研究 GC 和 Control.Dispose 之间的联系时，我发现了这个很好的答案

4

2 回答 2

1

在 [来自 DevExpress 的支持失败] 之后，我做了一些测试和代码，终于找到了解决方案。

诀窍是在UserControl处理之前清除控件。

可以修改DisposeUC 上的方法（此解决方案在某些情况下有效，但并非在所有情况下都有效）或隐藏 UC 而不是从面板中删除它并清除它Controls

解决方案1：

protected override void Dispose(bool disposing) {
  if (disposing && (components != null)) {
    components.Dispose();
  }

  Controls.Clear(); // <--- Add this line

  base.Dispose(disposing); 
  }

解决方案2：

向 UC 添加新方法：

public void ClearControls() {
  Controls.Clear();
}

在我原来的问题中替换这一行

panelControl1.Controls.Clear();

有了这个：

for (var ii = panelControl1.Controls.Count - 1; ii >= 0; --ii) { 
  var wdc = panelControl1.Controls[ii] as Components.WorkDisplayControl;
  wdc.Visible = false;
  wdc.ClearControls();
}

它的工作速度（至少）快了 20 倍，这已经足够了。

于 2012-05-24T15:15:37.007 回答
0

问题的原因与 DevExpress 和标准控件无关。但是，它与创建和销毁控制句柄有关。为了改进您的卡片的实施，尽可能避免这些操作。我建议您对卡片使用缓存：

void ShowCards(List<Work> workItems) {
    cardsPanel.SuspendLayout();
    CacheCards(cardsPanel.Controls);
    int y = 5;
    foreach(var work in workItems) {
        var card = GetCardFromCache(work);
        card.Top = y;
        card.Left = 10;
        y += card.Height + 5;
        cardsPanel.Controls.Add(card);
    }
    cardsPanel.ResumeLayout(true);
}
//
Stack<WorkDisplayControl> cache;
void CacheCards(Control.ControlCollection controls) {
    if(cache == null)
        cache = new Stack<WorkDisplayControl>();
    foreach(WorkDisplayControl wdc in controls)
        cache.Push(wdc);
    controls.Clear();
}
WorkDisplayControl GetCardFromCache(Work data) {
    WorkDisplayControl result = (cache.Count > 0) ?
        cache.Pop() : new WorkDisplayControl();
    result.InitData(data);
    return result;
}

优化卡片的下一步是减少使用的控制手柄的总数。由于您使用的是 DevExpress 控件，因此最适合您的选择是XtraLayoutControl。使用 XtraLayoutControl 可以显着减少控制句柄的总数。它只为您描述的布局创建 4 个句柄（在几个组框中带有标签的 3 个编辑器），而不是使用标准控件时的 8 个句柄。XtraLayoutControl 不会为编辑器的标签、组、选项卡创建句柄。另请查看XtraGrid LayoutView - 它提供了使用网格数据绑定架构和卡片布局虚拟化的好处，无需任何额外编码。

于 2012-05-28T04:39:20.457 回答