Changeset 12432

Timestamp:

06/12/15 11:14:39 (9 years ago)

Author:

pfleck

Message:

#2395
Use simple round-trip format for doubles because G17 prints some strange numbers (20.22 to 20.219999999999999999).
Some accuracy can still be lost on 64bit machines, but should be very rare and minimal.
double.MaxValue can still be parsed correctly, solving the problem of invalid value for the GUI.

Location:

trunk/sources/HeuristicLab.Data/3.3

Files:

• DoubleArray.cs (modified) (1 diff)
• DoubleMatrix.cs (modified) (1 diff)
• DoubleRange.cs (modified) (1 diff)
• DoubleValue.cs (modified) (1 diff)

trunk/sources/HeuristicLab.Data/3.3/DoubleArray.cs

@@ -56 +56 @@
     }
     protected virtual string GetValue(int index) {
-      return this[index].ToString("G17");
+      return this[index].ToString("r");
     }
     protected virtual bool SetValue(string value, int index) {

trunk/sources/HeuristicLab.Data/3.3/DoubleMatrix.cs

@@ -61 +61 @@
     }
     protected virtual string GetValue(int rowIndex, int columIndex) {
-      return this[rowIndex, columIndex].ToString("G17");
+      return this[rowIndex, columIndex].ToString("r");
     }
     protected virtual bool SetValue(string value, int rowIndex, int columnIndex) {

trunk/sources/HeuristicLab.Data/3.3/DoubleRange.cs

@@ -56 +56 @@
 
     public override string ToString() {
-      return string.Format("Start: {0:G17}, End: {1:G17}", Start, End);
+      return string.Format("Start: {0:r}, End: {1:r}", Start, End);
     }
 

trunk/sources/HeuristicLab.Data/3.3/DoubleValue.cs

@@ -48 +48 @@
 
     public override string ToString() {
-      return Value.ToString("G17");  // round-trip format
-      // Note that the round-trip format "r" can still lead to different values when run on an 64-bit machine, therefore using G17 instead.
-      // (https://msdn.microsoft.com/en-us/library/kfsatb94.aspx)
+      return Value.ToString("r");  // round-trip format
     }