Problemas com if

Programação Java
#1

da danod erro no código abaixo…como sou iniciante não consegui resolver:

import java.awt.;
import java.awt.event.;
import javax.swing.*;

public class OtraCalc extends JFrame {
private JRadioButton Soma, Mult, Sub, Div;
private JButton Calcular, Limpar;
private JTextField txt1, txt2, txt3;
private ButtonGroup Operacao;
private JLabel Label1;
private Container container;
double Val1, Val2, RSoma, RSub, RMult, RDiv;

public OtraCalc() {
super(“Cansei de faze calculadora”);

Container container = getContentPane();
container.setLayout( new FlowLayout() );

txt1 = new JTextField("",10);
container.add(txt1);

txt2 = new JTextField("",10);
container.add(txt2);

Soma = new JRadioButton("+", true);
container.add(Soma);

Sub = new JRadioButton("-", false);
container.add(Sub);

Mult = new JRadioButton("*", false);
container.add(Mult);

Div = new JRadioButton("/", false);
container.add(Div);

Operacao = new ButtonGroup();
Operacao.add(Soma);
Operacao.add(Sub);
Operacao.add(Mult);
Operacao.add(Div);

Calcular = new JButton(“Calcular”);
add.ActionListener(

new ActionListener() {

public void actionPerformed(ActionEvent actionevent) {
Val1 = Double.parseDouble(txt1.getText());
Val2 = Double.parseDouble(txt2.getText());
RSoma = Val1 + Val2;
RSub = Val1 - Val2;
RMult = Val1 * Val2;

if (event.getSource() == Soma) {
txt3.getText(RSoma);
}
if (event.getSource() == Sub) {
txt3.getText(RSub);
}
if (event.getSource() == Mult) {
txt3.getText(RMult);
}
if (event.getSource() == Div){
RDiv = Val1 / Val2;
txt3.getText(RDiv);
}

}
}
);

container.add(Calcular);
setSize( 300, 300);
setVisible( true );
}

public static void main(String args[]) {
OtraCalc application = new OtraCalc();
application.addWindowListener(
new WindowAdapter() {
public void WindowClosing( WindowEvent windowevent) {;
System.exit( 0 );
}
}

);
}
}

ME AJUDEMMMMMMMM!!!

#2

Olá gk9,

Qual é o erro que tá dando?

#3

Colega, seu primeiro problema com seus if’s está nessa linha:

public void actionPerformed(ActionEvent actionevent)

Você declarou o parâmetro com o nome de actionevent, mas nos if’s está testando event.

Depois disso, o que está estranho é você usar o txt3.getText(suaVariavel). Se o que você quer é fazer o valor da sua variável aparecer no txt3 você tem que usar o método setText em vez do getText. Aí tem outro problema, porque você tem que passar uma String pro setText e suas variáveis são Double… Deu pra resolver usando um DecimalFormat.

Agora o que não deu pra sacar (porque também ainda não manjo bem dessa parte) é o tratamento de evento. O resultado deveria aparecer em txt3 quando você clicar no botão Calcular, não é? Mas tá meio estranha essa implementação do tratamento de evento aí…
Bom, de todo jeito, vou colocar aí o teu código com umas modificações que eu fiz… Compilar compila, falta acertar essa parada do botão.

[code]
import java.awt.;
import java.awt.event.;
import javax.swing.*;
import java.text.DecimalFormat;

public class OtraCalc extends JFrame {
private JRadioButton Soma, Mult, Sub, Div;
private JButton Calcular, Limpar;
private JTextField txt1, txt2, txt3;
private ButtonGroup Operacao;
private JLabel Label1;
private Container container;
double Val1, Val2, RSoma, RSub, RMult, RDiv;

public OtraCalc() {
super(“Cansei de faze calculadora”);

Container container = getContentPane();
container.setLayout( new FlowLayout() );

txt1 = new JTextField("",10);
container.add(txt1);

txt2 = new JTextField("",10);
container.add(txt2);

txt3 = new JTextField("",10);
container.add(txt3);

Soma = new JRadioButton("+", true);
container.add(Soma);

Sub = new JRadioButton("-", false);
container.add(Sub);

Mult = new JRadioButton("*", false);
container.add(Mult);

Div = new JRadioButton("/", false);
container.add(Div);

Operacao = new ButtonGroup();
Operacao.add(Soma);
Operacao.add(Sub);
Operacao.add(Mult);
Operacao.add(Div);

Calcular = new JButton("Calcular");

Calcular.addActionListener(

new ActionListener() {

	  public void actionPerformed(ActionEvent event) {
		Val1 = Double.parseDouble(txt1.getText());
		Val2 = Double.parseDouble(txt2.getText());
		RSoma = Val1 + Val2;
		RSub = Val1 - Val2;
		RMult = Val1 * Val2;

		DecimalFormat resultado = new DecimalFormat("###,###.###");

		if (event.getSource() == Soma) {
			txt3.setText(resultado.format(RSoma));
		}
		if (event.getSource() == Sub) {
			txt3.setText(resultado.format(RSub));
		}
		if (event.getSource() == Mult) {
			txt3.setText(resultado.format(RMult));
		}
		if (event.getSource() == Div){
			RDiv = Val1 / Val2;
			txt3.setText(resultado.format(RDiv));
		}

	}
}
);

container.add(Calcular);
setSize( 300, 300);
setVisible( true );
}

public static void main(String args[]) {
	OtraCalc application = new OtraCalc();
	application.addWindowListener(
	new WindowAdapter() {
	public void WindowClosing( WindowEvent windowevent) {;
		System.exit( 0 );
	}
}

);
}

}
[/code][/b]

#4

Ola…

Como adicionamos um “ActionListener” a um botao o evento gerado ao pressionar o mesmo nunca trara informações sobre os RadioButtons que definem a operação (soma, sub…). Neste caso, dentro do actionPerformed, devemos verificar a situação de cada RadioButton com o metodo “isSelected()” e em caso positivo carregar o resultado na tela. Segue o codigo modificado baseado nas alterações do ADEMILTON. Até mais…

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.text.DecimalFormat;

public class OtraCalc extends JFrame {
private JRadioButton Soma, Mult, Sub, Div;
private JButton Calcular, Limpar;
private JTextField txt1, txt2, txt3;
private ButtonGroup Operacao;
private JLabel Label1;
private Container container;
double Val1, Val2, RSoma, RSub, RMult, RDiv;

public OtraCalc() {
   super("Cansei de faze calculadora");

   Container container = getContentPane();
   container.setLayout( new FlowLayout() );

   txt1 = new JTextField("",10);
   container.add(txt1);

   txt2 = new JTextField("",10);
   container.add(txt2);

   txt3 = new JTextField("",10);
   container.add(txt3);

   Soma = new JRadioButton("+", true);
   container.add(Soma);

   Sub = new JRadioButton("-", false);
   container.add(Sub);

   Mult = new JRadioButton("*", false);
   container.add(Mult);

   Div = new JRadioButton("/", false);
   container.add(Div);

   Operacao = new ButtonGroup();
   Operacao.add(Soma);
   Operacao.add(Sub);
   Operacao.add(Mult);
   Operacao.add(Div);

   Calcular = new JButton("Calcular");

   Calcular.addActionListener(

   new ActionListener() {

        public void actionPerformed(ActionEvent event) {
         Val1 = Double.parseDouble(txt1.getText());
         Val2 = Double.parseDouble(txt2.getText());
         RSoma = Val1 + Val2;
         RSub = Val1 - Val2;
         RMult = Val1 * Val2;
         RDiv = Val1 / Val2;
         
         DecimalFormat resultado = new DecimalFormat("###,###.###");

         if (Soma.isSelected()) { //somar
            txt3.setText(resultado.format(RSoma));
         }
         if (Sub.isSelected()) { //subtrair
            txt3.setText(resultado.format(RSub));
         }
         if (Mult.isSelected()) { //multiplicar
            txt3.setText(resultado.format(RMult));
         }
         if (Div.isSelected()){ //dividir
            txt3.setText(resultado.format(RDiv));
         }
         

      }
   }
   );

   container.add(Calcular);
   setSize( 300, 300);
   setVisible( true );
   }

   public static void main(String args[]) {
      OtraCalc application = new OtraCalc();
      application.addWindowListener(
      new WindowAdapter() {
      public void WindowClosing( WindowEvent windowevent) {;
         System.exit( 0 );
      }
   }

   );
   }
}
#5