PMD / Issues / #1361 ShortVariable and ShortMethodName configuration

#1361 ShortVariable and ShortMethodName configuration

Milestone: PMD-5.4.0

Status: closed

Owner: Andreas Dangel

Labels: rules (1)

Module: PMD

Priority: 3-Major

Type: Feature-Request

Affects version:

Ruleset / Rule: java-naming / ShortVariable and ShortMethodName

Updated: 2015-05-20

Created: 2010-05-18

Creator: Fabrice Daugan

Private: No

"ShortVariable" and "ShortMethodName" rules are not configurable as they should be.
Instead of :
//VariableDeclaratorId[string-length(@Image) < 3]

Use :
//VariableDeclaratorId[string-length(@Image) < $minimum]

With :
<properties>
<property name="minimum" description="The variable length reporting threshold" value="3"/>
</properties>

Discussion

Roman Muntyanu - 2010-10-04

Please as well consider adding configurable exclusions list because variables like "id" are meaningful, yet short, ending up with a ton of errors.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

blundell - 2015-05-19

does this still apply? ShortMethodName is not configurable from what I can fathom (through xml ruleset file)

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2015-05-20

That's true - both rules are not configurable yet.
I'll move this as a "feature-request" type issue...

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2015-05-20

Ticket moved from /p/pmd/feature-requests/546/

Can't be converted:

_milestone:
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2015-05-20

Module: --> PMD

Milestone: --> New Tickets

Priority: 5 --> 3-Major

Type: --> Feature-Request

Affects version: -->

Ruleset / Rule: --> java-naming / ShortVariable and ShortMethodName
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2015-05-20

status: open --> closed

assigned_to: Andreas Dangel

Milestone: New Tickets --> PMD-5.4.0
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2015-05-20

This will be implemented for pmd 5.4.0.

Commit: https://github.com/pmd/pmd/commit/869d4c7005e2293297a3f42c4a30530fb8e9e396

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.