Эффективный параметр, проходящий в методе в java

Question

У меня есть такая подпись метода.

public void calculateFinalInvoice(int a, int b, int c, int d, int e, int f, int g, int h, int i, InvoiceDO invoiceDO) { 
    // TO DO 

} 

Передаю некоторые параметры. Так что лучше передать все эти параметры производительности. Или 1) поддерживать класс для всех этих параметров 2) поддерживать HashMap для всех этих параметров 3) для этого необходимо использовать ArrayList.

Можете ли вы предложить, какой из них лучше достичь производительности.

Answer 1

Какую версию Java вы используете? Что вы делаете с этими параметрами. Вы могли бы быть лучше перемещая DAO к передней части списка аргументов, а затем с помощью varargs пройти однако многие int сек вы хотите:

calculateFinalInvoice(InvoiceDO invoiceDO, int args...) { 
} 

Answer 2

вы можете просто передать массив Int

calculateFinalInvoice(int[] values, InvoiceDO invoiceDO) { } 

Answer 3

Do не беспокойтесь о микроэффективности управления такими вещами. Код для ясности и после завершения кодирования, при необходимости, заботиться об эффективности. В этом случае вам может потребоваться переосмыслить ваш дизайн и создать дополнительную структуру для вашего кода. Например, вы можете использовать массивы:

public void calcualteFinalInvoices(int [] invoices, InvoiceDO action){ 
    //... 
} 

Answer 4

Забудьте о производительности. Это не проблема. Проблема заключается в том, что точно так же, как формула для инженера не должна превышать 1 дюйм, метод не должен содержать более трех аргументов. Если вам нужно больше, измените свой дизайн.

Возможно, вам необходимо создать специальный класс параметров и передать его экземпляр методу. Или поместите метод в другой класс, который хранит некоторые данные в виде полей экземпляров ...

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

Answer 5

Я бы осмелился дать вам более ООП, более читаемый подход.

Ваш образец кода сродни вызову trim("my string") вместо "my string".trim(). Это процедурное программирование, в котором код работает с пассивными данными (trim, работающий на "my string"). В ООП мы вместо этого отправляем сообщения на объекты для выполнения запросов, например, в "my string".trim()".

Таким образом, альтернатива идентифицирует Invoice как концепцию, которая может быть запрошена для ее значения или содержания. Затем мы используем method chaining и encapsulation, чтобы получить наиболее читаемую и удобную для пользователя программу. Если разные переменные имеют особые значения, вы можете украсить класс Invoice со специальными версиями add, например addBalance и так далее.

class Invoice { 
    private final List<Integer> values = new ArrayList<Integer>(); 

    public Invoice add(int value) { 
     values.add(value); 
     return this; 
    } 

    public double value() { 
     // Use the values to compute the final invoice. 
     return computedFinalInvoice; 
    } 
} 

Использование так:

double finalInvoiceValue = new Invoice() 
    .add(1) 
    .add(2) 
    .add(100) 
    .value();