Long.parseLong error when parsing a string from a SAXparser class (Java)

Question

I have a problem but I don't know how to solve it: I need to get a Long number from a String (I get this string in a SAXparser class I wrote to get some fields from an XML file).

Here you are the problematic code:

if (bprice) {
String price = new String(ch, start, length);
System.out.println("Product price: " + price);
bprice = false;
if (price.equalsIgnoreCase("0"))
product.price = null;
else product.price = Long.parseLong(price);
}

(product is an instance from an object where I store all the product information. I got the error in the last line)

I debugged it and I got this:

Exception occurred in target VM: For input string: "1.04" java.lang.NumberFormatException: For input string: "1.04" at java.lang.NumberFormatException.forInputString(NumberFormatException.java:48) at java.lang.Long.parseLong(Long.java:419) at java.lang.Long.parseLong(Long.java:468) at parsers.LectorXML.characters(LectorXML.java:325) at com.sun.org.apache.xerces.internal.parsers.AbstractSAXParser.characters(AbstractSAXParser.java:538) at com.sun.org.apache.xerces.internal.impl.XMLDocumentFragmentScannerImpl.scanDocument(XMLDocumentFragmentScannerImpl.java:464) at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse(XML11Configuration.java:808) at com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse(XML11Configuration.java:737) at com.sun.org.apache.xerces.internal.parsers.XMLParser.parse(XMLParser.java:119) at com.sun.org.apache.xerces.internal.parsers.AbstractSAXParser.parse(AbstractSAXParser.java:1205) at parsers.LectorXML.leer(LectorXML.java:71) at parsers.LeerXML.main(LeerXML.java:20) <

What's your question? It's quite obvious that 1.04 is not a valid long number. — JB Nizet
– JB Nizet, Commented Nov 13, 2012 at 21:44
You should not use long or floating point numbers to store prices, unless the prices are expressed in cents (104 instead of 1.04). The most precise type for storing monetary data is BigDecimal — Sergey Kalinichenko
– Sergey Kalinichenko, Commented Nov 13, 2012 at 21:46
@Alberto Double will accumulate errors when you do math with it because its representation of decimal fractions is not precise. — Sergey Kalinichenko
– Sergey Kalinichenko, Commented Nov 14, 2012 at 11:43
Thanks for your reply @dasblinkenlight. I changed my code this morning and I am using BigDecimal instead Double for the prices. I experienced it is slower than working with Doubles — Alberto
– Alberto, Commented Nov 14, 2012 at 16:18

JB Nizet · Accepted Answer · 2012-11-13 21:45:11Z

2

1.04 is not a valid long. It is in double format.

You need to use Double.parseDouble(String); instead.

Example:

Double.parseDouble(precio1);

edited Nov 13, 2012 at 21:45

JB Nizet

694k94 gold badges1.3k silver badges1.3k bronze badges

answered Nov 13, 2012 at 21:43

kosa

66.7k15 gold badges134 silver badges170 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Alberto Over a year ago

Oh my god!!! After 6 hours coding I didn't see that!! Ohhhhhh damn it! haha. I'm soooooooooooooo sorry for my stupid question... :(. Thank you very much @Nambari

Alberto Over a year ago

I tried to accept it but I still had to wait 11 minutes when I tried to (still 4 minutes from now to be able to accept it). I am going to have dinner and I will accept it after that, ok? Thank you very much and I am sorry again!

kosa Over a year ago

@Alberto: That was just regular note because you are new user to SO. No force (or) time lines on accepting answers.

Collectives™ on Stack Overflow

Long.parseLong error when parsing a string from a SAXparser class (Java)

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related