Merge pull request #163 from raynigon/master

Handle parsing for localised value formats

Author: jenetics

jpx/src/main/java/io/jenetics/jpx/GPX.java

@@ -1490,6 +1490,7 @@ public void write(final GPX gpx, final Result result)
 				try (output) {
 					final var format = NumberFormat.getNumberInstance(ENGLISH);
 					format.setMaximumFractionDigits(_maximumFractionDigits);
+					format.setGroupingUsed(false);
 					final Function<Number, String> formatter = value ->
 						value != null ? format.format(value) : null;
 

jpx/src/main/java/io/jenetics/jpx/Length.java

@@ -20,6 +20,7 @@
 package io.jenetics.jpx;
 
 import static java.lang.String.format;
+import static java.util.Locale.ENGLISH;
 import static java.util.Objects.requireNonNull;
 
 import java.io.DataInput;
@@ -29,6 +30,8 @@
 import java.io.ObjectInputStream;
 import java.io.Serial;
 import java.io.Serializable;
+import java.text.NumberFormat;
+import java.text.ParseException;
 
 /**
  * Extent of something along its greatest dimension or the extent of space
@@ -220,11 +223,17 @@ public static Length of(final double length, final Unit unit) {
 	}
 
 	static Length parse(final String value) {
-		final String length = Strings.trim(value);
+		final Double length = parseDouble(value);
+		return Length.of(length, Unit.METER);
+	}
 
-		return length != null
-			? Length.of(Double.parseDouble(length), Unit.METER)
-			: null;
+	private static Double parseDouble(final String value) {
+		NumberFormat formatter = NumberFormat.getNumberInstance(ENGLISH);
+		try {
+			return formatter.parse(Strings.trim(value)).doubleValue();
+		} catch (ParseException e) {
+			throw new NumberFormatException("Unable to parse " + value);
+		}
 	}
 
 	/* *************************************************************************

jpx/src/test/java/io/jenetics/jpx/GPXTest.java

@@ -23,6 +23,7 @@
 import static java.util.Arrays.asList;
 import static org.assertj.core.api.Assertions.assertThat;
 import static io.jenetics.jpx.ListsTest.revert;
+import static org.assertj.core.api.Assertions.assertThatNoException;
 
 import nl.jqno.equalsverifier.EqualsVerifier;
 
@@ -154,6 +155,19 @@ public void readFromDocument() throws Exception {
 		assertThat(gpx2).isEqualTo(gpx);
 	}
 
+	@Test
+	public void writeThenReadDocument() {
+		GPX gpx = GPX.builder()
+			.addWayPoint(wp ->
+				wp.ele(1234.5).build(1.2, 3.4)
+			)
+			.build();
+		String gpxString = GPX.Writer.DEFAULT.toString(gpx);
+		assertThatNoException().isThrownBy(() -> {
+			GPX.Reader.DEFAULT.fromString(gpxString);
+		});
+	}
+
 	//@Test
 	public void print() throws IOException {
 		final GPX gpx = nextGPX(new Random(6123)).toBuilder()