You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@stanbol.apache.org by "KuiLIU (JIRA)" <ji...@apache.org> on 2018/04/19 08:54:00 UTC

[jira] [Closed] (STANBOL-1469) Inconsistent method name

     [ https://issues.apache.org/jira/browse/STANBOL-1469?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

KuiLIU closed STANBOL-1469.
---------------------------
    Resolution: Incomplete

> Inconsistent method name
> ------------------------
>
>                 Key: STANBOL-1469
>                 URL: https://issues.apache.org/jira/browse/STANBOL-1469
>             Project: Stanbol
>          Issue Type: Improvement
>            Reporter: KuiLIU
>            Priority: Major
>         Attachments: rename-methods.patch
>
>
> The following method is name "select" which seems inconsistent with the method body code.
> "select" is prone to an action of selecting something.
> But the method is checking whether the status of the parameter "collection" is specified or not, which is a query.
> So, the name "isSelected" should be more clear than "select" to describe the method.
> {code:java}
> public boolean select(Collection collection) {
>         for (String feature : features) {
>             Status status = FeaturesHelper.getFeatureStatus(collection, feature);
>             if (status == Status.SPECIFIED)
>                 return true;
>         }
>         return false;
>     }
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)