Changeset 7468 for branches/ChangeDatasetOfRegressionModel/HeuristicLab.Problems.DataAnalysis.Views/3.4/Solution Views/RegressionSolutionView.cs

Timestamp:

02/15/12 10:40:17 (13 years ago)

Author:

mkommend

Message:

#1758: Minor code improvements (variable naming, return conditions)

File:

• branches/ChangeDatasetOfRegressionModel/HeuristicLab.Problems.DataAnalysis.Views/3.4/Solution Views/RegressionSolutionView.cs (modified) (2 diffs)

branches/ChangeDatasetOfRegressionModel/HeuristicLab.Problems.DataAnalysis.Views/3.4/Solution Views/RegressionSolutionView.cs

@@ -45 +45 @@
     protected override void SetEnabledStateOfControls() {
       base.SetEnabledStateOfControls();
-      btnImport.Enabled = !Locked && !ReadOnly && Content != null;
+      importButton.Enabled = !Locked && !ReadOnly && Content != null;
     }
 
@@ -62 +62 @@
     #endregion
 
-    private void btnImport_Click(object sender, System.EventArgs e) {
-      if (openFileDialog.ShowDialog(this) == DialogResult.OK) {
-        object hlFile = null;
-        try {
-          hlFile = XmlParser.Deserialize(openFileDialog.FileName);
-        }
-        catch (Exception ex) {
-          ErrorHandling.ShowErrorDialog(this, ex);
-        }
+    private void importButton_Click(object sender, EventArgs e) {
+      if (openFileDialog.ShowDialog(this) != DialogResult.OK) return;
 
-        if (hlFile != null) {
-          IRegressionProblemData problemData = null;
-          if (hlFile is RegressionProblemData) {
-            problemData = (RegressionProblemData)hlFile;
-          } else if (hlFile is IRegressionProblem) {
-            problemData = ((IRegressionProblem)hlFile).ProblemData;
-          } else if (hlFile is IRegressionSolution) {
-            problemData = ((IRegressionSolution)hlFile).ProblemData;
-          }
+      object hlFile = null;
+      try {
+        hlFile = XmlParser.Deserialize(openFileDialog.FileName);
+      }
+      catch (Exception ex) {
+        ErrorHandling.ShowErrorDialog(this, ex);
+        return;
+      }
 
-          if (problemData == null) {
-            ErrorHandling.ShowErrorDialog(this,
-              new NullReferenceException("The problem data is null." + Environment.NewLine
-                                       + "The .hl-file is no RegressionProblemData or RegressionProblem."));
-          } else {
-            StringBuilder message = new StringBuilder();
-            if (!problemData.TargetVariable.Equals(Content.ProblemData.TargetVariable))
-              message.AppendLine("The target variables are not matching. Old target variable: '"
-                               + Content.ProblemData.TargetVariable
-                               + "'. New targetvariable: '" + problemData.TargetVariable + "'");
+      IRegressionProblemData problemData = null;
+      if (hlFile is IRegressionProblemData) {
+        problemData = (IRegressionProblemData)hlFile;
+      } else if (hlFile is IRegressionProblem) {
+        problemData = ((IRegressionProblem)hlFile).ProblemData;
+      } else if (hlFile is IRegressionSolution) {
+        problemData = ((IRegressionSolution)hlFile).ProblemData;
+      }
 
-            List<string> variables = problemData.InputVariables.Select(x => x.Value).ToList();
+      if (problemData == null) {
+        ErrorHandling.ShowErrorDialog(this,
+          new NullReferenceException("The problem data is null." + Environment.NewLine
+                                   + "The .hl-file is no RegressionProblemData or RegressionProblem."));
+        return;
+      }
 
-            foreach (var item in Content.ProblemData.InputVariables.CheckedItems) {
-              if (!variables.Contains(item.Value.Value))
-                message.AppendLine("Input variable '" + item.Value.Value + "' is not in the new problem data.");
-            }
+      StringBuilder message = new StringBuilder();
+      if (!problemData.TargetVariable.Equals(Content.ProblemData.TargetVariable))
+        message.AppendLine("The target variables are not matching. Old target variable: '"
+                         + Content.ProblemData.TargetVariable
+                         + "'. New targetvariable: '" + problemData.TargetVariable + "'");
 
-            if (message.Length != 0)
-              ErrorHandling.ShowErrorDialog(this, new Exception(message.ToString()));
-            else
-              Content.ProblemData = problemData;
-          }
-        }
+      List<string> variables = problemData.InputVariables.Select(x => x.Value).ToList();
+      foreach (var item in Content.ProblemData.InputVariables.CheckedItems) {
+        if (!variables.Contains(item.Value.Value))
+          message.AppendLine("Input variable '" + item.Value.Value + "' is not in the new problem data.");
       }
+
+      if (message.Length != 0)
+        ErrorHandling.ShowErrorDialog(this, new Exception(message.ToString()));
+      else
+        Content.ProblemData = problemData;
+
     }
   }