There are two things in Droids that irk me a little

(1) There are lots of places where output is printed to stdout bypassing
the logging framework. I personally think this is suboptimal. One should
be able to add additional details such as timestamps, execution thread
context by configuring the log appender rather than programmatically in
the Droids code. IMHO well-behaved applications ought not write directly
to the console, at the very least in order to to play nicely with
managed environments such as component containers.

(2) Retrhrowing exceptions as plain RuntimeException is nasty. I do not
intend to provoke another instance of checked vs unchecked exceptions
discussion. There are pros and cons to both approaches. However, one
ought not rethrown exceptions as plain RuntimeException. We should
derive a subclass of RuntimeException, call it DroidsRuntimeException or
re-throw exceptions as a typed exception that can be differentiated from
other runtime exceptions.

Oleg

Reply via email to