Ist es gut, minimale Parameter zu übergeben?

Question

Angenommen, ich habe ein Objekt Auto, mit fünf Parametern, {Zahlen, Farbe, Kilometerstand, Pferdestärken, maxSpeed}. Ich habe eine Methode, die 3 dieser Werte benötigt. Welche der beiden Optionen gilt als Best Practice? Ist es besser, umschließendes Objekt zu übergeben und die Anzahl der Parameter zu reduzieren, ODER einfach nur minimale Daten an eine Methode zu übergeben (zB: bei Methode 2 wird auf Zahlen und Farbe nicht zugegriffen)?

1. Option 1 Pass ganzes Objekt:

   void compute(Car c, Person p) { 
       return c.mileage + c.horsepower + c.maxSpeed + p.age; 
   } 
   

2. Option 2: gerade in den Verfahren Werte zu.

   void compute(int mileage, int horsepower, int maxSpeed, int age) { 
       return mileage + horsepower + maxSpeed + age.; 
   } 
   

Hinweis: davon ausgehen, aus irgendeinem Grund, berechnen kann nicht ein Teil der Car-Klasse sein. Bitte antworten Sie mit dieser Annahme.

Answer 1

Es ist besser, das eingeschlossene Objekt zu übergeben, da jede Veränderung in der Anzahl der Parameter bei der Berechnung nicht erforderlich, die Methodensignatur in Zukunft beeinflussen würde.

Sie können tatsächlich eine Schnittstelle Typ, sagen, Vehicle stattdessen akzeptieren. Dies ermöglicht, dass die gleiche Methode später Daten auf Bike, Jet usw. berechnen kann.

public int compute(Vehicle veh) { 
    return veh.computePerformance(); 
} 

wo Car implementiert das Verfahren als

public int computePerformance() { 
    return mileage + horsepower + maxSpeed; 
} 

Die Idee ist, Code zu schreiben, der sich leicht auf Dehnbarkeit verleiht.

Answer 2

Ich würde eine dritte Option vorschlagen: Fügen Sie eine Methode "compute" zu Auto hinzu.

Dies heißt "Sag, frag nicht".

Siehe http://martinfowler.com/bliki/TellDontAsk.html

Answer 3

Ich schlage zweite Option vor.

Warum?

Annahme eines: Ihre Rechenmethode wird von einer anderen Klasse verwendet Kilometer zu berechnen sagte 'Klasse Eigentümer

Annahme zwei: Angenommen, morgen Ihre Rechenmethode eines mehr benötigt Attribut aus der Klasse Auto zu berechnen das Meilenalter.

Wenn Sie nach Option eins gehen: dann müssen Sie einen weiteren Parameter in method compute einführen, was zu Kompilierungsproblemen in anderen Klassen führen würde, die Ihre Methode aufrufen. In diesem Fall 'Klasseninhaber'.

Wenn Sie nach Option zwei gehen: Sie haben die Flexibilität, um Ihre Implementierung der Berechnungsmethode zu aktualisieren, ohne irgendeine andere Klasse zu beeinflussen, die diese Methode verwendet.