You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2013/03/27 14:04:25 UTC
svn commit: r1461559 -
/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/math/NumberUtils.java
Author: ggregory
Date: Wed Mar 27 13:04:25 2013
New Revision: 1461559
URL: http://svn.apache.org/r1461559
Log:
Statements unnecessarily nested within else clauses.
Modified:
commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/math/NumberUtils.java
Modified: commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/math/NumberUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/math/NumberUtils.java?rev=1461559&r1=1461558&r2=1461559&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/math/NumberUtils.java (original)
+++ commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/math/NumberUtils.java Wed Mar 27 13:04:25 2013
@@ -556,52 +556,49 @@ public class NumberUtils {
throw new NumberFormatException(str + " is not a valid number.");
}
+ }
+ //User doesn't have a preference on the return type, so let's start
+ //small and go from there...
+ if (expPos > -1 && expPos < str.length() - 1) {
+ exp = str.substring(expPos + 1, str.length());
} else {
- //User doesn't have a preference on the return type, so let's start
- //small and go from there...
- if (expPos > -1 && expPos < str.length() - 1) {
- exp = str.substring(expPos + 1, str.length());
- } else {
- exp = null;
+ exp = null;
+ }
+ if (dec == null && exp == null) {
+ //Must be an int,long,bigint
+ try {
+ return createInteger(str);
+ } catch (final NumberFormatException nfe) { // NOPMD
+ // ignore the bad number
}
- if (dec == null && exp == null) {
- //Must be an int,long,bigint
- try {
- return createInteger(str);
- } catch (final NumberFormatException nfe) { // NOPMD
- // ignore the bad number
- }
- try {
- return createLong(str);
- } catch (final NumberFormatException nfe) { // NOPMD
- // ignore the bad number
- }
- return createBigInteger(str);
-
- } else {
- //Must be a float,double,BigDec
- final boolean allZeros = isAllZeros(mant) && isAllZeros(exp);
- try {
- final Float f = createFloat(str);
- if (!(f.isInfinite() || (f.floatValue() == 0.0F && !allZeros))) {
- return f;
- }
- } catch (final NumberFormatException nfe) { // NOPMD
- // ignore the bad number
- }
- try {
- final Double d = createDouble(str);
- if (!(d.isInfinite() || (d.doubleValue() == 0.0D && !allZeros))) {
- return d;
- }
- } catch (final NumberFormatException nfe) { // NOPMD
- // ignore the bad number
- }
-
- return createBigDecimal(str);
+ try {
+ return createLong(str);
+ } catch (final NumberFormatException nfe) { // NOPMD
+ // ignore the bad number
+ }
+ return createBigInteger(str);
+ }
+ //Must be a float,double,BigDec
+ final boolean allZeros = isAllZeros(mant) && isAllZeros(exp);
+ try {
+ final Float f = createFloat(str);
+ if (!(f.isInfinite() || (f.floatValue() == 0.0F && !allZeros))) {
+ return f;
}
+ } catch (final NumberFormatException nfe) { // NOPMD
+ // ignore the bad number
}
+ try {
+ final Double d = createDouble(str);
+ if (!(d.isInfinite() || (d.doubleValue() == 0.0D && !allZeros))) {
+ return d;
+ }
+ } catch (final NumberFormatException nfe) { // NOPMD
+ // ignore the bad number
+ }
+
+ return createBigDecimal(str);
}
/**