Hi, Problem solved. Before checking through FindBugs I didn't build the project. That was the issue. Thanks all.
On Wed, Aug 12, 2015 at 3:39 PM, Nirmal Fernando <[email protected]> wrote: > > > On Wed, Aug 12, 2015 at 3:39 PM, Nirmal Fernando <[email protected]> wrote: > >> >> >> On Wed, Aug 12, 2015 at 12:44 PM, Bimali Dayananda <[email protected]> >> wrote: >> >>> Hi, >>> >>> My code is as below. >>> >>> try{ >>> configurationElement = AXIOMUtil.stringToOM(xmlFile); >>> }catch (XMLStreamException e) { >>> log.error("Unable to load files "); >>> } >>> >>> When I analyze it through FindBugs I get the following bug report. >>> >>> "Exception is caught when the exception is not caught. >>> >>> 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 ... >>> }" >>> >>> I tried the solution they have suggested. But it is not working. >>> >>> Any idea of this? >>> >>> >> Why do you need to catch RuntimeExceptions ? >> >> It's an anti-pattern to catch generic Exception class, if no Exception is >> thrown from the method, don't catch. >> >>> Thanks. >>> >>> >>> >>> >>> >>> >>> >>> -- >>> Bimali Dayananda >>> Intern-Software Engineering >>> Mobile : +94(77) 571 6618 >>> [email protected] >>> >>> _______________________________________________ >>> Dev mailing list >>> [email protected] >>> http://wso2.org/cgi-bin/mailman/listinfo/dev >>> >>> >> >> >> -- >> >> Thanks & regards, >> Nirmal >> >> Team Lead - WSO2 Machine Learner >> Associate Technical Lead - Data Technologies Team, WSO2 Inc. >> Mobile: +94715779733 >> Blog: http://nirmalfdo.blogspot.com/ >> >> >> > > > -- > > Thanks & regards, > Nirmal > > Team Lead - WSO2 Machine Learner > Associate Technical Lead - Data Technologies Team, WSO2 Inc. > Mobile: +94715779733 > Blog: http://nirmalfdo.blogspot.com/ > > > -- Bimali Dayananda Intern-Software Engineering Mobile : +94(77) 571 6618 [email protected]
_______________________________________________ Dev mailing list [email protected] http://wso2.org/cgi-bin/mailman/listinfo/dev
