The first thing we read after the if the condition is the IF

TL;DR: You have the important else condition on the else.

Problems

• Readability

Solutions

1. Swap the conditions.

Context

It is not as straightforward as it appears to write IF clauses in an elegant manner.

There are lots of variants and choices.

We need to pay special attention to readability.

Sample Code

Wrong

fun addToCart(item: Any) {
    if (!cartExists()) {
        // Condition is negated
        this.createCart();
        this.cart.addItem(Item);
        // Repeated Code
    }
    else {
        // Normal case is on the else clause
        this.cart.addItem(Item);
    }
}

Right

fun addToCart(item: Any) {
    if (cartExists()) {
        this.cart.addItem(Item);
    }
    else {      
        this.createCart();
        this.cart.addItem(Item);
    }
}   

fun addToCartShorter(item: Any) {
    if (!cartExists()) {
        this.createCart();
    }
    this.cart.addItem(Item);    
}

Detection

[X] Semi-Automatic

We can find negated expressions on IF conditions and check for this anti-pattern.

Tags

• IFs

Conclusion

We need to read code like prose.

Humans read the standard case first and the exceptional one after it.

Relations

Code Smell 51 - Double Negatives

Maxi Contieri ・ Dec 24 '20

#codenewbie #tutorial #programming #oop

Code Smell 156 - Implicit Else

Maxi Contieri ・ Aug 6 '22

#javascript #webdev #beginners #programming

More Info

• Know the Code

Disclaimer

Code Smells are just my opinion.

Credits

Photo by Karol Kasanicky on Unsplash

Beauty is more important in computing than anywhere else in technology because software is so complicated. Beauty is the ultimate defense against complexity.

D. Gelernter

Software Engineering Great Quotes

Maxi Contieri ・ Dec 28 '20

#codenewbie #programming #quotes #software

This article is part of the CodeSmell Series.

How to Find the Stinky parts of your Code

Maxi Contieri ・ May 21 '21

#codenewbie #tutorial #codequality #beginners