Hi all,

I was looking at the code for PageParameters and then StringValue in 1.2.1

StringValue is throwing a checked exception 
StringValueConversionException for string converted to another type.

While I understand the motivation, it seems to me it would remove some 
slight handling annoyance by putting it as an unchecked (ie: runtime) 
exception.

Strictly speaking if such thing happen this is due to a mistake by 
storing invalid types and therefore there is very little chance it to 
recover from it in any sort of way. So you basically won't be able to do 
anything worth it with the checked exception.

If however someone is doing some clever trick and handling multiple 
types in a strange way, then I guess, it makes sense for this person to 
try/catch unchecked exception and try to recover by asking another type, 
etc...

So I'm voting to have StringValueConversionException extends 
RuntimeException instead of Exception :)

Maybe I have missed something obvious though.

WDYT ?

Stephane


-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Wicket-develop mailing list
Wicket-develop@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-develop

Reply via email to