Хорошо ли передавать минимальные параметры?

Question

Предположим, у меня есть автомобиль с пятью параметрами, {numwheels, цвет, пробег, лошадиная сила, maxSpeed}. У меня есть метод, который нуждается в 3 из этих значений. Какой из двух вариантов считается лучшей практикой? Лучше ли передавать охватывающий объект и уменьшать количество параметров, ИЛИ просто передавать минимально-минимальные данные в метод (например: числовые символы и цвет не будут доступны в методе 2)?

1. Вариант 1 проход весь объект:

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

2. Вариант 2: передать все значение методы.

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

Примечание: предположим, по какой-то причине, вычислить не может быть частью класса Car. Пожалуйста, ответьте на это предположение.

Answer 1

Это лучше передать прилагаемое объект, поскольку любое изменение числа параметров, необходимых при расчете не будет влиять на сигнатуру метода в будущем.

Вы можете, фактически, принять интерфейс, скажем, Vehicle. Это позволяет одному и тому же методу вычислить данные по Bike, Jet и т. Д. Далее.

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

, где Car реализует метод как

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

Идея заключается в том, чтобы писать код, который легко поддается расширяемости.

Answer 2

Я бы предложил третий вариант: добавьте метод «вычислить» в «Автомобиль».

Это называется «Скажи, не спрашивай».

См http://martinfowler.com/bliki/TellDontAsk.html

Answer 3

Предлагаю второй вариант.

Почему?

Допущение один: Ваш метод вычисления используется какой-либо другой класс, чтобы вычислить пробег сказать «класс Владелец»

Успенская два: Предположим, что завтра ваш метод вычисления необходим еще один атрибут из класса Car, чтобы вычислить возраст мили.

Если вы идете по первому варианту: тогда вам придется ввести еще один параметр в вычислении метода, который приведет к проблемам компиляции в других классах, вызывающих ваш метод. В этом случае «владелец класса».

Если вы переходите по второму варианту: у вас есть гибкость , чтобы обновить реализацию метода вычислений, не затрагивая какой-либо другой класс, который использует этот метод.