[OFBIZ-9682] [FB] Package org.apache.ofbiz.service.config.model

• Type: Sub-task
• Status: Closed
• Priority: Minor
• Resolution: Implemented
• Affects Version/s: Trunk
• Fix Version/s: 17.12.01
• Component/s: ALL APPLICATIONS, ALL COMPONENTS
• Assignee: Michael Brohl
• Reporter: Julian Leichert
• Created: 07/Sep/17 10:42
• Updated: 10/Oct/17 13:47
• Resolved: 10/Oct/17 13:47

Description

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 ... }