Changes between Version 7 and Version 8 of GeoNetworkException
- Timestamp:
- 12/02/08 02:44:23 (16 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
GeoNetworkException
v7 v8 13 13 14 14 15 '''Chosen option 1(disputed)''':16 The application is well unit tested. It is important that the code remains clean. 15 '''Chosen option 2 (disputed)''': 16 In the ESA chat on December 1st 2008, option 2 was preferred by Heikki & Jose. Erik does not agree but accepts his is a minority position on this issue. 17 17 18 18 For now only one exception is implemented (org.geonetwork.domain.ebxml.exception.GeoNetworkException). We can add and diversify more of course. … … 21 21 Generally speaking, do not throw a !RuntimeException or create a subclass of !RuntimeException simply because you don't want to be bothered with specifying the exceptions your methods can throw. 22 22 23 Erik: +1. Since I like the minimalistic approach I prefer 1. Another advantage is that you have always the choice to catch the exception. In this project does it mean that the exception will be catched in the service layer. In all other layers you don't see boilerplate code, only code which makes sense.23 Erik: +1. The application is well unit tested. It is important that the code remains clean. Since I like the minimalistic approach I prefer 1. Another advantage is that you have always the choice to catch the exception. In this project does it mean that the exception will be catched in the service layer. In all other layers you don't see boilerplate code, only code which makes sense.