Strategie per il paramento dei parametri

Methods with many parameters are often unavoidable.

Da quando?

Voglio dire che potrebbe essere inevitabile avere un paio di metodi come questo in una base di codice non banale, ma la vasta maggioranza dei tuoi metodi avrà uno o due parametri.

Avere una barca di parametri è un segno che:  1. La tua funzione sta facendo troppo.  2. Il carico della tua barca di parametri separati è molto più coeso di quanto pensassi.

Il modo in cui risolvi il problema dipende da quale segno è applicabile. Se la tua funzione sta facendo troppo, allora suddividerla creerà naturalmente meno parametri, anche se una funzione di orchestrazione ne ha troppi. Il codice più isolato rende anche più facile vedere cosa sta succedendo per semplificare ulteriormente.

Se i tuoi parametri sono più coesivi, allora è meglio aggregarli (o almeno alcuni di essi) in un utile pacchetto / classe / struttura poiché sarebbe usato al di fuori di questa sola funzione.

Ma almeno personalmente, trovo che gli oggetti parametro siano vili. Se i parametri sono coesivi, dovrebbero essere stati in una classe dall'inizio. Se li stai trasformando in una classe "per rendere questa funzione più carina", allora ti manca il punto. Risolvi il problema, non il sintomo.

Il fattore chiave qui è quanto coesistono i parametri.

Se hai un gruppo di parametri logicamente raggruppati e dovrebbe appartenere a una classe, allora crea una classe per loro. Se si verificano insieme negli elenchi di parametri ma non sono raggruppati in modo logico, tenerli separati.

Mantieni questo metodo così com'è:

function getDistance(Point p1, Point p2) { ... }

Questo potrebbe essere meglio servito con un oggetto per raggruppare i parametri:

function manufacture(Make make, Model model, Year year) { ... }

Questi parametri funzionano tutti insieme per descrivere qualcosa:

class VehicleDescriptor {
  Make make;
  Model model;
  Year year;
}

function manufacture(VehicleDescriptor v) { ... }