In the spirit of the 10,000+ Hive Hackathon I am running, I am going to try to put out some tips and tricks over the next few days.

I also have a long-running Python Tips Series you can find here.
Look at the footer for links to the previous entries.

What is a Guard Clause

A guard clause a programming idiom that you are likely already using but are not aware there is a name for it. It is also something you are not likely using effectively.

A guard clause exits a loop or function when certain conditions are met early on.

Example 1

let inventory = 1
let cart = []

function addToCart(item){
  if (inventory > 0) {
    cart.push(item)
  }
}

addToCart('Book')
console.log(cart)

This simple code creates a variable to simulate an inventory and a shopping cart. There is an addToCart() function to add items to the cart. Before adding an item to the cart, it checks to see if the inventory is empty.

This code works perfectly fine and is easy to read. The problem comes in when the business rules become more complex and you start expanding the if statement.

Example 2

let inventory = 1
let cart = []
let userIsLoggedIn = true

function addToCart(item){
  if (userIsLoggedIn) {
    if (inventory > 0) {
      cart.push(item)
    }
  }

}

addToCart('Book')
console.log(cart)

While this is still relatively simple, you have likely coded far more complex examples many times. These types of nested if structures become very difficult to manage and change in the future.

Introducing the guard clause

A guard clause will exit your loop or function immediately. This avoids additional indents to your core condition statements making them far easier to read and maintain.

let inventory = 1
let cart = []
let userIsLoggedIn = true

function addToCart(item){
  if (!userIsLoggedIn || !inventory) return
  cart.push(item)
}

addToCart('Book')
console.log(cart)

By extracting the initial sanity checks into a dedicated if statement that exits immediately, your code is cleaner and far easier to maintain.

These examples are oversimplified, and real-life examples are far more difficult to work with.

Here's an example of an if cluster bomb from Reddit

const returnLinesRequiredData = (lineObj, id, key) => {

  if (id) {
    if (id < lineObj.stations.length) {
      if (key) {
        if (lineObj.stations[id].hasOwnProperty(key)) {
          return lineObj.stations[id][key];
        }
        return errorMsg;
      }
      return lineObj.stations[id];
    }
    return errorMsg;
  }
  return lineObj;

};

_Source

Try to look over your existing code and see where you can implement guard clauses to improve the readability of your code.

---

Securely chat with me on Keybase

Why you should vote me as witness