DEV Community

Shameel Uddin
Shameel Uddin

Posted on

How we misuse naming boolean and how they should be named

#programming #beginners #tutorial #typescript

Boolean is a pretty cool data type which we often use in our conditionals.
The blog explains how they should be named properly for the sake of clean coding practices in our codebase.

Positive over Negative

Banish the confusion of double negatives! Instead of drowning in a sea of "not," embrace clarity.

❌ if (!isNotUser)

✅ if (isUser)

Adjectives, Not Actions

Break free from mundane verbs and nouns! Elevate your boolean game with descriptive adjectives or short, snappy phrases.

❌ if (sendEmail)

✅ if (emailIsSent)

Present Tense

Time-traveling through tenses adds unnecessary noise. Stick to the present tense for clean, coherent code.

❌ if (hasBeenSubscribed)

✅ if (isSubscribe)

Proper use of Prefix

Boost readability with prefixes like is/has/should/can. Your code will thank you later!

❌ if (membership)

✅ if (hasMembership)

Avoid confusing true/false - Employ Enums!

Wave goodbye to mysterious true/false values. Embrace the power of enums for self-documenting code.

❌ bookMeeting(customer, true)

❌ bookMeeting(customer, false)

✅ bookMeeting(customer, Discount.Applied)

✅ bookMeeting(customer, Discount.NotApplied)

Happy coding! 🎉💻✨

Follow me for more such content:
LinkedIn: https://www.linkedin.com/in/shameeluddin/
Github: https://github.com/Shameel123

Top comments (10)

Collapse
 
framemuse profile image
Valery Zinchenko • • Edited

I personally love to use past tense for boolean, so it's reading like natural language

if (subscribed)
Enter fullscreen mode Exit fullscreen mode
Collapse
 
aarone4 profile image
Aaron Reese •

your first example would be better as
WRONG
if (isNotUser)
RIGHT
if (!isUser)

isNotUser evaluates to TRUE which is a confusing. Personally I don't like the ! logic reversal and will often write a separate not(boolean) function that reverses the boolean value

not(myBoolean) {
return !myBoolean
}

if (not(isUser)) {
...
}

Collapse
 
chukwuma1976 profile image
Chukwuma Anyadike •

Nicely done. This makes code simple to read. I like simplicity. These are the "little" things we take for granted.

Collapse
 
lundjrl profile image
James Robert Lund III •

We started doing this a few projects ago and it makes debugging so much easier.

Collapse
 
vojtech profile image
Vojtech Mašek •

In JS/TS I'd recommend setting up a naming lint via ESLint.

        "@typescript-eslint/naming-convention": [
          "warning",
          {
            "selector": "default",
            "format": ["camelCase"]
          },
          {
            "selector": "variable",
            "format": ["camelCase", "UPPER_CASE"]
          },
          {
            "selector": "typeLike",
            "format": ["PascalCase"]
          },
          {
            "selector": "classProperty",
            "format": ["camelCase"]
          },
          {
            "selector": "enumMember",
            "format": ["PascalCase"]
          },
          {
            "selector": "parameter",
            "modifiers": ["unused"],
            "format": null,
            "custom": {
              "regex": "^(_+|[a-z][a-zA-Z0-9]*)$",
              "match": true
            }
          },
          {
            "selector": ["variable", "typeProperty"],
            "format": ["PascalCase"],
            "types": ["boolean"],
            "prefix": [
              "are",
              "can",
              "contains",
              "has",
              "includes",
              "is",
              "should",
              "use",
              "will"
            ],
            "filter": {
              "regex": "^(loading|disabled|checked|production|invalid)$",
              "match": false
            }
          },
          {
            "selector": "objectLiteralProperty",
            "modifiers": ["requiresQuotes"],
            "format": null
          }
        ]
Enter fullscreen mode Exit fullscreen mode

Notice the prefix configuration for boolean variable names

Collapse
 
shameel profile image
Shameel Uddin •

Now that's something cool. Thanks for sharing ✌️

Collapse
 
cappe987 profile image
Casper •

sendEmail and emailIsSent don't have the same meaning, different tenses. You probably want something like shouldSendEmail

Collapse
 
soorajsnblaze333 profile image
Sooraj (PS) • • Edited

Good article, but I personally think for the 3rd one I would lose context if I were to say “isSubscribe” rather than “subscribed” or “hasSubscribed”

And also doesn’t number 3 contradict number 2 ? You have mentioned to stick to present tense but you have used a past tense “sent” in 2.

Collapse
 
htho profile image
Hauke T. •

Regarding the last:

Where applicable I to write two functions:

bookRegularMeeting(customer)
bookDiscountedMeeting(customer)

It is simpler than managing an additional enum.

Collapse
 
shameel profile image
Shameel Uddin •

Actually the last one means to have one function which handles two cases with a boolean conditional.

Your approach would require to have two separate functions. The approach is handy if they employ drastic different logic but not as much if logic is nearly the same.

Read next

vinyldavyl profile image

The Ultimate Guide to Styling with CSS-in-JS Using Styled Components

Vinyl-Davyl -

virtualsobriety profile image

SCRIMBA - LEARN JAVASCRIPT - from a beginners perspective

VirtualSobriety -

scorcism profile image

JavaScript Objects | Complete Guide

Abhishek Pathak -

daunderworks profile image

Building a monolith in Go - Layout

Doug Bridgens -