Menu

#10 Request for Checked Exception

closed
nobody
None
5
2008-05-13
2008-05-05
No

Request for Checked Exception
1440592 : WsmoFactory::createIRI

.. Should throw a *checked* exception instead of a runtime one.

The invalid IRI was supplied by the user to he should handle the error, instead of just propagating it out of context (which will happen with a runtime exception).

-----

I've opened a bug report for WsmoFactory to throw a *checked* exception when creating IRIs, currently it just throws only IllegalArgumentException which is:

- Incorrect in the case of validating the IRI (e.g. not just checking for empty arguments, etc).

- Makes it very difficult to handle the error in the appropriate context (it is just propagated out of context)

(There are probably other places where a checked exception is more appropriate too. Generally, the exception model of wsmo4j needs major redesign)
-----
Would be fine with me, but shouldn't we then also have a look at the other exceptions?
-----

Good point. There gave been an RFE for refactoring exceptions hanging for a long time.
Maybe it's time to refactor exception generally (with special attention to what is a runtime and what is a checked exception)

Discussion

  • Alamgir Farouk

    Alamgir Farouk - 2008-05-05
    • status: open --> closed
     
  • Alamgir Farouk

    Alamgir Farouk - 2008-05-05

    Logged In: YES
    user_id=15963
    Originator: YES

    On the instruction of Mick and Barry it is decided that we will continue to throw unchecked exceptions.

    AF

     
  • Alamgir Farouk

    Alamgir Farouk - 2008-05-13
    • labels: 1101099 -->
     

Log in to post a comment.

MongoDB Logo MongoDB