...
Although it advises against depending on parentheses for specifying evaluation order, guideline "EXP08-J. Do not write more than once to the same variable within an expression" applies only to expressions that contain side effects.
Noncompliant Code Example
The intent of the expression in this noncompliant code example is to add the variable OFFSET
to the result of the bitwise logical AND between x
and MASK
.
...
Code Block |
---|
x & (1337 - 1337) |
Compliant Solution
This compliant solution uses parentheses to ensure that the expression evaluates as intended.
Code Block | ||
---|---|---|
| ||
public static final int MASK = 1337; public static final int OFFSET = -1337; public static int computeCode(int x) { return (x & MASK) + OFFSET; } |
Exceptions
EXP06-EX0: Parentheses may be omitted from mathematical expressions that follow the algebraic precedence rules. For instance, consider the expression:
...
Code Block | ||
---|---|---|
| ||
x + (y * z) |
Risk Assessment
Mistakes regarding precedence guidelines can cause an expression to be evaluated in an unintended way. This can lead to unexpected and abnormal program behavior.
Guideline | Severity | Likelihood | Remediation Cost | Priority | Level |
---|---|---|---|---|---|
EXP06-J | low | probable | medium | P4 | L3 |
Automated Detection
Detection of all expressions using low-precedence operators without parentheses is straightforward. Determining the correctness of such uses is infeasible in the general case; heuristic warnings could be useful.
Related Guidelines
C Coding Standard: "EXP00-C. Use parentheses for precedence of operation"
C++ Secure Coding Standard: "EXP00-CPP. Use parentheses for precedence of operation"
Bibliography
<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="f88044f16ad7879c-33f2c0d7-487044f5-b2d588c4-c8ea43d3fd84f36b8c3e0b10"><ac:plain-text-body><![CDATA[ | [[ESA 2005 | AA. Bibliography#ESA 05]] | Rule 65: Use parentheses to explicitly indicate the order of execution of numerical operators | ]]></ac:plain-text-body></ac:structured-macro> | |
<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="42713ffada8c2e71-b377373a-438e4f36-b8ddaa87-22fb7bad6a88ad141fe50a35"><ac:plain-text-body><![CDATA[ | [[Tutorials 2008 | AA. Bibliography#Tutorials 08]] | [Expressions, Statements, and Blocks | http://java.sun.com/docs/books/tutorial/java/nutsandbolts/expressions.html] | ]]></ac:plain-text-body></ac:structured-macro> |
| |||||
<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="4b17efcfa1a2d511-a7ed8ccb-48f7436d-a6668767-543eabcb188b1adab441e296"><ac:plain-text-body><![CDATA[ | [[Rogue 2000 | AA. Bibliography#Rogue 2000]] | Rule 77: Clarify the order of operations with parentheses | ]]></ac:plain-text-body></ac:structured-macro> |
...