Common static analysis issues
org.openrewrite.staticanalysis.CommonStaticAnalysis
Resolve common static analysis issues (also known as SAST issues).
Recipe source
GitHub, Issue Tracker, Maven Central
This recipe is composed of more than one recipe. If you want to customize the set of recipes this is composed of, you can find and copy the GitHub source for the recipe from the link above.
This recipe is available under the Moderne Source Available License.
Definition
- Recipe List
- Yaml Recipe List
- Constructors of an
abstractclass should not be declaredpublic - Atomic Boolean, Integer, and Long equality checks compare their values
new BigDecimal(double)should not be usedBigDecimalrounding constants toRoundingModeenums- Boolean checks should not be inverted
- CaseInsensitive comparisons do not alter case
- Catch clause should do more than just rethrow
- Chain
StringBuilder.append()calls - 'Collection.toArray()' should be passed an array of the proper type
- Covariant equals
- Default comes last
- Remove empty blocks
- Equals avoids null
- Explicit initialization
Externalizableclasses have no-arguments constructor- Finalize private fields
- Fall through
- Finalize classes with private constructors
- Fix
String#formatandString#formattedexpressions forloop counters incremented in update- Use
indexOf(String, int) indexOf()replaceable bycontains()indexOfshould not compare greater than zero- Inline variable
- Use
Collection#isEmpty()instead of comparingsize() - Simplify lambda blocks to expressions
- Standardize method name casing
switchstatements should have at least 3caseclauses- Modifier order
- No multiple variable declarations
- Fix missing braces
- Nested enums are not static
- Change
StringBuilderandStringBuffercharacter constructor argument toString - No double brace initialization
- Use
Collections#emptyList(),emptyMap(), andemptySet() - Use comparison rather than equality checks in for conditions
- Remove
finalize()method - No primitive wrappers for #toString() or #compareTo(..)
- Jump statements should not be redundant
- Unnecessary
String#toString - Unnecessary
String#valueOf(..) finalize()calls super- Prefer
System.getProperty("user.home")overSystem.getenv("HOME") - Use primitive wrapper
valueOfmethod - Redundant file creation
- Remove extra semicolons
- Remove redundant null checks before instanceof
- Remove redundant null checks before literal equals
- Rename methods named
hashcode,equal, ortostring - Replace
A.class.isInstance(a)witha instanceof A - Use method references in lambda
- Replace
StringBuilder#appendwithString - Simplify
Arrays.asList(..)with varargs - Simplify boolean expression
- Simplify boolean return
- Static methods need not be final
- Use
String.equals()onStringliterals - Unnecessary close in try-with-resources
- Unnecessary explicit type arguments
- Remove unnecessary parentheses
- Remove
@Nullableand@CheckForNullannotations from primitives - Unnecessary
returnas last statement in void method - Upper case literal suffixes
- Use the diamond operator
- No C-style array declarations
- Use %n instead of \n in format strings
- Prefer
whileoverforloops - Write octal values as decimal
- Structural equality tests should use
==or!= itshouldn't be used as a lambda parameter name- Replace
Char#toInt()withChar#code - Custom import order
---
type: specs.openrewrite.org/v1beta/recipe
name: org.openrewrite.staticanalysis.CommonStaticAnalysis
displayName: Common static analysis issues
description: |
Resolve common static analysis issues (also known as SAST issues).
recipeList:
- org.openrewrite.staticanalysis.AbstractClassPublicConstructor
- org.openrewrite.staticanalysis.AtomicPrimitiveEqualsUsesGet
- org.openrewrite.staticanalysis.BigDecimalDoubleConstructorRecipe
- org.openrewrite.staticanalysis.BigDecimalRoundingConstantsToEnums
- org.openrewrite.staticanalysis.BooleanChecksNotInverted
- org.openrewrite.staticanalysis.CaseInsensitiveComparisonsDoNotChangeCase
- org.openrewrite.staticanalysis.CatchClauseOnlyRethrows
- org.openrewrite.staticanalysis.ChainStringBuilderAppendCalls
- org.openrewrite.staticanalysis.CollectionToArrayShouldHaveProperType
- org.openrewrite.staticanalysis.CovariantEquals
- org.openrewrite.staticanalysis.DefaultComesLast
- org.openrewrite.staticanalysis.EmptyBlock
- org.openrewrite.staticanalysis.EqualsAvoidsNull
- org.openrewrite.staticanalysis.ExplicitInitialization
- org.openrewrite.staticanalysis.ExternalizableHasNoArgsConstructor
- org.openrewrite.staticanalysis.FinalizePrivateFields
- org.openrewrite.staticanalysis.FallThrough
- org.openrewrite.staticanalysis.FinalClass
- org.openrewrite.staticanalysis.FixStringFormatExpressions
- org.openrewrite.staticanalysis.ForLoopIncrementInUpdate
- org.openrewrite.staticanalysis.IndexOfChecksShouldUseAStartPosition
- org.openrewrite.staticanalysis.IndexOfReplaceableByContains
- org.openrewrite.staticanalysis.IndexOfShouldNotCompareGreaterThanZero
- org.openrewrite.staticanalysis.InlineVariable
- org.openrewrite.staticanalysis.IsEmptyCallOnCollections
- org.openrewrite.staticanalysis.LambdaBlockToExpression
- org.openrewrite.staticanalysis.MethodNameCasing
- org.openrewrite.staticanalysis.MinimumSwitchCases
- org.openrewrite.staticanalysis.ModifierOrder
- org.openrewrite.staticanalysis.MultipleVariableDeclarations
- org.openrewrite.staticanalysis.NeedBraces
- org.openrewrite.staticanalysis.NestedEnumsAreNotStatic
- org.openrewrite.staticanalysis.NewStringBuilderBufferWithCharArgument
- org.openrewrite.staticanalysis.NoDoubleBraceInitialization
- org.openrewrite.staticanalysis.NoEmptyCollectionWithRawType
- org.openrewrite.staticanalysis.NoEqualityInForCondition
- org.openrewrite.staticanalysis.NoFinalizer
- org.openrewrite.staticanalysis.NoPrimitiveWrappersForToStringOrCompareTo
- org.openrewrite.staticanalysis.NoRedundantJumpStatements
- org.openrewrite.staticanalysis.NoToStringOnStringType
- org.openrewrite.staticanalysis.NoValueOfOnStringType
- org.openrewrite.staticanalysis.ObjectFinalizeCallsSuper
- org.openrewrite.staticanalysis.PreferSystemGetPropertyOverGetenv
- org.openrewrite.staticanalysis.PrimitiveWrapperClassConstructorToValueOf
- org.openrewrite.staticanalysis.RedundantFileCreation
- org.openrewrite.staticanalysis.RemoveExtraSemicolons
- org.openrewrite.staticanalysis.RemoveRedundantNullCheckBeforeInstanceof
- org.openrewrite.staticanalysis.RemoveRedundantNullCheckBeforeLiteralEquals
- org.openrewrite.staticanalysis.RenameMethodsNamedHashcodeEqualOrToString
- org.openrewrite.staticanalysis.ReplaceClassIsInstanceWithInstanceof
- org.openrewrite.staticanalysis.ReplaceLambdaWithMethodReference
- org.openrewrite.staticanalysis.ReplaceStringBuilderWithString
- org.openrewrite.staticanalysis.SimplifyArraysAsList
- org.openrewrite.staticanalysis.SimplifyBooleanExpression
- org.openrewrite.staticanalysis.SimplifyBooleanReturn
- org.openrewrite.staticanalysis.StaticMethodNotFinal
- org.openrewrite.staticanalysis.StringLiteralEquality
- org.openrewrite.staticanalysis.UnnecessaryCloseInTryWithResources
- org.openrewrite.staticanalysis.UnnecessaryExplicitTypeArguments
- org.openrewrite.staticanalysis.UnnecessaryParentheses
- org.openrewrite.staticanalysis.UnnecessaryPrimitiveAnnotations
- org.openrewrite.staticanalysis.UnnecessaryReturnAsLastStatement
- org.openrewrite.staticanalysis.UpperCaseLiteralSuffixes
- org.openrewrite.staticanalysis.UseDiamondOperator
- org.openrewrite.staticanalysis.UseJavaStyleArrayDeclarations
- org.openrewrite.staticanalysis.UsePortableNewlines
- org.openrewrite.staticanalysis.WhileInsteadOfFor
- org.openrewrite.staticanalysis.WriteOctalValuesAsDecimal
- org.openrewrite.kotlin.cleanup.EqualsMethodUsage
- org.openrewrite.kotlin.cleanup.ImplicitParameterInLambda
- org.openrewrite.kotlin.cleanup.ReplaceCharToIntWithCode
- org.openrewrite.staticanalysis.CustomImportOrder
Examples
Example 1
CommonStaticAnalysisIssuesPerformanceTest#indexOfOnList
- java
- Diff
Before
import java.util.List;
class Test {
static boolean hasIndex(List<String> strList, String str) {
if (strList.indexOf(str) > 0) {
}
return strList.indexOf(str) > 0;
}
}
After
import java.util.List;
class Test {
static boolean hasIndex(List<String> strList, String str) {
strList.indexOf(str);
return strList.indexOf(str) >= 1;
}
}
@@ -5,3 +5,2 @@
class Test {
static boolean hasIndex(List<String> strList, String str) {
- if (strList.indexOf(str) > 0) {
- }
- return strList.indexOf(str) > 0;
+ strList.indexOf(str);
+ return strList.indexOf(str) >= 1;
}
Example 2
CommonStaticAnalysisIssuesPerformanceTest#indexOfOnList
- java
- Diff
Before
import java.util.List;
class Test {
static boolean hasIndex(List<String> strList, String str) {
if (strList.indexOf(str) > 0) {
}
return strList.indexOf(str) > 0;
}
}
After
import java.util.List;
class Test {
static boolean hasIndex(List<String> strList, String str) {
strList.indexOf(str);
return strList.indexOf(str) >= 1;
}
}
@@ -5,3 +5,2 @@
class Test {
static boolean hasIndex(List<String> strList, String str) {
- if (strList.indexOf(str) > 0) {
- }
- return strList.indexOf(str) > 0;
+ strList.indexOf(str);
+ return strList.indexOf(str) >= 1;
}
Usage
This recipe has no required configuration options. It can be activated by adding a dependency on org.openrewrite.recipe:rewrite-static-analysis in your build file or by running a shell command (in which case no build changes are needed):
- Gradle
- Gradle init script
- Maven POM
- Maven Command Line
- Moderne CLI
- Add the following to your
build.gradlefile:
plugins {
id("org.openrewrite.rewrite") version("latest.release")
}
rewrite {
activeRecipe("org.openrewrite.staticanalysis.CommonStaticAnalysis")
setExportDatatables(true)
}
repositories {
mavenCentral()
}
dependencies {
rewrite("org.openrewrite.recipe:rewrite-static-analysis:2.20.0")
}
- Run
gradle rewriteRunto run the recipe.
- Create a file named
init.gradlein the root of your project.
initscript {
repositories {
maven { url "https://plugins.gradle.org/m2" }
}
dependencies { classpath("org.openrewrite:plugin:7.19.0") }
}
rootProject {
plugins.apply(org.openrewrite.gradle.RewritePlugin)
dependencies {
rewrite("org.openrewrite.recipe:rewrite-static-analysis:2.20.0")
}
rewrite {
activeRecipe("org.openrewrite.staticanalysis.CommonStaticAnalysis")
setExportDatatables(true)
}
afterEvaluate {
if (repositories.isEmpty()) {
repositories {
mavenCentral()
}
}
}
}
- Run the recipe.
gradle --init-script init.gradle rewriteRun
- Add the following to your
pom.xmlfile:
<project>
<build>
<plugins>
<plugin>
<groupId>org.openrewrite.maven</groupId>
<artifactId>rewrite-maven-plugin</artifactId>
<version>6.22.1</version>
<configuration>
<exportDatatables>true</exportDatatables>
<activeRecipes>
<recipe>org.openrewrite.staticanalysis.CommonStaticAnalysis</recipe>
</activeRecipes>
</configuration>
<dependencies>
<dependency>
<groupId>org.openrewrite.recipe</groupId>
<artifactId>rewrite-static-analysis</artifactId>
<version>2.20.0</version>
</dependency>
</dependencies>
</plugin>
</plugins>
</build>
</project>
- Run
mvn rewrite:runto run the recipe.
You will need to have Maven installed on your machine before you can run the following command.
mvn -U org.openrewrite.maven:rewrite-maven-plugin:run -Drewrite.recipeArtifactCoordinates=org.openrewrite.recipe:rewrite-static-analysis:RELEASE -Drewrite.activeRecipes=org.openrewrite.staticanalysis.CommonStaticAnalysis -Drewrite.exportDatatables=true
You will need to have configured the Moderne CLI on your machine before you can run the following command.
mod run . --recipe CommonStaticAnalysis
If the recipe is not available locally, then you can install it using:
mod config recipes jar install org.openrewrite.recipe:rewrite-static-analysis:2.20.0
See how this recipe works across multiple open-source repositories
Run this recipe on OSS repos at scale with the Moderne SaaS.

The community edition of the Moderne platform enables you to easily run recipes across thousands of open-source repositories.
Please contact Moderne for more information about safely running the recipes on your own codebase in a private SaaS.
Data Tables
- SourcesFileResults
- SourcesFileErrors
- RecipeRunStats
Source files that had results
org.openrewrite.table.SourcesFileResults
Source files that were modified by the recipe run.
| Column Name | Description |
|---|---|
| Source path before the run | The source path of the file before the run. null when a source file was created during the run. |
| Source path after the run | A recipe may modify the source path. This is the path after the run. null when a source file was deleted during the run. |
| Parent of the recipe that made changes | In a hierarchical recipe, the parent of the recipe that made a change. Empty if this is the root of a hierarchy or if the recipe is not hierarchical at all. |
| Recipe that made changes | The specific recipe that made a change. |
| Estimated time saving | An estimated effort that a developer to fix manually instead of using this recipe, in unit of seconds. |
| Cycle | The recipe cycle in which the change was made. |
Source files that errored on a recipe
org.openrewrite.table.SourcesFileErrors
The details of all errors produced by a recipe run.
| Column Name | Description |
|---|---|
| Source path | The file that failed to parse. |
| Recipe that made changes | The specific recipe that made a change. |
| Stack trace | The stack trace of the failure. |
Recipe performance
org.openrewrite.table.RecipeRunStats
Statistics used in analyzing the performance of recipes.
| Column Name | Description |
|---|---|
| The recipe | The recipe whose stats are being measured both individually and cumulatively. |
| Source file count | The number of source files the recipe ran over. |
| Source file changed count | The number of source files which were changed in the recipe run. Includes files created, deleted, and edited. |
| Cumulative scanning time (ns) | The total time spent across the scanning phase of this recipe. |
| Max scanning time (ns) | The max time scanning any one source file. |
| Cumulative edit time (ns) | The total time spent across the editing phase of this recipe. |
| Max edit time (ns) | The max time editing any one source file. |