create reusable parseValue method for ratio limits

1 files changed, 13 insertions, 20 deletions

diff --git a/org.jacoco.report/src/org/jacoco/report/check/Limit.java b/org.jacoco.report/src/org/jacoco/report/check/Limit.java
index 1e2634aa..2b25e256 100644
--- a/org.jacoco.report/src/org/jacoco/report/check/Limit.java
+++ b/org.jacoco.report/src/org/jacoco/report/check/Limit.java
@@ -124,13 +124,7 @@ public class Limit {
 	 *            checked
 	 */
 	public void setMinimum(final String minimum) {
-		if (minimum == null) {
-			this.minimum = null;
-		} else if (isPercent(minimum)) {
-			this.minimum = percentToBigDecimal(minimum);
-		} else {
-			this.minimum = new BigDecimal(minimum);
-		}
+		this.minimum = parseValue(minimum);
 	}
 
 	/**
@@ -151,22 +145,21 @@ public class Limit {
 	 *            checked
 	 */
 	public void setMaximum(final String maximum) {
-		if (maximum == null) {
-			this.maximum = null;
-		} else if (isPercent(maximum)) {
-			this.maximum = percentToBigDecimal(maximum);
-		} else {
-			this.maximum = new BigDecimal(maximum);
-		}
+		this.maximum = parseValue(maximum);
 	}
 
-	private static boolean isPercent(final String value) {
-		return value.trim().endsWith("%");
-	}
+	private static BigDecimal parseValue(final String value) {
+		if (value == null) {
+			return null;
+		}
+		
+		final String trimmedValue = value.trim();
+		if (trimmedValue.endsWith("%")) {
+			final String percent = trimmedValue.substring(0, trimmedValue.length() - 1);
+			return new BigDecimal(percent).movePointLeft(2);
+		}
 
-	private static BigDecimal percentToBigDecimal(final String value) {
-		final String percent = value.trim().substring(0, value.length() - 1);
-		return new BigDecimal(percent).movePointLeft(2);
+		return new BigDecimal(trimmedValue);
 	}
 
 	String check(final ICoverageNode node) {