Maxi Contieri

Posted on Sep 26, 2021 • Originally published at maximilianocontieri.com

Code Smell 87 - Inconsistent Parameters Sorting

#programming #webdev #codenewbie #functional

Be consistent with the parameters you use. Code is prose.

TL;DR: Don't confuse you readers. Keep the order.

Problems

Readability
Consistency

Solutions

Refactor and change parameters order.
Use named parameters

Sample Code

Wrong

function giveFirstDoseOfVaccine(person, vaccine) {
  //
}

function giveSecondDoseOfVaccine(vaccine, person) {
  //
}


giveFirstDoseOfVaccine(jane, pfizer);
giveSecondDoseOfVaccine(jane, pfizer);  //Unnoticed mistake

Right

function giveFirstDoseOfVaccine(person, vaccine) {
  //
}

function giveSecondDoseOfVaccine(person, vaccine) {
  //
}


giveFirstDoseOfVaccine(jane, pfizer);
giveSecondDoseOfVaccine(jane, pfizer);  //jane is immunized

Detection

Some very smart linters may be able to compare arguments and hint for possible mistakes.

Conclusion

This is a very simple smell.

Readability is very important to avoid mistakes.

Relations

Code Smell 10 - Too Many Arguments

Maxi Contieri ・ Oct 29 '20

#codenewbie #beginners #100daysofcode #oop

Credits

Photo by Lance Grandahl on Unsplash

Computers are good at following instructions, but not at reading your mind.

Donald Knuth

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

Top comments (1)

James Robb • Sep 30 '21

Usually for cases like this I would create a factory to avoid the ambiguity, something like:

type Vaccine = "pfizer" | "moderna"
type Patient = {
    name: string;
}
type PatientWithCoronaVaccineDetails = Patient & {
    vaccine: Vaccine;
    doses: number;
}

function vaccineFactoryForPatient(patient: PatientWithCoronaVaccineDetails) {
  // ...

  return {
    giveFirstDoseOfVaccine() {
        // ...
    },
    giveSecondDoseOfVaccine() {
        // ...
    }
  }
}
}

This way we can avoid this category of issues in the first place.

DEV Community