Discussion :

[swann7]

Dans le cadre de mise en place de qualité nous aimerions mettre en place quelques règles PMD. Ma question est donc simple :

Quelles sont, pour vous, les meilleures règles PMD, celles qui sont indispensables ?

Le but est de commencer en douceur.

[swann7]

Je vais donc me répondre à moi même en vous partageant ce que nous avons décidé.

Nous avons définis des règles incontournables :

• NcssMethodCount (50 instructions)
• NcssConstructorCount (50 instructions)
• SwitchDensity
• PreserveStackTrace
• UnusedImports
• SystemPrintln
• AvoidPrintStackTrace

Et des règles également très importantes :

• UnconditionalIfStatement
• UselessOverridingMethod
• UselessOperationOnImmutable
• MisplacedNullCheck
• SimplifyBooleanReturns
• SimplifyBooleanExpressions
• AvoidReassigningParameters
• PositionLiteralsFirstInComparisons
• UnnecessaryLocalBeforeReturn
• SingularField
• MethodWithSameNameAsEnclosingClass
• UnusedImports
• UnusedPrivateField
• UnusedLocalVariable