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)