What I mean is: some boolean flags are perfect for the real world phenomenon they are representing e.g. is_light_on makes you understand perfectly that when it is true the light is on and when it is false the light is off.
There are other cases in which if you didn’t write the code and you don’t read any additional documentation, everything is not clear just by looking at the variable name e.g. is_person_standing, when true it’s clear what that means but when false, is the person sitting? Lying? Kneeling?
I’m obviously not talking about cases in which there are more states, boolean would of course not be a good solution in those cases. I’m talking about programs in which there are only two states but it’s not obvious, without external knowledge, which ones they are.
if the states aren’t obvious, use an enum with two values, and name them both. Thats what enums are for.
Our IT often use a Boolean as a shortcut for figuring out things in code. For example, if there’s a charge we don’t apply to some customers, instead of setting it to zero, they’ll have a Boolean on the customer to decide whether they skip that part of the calculation. On top of this, they then name it in a way that limits how many records they have to update, this leads to many settings phrased in the negative, such as “Don’t apply extra leg charges”. As an extra layer on this, more recently they were made aware of the confusion this causes for staff and their solution was to change how end users are the question, which causes the “yes/no” in the interface to read the opposite of the “true/false” in the database
I would use an Enum if available in the language:
- More meaningful
- Extendable
- Lower chance of misuse
- No naming problem
PolicyCancelledByCarrier
If canx notice has been received, processed, blah blah, eventually it’s set to true.
If/when a reinstatement is received, set to false.
Zero ambiguity, something along that line saved my tail when working with devs in different countries with different insurance customs.
Carrier sent letter telling policy holder to get bent because “fuck you, pay me?” Field is true.
Otherwise, or with reinstatement letter, field true.
One should not use boolean just because variable has only two states.
I believe when you use boolean when enum should be used is called “boolean blindness”.
Eg: isFemale instead of enum Sex {MALE;FEMALE} It also gives you an option to simply extend code if requirements change and there are more than two options.
