[jira] [Commented] (OFBIZ-7272) Enforce noninstantiability to all Utility classes

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

[jira] [Commented] (OFBIZ-7272) Enforce noninstantiability to all Utility classes

Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-7272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15441292#comment-15441292 ]

Rishi Solanki commented on OFBIZ-7272:
--------------------------------------

[~arunpati]: Thanks for your help and committing all the changes. I have looked at all possible classes and seems to me all the classes we could think of have been covered now. No new classes from our side, I think we can close this ticket. In case we found any more class then we will reopen this one. Thanks!

> Enforce noninstantiability to all Utility classes
> -------------------------------------------------
>
>                 Key: OFBIZ-7272
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-7272
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: ALL COMPONENTS
>    Affects Versions: Trunk
>            Reporter: Rishi Solanki
>            Assignee: Arun Patidar
>         Attachments: OFBIZ-7272.patch
>
>
> I have discusses this in past on mailing list, that Utility, helper, worker classes which contains only static methods should not be instantiable. Also we can restrict it by adding a private constructor.
> Currently OFBiz has many such classes which should have private constructor. Identify all the *Worker.java, *.Helper.java, and *Util.java which do not have any non static method/member are eligible for having a private constructor.
> Reference why we should do this : http://www.informit.com/articles/article.aspx?p=1216151&seqNum=4



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)