Changeset 4200 for trunk/sources/HeuristicLab.Optimization.Views/3.3/RunCollectionTabularView.cs

Timestamp:

08/11/10 17:40:00 (14 years ago)

Author:

mkommend

Message:

improved the performance of RunCollections by caching some properties and corrected some bugs in the RunCollectionTabularView (ticket #1144)

File:

• trunk/sources/HeuristicLab.Optimization.Views/3.3/RunCollectionTabularView.cs (modified) (7 diffs)

trunk/sources/HeuristicLab.Optimization.Views/3.3/RunCollectionTabularView.cs

@@ -31 +31 @@
   [View("RunCollection Tabular View")]
   [Content(typeof(RunCollection), false)]
-  public partial class RunCollectionTabularView : StringConvertibleMatrixView {
+  public sealed partial class RunCollectionTabularView : StringConvertibleMatrixView {
+    private int[] runToRowMapping;
     public RunCollectionTabularView() {
       InitializeComponent();
-      this.dataGridView.RowHeaderMouseDoubleClick += new DataGridViewCellMouseEventHandler(dataGridView_RowHeaderMouseDoubleClick);
       base.ReadOnly = true;
     }
@@ -51 +51 @@
       base.OnContentChanged();
       if (Content != null) {
-        foreach (IRun run in Content)
-          UpdateRun(run);
-      }
-    }
-
+        runToRowMapping = Enumerable.Range(0, Content.Count).ToArray();
+        UpdateRowAttributes();
+      }
+    }
+
+    #region events
     protected override void RegisterContentEvents() {
       base.RegisterContentEvents();
@@ -63 +64 @@
       RegisterRunEvents(Content);
     }
-    protected virtual void RegisterRunEvents(IEnumerable<IRun> runs) {
+    private void RegisterRunEvents(IEnumerable<IRun> runs) {
       foreach (IRun run in runs)
         run.Changed += new EventHandler(run_Changed);
@@ -74 +75 @@
       DeregisterRunEvents(Content);
     }
-    protected virtual void DeregisterRunEvents(IEnumerable<IRun> runs) {
+    private void DeregisterRunEvents(IEnumerable<IRun> runs) {
       foreach (IRun run in runs)
         run.Changed -= new EventHandler(run_Changed);
@@ -81 +82 @@
       DeregisterRunEvents(e.OldItems);
       RegisterRunEvents(e.Items);
+      OnContentChanged();
     }
     private void Content_ItemsRemoved(object sender, HeuristicLab.Collections.CollectionItemsChangedEventArgs<IRun> e) {
@@ -96 +98 @@
       }
     }
+    #endregion
 
     private void UpdateRun(IRun run) {
-      int rowIndex = Content.ToList().IndexOf(run);
-      rowIndex = virtualRowIndizes[rowIndex];
+      int runIndex = GetIndexOfRun(run);
+      int rowIndex = runToRowMapping[runIndex];
       this.dataGridView.Rows[rowIndex].Visible = run.Visible;
       this.dataGridView.Rows[rowIndex].DefaultCellStyle.ForeColor = run.Color;
-      this.rowsTextBox.Text = this.Content.Count(r => r.Visible).ToString();
+      this.UpdateRowHeaders();
+    }
+
+    private int GetIndexOfRun(IRun run) {
+      int i = 0;
+      foreach (IRun actualRun in Content) {
+        if (actualRun == run)
+          return i;
+        i++;
+      }
+      throw new ArgumentException("Run " + run.Name + "could not be found in the RunCollection.");
     }
 
     private void dataGridView_RowHeaderMouseDoubleClick(object sender, DataGridViewCellMouseEventArgs e) {
       if (e.RowIndex >= 0) {
-        IRun run = Content.ElementAt(virtualRowIndizes[e.RowIndex]);
+        IRun run = Content.ElementAt(runToRowMapping.ToList().IndexOf(e.RowIndex));
         IContentView view = MainFormManager.MainForm.ShowContent(run);
         if (view != null) {
@@ -124 +137 @@
         Array.Sort(newSortedIndex, rowComparer);
       }
+
+      runToRowMapping = new int[newSortedIndex.Length];
+      int i = 0;
+      foreach (int runIndex in newSortedIndex) {
+        runToRowMapping[runIndex] = i;
+        i++;
+      }
+      UpdateRowAttributes();
       return newSortedIndex;
+    }
+
+    private void UpdateRowAttributes() {
+      int runIndex = 0;
+      foreach (IRun run in Content) {
+        int rowIndex = this.runToRowMapping[runIndex];
+        this.dataGridView.Rows[rowIndex].Visible = run.Visible;
+        this.dataGridView.Rows[rowIndex].DefaultCellStyle.ForeColor = run.Color;
+        runIndex++;
+      }
     }