Defect Indicators: A string literal has been hard-coded into a statement that influences the flow of a program (e.g., in an If, While, or For statement).
Risks: If strings are embedded in code in this way, it increases the time needed to make maintenance changes, and increases the risk of error.
Assessment: Mark all binary terms that involve a string literal. An easy way to do this is by looking for the keywords and symbols that can precede binary terms in the language you are using. For example:
- If, While, Until, etc.
- And, Or, &&, ||, etc.
Remedy: Isolate a single copy of the string and refer to it. For example isolate the literal into:
- a local constant or enumerated type
- a global constant or enumerated type
- a constant class
- an initialisation file
Note: If your programming language does not support constants, simulate this—for example, by declaring a variable and initialising it at the beginning of the program.
Don't Repeat Yourself (DRY):
The DRY principle: Don't Repeat Yourself.