Larry Stone wrote:
> It _should_ never get an NPE over a missing configuration key.  Given
> our sparse testing resources it behooves us to make the code as
> resilient as possible.

I'll disagree slightly - it isn't necessarily wrong for code to blow up. 
In cases like this, it's probably better for the code to blow up, than 
to quietly pretend to work, with only a vigilant pair of eyes on the log 
file realising if it isn't.

But it should blow up with something more useful than an NPE - testing 
the returned value for null, or catching the NPE and rethrowing a more 
descriptive exception.

G
This email has been scanned by Postini.
For more information please visit http://www.postini.com


-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
DSpace-tech mailing list
DSpace-tech@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dspace-tech

Reply via email to