Java returning 0 in multiplication

Question

The code is even simple to calculate a seller’s yield by multiplying in the case of 150000 * 0.1 it returns 0

code /

public class FuncionarioComissionado {
//Strings private para poder usar Get/Set
//Atributos
private String nome;
private String CPF;
private double taxaComissao;
private double vendasBrutas;
private double rendimentos;


// Criação de um construtor para a classe FuncionarioComissionado
// Construtor
public FuncionarioComissionado(String nome, String CPF, double taxaComissao, double vendasBrutas) {
	this.nome = nome;
	this.CPF = CPF;
	this.taxaComissao = taxaComissao;
	this.vendasBrutas = vendasBrutas;
}
// Calculo dos Rendimentos
//Metodo
private void Rendimentos(){
	rendimentos = (vendasBrutas * (taxaComissao / 100));
}

// Criação de Get's para cada Atributo
// Metodos
public double getRendimentos() {
	return rendimentos;
}
public String getNome() {
	return nome;
}
public String getCPF() {
	return CPF;
}
public double getTaxaComissao() {
	return taxaComissao / 100;
}
public double getVendasBrutas() {
	return vendasBrutas;
}
}

test /

FuncionarioComissionado f1 = new FuncionarioComissionado("Guilherme", "12345678921",10,125000);
    System.out.println(f1.getNome());
    System.out.println(f1.getCPF());
    System.out.println(f1.getTaxaComissao());
    System.out.println(f1.getVendasBrutas());
    System.out.println(f1.getRendimentos());

Answer 1

Returns 0 because your method getRendimentos() is simply returning the initialization value of the variable rendimentos, at any time of the code you perform calculation.

I suggest two modifications to the code:

• do not start method names with uppercase letter, java convention asks that only the name of the constructor can be capitalized, and the other methods follow the format camelcase;

• if you need to store the proceeds separately, first call the method you calculate and then call the getter from it.

With these changes, the class stays this way:

class FuncionarioComissionado {
    //Strings private para poder usar Get/Set
    //Atributos
    private String nome;
    private String CPF;
    private double taxaComissao;
    private double vendasBrutas;
    private double rendimentos;


    // Criação de um construtor para a classe FuncionarioComissionado
    // Construtor
    public FuncionarioComissionado(String nome, String CPF, double taxaComissao, double vendasBrutas) {
        this.nome = nome;
        this.CPF = CPF;
        this.taxaComissao = taxaComissao;
        this.vendasBrutas = vendasBrutas;
    }

    // Calculo dos Rendimentos
    //Metodo
    public void calculaRendimentos(){
        rendimentos = (vendasBrutas * (taxaComissao / 100));
    }

    // Criação de Get's para cada Atributo
    // Metodos
    public double getRendimentos() {
        return rendimentos;
    }
    public String getNome() {
        return nome;
    }
    public String getCPF() {
        return CPF;
    }
    public double getTaxaComissao() {
        return taxaComissao / 100;
    }
    public double getVendasBrutas() {
        return vendasBrutas;
    }
}

The test:

class Teste {

    public static void main (String[] args) {

    FuncionarioComissionado f1 = new FuncionarioComissionado("Guilherme", "12345678921",10,125000);
    System.out.println(f1.getNome());
    System.out.println(f1.getCPF());
    System.out.println(f1.getTaxaComissao());
    System.out.println(f1.getVendasBrutas());
    f1.calculaRendimentos();
    System.out.println(f1.getRendimentos());

    }
}

See working on ideone