diff options
author | TomyLobo <tomylobo@nurfuerspam.de> | 2012-02-25 16:25:42 +0100 |
---|---|---|
committer | EvilSeph <evilseph@gmail.com> | 2012-02-28 17:20:51 -0500 |
commit | 07f964813c71c7bf0dd5d4c1b7605242979f1fec (patch) | |
tree | 74ae7b4439199a100d2d5ba1bc20ca9733bd5d46 /src | |
parent | 7d51bc7834903d5c36e1472a45e7ad4673bcad27 (diff) | |
download | bukkit-07f964813c71c7bf0dd5d4c1b7605242979f1fec.tar bukkit-07f964813c71c7bf0dd5d4c1b7605242979f1fec.tar.gz bukkit-07f964813c71c7bf0dd5d4c1b7605242979f1fec.tar.lz bukkit-07f964813c71c7bf0dd5d4c1b7605242979f1fec.tar.xz bukkit-07f964813c71c7bf0dd5d4c1b7605242979f1fec.zip |
Reduced excessive exception handling in NumberConversions. Addresses BUKKIT-825
This also allows, for instance, to parse complex numbers with imaginary part=0, if the according toString method omits zero components.
This also saves some unboxing (Foo.valueOf returns a wrapper, while Foo.parseFoo returns a primitive)
Diffstat (limited to 'src')
-rw-r--r-- | src/main/java/org/bukkit/util/NumberConversions.java | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/src/main/java/org/bukkit/util/NumberConversions.java b/src/main/java/org/bukkit/util/NumberConversions.java index b6cf1009..f70cc314 100644 --- a/src/main/java/org/bukkit/util/NumberConversions.java +++ b/src/main/java/org/bukkit/util/NumberConversions.java @@ -13,8 +13,8 @@ public final class NumberConversions { int result = 0; try { - result = Integer.valueOf((String) object); - } catch (Throwable ex) {} + result = Integer.parseInt(object.toString()); + } catch (NumberFormatException ex) {} return result; } @@ -27,8 +27,8 @@ public final class NumberConversions { float result = 0; try { - result = Float.valueOf((String) object); - } catch (Throwable ex) {} + result = Float.parseFloat(object.toString()); + } catch (NumberFormatException ex) {} return result; } @@ -41,8 +41,8 @@ public final class NumberConversions { double result = 0; try { - result = Double.valueOf((String) object); - } catch (Throwable ex) {} + result = Double.parseDouble(object.toString()); + } catch (NumberFormatException ex) {} return result; } @@ -55,8 +55,8 @@ public final class NumberConversions { long result = 0; try { - result = Long.valueOf((String) object); - } catch (Throwable ex) {} + result = Long.parseLong(object.toString()); + } catch (NumberFormatException ex) {} return result; } @@ -69,8 +69,8 @@ public final class NumberConversions { short result = 0; try { - result = Short.valueOf((String) object); - } catch (Throwable ex) {} + result = Short.parseShort(object.toString()); + } catch (NumberFormatException ex) {} return result; } @@ -83,8 +83,8 @@ public final class NumberConversions { byte result = 0; try { - result = Byte.valueOf((String) object); - } catch (Throwable ex) {} + result = Byte.parseByte(object.toString()); + } catch (NumberFormatException ex) {} return result; } |