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)