[jira] [Created] (OFBIZ-10091) Unnecessary throws clauses in renderer classes?

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Created] (OFBIZ-10091) Unnecessary throws clauses in renderer classes?

Nicolas Malin (Jira)
Michael Brohl created OFBIZ-10091:
-------------------------------------

             Summary: Unnecessary throws clauses in renderer classes?
                 Key: OFBIZ-10091
                 URL: https://issues.apache.org/jira/browse/OFBIZ-10091
             Project: OFBiz
          Issue Type: Improvement
          Components: framework
    Affects Versions: Trunk
            Reporter: Michael Brohl
            Priority: Minor


During the refactoring work I found that there are several classes in org.apache.ofbiz.widget.renderer.macro which state in their method declarations that they throw TemplateException and/or IOException.

In many cases, there is either no evidence of these Exceptions or they are catched inside the methods.

One example is the MacroFormRenderer constructor which says it throws TemplateException but does not.

The method declarations in the interfaces of org.apache.ofbiz.widget.renderer all say that IOException is thrown.

It might be that I overlook something and I did not dig much deeper into the code so I first want to ask what the reasons are or if it would not be reasonable to make a difference in interfaces and implementations and only declare that a method throws a specific exception if it really does?

Thanks for your help,
regards,
Michael



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)