Empty statements represented by a semicolon ;
are statements that do not perform any operation. They are often the result of a typo or
a misunderstanding of the language syntax. It is a good practice to remove empty statements since they don’t add value and lead to confusion and
errors.
Code examples
Noncompliant code example
void doSomething() {
; // Noncompliant - was used as a kind of TODO marker
}
void f() {
if (complicated.expression.foo()); // Noncompliant - the condition doesn't apply to bar
bar();
}
void f() {
if (complicated.expression.foo())
bar();
else ; // Noncompliant else is empty
buzz();
}
Compliant solution
void doSomething() {
}
void f() {
if (complicated.expression.foo()) {
bar();
}
}
void f() {
if (complicated.expression.foo())
bar();
else
buzz();
}