KuiLIU created STANBOL-1469:
-------------------------------

             Summary: Inconsistent method name
                 Key: STANBOL-1469
                 URL: https://issues.apache.org/jira/browse/STANBOL-1469
             Project: Stanbol
          Issue Type: Improvement
            Reporter: KuiLIU
         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)

Reply via email to