Retorna dobrado

4 de julho de 2011 3 respostas

ozzyblank 4 de julho de 2011

boa noite pessoal estou com dificuldades de somar um retorno, seria o seguindo é um metodo que recebe um valor e toda vez que ele receber um valor ele retorna o dobro o que esta errado segue o codigo.

ublic class SLVeiculos  {
	
	String novoOuUsado, novo = "n", usado= "u";
	 double valorVenda,SomaComissaoVeiculo,valorAcessorio,valorFinanciado,porcentagem,comissaoFinanciamento,valorTotal,valorTotal2;	
	
	
	public double comissaoVeiculo (double valorVenda){
		
		this.valorVenda = 0;
		this.valorVenda = valorVenda;
		this.SomaComissaoVeiculo =0;
		if(novoOuUsado == null){
			JOptionPane.showMessageDialog(null,("Selecione um Tipo de Venda!!! "));// mensagem para o usuario
		}
		
		if(this.novoOuUsado.equalsIgnoreCase(novo)){
			this.SomaComissaoVeiculo = valorVenda * 0.004;
			valorTotal = SomaComissaoVeiculo;
			valorTotal2 =  valorTotal + valorTotal2;// aqui que quero que retorne a soma dos valores.
		
			
		}
		if(this.novoOuUsado.equalsIgnoreCase(usado)){
			this.SomaComissaoVeiculo = valorVenda * 0.005;
		}
		/*System.out.println(SomaComissaoVeiculo);*/
		return SomaComissaoVeiculo;
		
	}

seria bem simples o usuario digita um valor de venda de veiculos sendo do tipo novo seria 80000

o metodo faz o calculo 80000 * 0.004 = 320

dai ele digita um novo valor digamos 30000*0.004 = 120

entao no “TESTE” seria 320 + 120 retornaria 440

só que o programa esta fazendo o seguinte

o primeiro ele esta retornando ao inves de 320 ele retorna 640

e o segundo 760 que dai seria o correto se o primeiro fosse 640 entenderam ???

obrigado a quem me ajudar.

3 Respostas

redr4gon 4 de jul. de 2011

ao invés de

JOptionPane.showMessageDialog(null,("Selecione um Tipo de Venda!!! "));

faça

novoOuUsado = OptionPane.showInputMessageDialog("Selecione um Tipo de Venda!!! ");

ali mude… ao invés de

valorTotal2 =  valorTotal + valorTotal2;// aqui que quero que retorne a soma dos valores.

faça

valorTotal2 =  valorTotal + ValorVenda;// aqui que quero que retorne a soma dos valores.

nem testei nem nada… da uma olhada e ve se ajudou algo

ViniGodoy 5 de jul. de 2011

Tem várias coisas estranhas no seu código.

1. Por que você declarar um monte de variáveis que não usa?
2. O que o método realmente deve retornar? Por que, ao final, você está retornando apenas "SomaComissaoVeiculo", mas há uma série de cálculos intermediários que aparentemente não estão fazendo nada.
3. Declare variáveis no ponto mais próximo possível onde você irá utiliza-las.
4. Por que sua função altera outras propriedades da classe?

Vou postar aqui o que entendi que o seu código deveria ser:

public class SLVeiculos  {	   
   double valorVenda, somaComissaoVeiculo,valorAcessorio, valorFinanciado, porcentagem, comissaoFinanciamento, valorTotal;	
   public double comissaoVeiculo (double valorVenda){
      //Perguntamos o tipo de venda
      int novo = JOptionPane.YES_OPTION;
      int usado = JOptionPane.NO_OPTION;
      Object[] options = {"Novo", "Usado"};
      int novoOuUsado = new JOptionPane.showOptionDialog(null, "Selecione um tipo de venda", "Tipo de venda", 
         JOptionPane.YES_NO_OPTION,   JOptionPane.QUESTION_MESSAGE,   null, options, options[0]);

      if (novoOuUsado == JOptionPane.CANCEL_OPTION)
          return -1;
        
      //1. Vamos colocar a verificação no início para não mexer em valores caso a chamada de método seja inválida
      if(novoOuUsado == null){
         JOptionPane.showMessageDialog(null,"Selecione um Tipo de Venda!!! ");
         return; //2. Caímos fora do método.
      }

      //2. Definimos o valor da venda. Isso não deveria estar aqui, já que o nome desse 
     // método indica que ele só calcularia a comissão do veículo.
      this.valorVenda = valorVenda;
      //3. Definimos o percentual da comissão		
      if(novoOuUsado == novo){
         this.somaComissaoVeiculo = valorVenda * 0.005;
      }
      else if(novoOuUsado == usado){
         this.somaComissaoVeiculo = valorVenda * 0.004;
      }
      //4. Calculamos o valor total
      valorTotal = valorVenda + somaComissaoVeiculo;
      return somaComissaoVeiculo;
   }
}

ozzyblank 5 de jul. de 2011

Boa Tarde tentei fazer o que voces falara mais sem sucesso, o codigo a cima nao é completo segue o codigo completo para voces entenderem

CLASSE SLVEICULOS ( CODIGO)

import javax.swing.JOptionPane;


public class SLVeiculos  {
	
	String novoOuUsado, novo = "n", usado= "u";
	 double valorVenda,SomaComissaoVeiculo,valorAcessorio,valorFinanciado,porcentagem,comissaoFinanciamento,valorTotal,valorTotal2,valorTotal3;	
	
	
	public double comissaoVeiculo (double valorVenda){
		
		this.valorVenda = 0;
		this.valorVenda = valorVenda;
		this.SomaComissaoVeiculo =0;
		if(novoOuUsado == null){
			JOptionPane.showMessageDialog(null,("Selecione um Tipo de Venda!!! "));// mensagem para o usuario
		}
		
		if(this.novoOuUsado.equalsIgnoreCase(novo)){
			this.SomaComissaoVeiculo = valorVenda * 0.004; 
			this.valorTotal=0;
			this.valorTotal = SomaComissaoVeiculo; 
			this.valorTotal2=0;
           this.valorTotal2 =valorTotal ;
            this.valorTotal3+=  valorTotal2;
            
            
		}
		if(this.novoOuUsado.equalsIgnoreCase(usado)){
			this.SomaComissaoVeiculo = valorVenda * 0.005;
		}
		/*System.out.println(SomaComissaoVeiculo);*/
		return SomaComissaoVeiculo;
		
	}
	
	public double comissaoAcessorio(double valorAcessorio){
		this.valorAcessorio =0;
		this.valorAcessorio = valorAcessorio * 0.05;
		/*System.out.println(this.valorAcessorio);*/
		return  this.valorAcessorio;
	}
	
	public double comissaoFinanciamento(double valorFinanciado){
		this.valorFinanciado =0;
		this.comissaoFinanciamento =0;
		this.valorFinanciado = valorFinanciado;
		
		
		/*System.out.println("Valor Comissao: " + this.comissaoFinanciamento);*/
		return valorFinanciado;
		
		
	}
	
	public double comissaoFinanciamento2(double porcentagem){
		this.porcentagem =0;
		this.porcentagem = porcentagem;
		this.valorFinanciado =valorFinanciado;
		this.comissaoFinanciamento = this.valorFinanciado * this.porcentagem * 0.20;
		return comissaoFinanciamento;
	}
	
	public String mostra(){
		System.out.println("Comissao da Venda do Veiculo: "  + this.SomaComissaoVeiculo+"   "   + "Comissao Venda Acessorios: "  + this.valorAcessorio+"   "    + 
				"Comissao do Financiamento: " +" " + comissaoFinanciamento);
		return "Comissao da Venda do Veiculo:  " + this.SomaComissaoVeiculo + " \n" +"-------------------------------------------------------------------------------" +"\n"   + "Comissao Venda Acessorios: "  + this.valorAcessorio  + "\n" +"-------------------------------------------------------------------------------" +"\n"   + 
		"Comissao do Financiamento:  "  + comissaoFinanciamento +"\n" + "-------------------------------------------------------------------------------" + "\n" +"\n"+"\n" + "teste: " + valorTotal3;
	}

	public void setNovoOuUsado(String novoOuUsado) {
		this.novoOuUsado = novoOuUsado;
	}

	public void setValorVenda(double valorVenda) {
		this.valorVenda = valorVenda;
	}

	public void setValorAcessorio(double valorAcessorio) {
		this.valorAcessorio = valorAcessorio;
	}

	
	
}


CLASSE ARRAY


public class Array {
	
	SLVeiculos [] sl;
	 
 	
	 public Array(){
		 this.sl = new SLVeiculos [50];
		 }
	 
	 
	 void adicionaArray(SLVeiculos f ){
	 		for(int x = 0; x < this.sl.length; x++ ){
				if(sl[x] == null){	
					this.sl[x] = f;
					break;
				}
			}
	 }		
	 public String verificar(){
			for(int x = 0; x < this.sl.length; x++){  
				if(this.sl[x] == null){  
					System.out.println("Nao possui DADOS !!!");  
				}
				else{
				
				
				return this.sl[x].mostra(); 
				}
			}return null; 
			 
		}

}


CLASSE INTERFACE

import java.awt.Image;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.ButtonGroup;
import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JRadioButton;
import javax.swing.JScrollBar;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
import javax.swing.JTextField;
import javax.swing.ScrollPaneConstants;


public class Interface extends JFrame implements ActionListener  {
	
	SLVeiculos sl = new SLVeiculos();
	Array a1 = null;
	Array sl2  = new Array();
	JScrollPane scroll;
	
	
	
	
	
	
	
	
	
	 
	
	
	
	
	JButton botaovendaveiculo;
	JButton botaovendaacessorio;
	JButton botaovendafinanciamento;
	JButton botaoResultado;
	JButton botaoSair;
	JButton botaoSalvar;
	JButton botaoLimpa; 
	JButton botaoOK;
	
	JRadioButton novo,usado;
	ButtonGroup botaogrupo;
	
	 JLabel lTexto = new JLabel(new ImageIcon("C:/Users/Juliano2/Desktop/SLVeiculos.jpg"));
	
	
	JTextField vendaveiculo2 = new JTextField();
	JTextField vendaacessorio2 = new JTextField();
	JTextField vendafinanciamento2 = new JTextField();
	JTextField Resultado2 = new JTextField();
	JTextField tipo = new JTextField();
	JTextField porcentofinanciamento2 = new JTextField();
	
	JTextArea txt = new JTextArea();
	
	Interface(SLVeiculos slv){
		this.sl = slv;
		
		 scroll = new JScrollPane(txt);
	
	
		
		 
		 
		 
		 
		
		 
		 
		 
		 
		 
		 
		 
		 
		 
		 
		 
		 
		 
		 
		 
		 
		 
		 
		 
		 
		 
	
		
		
		setTitle("SL Veiculos Comissoes");
		setSize(800,820);
		setResizable(false);
		
		//Setando as Escritas
		JLabel vendaveiculo = new JLabel("Valor Venda Veiculo: ");
		JLabel vendaacessorio = new JLabel("Valor Venda Acessorio: ");
		JLabel vendafinanciamento = new JLabel("Valor Venda Financiamento: ");
		JLabel tipo2 = new JLabel("Tipo de Venda:  ");
		JLabel porcentofinanciamento = new JLabel("Porcento Financiamento: ");
		
		
		
		
		
	     Resultado2 = new JTextField();
		
		// os Botoes OK
		botaovendaveiculo = new JButton("OK");
		botaovendaacessorio = new JButton ("OK");
		botaovendafinanciamento = new JButton ("OK");
		botaoResultado = new JButton ("Resultado ");
		botaoSair = new JButton ("Sair");
		botaoSalvar = new JButton ("Salvar");
		botaoLimpa = new JButton ("Limpar");
		botaoOK = new JButton("OK");
		
		novo = new JRadioButton("Novo");
		usado = new JRadioButton("Usado");
		botaogrupo = new ButtonGroup();
		
		botaogrupo.add(usado);//é preciso colocar separado para funcionar ou seja é necessario colocar em buttongroup
		botaogrupo.add(novo);
		
		
		//setando nullo o Layout
		getContentPane().setLayout(null);
		
		
		
		//Setando os lugares dos objetos
		vendaveiculo.setBounds(20, 60,200, 20);
		vendaacessorio.setBounds(20, 100, 150, 50);
		vendafinanciamento.setBounds(20, 155, 200, 50);
		porcentofinanciamento.setBounds(200, 155, 200, 50);
		
		vendaveiculo2.setBounds(20,80,150,20);
		tipo2.setBounds(18,10,200,20);
		
		vendaacessorio2.setBounds(20,135,150,20);
		vendafinanciamento2.setBounds(20, 190, 150, 20);
		porcentofinanciamento2.setBounds(200, 190, 150, 20);
		//txt.setBounds(10, 300, 773, 200);
		botaoResultado.setBounds(10, 240, 100, 50);
		//botaoSalvar.setBounds(160, 240, 100, 50);
		novo.setBounds(110, -5, 60, 50);
		usado.setBounds(180, -5, 100,50);
		lTexto.setBounds(390, 13, 400, 100);
		scroll.setBounds(10, 300, 775, 470);
		botaoLimpa.setBounds(160, 240, 100, 50);
		botaoOK.setBounds(360, 185, 53, 30);
		
		
		
		
		
		
		
		
		
		
		
		
		
		
		
		
		getContentPane().add(vendaveiculo);
		getContentPane().add(vendaveiculo2);
		getContentPane().add(tipo);
		getContentPane().add(tipo2);
		getContentPane().add(vendaacessorio);
		getContentPane().add(vendaacessorio2);
		getContentPane().add(vendafinanciamento);
		getContentPane().add(vendafinanciamento2);
		getContentPane().add(porcentofinanciamento);
		getContentPane().add(porcentofinanciamento2);
		//getContentPane().add(Resultado2);
		getContentPane().add(botaoResultado);
		//getContentPane().add(botaoSalvar);
		getContentPane().add(novo);
		getContentPane().add(usado);
		getContentPane().add(lTexto);
		//getContentPane().add(txt);
		getContentPane().add(scroll);
		 getContentPane().add(botaoLimpa);
		 getContentPane().add(botaoOK);
		
		 
		 
		 
		 
		 scroll.setVerticalScrollBarPolicy(ScrollPaneConstants.VERTICAL_SCROLLBAR_ALWAYS);//tornar visivel a barra de rolagem
		
		
		
		
		
		
		vendaveiculo2.addActionListener(this);
		tipo.addActionListener(this);
		vendaacessorio2.addActionListener(this);
		vendafinanciamento2.addActionListener(this);
		porcentofinanciamento2.addActionListener(this);
		//Resultado2.addActionListener(this);
		botaoResultado.addActionListener(this);
		//botaoSalvar.addActionListener(this);
		novo.addActionListener(this);
		usado.addActionListener(this);
		 botaoLimpa.addActionListener(this); 
		 botaoOK.addActionListener(this); 
		
		
		
		
		
		
		
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setVisible(true);
}
	
	
	
	
	public void actionPerformed (ActionEvent e){
		                
            		  
		if(e.getSource() == vendaveiculo2 ){   
            this.sl.comissaoVeiculo(Double.parseDouble(vendaveiculo2.getText())); //Double.parseDouble faz mudar para double o resultado
            
           
        }   
		
		/*if(e.getSource() == Resultado2){
			Resultado2.setText(this.sl.mostra());
			
			
		
			
			
		}*/
		
		if(e.getSource() == botaoResultado){
			sl.setValorAcessorio(sl.valorAcessorio);
			sl.comissaoFinanciamento(sl.valorFinanciado);
				sl.comissaoVeiculo(sl.valorVenda);
					sl.comissaoFinanciamento2(sl.porcentagem);
						sl2.adicionaArray(sl);
			txt.setText((txt.getText().toString()) + sl2.verificar());
			
		 
		}

			
			
			
			
		/*if(e.getSource() == tipo){
			this.sl.setNovoOuUsado(tipo.getText());
		} se USA PARA DIGITAR O n OU u FOI MUDADO PARA RADIOBOTTOM
		*/
		if(e.getSource() == vendaacessorio2){
			this.sl.comissaoAcessorio(Double.parseDouble(vendaacessorio2.getText()));
		}
		if(e.getSource() == vendafinanciamento2 ){
			this.sl.comissaoFinanciamento(Double.parseDouble(vendafinanciamento2.getText()));
		}
		if(e.getSource()== porcentofinanciamento2 ){
			this.sl.comissaoFinanciamento2(Double.parseDouble(porcentofinanciamento2.getText()));
		}
		
		/*if(e.getSource() == botaoSalvar){
			sl.setValorAcessorio(sl.valorAcessorio);
				sl.comissaoFinanciamento(sl.valorFinanciado);
					sl.comissaoVeiculo(sl.valorVenda);
						sl.comissaoFinanciamento2(sl.porcentagem);
							sl2.adicionaArray(sl);
			}	 
			// salvar em um array
			*/
			
		
		
		if(e.getSource() == novo){
			this.sl.setNovoOuUsado(this.sl.novo);
	}
		if(e.getSource() == usado){
			this.sl.setNovoOuUsado(this.sl.usado);
	}	
		   if(e.getSource() == botaoLimpa){  
	            this.sl.comissaoFinanciamento =0;  
	               this.sl.porcentagem = 0;  
	                 this.sl.novoOuUsado = null;  
	                   this.sl.valorFinanciado =0;  
	                     this.sl.valorVenda =0;  
	                       this.sl.valorAcessorio =0;  
	    }  
		   if(e.getSource() == botaoOK){
			   this.sl.comissaoVeiculo(Double.parseDouble(vendaveiculo2.getText()));
				this.sl.comissaoFinanciamento(Double.parseDouble(vendafinanciamento2.getText()));
				this.sl.comissaoFinanciamento2(Double.parseDouble(porcentofinanciamento2.getText()));
			   this.sl.comissaoAcessorio(Double.parseDouble(vendaacessorio2.getText()));
			
				
			   
				
		}
		
}	
	
	
	
	
	 
	
	
	
	
	
	
	
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		
		 SLVeiculos sl10 = new SLVeiculos();
		
		  Interface add = new Interface(sl10); 
		
		  

	}

}

conseguiram entender o que esta acontecendo ?

obrigado assim mesmo pelas respostas.

Criado 4 de julho de 2011

Ultima resposta 5 de jul. de 2011

Respostas 3

Participantes 3

3 Respostas

Topicos relacionados