I'd introduced a Hibernate user type into the code base a few days ago. I'd based it on an example in the (more or less) wonderful book Hibernate in Action. Naturally, however, I'd copied the example with a bug in it!
FWIW, the example is listing 6.1, on page 204. The code in question reads:
public Object nullSafeGet(ResultSet resultSet, String[] names, Object owner) throws HibernateException, SQLException {
if (resultSet.wasNull()) return null;
BigDecimal valueInUSD = resultSet.getBigDecimal(names[0]);
return new MonetaryAmount(valueInUSD, Currency.getInstance("USD"));
}
The problem is that the first and second lines of the method are the wrong way around. ResultSet.wasNull(), btw, returns true if the last value extracted from the result set is null. As written, this will fail randomly if the previous column had contained null - hardly what's desired. (It could also fail if the column intended is null, but that's probably not as odd to try and diagnose) The reason to use ResultSet.wasNull(), instead of just looking at the returned value, is that the JDBC drive may not actually give you a null! For example, if you're trying to get back a primitive (say, using ResultSet.getDouble(), instead of getBigDecimal()), you're not going to get a null back, are you?
You can see this errata notice, and some more, at http://forum.hibernate.org/viewtopic.php?t=935347
Comments (3)
Yep, I can read the examples now. Just wish I could understand them :-)
Posted by ZenTiger | April 5, 2005 7:32 PM
Posted on April 5, 2005 19:32
Ah, well... that's a different problem. :)
If you have questions about any particular example, feel free to ask.
Posted by Robert | April 5, 2005 8:39 PM
Posted on April 5, 2005 20:39
I was bitten by the same thing a few days ago.
Posted by Twice | July 19, 2005 10:53 PM
Posted on July 19, 2005 22:53