leandro-bsf / Comex

0 stars 0 forks source link

Criar método getValorTotal em PedidoCsv #4

Open cacio-costa opened 2 years ago

cacio-costa commented 2 years ago

Leandrão, as linhas 10 e 11 calculam o valor total do PedidoCsv. O que essas linhas fazem? Pedem informações do Pedido (usando métodos get) para calcular o total.

Nesse caso, seria melhor criar um novo método em PedidoCsv que devolvesse esse cálculo:

public class PedidoCsv {
    // código existente omitido

    public double getValorTotal() {
        return Integer.valueOf(this.quantidade) * Double.valueOf(this.valor);
    }
}

Dessa forma, a gente distribui os comportamentos de maneira adequada nas classes corretas. E o Comparator ficaria mais simples:

double totalPd1 = p1.getValorTotal();
double totalPd2 = p2.getValorTotal();

// código omitido

https://github.com/leandro-bsf/Comex/blob/cc2619ebceea6a87bbef054771b28d1b16e1b239/comex/src/br/com/comex/csv/PedMaxMin.java#L10