e.getSource?

[luigimaldini]

Hola amigos llevo todo el dia intentando encontrar el error de porque no funciona el boton como deberia, pongo el codigo:

public class Boton extends JFrame implements ActionListener {

public Boton(){
super("MI VENTANA");
setSize(400,300);
setDefaultCloseOperation(EXIT_ON_CLOSE);
JButton boton1;
JButton boton2;

boton1 = new JButton("ACEPTAR");
boton2 = new JButton("CANCELAR");


FlowLayout s = new FlowLayout();
setLayout(s);
add(boton1);
add(boton2);

setVisible(true);

boton1.addActionListener(this);
boton2.addActionListener(this);
}

public void actionPerformed(ActionEvent e){

if (e.getSource()==boton1)

System.exit(0);

cuando Pulso el boton no se ejecuta lo que esta en el metodo actioPerformed, lo raro es que si cambio eso por: System.out.println("funciono") si funciona, no me evalua el if (e.getSource()==boton1) y no se el porque por mas vuelta que le doy, gracias y un saludo.

[chuidiang]

Buenas:

Código java:

public Boton(){
   super("MI VENTANA");
   setSize(400,300);
   setDefaultCloseOperation(EXIT_ON_CLOSE);
   JButton boton1;
   JButton boton2;
 

Las variable boton1 y boton2 son locales al método (constructor supongo) Boton(). Por ello, NO son accesibles desde el actionPerformed. Seguramente como atributo de la clase tienes definida otra variable boton1.

Se bueno.

[luigimaldini]

Amigo no tengo definidias mas variables boto1 y boton2, solo las que estan en el constructor, y porque no son accesible? creo que deberia de funcionar, en fin gracias y espero tu respuesta.saludos.

Arreglado amigo, saque las variables del constructor y ya funciona el codigo, gracias y un saludo.

[afrika026]

la variable "JButton boton1" la estas declarando como local en el constructor, tenes que declararla fuera del constructor.

El codigo seria así:

public class Boton extends JFrame implements ActionListener {
//las declaras en la clase y no dentro del constructor

private JButton boton1;
private JButton boton2;

public Boton(){
super("MI VENTANA");
setSize(400,300);
setDefaultCloseOperation(EXIT_ON_CLOSE);
boton1 = new JButton("ACEPTAR");
boton2 = new JButton("CANCELAR");


FlowLayout s = new FlowLayout();
setLayout(s);
add(boton1);
add(boton2);

setVisible(true);

boton1.addActionListener(this);
boton2.addActionListener(this);
}

public void actionPerformed(ActionEvent e){

if (e.getSource()==boton1)

System.exit(0);

[luigimaldini]

Gracias, ya esta arreglado,saludos.

[afrika026]

comenta si encontrastes la solucion con mi explicación o sino como lo solucionastes...

[luigimaldini]

Si amigo, lo solucione sacando y declarando las 2 variables fuera del constructor como bien decis. gracias de nuevo.