You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Josh Elser (JIRA)" <ji...@apache.org> on 2016/11/22 18:48:58 UTC

[jira] [Commented] (ACCUMULO-4525) Replace meaningless method names

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

Josh Elser commented on ACCUMULO-4525:
--------------------------------------

bq. Tests are a good place to learn how a class works and choosing brevity over clarity greater hinders the readability of a Test.

+1, well said.

> Replace meaningless method names
> --------------------------------
>
>                 Key: ACCUMULO-4525
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4525
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: Michael Miller
>            Assignee: Michael Miller
>            Priority: Minor
>             Fix For: 1.7.3, 1.8.1, 2.0.0
>
>
> There are a few abbreviated method names reused throughout the code that are meaningless to new users. Method names such as "nk", "nf" or "ane" are not helpful when trying to determine how a class works. This seems to only occur in Tests but I think it would improve readability a lot, especially for newbies. I for one think Tests are a good place to learn how a class works and choosing brevity over clarity greater hinders the readability of a Test.



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