title: PMD Issues in Code - Are they critical?
body: <p>We have a number of issues that PMD has identified for us - a lot of which seems nit picky. I have a list below - may I ask the hive mind - if you had to pick the top 5 crticial rules out of below list what would they be? 
  I am trying to balance effort with criticality of what would be bad if it made to production.</p>
  
  <p>Thanks in advance!</p>
  
  <p>AssignmentInOperand</p>
  
  <p>AvoidDeeplyNestedIfStmts</p>
  
  <p>AvoidDuplicateLiterals</p>
  
  <p>AvoidInstantiatingObjectsInLoops</p>
  
  <p>ClassWithOnlyPrivateConstructorsShouldBeFinal</p>
  
  <p>CollapsibleIfStatements</p>
  
  <p>CyclomaticComplexity</p>
  
  <p>EmptyIfStmt</p>
  
  <p>ExcessiveMethodLength</p>
  
  <p>ExcessivePublicCount</p>
  
  <p>ImmutableField</p>
  
  <p>InefficientStringBuffering</p>
  
  <p>InsufficientStringBufferDeclaration</p>
  
  <p>LocalVariableCouldBeFinal</p>
  
  <p>LooseCoupling</p>
  
  <p>MethodArgumentCouldBeFinal</p>
  
  <p>NcssMethodCount</p>
  
  <p>NPathComplexity</p>
  
  <p>PackageCase</p>
  
  <p>PositionLiteralsFirstInComparisons</p>
  
  <p>SignatureDeclareThrowsException</p>
  
  <p>SingularField</p>
  
  <p>TooManyFields</p>
  
  <p>UncommentedEmptyConstructor</p>
  
  <p>UncommentedEmptyMethod</p>
  
  <p>UnusedImports</p>
  
  <p>UnusedPrivateField</p>
  
  <p>UseSingleton</p>