Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

Code Block
bgColor#ffcccc
if(age>=18)
{
        printf("Of legal voting age"takevote(personID);
}
/*Various Processing code*/
if (age<=18)
{
	checkSchoolEnrollment(personID);
}

Compliant Code:

Code Block
bgColor#ccccff
if(age>=VOTINGADULT_AGE)
{
        printf("Of legal voting age"takevote(personID);
}
/*Various Processing code*/
if (age<=ADULT_AGE)
{	checkSchoolEnrollment(personID);
}  

In the compliant code it is easy to check if the user is an adult and process accordingly. If the definition of adult changes during iterations of the codebase it is much simpler to replace the value for ADULT_AGE then search for instance of 18 and see if they're appropriate for change.

Risk Assessment

Mistakes regarding numeric values can cause unintended consequences if changes are not made uniformly

...