If a constant value is given for an identifier, do not diminish the maintainability of the code in which it is used by assuming its value in expressions. Simply giving the constant a name is not enough to ensure modifiability; you must be careful to always use the name, and remember that the value can change. This recommendation is related to DCL06-C. Use meaningful symbolic constants to represent literal values.
Noncompliant Code Example
stdio.h defines the
BUFSIZ macro, which expands to an integer constant expression that is the size of the buffer used by the
setbuf() function. This noncompliant code example defeats the purpose of defining
BUFSIZ as a constant by assuming its value in the following expression:
The programmer's assumption underlying this code is that "everyone knows that
BUFSIZ equals 512," and right-shifting 9 bits is the same (for positive numbers) as dividing by 512. However, if
BUFSIZ changes to 1024 on some systems, modifications are difficult and error prone.
This compliant solution uses the identifier assigned to the constant value in the expression:
Most modern C compilers will optimize this code appropriately.
Assuming the value of an expression diminishes the maintainability of code and can produce unexpected behavior under any circumstances in which the constant changes.
|Axivion Bauhaus Suite|
C3120, C3121, C3122, C3123, C3131, C3132
|LDRA tool suite|
|SEI CERT C++ Coding Standard||VOID EXP07-CPP. Do not diminish the benefits of constants by assuming their values in expressions|
|[Plum 1985]||Rule 1-5|