Code Smell 62 - Flag Variables

Flags indicate what happened. Unless their name is too generic.

Problems

• Readability

• Maintainability

• Coupling

Solutions

1. Use meaningful names

2. Try to avoid flags. They generate coupling.

Sample Code

Wrong

Right

Detection

We can search all the code for bad named flags.

Tags

• Readability

Conclusion

Flags are widespread on production code. We should restrict their usage and use clear and intention revealing names.

Relations

Code Smell 51 - Double Negatives

Maxi Contieri ・ Dec 24 '20

#codenewbie #tutorial #programming #oop

Code Smell 07 - Boolean Variables

Maxi Contieri ・ Oct 26 '20

#codenewbie #tutorial #beginners

More Info

Wikipedia

---

If you lie to the compiler, it will get its revenge.

Henry Spencer

---

Comments

[Michael Currin]

You suggested to restrict usage but didn't say how or provide and example.

Can I provide an alternative without the flag? Which is shorter and more readable.

while(true) {
  $elementSatisfies = doSomething();
  if (!$elementSatisfies) {
    break;
  }
}

Or even

while(doSomething()) {
  pass
}