On 11/16/06, Allen Gilliland <[EMAIL PROTECTED]> wrote:
Just going to resend this one since I didn't hear back from anyone.  I
plan to make this change while I am cleaning up the business layer a bit.

Basically, we don't need to throw a RollerException from every call to
Roller.getXXXManager() because 1) it's incredibly unlikely that will
cause a problem and if it does then we can throw a RuntimeException
instead and 2) none of the current implementation throws the exception
anyways.  So right now it's just forcing lots of try/catch blocks which
are of no value.

Sounds good, and very Spring-like. ;-)

Matt


-- Allen


Allen Gilliland wrote:
> I've started notice a little nit pick that I have about the fact that we
> throw a RollerException from all of the Roller.getXXXManager() methods.
>  I find myself constantly feeling that this is a totally useless
> exception which often causes the code to be overly littered with
> try/catch blocks which ultimately have no real use.
>
> Would anyone object to changing these methods to not throw an exception?
>  Does anyone see any value in having these methods throw that exception?
>
> Also, just to be clear, this work wouldn't be done until after the 3.1
> release is finalized.
>
> -- Allen



--
http://raibledesigns.com

Reply via email to