1

O codigo é até simples fazer o calculo do rendimento de um vendedor porem na multiplicação no caso de 150000 * 0,1 ele retorna 0

codigo /

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;
}
}

teste /

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());
8
  • Em que ponto o erro ocorre?
    – user28595
    6/10/2018 às 23:47
  • tentando descobrir mas o erro seria o getRendimentos retornando 0 ao invés de (125000 * 0,1) 6/10/2018 às 23:48
  • Voce nao está calculando rendimentos, só está retornando um valor vazio da variavel.
    – user28595
    6/10/2018 às 23:51
  • Esta conta nunca chega a ocorrer, pq vc simplesmente nao a efetua.
    – user28595
    6/10/2018 às 23:51
  • Isso não efetuaria a conta? private void Rendimentos(){ rendimentos = (vendasBrutas * (taxaComissao / 100)); 6/10/2018 às 23:52

1 Resposta 1

3

Retorna 0 porque seu método getRendimentos() está simplesmente retornando o valor de inicialização da variável rendimentos, em momento algum do código você efetua cálculo.

Sugiro duas modificações no código:

  • não inicie nome de métodos com letra maiúscula, a convenção do java pede que somente o nome do construtor possa ser maiúsculo, e os demais métodos seguem o formato camelcase;

  • se precisa armazenar os rendimentos a parte, primeiro chame o método que calcula e depois chame o getter dele.

Com essas alteraçoes, a classe fica desta forma:

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;
    }
}

O teste:

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());

    }
}

Veja em funcionamento no ideone

4
  • 1
    Obrigado eu entendi o que tava faltando com a ajuda do seus comentarios, porém errei na construção do codigo, faltou os () em f1.calculaRendimentos(); 7/10/2018 às 0:06
  • Melhor ainda seria calcular os rendimentos no construtor, pois não há razão para deixar isso para depois quando um método específico for chamado. 7/10/2018 às 2:03
  • Ideal seria mas não sei como fazer isto >.< 7/10/2018 às 2:21
  • @guilhermecabrals Basta colocar a linha rendimentos = (vendasBrutas * (taxaComissao / 100)); no final do construtor e então você pode deletar o método calculaRendimentos (ou Rendimentos de acordo com seu código original) e todas as chamadas a ele. 7/10/2018 às 2:33

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .