Andreas Schaefer wrote: > Hi Dain > > >>That is no excuse. The exceptional case is the MBeanServer is having >>issues. So to quote you "get off your fat ass". This code is >>confusing, and just plane wrong. Fix it. > > > No, it is an exception when no server is available therefore this > is correct or do you check every String parameter if it is null > before applying a method on it ? > > BTW Alex problem is not there !
If I expect a String to be null, I check. I ALWAYS call hasNext before calling next, and I check if an Set isEmpty when is a normal result. This is just bad code. Fix it. -dain ------------------------------------------------------- This sf.net email is sponsored by: OSDN - Tired of that same old cell phone? Get a new here for FREE! https://www.inphonic.com/r.asp?r=sourceforge1&refcode1=vs3390 _______________________________________________ Jboss-development mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/jboss-development
