Code Smell 17 - Global Functions

Code Smell 17 - Global Functions

Maxi Contieri
·Nov 5, 2020·

1 min read

Subscribe to my newsletter and never miss my upcoming articles

Play this article

Discouraged by Object-Oriented Programmings, Many mixed languages support it. And developers abuse them.

TL;DR: Global function bring a lot of coupling. Don't use them.

Problems

  • Coupling

  • Readability

  • Maintainability

  • Testability

Solutions

  • Wrap the function in a context object.

Examples

  • External Resources Access, Database access, Time and Operating System resources.

Sample Code

Wrong

<?

class Employee {
    function taxesPayedUntilToday() {
        return database()->select(
            "SELECT TAXES FROM EMPLOYEE".
            " WHERE ID = " . $this->id() .
            " AND DATE < " . currentDate());
    }
}
<?

final class EmployeeTaxesCalculator {
    function taxesPayedUntilToday($context) {
        return $context->SelectTaxesForEmployeeUntil(
            $this->ssn,
            $context->currentDate());
    }
}

Detection

Many modern languages avoid them. For the permissive ones, scope rules can be applied and automatically checked.

Tags

  • Global

Conclusion

Structured programming considers global functions harmful. Yet, we can observe some bad practices cross paradigm boundaries.

Relations

  • Singleton and Classes are global points of access.

More Info

Credits

Photo by Mae Mu on Unsplash


The road to programming hell is paved with global variables.

Steve McConnell


This article is part of the CodeSmell Series.

Last update: 2021/06/26

 
Share this

Impressum

Technical Opinions are my own. I don't have the revealed truth.

Software Design is a creative activity. These are hints and not rigid rules.

I write on BackEnd Business Systems and OOP Design. My advice/experience might not suit other systems.

You can write me at info(at)maximilianocontieri.com