DEV Community

loading...
Cover image for Code Smell 09 - Dead Code

Code Smell 09 - Dead Code

Maxi Contieri
Learn something new every day. - I am a senior software engineer working in industry, teaching and writing on software design, SOLID principles, DDD and TDD.
Originally published at maximilianocontieri.com Updated on ・1 min read

Code that is no longer used or needed.

Problems

  • Maintainability

Solutions

  • Remove the code
  • KISS

Examples

  • Gold plating code or Yagni code.

Exceptions

  • Avoid metaprogramming. When used it is very difficult to find references to the code.

Sample Code

Wrong

class Robot {   
  walk(){
    //...
    }
  serialize(){
    //..  
  }
  persistOnDatabase(database){
    //..  
  }
}

Enter fullscreen mode Exit fullscreen mode

Right

class Robot {   
  walk(){
    //...
    }  
}
Enter fullscreen mode Exit fullscreen mode

Detection

Coverage tools can find dead code (uncovered) if you have a great suite of tests.

Tags

  • Unnecessary

Conclusion

Remove dead code for simplicity.
If you are uncertain of code you can temporary disable it using Feature Toggle.
Removing code is always more rewarding than adding.

More info

Credits

Photo by Ray Shrewsberry on Pixabay


This article is part of the CodeSmell Series.

Last update: 2021/06/11

Discussion (0)