Code Smell 107 - Variables Reuse

Reusing variables makes scopes and boundaries harder to follow

TL;DR: Don't read and write the same variable for different purposes

Problems

• Readability

• Hidden problems

Solutions

1. Don't reuse variables

2. Extract Method to isolate scopes

Context

When programming a script it is common to reuse variables.

This leads to confusion and makes debugging harder.

We should narrow the scope as much as possible.

Sample Code

Wrong

// print line total
double total = item.getPrice() * item.getQuantity();
System.out.println("Line total: " + total );

// print amount total 
total = order.getTotal() - order.getDiscount();
System.out.println( "Amount due: " + total );

// variable is reused

Right

function printLineTotal() {
  double total = item.getPrice() * item.getQuantity();
  System.out.println("Line total: " + total );
}

function printAmountTotal() {
  double total = order.getTotal() - order.getDiscount();
  System.out.println( "Amount due: " + total );
}

Detection

[X] Automatic

Linters can use the parse tree to find variable definition and usages.

Tags

• Readability

Conclusion

Avoid reusing variable names. Use more specific and different names.

Relations

Code Smell 03 - Functions Are Too Long

Maxi Contieri ・ Oct 22 '20 ・ 1 min read

#codenewbie #beginners #programming #100daysofcode

More Info

Refactoring 002 - Extract Method

Maxi Contieri ・ Nov 25 ・ 2 min read

#refactoring #oop #webdev #codenewbie

Credits

Photo by Sigmund on Unsplash

Simplicity before generality, use before reuse.

Kevlin Henney

Software Engineering Great Quotes

Maxi Contieri ・ Dec 28 '20 ・ 13 min read

#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 ・ 4 min read

#codenewbie #tutorial #codequality #beginners