| 1 | <?xml version="1.0"?> |
|---|
| 2 | <ruleset name="jcoderzruleset"> |
|---|
| 3 | <!-- using PMD 3.5 --> |
|---|
| 4 | <description> |
|---|
| 5 | This ruleset checks the jCoderZ Java code for bad stuff |
|---|
| 6 | </description> |
|---|
| 7 | <!-- priorities are maped to JcReport priorities as follows: |
|---|
| 8 | 5: INFO |
|---|
| 9 | 4: CODE_STYLE |
|---|
| 10 | 3: DESIGN |
|---|
| 11 | 2: WARNING |
|---|
| 12 | 1: ERROR |
|---|
| 13 | --> |
|---|
| 14 | <!-- =================================================================== --> |
|---|
| 15 | <!--basic--> |
|---|
| 16 | <!-- =================================================================== --> |
|---|
| 17 | <rule ref="rulesets/basic.xml/AvoidDecimalLiteralsInBigDecimalConstructor"> |
|---|
| 18 | <!-- CHECKME: 3.5 --> |
|---|
| 19 | <priority>4</priority> |
|---|
| 20 | </rule> |
|---|
| 21 | <rule ref="rulesets/basic.xml/AvoidThreadGroup"> |
|---|
| 22 | <!-- CHECKME: 3.6 --> |
|---|
| 23 | <priority>4</priority> |
|---|
| 24 | </rule> |
|---|
| 25 | <rule ref="rulesets/basic.xml/AvoidUsingOctalValues"> |
|---|
| 26 | <!-- CHECKME: 3.9 --> |
|---|
| 27 | <priority>4</priority> |
|---|
| 28 | </rule> |
|---|
| 29 | <rule ref="rulesets/basic.xml/BigIntegerInstantiation"> |
|---|
| 30 | <!-- CHECKME: 3.9 --> |
|---|
| 31 | <priority>4</priority> |
|---|
| 32 | </rule> |
|---|
| 33 | <!-- <rule ref="rulesets/basic.xml/BooleanInstantiation"/> --> |
|---|
| 34 | <rule ref="rulesets/basic.xml/ClassCastExceptionWithToArray"> |
|---|
| 35 | <priority>1</priority> |
|---|
| 36 | </rule> |
|---|
| 37 | <rule ref="rulesets/basic.xml/BrokenNullCheck"> |
|---|
| 38 | <!-- CHECKME: 3.8 --> |
|---|
| 39 | <priority>2</priority> |
|---|
| 40 | </rule> |
|---|
| 41 | <!-- <rule ref="rulesets/basic.xml/CollapsibleIfStatements"/> --> |
|---|
| 42 | <!-- <rule ref="rulesets/basic.xml/DoubleCheckedLocking"/> --> |
|---|
| 43 | <!-- <rule ref="rulesets/basic.xml/EmptyCatchBlock"/> --> |
|---|
| 44 | <!-- <rule ref="rulesets/basic.xml/EmptyFinallyBlock"/> --> |
|---|
| 45 | <!-- <rule ref="rulesets/basic.xml/EmptyIfStmt"/> --> |
|---|
| 46 | <!-- <rule ref="rulesets/basic.xml/EmptyStatementNotInLoop"/> --> |
|---|
| 47 | <!-- <rule ref="rulesets/basic.xml/EmptyStaticInitializer"/> --> |
|---|
| 48 | <!-- <rule ref="rulesets/basic.xml/EmptySwitchStatements"/> --> |
|---|
| 49 | <!-- <rule ref="rulesets/basic.xml/EmptySynchronizedBlock"/> --> |
|---|
| 50 | <!-- <rule ref="rulesets/basic.xml/EmptyTryBlock"/> --> |
|---|
| 51 | <!-- <rule ref="rulesets/basic.xml/EmptyWhileStmt"/> --> |
|---|
| 52 | <rule ref="rulesets/basic.xml/ForLoopShouldBeWhileLoop"> |
|---|
| 53 | <priority>4</priority> |
|---|
| 54 | </rule> |
|---|
| 55 | <rule ref="rulesets/basic.xml/JumbledIncrementer"> |
|---|
| 56 | <priority>4</priority> |
|---|
| 57 | </rule> |
|---|
| 58 | <rule ref="rulesets/basic.xml/MisplacedNullCheck"> |
|---|
| 59 | <priority>2</priority> |
|---|
| 60 | </rule> |
|---|
| 61 | <!-- <rule ref="rulesets/basic.xml/OverrideBothEqualsAndHashcode"/> Detected by Findbugs --> |
|---|
| 62 | <rule ref="rulesets/basic.xml/ReturnFromFinallyBlock"> |
|---|
| 63 | <priority>3</priority> |
|---|
| 64 | </rule> |
|---|
| 65 | <rule ref="rulesets/basic.xml/UnconditionalIfStatement"> |
|---|
| 66 | <priority>3</priority> |
|---|
| 67 | </rule> |
|---|
| 68 | <rule ref="rulesets/basic.xml/UnnecessaryConversionTemporary"> |
|---|
| 69 | <priority>3</priority> |
|---|
| 70 | </rule> |
|---|
| 71 | <rule ref="rulesets/basic.xml/UnnecessaryFinalModifier"> |
|---|
| 72 | <priority>4</priority> |
|---|
| 73 | </rule> |
|---|
| 74 | <rule ref="rulesets/basic.xml/UnnecessaryReturn"> |
|---|
| 75 | <priority>4</priority> |
|---|
| 76 | </rule> |
|---|
| 77 | <rule ref="rulesets/basic.xml/UnusedNullCheckInEquals"> |
|---|
| 78 | <!-- CHECKME: 3.5 --> |
|---|
| 79 | <priority>4</priority> |
|---|
| 80 | </rule> |
|---|
| 81 | <rule ref="rulesets/basic.xml/UselessOperationOnImmutable"> |
|---|
| 82 | <!-- CHECKME: 3.5 --> |
|---|
| 83 | <priority>2</priority> |
|---|
| 84 | </rule> |
|---|
| 85 | <rule ref="rulesets/basic.xml/UselessOverridingMethod"> |
|---|
| 86 | <!-- CHECKME: 3.5 --> |
|---|
| 87 | <priority>3</priority> |
|---|
| 88 | </rule> |
|---|
| 89 | <!-- =================================================================== --> |
|---|
| 90 | <!--braces--> |
|---|
| 91 | <!-- =================================================================== --> |
|---|
| 92 | <rule ref="rulesets/braces.xml/ForLoopsMustUseBraces"> |
|---|
| 93 | <priority>4</priority> |
|---|
| 94 | </rule> |
|---|
| 95 | <rule ref="rulesets/braces.xml/IfElseStmtsMustUseBraces"> |
|---|
| 96 | <priority>4</priority> |
|---|
| 97 | </rule> |
|---|
| 98 | <rule ref="rulesets/braces.xml/IfStmtsMustUseBraces"> |
|---|
| 99 | <priority>4</priority> |
|---|
| 100 | </rule> |
|---|
| 101 | <!-- <rule ref="rulesets/braces.xml/WhileLoopsMustUseBraces"/> --> |
|---|
| 102 | <!-- =================================================================== --> |
|---|
| 103 | <!--clone--> |
|---|
| 104 | <!-- =================================================================== --> |
|---|
| 105 | <rule ref="rulesets/clone.xml/CloneMethodMustImplementCloneable"> |
|---|
| 106 | <priority>2</priority> |
|---|
| 107 | </rule> |
|---|
| 108 | <rule ref="rulesets/clone.xml/CloneThrowsCloneNotSupportedException"> |
|---|
| 109 | <priority>2</priority> |
|---|
| 110 | </rule> |
|---|
| 111 | <rule ref="rulesets/clone.xml/ProperCloneImplementation"> |
|---|
| 112 | <priority>1</priority> |
|---|
| 113 | </rule> |
|---|
| 114 | <!-- =================================================================== --> |
|---|
| 115 | <!--codesize--> |
|---|
| 116 | <!-- =================================================================== --> |
|---|
| 117 | <!-- <rule ref="rulesets/codesize.xml/CyclomaticComplexity"/> --> |
|---|
| 118 | <!-- <rule ref="rulesets/codesize.xml/ExcessiveClassLength"/> --> |
|---|
| 119 | <!-- <rule ref="rulesets/codesize.xml/ExcessiveMethodLength"/>--> |
|---|
| 120 | <!-- <rule ref="rulesets/codesize.xml/ExcessiveParameterList"/> --> |
|---|
| 121 | <rule ref="rulesets/codesize.xml/ExcessivePublicCount"> |
|---|
| 122 | <priority>4</priority> |
|---|
| 123 | </rule> |
|---|
| 124 | <rule ref="rulesets/codesize.xml/NcssConstructorCount"> |
|---|
| 125 | <!-- CHECKME: 3.9 --> |
|---|
| 126 | <priority>4</priority> |
|---|
| 127 | </rule> |
|---|
| 128 | <rule ref="rulesets/codesize.xml/NcssMethodCount"> |
|---|
| 129 | <!-- CHECKME: 3.9 --> |
|---|
| 130 | <priority>4</priority> |
|---|
| 131 | </rule> |
|---|
| 132 | <rule ref="rulesets/codesize.xml/NcssTypeCount"> |
|---|
| 133 | <!-- CHECKME: 3.9 --> |
|---|
| 134 | <priority>4</priority> |
|---|
| 135 | </rule> |
|---|
| 136 | <rule ref="rulesets/codesize.xml/NPathComplexity"> |
|---|
| 137 | <!-- CHECKME: 3.9 --> |
|---|
| 138 | <priority>4</priority> |
|---|
| 139 | </rule> |
|---|
| 140 | <!-- <rule ref="rulesets/codesize.xml/TooManyFields"/> --> |
|---|
| 141 | <!-- =================================================================== --> |
|---|
| 142 | <!--controversial--> |
|---|
| 143 | <!-- =================================================================== --> |
|---|
| 144 | <!-- <rule ref="rulesets/controversial.xml/AssignmentInOperand"/> --> |
|---|
| 145 | <!-- <rule ref="rulesets/controversial.xml/AtLeastOneConstructor"/> --> |
|---|
| 146 | <rule ref="rulesets/controversial.xml/BooleanInversion"> |
|---|
| 147 | <!-- Checkme 3.5 --> |
|---|
| 148 | <priority>4</priority> |
|---|
| 149 | </rule> |
|---|
| 150 | <!-- <rule ref="rulesets/controversial.xml/CallSuperInConstructor"/> --> |
|---|
| 151 | <!-- <rule ref="rulesets/controversial.xml/DontImportSun"/> --> |
|---|
| 152 | <!-- <rule ref="rulesets/controversial.xml/DefaultPackage"/> --> |
|---|
| 153 | <!-- <rule ref="rulesets/controversial.xml/NullAssignment"/> --> |
|---|
| 154 | <!-- <rule ref="rulesets/controversial.xml/OnlyOneReturn"/> --> |
|---|
| 155 | <!-- <rule ref="rulesets/controversial.xml/SingularField"/> Good check but many false positives--> |
|---|
| 156 | <rule ref="rulesets/controversial.xml/SuspiciousOctalEscape"> |
|---|
| 157 | <priority>4</priority> |
|---|
| 158 | </rule> |
|---|
| 159 | <rule ref="rulesets/controversial.xml/UnnecessaryConstructor"> |
|---|
| 160 | <priority>4</priority> |
|---|
| 161 | </rule> |
|---|
| 162 | <rule ref="rulesets/controversial.xml/UnnecessaryParentheses"> |
|---|
| 163 | <priority>4</priority> |
|---|
| 164 | </rule> |
|---|
| 165 | <rule ref="rulesets/controversial.xml/UnusedModifier"> |
|---|
| 166 | <priority>4</priority> |
|---|
| 167 | </rule> |
|---|
| 168 | <!-- =================================================================== --> |
|---|
| 169 | <!--coupling--> |
|---|
| 170 | <!-- =================================================================== --> |
|---|
| 171 | <rule ref="rulesets/coupling.xml/CouplingBetweenObjects"> |
|---|
| 172 | <priority>3</priority> |
|---|
| 173 | </rule> |
|---|
| 174 | |
|---|
| 175 | <!-- <rule ref="rulesets/coupling.xml/ExcessiveImports"/> --> |
|---|
| 176 | <!-- <rule ref="rulesets/coupling.xml/LooseCoupling"/> --> |
|---|
| 177 | <!-- =================================================================== --> |
|---|
| 178 | <!--daa--> |
|---|
| 179 | <!-- =================================================================== --> |
|---|
| 180 | <!-- <rule ref="rulesets/daa.xml/FindDataFlowAnomalies"/> --> |
|---|
| 181 | <!-- =================================================================== --> |
|---|
| 182 | <!--design--> |
|---|
| 183 | <!-- =================================================================== --> |
|---|
| 184 | <!-- <rule ref="rulesets/design.xml/AccessorClassGeneration"/> --> |
|---|
| 185 | <rule ref="rulesets/design.xml/AssignmentToNonFinalStatic"> |
|---|
| 186 | <!-- CHECKME: 3.0 --> |
|---|
| 187 | <priority>3</priority> |
|---|
| 188 | </rule> |
|---|
| 189 | <!-- Covered by Checkstyle. (CS_INTERFACE_TYPE) |
|---|
| 190 | <rule ref="rulesets/design.xml/AvoidConstantsInterface"> |
|---|
| 191 | <priority>3</priority> |
|---|
| 192 | </rule> --> |
|---|
| 193 | <!-- <rule ref="rulesets/design.xml/AvoidDeeplyNestedIfStmts"/> --> |
|---|
| 194 | <rule ref="rulesets/design.xml/AvoidInstanceofChecksInCatchClause"> |
|---|
| 195 | <priority>4</priority> |
|---|
| 196 | </rule> |
|---|
| 197 | <rule ref="rulesets/design.xml/AvoidProtectedFieldInFinalClass"> |
|---|
| 198 | <priority>4</priority> |
|---|
| 199 | </rule> |
|---|
| 200 | <rule ref="rulesets/design.xml/AvoidReassigningParameters"> |
|---|
| 201 | <priority>4</priority> |
|---|
| 202 | </rule> |
|---|
| 203 | <!-- <rule ref="rulesets/design.xml/AvoidSynchronizedAtMethodLevel"/> --> |
|---|
| 204 | <rule ref="rulesets/design.xml/BadComparison"> |
|---|
| 205 | <priority>2</priority> |
|---|
| 206 | </rule> |
|---|
| 207 | <!-- <rule ref="rulesets/design.xml/CloseResource"/> --> |
|---|
| 208 | <!-- <rule ref="rulesets/design.xml/ConfusingTernary"/> --> |
|---|
| 209 | <rule ref="rulesets/design.xml/ConstructorCallsOverridableMethod"> |
|---|
| 210 | <priority>3</priority> |
|---|
| 211 | </rule> |
|---|
| 212 | <rule ref="rulesets/design.xml/DefaultLabelNotLastInSwitchStmt"> |
|---|
| 213 | <priority>4</priority> |
|---|
| 214 | </rule> |
|---|
| 215 | <!-- <rule ref="rulesets/design.xml/EqualsNull"/> --> |
|---|
| 216 | <rule ref="rulesets/design.xml/FinalFieldCouldBeStatic"> |
|---|
| 217 | <priority>3</priority> |
|---|
| 218 | </rule> |
|---|
| 219 | <rule ref="rulesets/design.xml/IdempotentOperations"> <!-- test --> |
|---|
| 220 | <priority>4</priority> |
|---|
| 221 | </rule> |
|---|
| 222 | <rule ref="rulesets/design.xml/ImmutableField"> |
|---|
| 223 | <priority>4</priority> |
|---|
| 224 | </rule> |
|---|
| 225 | <rule ref="rulesets/design.xml/InstantiationToGetClass"> |
|---|
| 226 | <priority>3</priority> |
|---|
| 227 | </rule> |
|---|
| 228 | <rule ref="rulesets/design.xml/MissingBreakInSwitch"> |
|---|
| 229 | <priority>4</priority> |
|---|
| 230 | </rule> |
|---|
| 231 | |
|---|
| 232 | <!-- <rule ref="rulesets/design.xml/MissingStaticMethodInNonInstantiatableClass"/> --> |
|---|
| 233 | <rule ref="rulesets/design.xml/NonCaseLabelInSwitchStatement"> |
|---|
| 234 | <priority>4</priority> |
|---|
| 235 | </rule> |
|---|
| 236 | <rule ref="rulesets/design.xml/NonStaticInitializer"> |
|---|
| 237 | <priority>4</priority> |
|---|
| 238 | </rule> |
|---|
| 239 | <rule ref="rulesets/design.xml/NonThreadSafeSingleton"> |
|---|
| 240 | <!-- CHECKME: 3.5 --> |
|---|
| 241 | <priority>3</priority> |
|---|
| 242 | </rule> |
|---|
| 243 | <rule ref="rulesets/design.xml/PositionLiteralsFirstInComparisons"> |
|---|
| 244 | <!-- CHECKME: 3.5 --> |
|---|
| 245 | <priority>4</priority> |
|---|
| 246 | </rule> |
|---|
| 247 | <!-- <rule ref="rulesets/design.xml/PreserveStackTrace"/> Great checker but many false positives 3.7 --> |
|---|
| 248 | <rule ref="rulesets/design.xml/OptimizableToArrayCall"> |
|---|
| 249 | <priority>4</priority> |
|---|
| 250 | </rule> |
|---|
| 251 | <rule ref="rulesets/design.xml/SimpleDateFormatNeedsLocale"> |
|---|
| 252 | <priority>4</priority> |
|---|
| 253 | </rule> |
|---|
| 254 | <rule ref="rulesets/design.xml/SimplifyBooleanExpressions"> |
|---|
| 255 | <priority>4</priority> |
|---|
| 256 | </rule> |
|---|
| 257 | <!-- <rule ref="rulesets/design.xml/SimplifyConditional"/> --> |
|---|
| 258 | <!-- <rule ref="rulesets/design.xml/SimplifyBooleanReturns"/> --> |
|---|
| 259 | <rule ref="rulesets/design.xml/SwitchDensity"> |
|---|
| 260 | <priority>4</priority> |
|---|
| 261 | </rule> |
|---|
| 262 | <!-- <rule ref="rulesets/design.xml/SwitchStmtsShouldHaveDefault"/> --> |
|---|
| 263 | <rule ref="rulesets/design.xml/UncommentedEmptyMethod"> |
|---|
| 264 | <!-- CHECKME: 3.5 --> |
|---|
| 265 | <priority>4</priority> |
|---|
| 266 | </rule> |
|---|
| 267 | <rule ref="rulesets/design.xml/UncommentedEmptyConstructor"> |
|---|
| 268 | <!-- CHECKME: 3.5 --> |
|---|
| 269 | <priority>4</priority> |
|---|
| 270 | </rule> |
|---|
| 271 | <rule ref="rulesets/design.xml/UnnecessaryLocalBeforeReturn"> |
|---|
| 272 | <!-- CHECKME: 3.5 --> |
|---|
| 273 | <priority>4</priority> |
|---|
| 274 | </rule> |
|---|
| 275 | <rule ref="rulesets/design.xml/UnsynchronizedStaticDateFormatter"> |
|---|
| 276 | <!-- CHECKME: 3.6 --> |
|---|
| 277 | <priority>2</priority> |
|---|
| 278 | </rule> |
|---|
| 279 | <rule ref="rulesets/design.xml/UseCollectionIsEmpty"> |
|---|
| 280 | <!-- CHECKME: 3.9 --> |
|---|
| 281 | <priority>3</priority> |
|---|
| 282 | </rule> |
|---|
| 283 | <rule ref="rulesets/design.xml/UseLocaleWithCaseConversions"> |
|---|
| 284 | <priority>4</priority> |
|---|
| 285 | </rule> |
|---|
| 286 | <rule ref="rulesets/design.xml/UseNotifyAllInsteadOfNotify"> |
|---|
| 287 | <priority>3</priority> |
|---|
| 288 | </rule> |
|---|
| 289 | <!-- <rule ref="rulesets/design.xml/UseSingleton"/> --> |
|---|
| 290 | |
|---|
| 291 | <!-- =================================================================== --> |
|---|
| 292 | <!--favorites--> |
|---|
| 293 | <!-- =================================================================== --> |
|---|
| 294 | <!-- =================================================================== --> |
|---|
| 295 | <!--finalizers--> |
|---|
| 296 | <!-- =================================================================== --> |
|---|
| 297 | <rule ref="rulesets/finalizers.xml/AvoidCallingFinalize"> |
|---|
| 298 | <priority>3</priority> |
|---|
| 299 | </rule> |
|---|
| 300 | <rule ref="rulesets/finalizers.xml/EmptyFinalizer"> |
|---|
| 301 | <priority>4</priority> |
|---|
| 302 | </rule> |
|---|
| 303 | <rule ref="rulesets/finalizers.xml/FinalizeDoesNotCallSuperFinalize"> |
|---|
| 304 | <priority>3</priority> |
|---|
| 305 | </rule> |
|---|
| 306 | <rule ref="rulesets/finalizers.xml/FinalizeOnlyCallsSuperFinalize"> |
|---|
| 307 | <priority>4</priority> |
|---|
| 308 | </rule> |
|---|
| 309 | <rule ref="rulesets/finalizers.xml/FinalizeOverloaded"> |
|---|
| 310 | <priority>3</priority> |
|---|
| 311 | </rule> |
|---|
| 312 | <rule ref="rulesets/finalizers.xml/FinalizeShouldBeProtected"> |
|---|
| 313 | <priority>4</priority> |
|---|
| 314 | </rule> |
|---|
| 315 | |
|---|
| 316 | <!-- =================================================================== --> |
|---|
| 317 | <!--imports--> |
|---|
| 318 | <!-- =================================================================== --> |
|---|
| 319 | <!-- <rule ref="rulesets/imports.xml/DontImportJavaLang"/> --> |
|---|
| 320 | <rule ref="rulesets/imports.xml/DuplicateImports"> |
|---|
| 321 | <priority>4</priority> |
|---|
| 322 | </rule> |
|---|
| 323 | <!-- <rule ref="rulesets/imports.xml/ImportFromSamePackage"/> --> |
|---|
| 324 | <!-- <rule ref="rulesets/imports.xml/UnusedImports"/> --> |
|---|
| 325 | <!-- =================================================================== --> |
|---|
| 326 | <!--javabeans--> |
|---|
| 327 | <!-- =================================================================== --> |
|---|
| 328 | <!-- <rule ref="rulesets/javabeans.xml/BeanMembersShouldSerialize"/> --> |
|---|
| 329 | <!-- <rule ref="rulesets/javabeans.xml/MissingSerialVersionUID"/> --> |
|---|
| 330 | <!-- =================================================================== --> |
|---|
| 331 | <!--junit--> |
|---|
| 332 | <!-- =================================================================== --> |
|---|
| 333 | <rule ref="rulesets/junit.xml/JUnitAssertionsShouldIncludeMessage"> |
|---|
| 334 | <priority>4</priority> |
|---|
| 335 | </rule> |
|---|
| 336 | <rule ref="rulesets/junit.xml/JUnitSpelling"> |
|---|
| 337 | <priority>4</priority> |
|---|
| 338 | </rule> |
|---|
| 339 | <rule ref="rulesets/junit.xml/JUnitStaticSuite"> |
|---|
| 340 | <priority>4</priority> |
|---|
| 341 | </rule> |
|---|
| 342 | <!-- many false positives <rule ref="rulesets/junit.xml/JUnitTestsShouldIncludeAssert"/> --> |
|---|
| 343 | <rule ref="rulesets/junit.xml/SimplifyBooleanAssertion"> |
|---|
| 344 | <!-- CHECKME: 3.6 --> |
|---|
| 345 | <priority>4</priority> |
|---|
| 346 | </rule> |
|---|
| 347 | <!-- <rule ref="rulesets/junit.xml/TestClassWithoutTestCases"/> --> |
|---|
| 348 | <rule ref="rulesets/junit.xml/UnnecessaryBooleanAssertion"> |
|---|
| 349 | <priority>4</priority> |
|---|
| 350 | </rule> |
|---|
| 351 | <rule ref="rulesets/junit.xml/UseAssertEqualsInsteadOfAssertTrue"> |
|---|
| 352 | <priority>4</priority> |
|---|
| 353 | </rule> |
|---|
| 354 | <rule ref="rulesets/junit.xml/UseAssertNullInsteadOfAssertTrue"> |
|---|
| 355 | <!-- CHECKME: 3.5 --> |
|---|
| 356 | <priority>4</priority> |
|---|
| 357 | </rule> |
|---|
| 358 | <rule ref="rulesets/junit.xml/UseAssertSameInsteadOfAssertTrue"> |
|---|
| 359 | <priority>4</priority> |
|---|
| 360 | </rule> |
|---|
| 361 | |
|---|
| 362 | <!-- =================================================================== --> |
|---|
| 363 | <!--logging-java--> |
|---|
| 364 | <!-- =================================================================== --> |
|---|
| 365 | <rule ref="rulesets/logging-java.xml/AvoidPrintStackTrace"> |
|---|
| 366 | <!-- CHECKME: 3.5 --> |
|---|
| 367 | <priority>3</priority> |
|---|
| 368 | </rule> |
|---|
| 369 | <rule ref="rulesets/logging-java.xml/LoggerIsNotStaticFinal"> |
|---|
| 370 | <priority>3</priority> |
|---|
| 371 | </rule> |
|---|
| 372 | <rule ref="rulesets/logging-java.xml/MoreThanOneLogger"> |
|---|
| 373 | <priority>3</priority> |
|---|
| 374 | </rule> |
|---|
| 375 | <rule ref="rulesets/logging-java.xml/SystemPrintln"> |
|---|
| 376 | <priority>3</priority> |
|---|
| 377 | </rule> |
|---|
| 378 | <!-- =================================================================== --> |
|---|
| 379 | <!-- Migration ruleset --> |
|---|
| 380 | <!-- =================================================================== --> |
|---|
| 381 | <rule ref="rulesets/migrating.xml/AvoidAssertAsIdentifier"> |
|---|
| 382 | <!-- CHECKME: 3.5 --> |
|---|
| 383 | <priority>3</priority> |
|---|
| 384 | </rule> |
|---|
| 385 | <rule ref="rulesets/migrating.xml/AvoidEnumAsIdentifier"> |
|---|
| 386 | <!-- CHECKME: 3.5 --> |
|---|
| 387 | <priority>3</priority> |
|---|
| 388 | </rule> |
|---|
| 389 | <!-- <rule ref="rulesets/migrating.xml/IntegerInstantiation"/> Disabled because it enforces jdk 1.5 code CHECKME: 3.5 --> |
|---|
| 390 | <rule ref="rulesets/migrating.xml/ReplaceEnumerationWithIterator"> |
|---|
| 391 | <!-- CHECKME: 3.5 --> |
|---|
| 392 | <priority>4</priority> |
|---|
| 393 | </rule> |
|---|
| 394 | <rule ref="rulesets/migrating.xml/ReplaceHashtableWithMap"> |
|---|
| 395 | <!-- CHECKME: 3.5 --> |
|---|
| 396 | <priority>4</priority> |
|---|
| 397 | </rule> |
|---|
| 398 | <rule ref="rulesets/migrating.xml/ReplaceVectorWithList"> |
|---|
| 399 | <!-- CHECKME: 3.5 --> |
|---|
| 400 | <priority>4</priority> |
|---|
| 401 | </rule> |
|---|
| 402 | <!-- =================================================================== --> |
|---|
| 403 | <!--naming--> |
|---|
| 404 | <!-- =================================================================== --> |
|---|
| 405 | <!-- <rule ref="rulesets/naming.xml/AbstractNaming"/> --> |
|---|
| 406 | <rule ref="rulesets/naming.xml/AvoidDollarSigns"> |
|---|
| 407 | <priority>4</priority> |
|---|
| 408 | </rule> |
|---|
| 409 | <!-- <rule ref="rulesets/naming.xml/AvoidFieldNameMatchingMethodName"/> --> |
|---|
| 410 | <!-- <rule ref="rulesets/naming.xml/AvoidFieldNameMatchingTypeName"/> --> |
|---|
| 411 | <!-- <rule ref="rulesets/naming.xml/ClassNamingConventions"/> --> |
|---|
| 412 | <!-- <rule ref="rulesets/naming.xml/LongVariable"/> --> |
|---|
| 413 | <!-- <rule ref="rulesets/naming.xml/MethodNamingConventions"/> --> |
|---|
| 414 | <!-- <rule ref="rulesets/naming.xml/MethodWithSameNameAsEnclosingClass"/> --> |
|---|
| 415 | <rule ref="rulesets/naming.xml/MisleadingVariableName"> |
|---|
| 416 | <!-- CHECKME: 3.5 --> |
|---|
| 417 | <priority>4</priority> |
|---|
| 418 | </rule> |
|---|
| 419 | <rule ref="rulesets/naming.xml/NoPackage"> |
|---|
| 420 | <!-- CHECKME: 3.5 --> |
|---|
| 421 | <priority>3</priority> |
|---|
| 422 | </rule> |
|---|
| 423 | <rule ref="rulesets/naming.xml/PackageCase"> |
|---|
| 424 | <!-- CHECKME: 3.5 --> |
|---|
| 425 | <priority>4</priority> |
|---|
| 426 | </rule> |
|---|
| 427 | <!-- <rule ref="rulesets/naming.xml/ShortMethodName"/> --> |
|---|
| 428 | <!-- <rule ref="rulesets/naming.xml/ShortVariable"/> --> |
|---|
| 429 | <rule ref="rulesets/naming.xml/SuspiciousConstantFieldName"> |
|---|
| 430 | <priority>4</priority> |
|---|
| 431 | </rule> |
|---|
| 432 | <!-- <rule ref="rulesets/naming.xml/SuspiciousEqualsMethodName"/> --> |
|---|
| 433 | <rule ref="rulesets/naming.xml/SuspiciousHashcodeMethodName"> |
|---|
| 434 | <priority>4</priority> |
|---|
| 435 | </rule> |
|---|
| 436 | <!-- <rule ref="rulesets/naming.xml/VariableNamingConventions"/> --> |
|---|
| 437 | <!-- =================================================================== --> |
|---|
| 438 | <!--newrules--> |
|---|
| 439 | <!-- =================================================================== --> |
|---|
| 440 | <!-- =================================================================== --> |
|---|
| 441 | <!--Optimization Rules --> |
|---|
| 442 | <!-- =================================================================== --> |
|---|
| 443 | <rule ref="rulesets/optimizations.xml/AvoidArrayLoops"> |
|---|
| 444 | <!-- CHECKME: 3.5 --> |
|---|
| 445 | <priority>4</priority> |
|---|
| 446 | </rule> |
|---|
| 447 | <!-- <rule ref="rulesets/optimizations.xml/AvoidInstantiatingObjectsInLoops"/> --> |
|---|
| 448 | <!-- <rule ref="rulesets/optimizations.xml/LocalVariableCouldBeFinal"/> --> |
|---|
| 449 | <!-- <rule ref="rulesets/optimizations.xml/MethodArgumentCouldBeFinal"/> --> |
|---|
| 450 | <rule ref="rulesets/optimizations.xml/SimplifyStartsWith"> |
|---|
| 451 | <priority>4</priority> |
|---|
| 452 | </rule> |
|---|
| 453 | <rule ref="rulesets/optimizations.xml/UnnecessaryWrapperObjectCreation"> |
|---|
| 454 | <!-- CHECKME: 3.8 --> |
|---|
| 455 | <priority>4</priority> |
|---|
| 456 | </rule> |
|---|
| 457 | <rule ref="rulesets/optimizations.xml/UseArraysAsList"> |
|---|
| 458 | <!-- Checkme: 3.5 --> |
|---|
| 459 | <priority>4</priority> |
|---|
| 460 | </rule> |
|---|
| 461 | <rule ref="rulesets/optimizations.xml/UseArrayListInsteadOfVector"> |
|---|
| 462 | <priority>4</priority> |
|---|
| 463 | </rule> |
|---|
| 464 | <rule ref="rulesets/optimizations.xml/UseStringBufferForStringAppends"> |
|---|
| 465 | <priority>4</priority> |
|---|
| 466 | </rule> |
|---|
| 467 | <!-- =================================================================== --> |
|---|
| 468 | <!--scratchpad--> |
|---|
| 469 | <!-- =================================================================== --> |
|---|
| 470 | <!-- =================================================================== --> |
|---|
| 471 | <!--Security Code Guidelines--> |
|---|
| 472 | <!-- =================================================================== --> |
|---|
| 473 | <!-- <rule ref="rulesets/sunsecure.xml/ArrayIsStoredDirectly"/> --> |
|---|
| 474 | <rule ref="rulesets/sunsecure.xml/MethodReturnsInternalArray"> |
|---|
| 475 | <priority>3</priority> |
|---|
| 476 | </rule> |
|---|
| 477 | <!-- =================================================================== --> |
|---|
| 478 | <!--strictexception--> |
|---|
| 479 | <!-- =================================================================== --> |
|---|
| 480 | <rule ref="rulesets/strictexception.xml/AvoidCatchingNPE"> |
|---|
| 481 | <priority>3</priority> |
|---|
| 482 | </rule> |
|---|
| 483 | <!-- <rule ref="rulesets/strictexception.xml/AvoidCatchingThrowable"/> --> |
|---|
| 484 | <!-- <rule ref="rulesets/strictexception.xml/AvoidRethrowingException"> |
|---|
| 485 | <priority>3</priority> |
|---|
| 486 | </rule> --> |
|---|
| 487 | <!-- CHECKME: nice checker but we don't want the RuntimeException |
|---|
| 488 | <rule ref="rulesets/strictexception.xml/AvoidThrowingRawExceptionTypes "/> |
|---|
| 489 | <rule ref="rulesets/strictexception.xml/AvoidThrowingNullPointerException"/> --> |
|---|
| 490 | <!-- <rule ref="rulesets/strictexception.xml/ExceptionAsFlowControl"/> --> |
|---|
| 491 | <rule ref="rulesets/strictexception.xml/SignatureDeclareThrowsException"> |
|---|
| 492 | <priority>3</priority> |
|---|
| 493 | </rule> |
|---|
| 494 | <!-- =================================================================== --> |
|---|
| 495 | <!--strings--> |
|---|
| 496 | <!-- =================================================================== --> |
|---|
| 497 | <rule ref="rulesets/strings.xml/AppendCharacterWithChar"> |
|---|
| 498 | <!-- CHECKME: 3.5 --> |
|---|
| 499 | <priority>5</priority> |
|---|
| 500 | </rule> |
|---|
| 501 | <rule ref="rulesets/strings.xml/AvoidDuplicateLiterals"> |
|---|
| 502 | <priority>5</priority> |
|---|
| 503 | </rule> |
|---|
| 504 | <rule ref="rulesets/strings.xml/ConsecutiveLiteralAppends"> |
|---|
| 505 | <!-- CHECKME: 3.5 --> |
|---|
| 506 | <priority>5</priority> |
|---|
| 507 | </rule> |
|---|
| 508 | <rule ref="rulesets/strings.xml/StringBufferInstantiationWithChar"> |
|---|
| 509 | <!-- CHECKME: 3.9 --> |
|---|
| 510 | <priority>5</priority> |
|---|
| 511 | </rule> |
|---|
| 512 | <rule ref="rulesets/strings.xml/StringInstantiation"> |
|---|
| 513 | <priority>5</priority> |
|---|
| 514 | </rule> |
|---|
| 515 | <!-- <rule ref="rulesets/strings.xml/StringToString"/> --> |
|---|
| 516 | <rule ref="rulesets/strings.xml/InefficientEmptyStringCheck"> |
|---|
| 517 | <!-- CHECKME: 3.6 --> |
|---|
| 518 | <priority>5</priority> |
|---|
| 519 | </rule> |
|---|
| 520 | <rule ref="rulesets/strings.xml/InefficientStringBuffering"> |
|---|
| 521 | <priority>5</priority> |
|---|
| 522 | </rule> |
|---|
| 523 | <rule ref="rulesets/strings.xml/InsufficientStringBufferDeclaration"> |
|---|
| 524 | <!-- CHECKME: 3.6 --> |
|---|
| 525 | <priority>5</priority> |
|---|
| 526 | </rule> |
|---|
| 527 | <rule ref="rulesets/strings.xml/UnnecessaryCaseChange"> |
|---|
| 528 | <!-- CHECKME: 3.5 --> |
|---|
| 529 | <priority>5</priority> |
|---|
| 530 | </rule> |
|---|
| 531 | <rule ref="rulesets/strings.xml/UseIndexOfChar"> |
|---|
| 532 | <!-- CHECKME: 3.5 --> |
|---|
| 533 | <priority>5</priority> |
|---|
| 534 | </rule> |
|---|
| 535 | <!-- <rule ref="rulesets/strings.xml/UselessStringValueOf"> |
|---|
| 536 | <priority>4</priority> |
|---|
| 537 | </rule> --> |
|---|
| 538 | <rule ref="rulesets/strings.xml/UseStringBufferLength"> |
|---|
| 539 | <!-- CHECKME: 3.5 --> |
|---|
| 540 | <priority>5</priority> |
|---|
| 541 | </rule> |
|---|
| 542 | <!-- =================================================================== --> |
|---|
| 543 | <!--unusedcode--> |
|---|
| 544 | <!-- =================================================================== --> |
|---|
| 545 | <!-- <rule ref="rulesets/unusedcode.xml/UnusedFormalParameter"/> --> |
|---|
| 546 | <!-- <rule ref="rulesets/unusedcode.xml/UnusedLocalVariable"/> --> |
|---|
| 547 | <!-- <rule ref="rulesets/unusedcode.xml/UnusedPrivateField"/> --> |
|---|
| 548 | <rule ref="rulesets/unusedcode.xml/UnusedPrivateMethod"> |
|---|
| 549 | <priority>5</priority> |
|---|
| 550 | </rule> |
|---|
| 551 | </ruleset> |
|---|