Code Smell 60 - Global Classes

mcsee

Maxi Contieri

Posted on January 31, 2021

Code Smell 60 - Global Classes

Classes are handy. We can call them and invoke them any time. Is this good?

TL;DR: Don't use your classes as a global point of access.

Problems

  • Coupling

  • Classes are global unless we use Namespaces.

  • Name polluting

  • Static Methods

  • Static Constants

  • Singletons

Solutions

  1. Use namespaces, module qualifiers or similar

  2. Avoid namespace polluting, keep the Global names as short as possible.

  3. Class single Responsibility is to create instances.

Sample Code

Wrong

<?

final class StringUtilHelper {
    static function reformatYYYYDDMMtoYYYYMMDD($date) {
    }
}

class Singleton {

}

final class DatabaseAccessor extends Singleton {

}
Enter fullscreen mode Exit fullscreen mode

Right

<?

namespace Date;

final class DateFormatter {

    function reformatYYYYDDMMtoYYYYMMDD(Date $date) {
    }
    //function is not static since class single responsibility is to create instances and not be a library of utils

}

namespace OracleDatabase;

class DatabaseAccessor {
    //Database is not a singleton and it is namespace scoped
}
Enter fullscreen mode Exit fullscreen mode

Detection

We can use almost any linter or create dependency rules searching for bad class references.

Tags

  • Globals

Conclusion

We should restrict our classes to small domains and expose just facades to the outside. This greatly reduces coupling.

Relations

More info

Credits

Photo by Alfons Morales on Unsplash


Write shy code — modules that don't reveal anything unnecessary to other modules and that don't rely on other modules' implementations.

Dave Thomas



This article is part of the CodeSmell Series.

💖 💪 🙅 🚩
mcsee
Maxi Contieri

Posted on January 31, 2021

Join Our Newsletter. No Spam, Only the good stuff.

Sign up to receive the latest update from our blog.

Related

Code Smell 60 - Global Classes
codenewbie Code Smell 60 - Global Classes

January 31, 2021

Code Smell 32 - Singletons
codenewbie Code Smell 32 - Singletons

November 23, 2020