Page History

Declaring multiple variables in a single declaration may could cause confusion regarding about the types of the variables and their initial values. In particular, do not declare any of the following in a single declaration:

• variables Variables of different types
• a A mixture of initialized and uninitialized variables

In general, you should declare each variable on its own line with an explanatory comment about the role of the variable. Although regarding its role. While not required for conformance with this guideguideline, this practice is also recommended in the Code Conventions for the Java Programming Language Conventions 2009 Section 6, §6.1, "Number Per Line" .

When more than one variable is declared in a single declaration, ensure that the type and initial value of the variable are self evident.

[Conventions 2009].

This guideline applies to:

• local Local variable declaration statements ([JLS §142013, §14.4)]
• field Field declarations ([JLS §82013, §8.3)]
• field Field (constant) declarations ([JLS §92013, §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.:

int i, j = 1;

Compliant Solution (

...

Initialization)

In this compliant solution, it is readily apparent that both i and j are initialized to 1.:

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.:

int i = 1, j = 1;

It is preferrable to declare Declaring each variable on a separate line is the preferred method. However, but multiple variables on one line can be are acceptable if when they are small trivial temporary variables , such as array indexesindices.

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 Object. However, it is easy to forget that arrays require special treatment when some of these methods are overridden.

public class Example<T> {
  private T a, b, c[], d;

  public Example(T in) {
    a = in;
    b = in;
    c = (T[]) new Object[10];
    d = in;
  }
}

When a an Object method of Object , such as toString(), is overridden, a programmer might could accidentally provide an implementation for type T that fails to consider that c is an array of T, rather than a reference to an object of type T.

public String toString() {
  return a.toString() + b.toString() + 
         c.toString() + d.toString();
}

However, the programmer's actual intent might could have been to invoke toString() on each individual element of the array c.

// 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 , as well as using and uses the preferred notation for array declaration.:

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;
  }
}

Exceptions

Applicability

Declaration of multiple variables per line can reduce code readability and lead to programmer confusion.

When more than one variable is declared in a single declaration, ensure that both the type and the initial value of each variable are self-evidentDCL01-EX1: Note that the declaration of a loop counter in a for statement is in violation of this recommendation because the declaration is not on its own line with an explanatory comment about the role of the variable. However, declaration of loop indices in for statements is not only a common idiom; it also provides the benefit of restricting the scope of the loop index to that of the for loop itself. These are sufficient reasons to relax this guideline in this specific case.

Declarations of loop indices should be included within a for statement even when this results in variable declarations that lack a comment about the purpose of the variable:

public class Example {
  void function() {
    int mx = 100; // Some max value

    for (int i = 0; i < mx; ++i ) {
      /* ... */
    }

  }
}

Risk Assessment

Declaration of multiple variables per line can reduce code readability and lead to programmer confusion.

Related Guidelines

C Secure Coding Standard: DCL04-C. Do not declare more than one variable per declaration

C++ Secure Coding Standard: DCL04-CPP. Do not declare more than one variable per declaration

Related Vulnerabilities

Search for vulnerabilities resulting from the violation of this guideline on the CERT website.

Bibliography

\[[Conventions 2009|AA. Bibliography#Conventions 09]\] Section 6.1, "Number Per Line"
\[[ESA 2005|AA. Bibliography#ESA 05]\] Rule 9: Put single variable definitions in separate lines.
\[[JLS 2005|AA. Bibliography#JLS 05]\] [Section 8.3, "Field Declarations"|http://java.sun.com/docs/books/jls/third_edition/html/classes.html#8.3], [Section 9.3, "Field (Constant) Declarations"|http://java.sun.com/docs/books/jls/third_edition/html/classes.html#9.3][Section 14.4, "Local Variable Declaration Statements"|http://java.sun.com/docs/books/jls/third_edition/html/statements.html#14.4] Section 6.1, "Declarations", Section 4.3.2, "The class Object"

Such declarations are not required to be in a separate line, and the explanatory comment may be omitted.

Automated Detection

Bibliography

...

Image Added Image Added Image AddedDCL00-J. Use visually distinct identifiers      03. Declarations and Initialization (DCL)      DCL02-J. Use meaningful symbolic constants to represent literal values in program logic