Do you have a function called Enable()
and another called Disable()
. Make those two private and make a third:
void Enable(bool enable) {
if ( enable )
Enable();
else
Disable();
}
// now change all your calling code from
if( x == y )
Enable();
else
Disable();
// to
Enable( x == y ); // 4 lines are now 1
As per my example above, four lines got compressed into one line but nothing actually changed. The algorithm is identical but you now have fewer lines to think about and worry about. And all of this multiplies. If you make that change 4 times you've removed 12 lines of potential bugs and oversights.
Comments: 0