Do not use a semicolon on the same line as after an if
, for
, or while
statement condition because this it typically indicates programmer error and can result in unexpected behavior.
Noncompliant Code Example
In this noncompliant code example, a semicolon is used on the same line as immediately following an if
statement. condition:
Code Block | ||
---|---|---|
| ||
if (a == b); {
/* ... */
}
|
The statements in the apparent body of the if
statement are always evaluated regardless of the result of the condition expression.
Compliant Solution
It is likely, in this example, that the semicolon was accidentally inserted.This compliant solution eliminates the semicolon and ensures that the body of the if
statement is executed only when the condition expression is true:
Code Block | ||
---|---|---|
| ||
if (a == b) {
/* ... */
}
|
Related Guidelines
Applicability
Placing a semicolon immediately following
...
...
an if
, for
, or while
...
<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="2d03788b-6a5d-4c79-b8ce-bddf4885c925"><ac:plain-text-body><![CDATA[
...
[ISO/IEC TR 24772:2010
...
http://www.aitcnet.org/isai/]
...
"Likely Incorrect Expression [java:KOA]"
...
]]></ac:plain-text-body></ac:structured-macro>
condition may result in unexpected behavior.
Automated Detection
Tool | Version | Checker | Description | ||||||
---|---|---|---|---|---|---|---|---|---|
PVS-Studio |
| V6063 | |||||||
SonarQube |
| EmptyStatementUsageCheck |
Bibliography
§2 |
Bibliography
<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="6fb6d93f-1bbb-4434-ab5a-ec9f3e7eeb81"><ac:plain-text-body><![CDATA[
[java:Hatton 1995]
.7.2, "Errors of |
Omission and |
]]></ac:plain-text-body></ac:structured-macro>
Addition" |
...
49. Miscellaneous (MSC) MSC07-J. Do not assume infinite heap space