Growing up near the Great Lakes, I’m well aware of the threat that Asian carp are posing on the polluted lakes we endearingly call the “North Coast”. It seems that these carp can’t be stopped. They’re breeding like crazy in an environment that isn’t equipped to keep their population in check. It’s so bad that underwater electric barriers have become viable solutions to the problem.

I can’t help but draw parallels between these carp and conditionals in code I’ve seen over the years. They’re always introduced for a good reason, whether they’re controlling parasites or controlling application flow. Once they start to spread, though, they tend to take over.

Conditionals spread

Are conditionals really as evil as an invading species of bottom feeders? To answer that question we need to look at how software changes over time. That requires us to think about how people change software over time.

People usually make the most obvious and convenient change. When there’s delivery pressure, we do what’s easiest. This is especially true when we aren’t confident that our change is isolated. When we’re worried that we may break other code, we do what’s most convenient and seems least “dangerous”.

Unfortunately, this means that conditionals tend to grow. What’s the easiest change when the complexity of a function is already through the roof? We add one more branch to the conditional and cross our fingers.

Conditionals are confusing

Conditionals mix multiple levels of abstraction in one neat little package. Here’s a list of things that I have to keep in my head while I’m reading a conditional:

  1. The shape of the conditional to understand the available paths through the application.
  2. The scope of multiple variables as they’re used within the conditional.
  3. The values that are used to determine which path to follow.
    * This often means understanding the structure of a deeply nested object.
  4. The actual code being executed within each path.
  5. How each of these specific actions affect the larger system since none of them have names.

Just thinking about all the things I need to think about leaves me exhausted. This is another reason why conditionals spread. After getting a grasp on all that information, I just don’t have any more room to plan a refactor!

Abstractions to keep the population at bay

Just like those carp, conditionals should be carefully culled. Unfortunately, Slack doesn’t have integrations for electric barriers or poison pills. There are a few patterns that I regularly use instead of conditionals. Before blindly adopting any of these techniques, ask if the code can be written without a conditional.

These 2 techniques build on All the Little Things. They are ways to design components that have a singular focus, rather than a list of responsibilities like we see above.

Each technique has a specific problem it solves. Each proposed alternative has a few important things in common.

  • The new abstraction has a single purpose. Choose and execute the appropriate path(s) through the application. That’s it. Fake paths can be provided for testing the abstraction so details about a specific path aren’t complicating the test.
  • Each path has the same 2 methods. One to test whether it should be executed, and one to execute the action. This allows each piece to be defined and tested in isolation. Also we gain valuable names describing each path, helping other developers understand its purpose.

Alternative to if / else if / else

The problem: We only want to execute one path through the application.

export default function walkDog (dog) {
  if (dog.hasEatenRecently()) {
    dog.walk(length: LONG_WALK)
  } else if ((time.readyForBed() && dog.hasWalkedRecently()) || time.aboutToLeave()) {
    dog.walk(length: JUST_POP_OUTSIDE)
  } else {
    dog.walk(length: AROUND_THE_BLOCK)
  }
}

The solution: Find the first matching path, then execute it.

import * as pooWalk from './poo-walk'
import * as beforeBed from './before-bed'
import * as aboutToLeave from './about-to-leave'
import * as normalWalk from './normal-walk'

const walkingPatterns = [ pooWalk, beforeBed, aboutToLeave, normalWalk ]

export default function walkDog (dog, walkingPatterns = walkingPatterns) {
  walkingPatterns
    .find(pattern => pattern.matches(dog))
    .execute(dog)
}

// ------------- ./normal-walk.js ------------- //
export function matches (shoppingCart) { ... }
export function execute (shoppingCart) { ... }

Alternative to multiple ifs

The problem: We only want to execute each appropriate path through the application.

export default function applyDiscounts (shoppingCart) {
  const discountedTotal = shoppingCart.total;

  if (shoppingCart.promotions && shoppingCart.promotions.length >= 0) {
    discountedTotal -= shoppingCart.promotions.each(promo => promo.apply(shoppingCart))
  }

  if (shoppingCart.user.credit) {
    discountedTotal -= shoppingCart.user.credit
  }

  return discountedTotal
}

The solution: Find all matching paths, then execute them.

import * as cartPromotions from  './cart-promotions'
import * as userCredits from  './user-credits'

const discountCalculators = [ cartPromotions, userCredits ]

export default function applyDiscounts (shoppingCart, discountCalculators = discountCalculators) {
  return discountCalculators
    .filter(calculator => calculator.canBeApplied(shoppingCart))
    .reduce((discountedTotal, calculator) => discountedTotal - calculator.apply(shoppingCart), shoppingCart.total)
}

// ------------- ./cart-promotions.js ------------- //
export function canBeApplied (shoppingCart) { ... }
export function apply (shoppingCart) { ... }

Sam Jones

Person An icon of a human figure Status
Double Agent
Hash An icon of a hash sign Code Name
Agent 0019
Location An icon of a map marker Location
Philadelphia, PA