imo you should add one point.
Instead of getMessage() which may return null, which in some cases may lead to useless output or worse NPEs, use toString, which returns the Throwable's classname + getLocalizedMessage if it is not null.
In this way you will always get at least the information which exception arised. |