Changeset 9041 for branches/RuntimeOptimizer/HeuristicLab.Optimization.Views/3.3/TimeLimitRunView.cs

Timestamp:

12/12/12 16:50:46 (11 years ago)

Author:

abeham

Message:

#1985:

• Moved code to format and parse timespans to a TimeSpanHelper class
• Added a dialog for defining arithmetic sequences of timespans

File:

• branches/RuntimeOptimizer/HeuristicLab.Optimization.Views/3.3/TimeLimitRunView.cs (modified) (11 diffs)

branches/RuntimeOptimizer/HeuristicLab.Optimization.Views/3.3/TimeLimitRunView.cs

@@ -84 +84 @@
       try {
         if (Content == null) {
-          timeLimitTextBox.Text = FormatTimeSpan(TimeSpan.FromSeconds(60));
+          timeLimitTextBox.Text = TimeSpanHelper.FormatNatural(TimeSpan.FromSeconds(60));
           snapshotsTextBox.Text = String.Empty;
           storeAlgorithmInEachSnapshotCheckBox.Checked = false;
@@ -91 +91 @@
           runsView.Content = null;
         } else {
-          timeLimitTextBox.Text = FormatTimeSpan(Content.MaximumExecutionTime);
-          snapshotsTextBox.Text = String.Join(" ; ", Content.SnapshotTimes.Select(x => FormatTimeSpan(x, true)));
+          timeLimitTextBox.Text = TimeSpanHelper.FormatNatural(Content.MaximumExecutionTime);
+          snapshotsTextBox.Text = String.Join(" ; ", Content.SnapshotTimes.Select(x => TimeSpanHelper.FormatNatural(x, true)));
           storeAlgorithmInEachSnapshotCheckBox.Checked = Content.StoreAlgorithmInEachSnapshot;
           algorithmViewHost.Content = Content.Algorithm;
@@ -137 +137 @@
           SuppressEvents = true;
           try {
-            timeLimitTextBox.Text = FormatTimeSpan(Content.MaximumExecutionTime);
+            timeLimitTextBox.Text = TimeSpanHelper.FormatNatural(Content.MaximumExecutionTime);
           } finally { SuppressEvents = false; }
           break;
@@ -144 +144 @@
           try {
             if (Content.SnapshotTimes.Any())
-              snapshotsTextBox.Text = String.Join(" ; ", Content.SnapshotTimes.Select(x => FormatTimeSpan(x, true)));
+              snapshotsTextBox.Text = String.Join(" ; ", Content.SnapshotTimes.Select(x => TimeSpanHelper.FormatNatural(x, true)));
             else snapshotsTextBox.Text = String.Empty;
             Content.SnapshotTimes.ItemsAdded += Content_SnapshotTimes_Changed;
@@ -184 +184 @@
       try {
         if (Content.SnapshotTimes.Any())
-          snapshotsTextBox.Text = string.Join(" ; ", Content.SnapshotTimes.Select(x => FormatTimeSpan(x, true)));
+          snapshotsTextBox.Text = string.Join(" ; ", Content.SnapshotTimes.Select(x => TimeSpanHelper.FormatNatural(x, true)));
         else snapshotsTextBox.Text = String.Empty;
       } finally { SuppressEvents = false; }
@@ -194 +194 @@
       if (SuppressEvents) return;
       TimeSpan ts;
-      if (!TryGetTimeSpanFromFormat(timeLimitTextBox.Text, out ts)) {
+      if (!TimeSpanHelper.TryGetFromNaturalFormat(timeLimitTextBox.Text, out ts)) {
         e.Cancel = true;
         errorProvider.SetError(timeLimitTextBox, "Please enter a valid time span, e.g. 60, 20s, 45 seconds, 3h, 1 hour, 4 d, 2 days");
@@ -200 +200 @@
         Content.MaximumExecutionTime = ts;
         e.Cancel = false;
-        errorProvider.SetError(timeLimitTextBox, String.Empty);
+        errorProvider.SetError(timeLimitTextBox, null);
       }
     }
@@ -207 +207 @@
       if (SuppressEvents) return;
       e.Cancel = false;
-      errorProvider.SetError(timeLimitTextBox, String.Empty);
+      errorProvider.SetError(snapshotsTextBox, null);
 
       var snapshotTimes = new ObservableList<TimeSpan>();
@@ -213 +213 @@
       foreach (Match m in matches) {
         TimeSpan value;
-        if (!TryGetTimeSpanFromFormat(m.Value, out value)) {
+        if (!TimeSpanHelper.TryGetFromNaturalFormat(m.Value, out value)) {
           e.Cancel = !snapshotsTextBox.ReadOnly && snapshotsTextBox.Enabled; // don't cancel an operation that cannot be edited
           errorProvider.SetError(snapshotsTextBox, "Error parsing " + m.Value + ", please provide a valid time span, e.g. 60, 20s, 45 seconds, 3h, 1 hour, 4 d, 2 days");
@@ -297 +297 @@
 
     private void sequenceButton_Click(object sender, EventArgs e) {
-      using (var dialog = new DefineArithmeticProgressionDialog(false, 1, Content.MaximumExecutionTime.TotalSeconds, 1)) {
+      using (var dialog = new DefineArithmeticTimeSpanProgressionDialog(TimeSpan.FromSeconds(1), Content.MaximumExecutionTime, TimeSpan.FromSeconds(1))) {
         if (dialog.ShowDialog() == DialogResult.OK) {
           if (dialog.Values.Any())
-            Content.SnapshotTimes = new ObservableList<TimeSpan>(dialog.Values.Select(TimeSpan.FromSeconds));
+            Content.SnapshotTimes = new ObservableList<TimeSpan>(dialog.Values);
           else Content.SnapshotTimes = new ObservableList<TimeSpan>();
         }
@@ -307 +307 @@
     #endregion
     #endregion
-
-    private string FormatTimeSpan(TimeSpan ts, bool abbrevUnit = false) {
-      if (ts.TotalSeconds < 180) return ts.TotalSeconds + (abbrevUnit ? "s" : " seconds");
-      if (ts.TotalMinutes < 180) return ts.TotalMinutes + (abbrevUnit ? "min" : " minutes");
-      if (ts.TotalHours < 96) return ts.TotalHours + (abbrevUnit ? "h" : " hours");
-      return ts.TotalDays + (abbrevUnit ? "d" : " days");
-    }
-
-    private bool TryGetTimeSpanFromFormat(string text, out TimeSpan result) {
-      double value;
-      result = TimeSpan.Zero;
-
-      var numberMatch = Regex.Match(text, @"\d+");
-      if (!numberMatch.Success) return false;
-
-      if (!double.TryParse(numberMatch.Value, out value))
-        return false;
-
-      var unitMatch = Regex.Match(text, @"[a-zA-Z]+$");
-      if (!unitMatch.Success) return false;
-
-      switch (unitMatch.Value) {
-        case "d":
-        case "day":
-        case "days": result = TimeSpan.FromDays(value); break;
-        case "h":
-        case "hour":
-        case "hours": result = TimeSpan.FromHours(value); break;
-        case "min":
-        case "minute":
-        case "minutes": result = TimeSpan.FromMinutes(value); break;
-        case "s":
-        case "second":
-        case "seconds": result = TimeSpan.FromSeconds(value); break;
-        default: return false;
-      }
-      return true;
-    }
   }
 }