You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ofbiz.apache.org by "Julian Leichert (JIRA)" <ji...@apache.org> on 2017/09/07 10:43:00 UTC
[jira] [Created] (OFBIZ-9682) [FB] Package
org.apache.ofbiz.service.config.model
Julian Leichert created OFBIZ-9682:
--------------------------------------
Summary: [FB] Package org.apache.ofbiz.service.config.model
Key: OFBIZ-9682
URL: https://issues.apache.org/jira/browse/OFBIZ-9682
Project: OFBiz
Issue Type: Sub-task
Affects Versions: Trunk
Reporter: Julian Leichert
Priority: Minor
ThreadPool.java:135, REC_CATCH_EXCEPTION
REC: Exception is caught when Exception is not thrown in new org.apache.ofbiz.service.config.model.ThreadPool(Element)
This method uses a try-catch block that catches Exception objects, but Exception is not thrown within the try block, and RuntimeException is not explicitly caught. It is a common bug pattern to say try { ... } catch (Exception e) { something } as a shorthand for catching a number of types of exception each of whose catch blocks is identical, but this construct also accidentally catches RuntimeException as well, masking potential bugs.
A better approach is to either explicitly catch the specific exceptions that are thrown, or to explicitly catch RuntimeException exception, rethrow it, and then catch all non-Runtime Exceptions, as shown below:
try {
...
} catch (RuntimeException e) {
throw e;
} catch (Exception e) {
... deal with all non-runtime exceptions ...
}
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)