LowFile-scopeJava Plug & play

Extract query strings to constants

Define SQL strings as static final constants or in a repository, not inline across methods.

Why this matters

Eases maintenance, reuse, and prevents typos.

Side-by-side examples engineers can pattern-match during review.

Avoid

conn.prepareStatement("SELECT * FROM orders WHERE id=?");

Prefer

private static final String SQL_GET_ORDER = "SELECT * FROM orders WHERE id=?";
conn.prepareStatement(SQL_GET_ORDER);

Bad

"UPDATE t SET..." scattered

Good

static final String SQL = "UPDATE t SET...";

From the same buckets as this rule.

Critical
Avoid Duplicating String Literals
Store repeated string literals in constants or variables to improve maintainability and reduce redundancy.
duplication-complexitymaintainability
Critical
Avoid Redundant `None` Comparisons
Identify cases where `x is None` or `x is not None` is used redundantly when `x` is always or never `None`. This may indicate a deeper logic issue.
duplication-complexityreadability-refactor+1
High
Avoid Functions That Always Return the Same Value
Identify functions that return the same value regardless of input. This may indicate poor design or a logic error.
duplication-complexityreadability-refactor+1