I wasn't necessarily saying that we should always use
IllegalArgumentException (although it can be applicable if the thing
being switched upon is an argument to the method). The idea was that
we could throw an exception of some sort in our default clause.
On Tue, Sep 4, 2012 at 10:44 AM, Sébastien Brisard
<sebastien.bris...@m4x.org> wrote:
> Hello,
>
> 2012/9/4 James Carman <ja...@carmanconsulting.com>:
>> Something like:
>>
>> throw new IllegalArgumentException("This should never happen because
>> we are so smart we thought of every possibility in our case
>> statement.");
>>
>> would suffice :)
>>
> Not that it really matters, since this is never going to occur, but I
> think Bloch throws AssertionError in such cases.
> I think that IllegalStateException would be more appropriate in this
> case. But it really is for the sake of the argument anyway...
>
> Sébastien
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org
