...
In general, you should declare each variable on its own line with an explanatory comment regarding its role. Although it is While not required for conformance with this guideline, this practice is also recommended in the Code Conventions for the Java Programming Language, Section 6§6.1, "Number Per Line" [Conventions 2009].
This guideline applies to
- local Local variable declaration statements [Java 2011JLS 2013, §14.4],
- field Field declarations [Java 2011JLS 2013, §8.3],
- field Field (constant) declarations [Java 2011JLS 2013, §9.3],
Noncompliant Code Example (Initialization)
This noncompliant code example might lead a programmer or reviewer to mistakenly believe that both i
and j
are initialized to 1. In fact, only j
is initialized; , while i
remains uninitialized.:
Code Block | ||
---|---|---|
| ||
int i, j = 1; |
Compliant Solution (Initialization)
In this compliant solution, it is readily apparent that both i
and j
are initialized to 1.:
Code Block | ||
---|---|---|
| ||
int i = 1; // purposePurpose of i... int j = 1; // purposePurpose of j... |
Compliant Solution (Initialization)
In this compliant solution, it is readily apparent that both i
and j
are initialized to 1.:
Code Block | ||
---|---|---|
| ||
int i = 1, j = 1; |
Declaring each variable on a separate line is the preferred method. However, multiple variables on one line are acceptable when they are trivial temporary variables such as array indices.
Noncompliant Code Example (Different Types)
In this noncompliant code example, the programmer declares multiple variables, including an array, on the same line. All instances of the type T
have access to methods of the Object
class. However, it is easy to forget that arrays require special treatment when some of these methods are overridden.
...
Code Block |
---|
public String toString() { return a.toString() + b.toString() + c.toString() + d.toString(); } |
...
Code Block |
---|
// Correct functional implementation
public String toString(){
String s = a.toString() + b.toString();
for (int i = 0; i < c.length; i++){
s += c[i].toString();
}
s += d.toString();
return s;
}
|
Compliant Solution (Different Types)
This compliant solution places each declaration on its own line and uses the preferred notation for array declaration.:
Code Block | ||
---|---|---|
| ||
public class ExampleExample<T> { private T a; // purposePurpose of a... private T b; // purposePurpose of b... private T[] c; // purposePurpose of c[]... private T d; // purposePurpose of d... public Example(T in){ a = in; b = in; c = (T[]) new Object[10]; d = in; } } |
Applicability
Declaration of multiple variables per line can reduce code readability and lead to programmer confusion.
...
Declarations of loop indices should be included within a for
statement even if it when this results in variable declarations that do not include lack a comment about the purpose of the variable:
...
Such declarations are not required to be in a separate line, and the explanatory comment may also be omitted.
Automated Detection
Tool | Version | Checker | Description | ||||||
---|---|---|---|---|---|---|---|---|---|
Parasoft Jtest |
| CERT.DCL52.MVOS | Do not declare multiple variables in one statement | ||||||
SonarQube |
| S1659 |
Bibliography
§6.1, "Number Per Line" | |
[ESA 2005] | Rule 9, Put Single Variable Definitions in Separate Lines |
[JLS |
2013] | §4.3.2, "The |
...
...