Changeset 12613

Timestamp:

07/06/15 16:49:35 (9 years ago)

Author:

ascheibe

Message:

#2270 fixed NullReferenceException in statistical tests view by introducing a flag to suppress firing ui events

File:

• trunk/sources/HeuristicLab.Analysis.Statistics.Views/3.3/StatisticalTestsView.cs (modified) (5 diffs)

trunk/sources/HeuristicLab.Analysis.Statistics.Views/3.3/StatisticalTestsView.cs

@@ -41 +41 @@
     private double[][] data;
     private bool suppressUpdates = false;
+    private bool initializing = false;
 
     public double SignificanceLevel {
@@ -77 +78 @@
 
       if (Content != null) {
+        initializing = true;
         UpdateResultComboBox();
         UpdateGroupsComboBox();
         RebuildDataTable();
+        FillCompComboBox();
+        ResetUI();
+        CalculateValues();
+        initializing = false;
       }
       UpdateCaption();
@@ -135 +141 @@
         if (!suppressUpdates) RebuildDataTable();
       }
+    }
+
+    private void openBoxPlotToolStripMenuItem_Click(object sender, EventArgs e) {
+      RunCollectionBoxPlotView boxplotView = new RunCollectionBoxPlotView();
+      boxplotView.Content = Content;
+      boxplotView.SetXAxis(groupComboBox.SelectedItem.ToString());
+      boxplotView.SetYAxis(resultComboBox.SelectedItem.ToString());
+
+      boxplotView.Show();
+    }
+
+    private void groupCompComboBox_SelectedValueChanged(object sender, EventArgs e) {
+      if (initializing) return;
+      string curItem = (string)groupCompComboBox.SelectedItem;
+      CalculatePairwise(curItem);
+    }
+
+    private void resultComboBox_SelectedValueChanged(object sender, EventArgs e) {
+      if (initializing) return;
+      RebuildDataTable();
+      ResetUI();
+      CalculateValues();
+    }
+
+    private void groupComboBox_SelectedValueChanged(object sender, EventArgs e) {
+      if (initializing) return;
+      RebuildDataTable();
+      FillCompComboBox();
+      ResetUI();
+      CalculateValues();
     }
     #endregion
@@ -272 +308 @@
       pValTextBox.Text = string.Empty;
       equalDistsTextBox.Text = string.Empty;
-    }
-
-    private void resultComboBox_SelectedValueChanged(object sender, EventArgs e) {
-      RebuildDataTable();
-      ResetUI();
-      CalculateValues();
-    }
-
-    private void groupComboBox_SelectedValueChanged(object sender, EventArgs e) {
-      RebuildDataTable();
-      FillCompComboBox();
-      ResetUI();
-      CalculateValues();
     }
 
@@ -484 +507 @@
       return newData;
     }
-
-    private void openBoxPlotToolStripMenuItem_Click(object sender, EventArgs e) {
-      RunCollectionBoxPlotView boxplotView = new RunCollectionBoxPlotView();
-      boxplotView.Content = Content;
-      boxplotView.SetXAxis(groupComboBox.SelectedItem.ToString());
-      boxplotView.SetYAxis(resultComboBox.SelectedItem.ToString());
-
-      boxplotView.Show();
-    }
-
-    private void groupCompComboBox_SelectedValueChanged(object sender, EventArgs e) {
-      string curItem = (string)groupCompComboBox.SelectedItem;
-      CalculatePairwise(curItem);
-    }
   }
 }