Your classes are abstract, final, or undefined

TL;DR: If your language has the right tool, your classes should be either abstract or final.

Problems

• Subclassification for Code Reuse

• Classes with just one concrete subclass

• Liskov Substitution Violation

• [Yo-Yo] Problem

Solutions

1. Declare all your leaf classes as final and the rest of them abstract.

Context

Managing hierarchies and composition is the main task of a good software designer.

Keeping hierarchies healthy is crucial to favor cohesion and avoid coupling.

Sample Code

Wrong

public class Vehicle
{
  // class is not a leaf. Therefore it should be abstract

  //an abstract method that only declares, but does not define the start 
  //functionality because each vehicle uses a different starting mechanism
  abstract void start();
}

public class Car extends Vehicle
{
  // class is leaf. Therefore it should be final
}

public class Motorcycle extends Vehicle
{
  // class is leaf. Therefore it should be final
}

Right

abstract public class Vehicle
{
  // class is not a leaf. Therefore it is be abstract  

  //an abstract method that only declares, but does not define the start 
  //functionality because each vehicle uses a different starting mechanism
  abstract void start();
}

final public class Car extends Vehicle
{
  // class is leaf. Therefore it is final
}

final public class Motorcycle extends Vehicle
{
  // class is leaf. Therefore it is final
}

Detection

[X] Automatic

Since this is enforced by static analysis, we can't do it with most available tools.

Tags

• Subclassification

Conclusion

We should look back at our classes and start qualifying them either as abstract or final.

There are no valid cases for two concrete classes, one subclassifying the other.

Relations

Code Smell 11 - Subclassification for Code Reuse

Maxi Contieri ・ Oct 30 '20

#oop #codenewbie #tutorial

Code Smell 136 - Classes With just One Subclass

Maxi Contieri ・ May 28 '22

Code Smell 37 - Protected Attributes

Maxi Contieri ・ Nov 29 '20

#oop #webdev #tutorial #codenewbie

Code Smell 58 - Yo-yo Problem

Maxi Contieri ・ Jan 24 '21

#codenewbie #codesmell #tutorial #webdev

More Info

Coupling: The one and only software design problem

Maxi Contieri ・ Feb 6 '21

#webdev #programming #oop #tutorial

Deep Subclasses

Disclaimer

Code Smells are just my opinion.

Credits

Photo by William Bossen on Unsplash

When the final design seems too simple for the amount of work you've put in, then you know you're done.

Brady Clark

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