Thread: Re: ActionListener help

Originally Posted by Junky

A solution is not to put any code into the actionPerfomed method other than a call to another method in the enclosing class. This method should have access to everything you need. Something like...

class Foo {
    private JTextfield text;
    private JButton button;
 
    Foo() {
        text = new JTextField();
        button = new JButton();
        button.addACtionListener(new ActionListener() {
            public void actionPerformed(ActionEvent ae) {
                someMethod();
            }
        });
    }
 
    private void someMethod() {
        // do what you want here
    }
}

Above is what I'm trying do

I have this and it works

switchHLbut.addActionListener(new ActionListener()
{    
 public void actionPerformed(ActionEvent e)
	{
   if(setBg == true){
            switchHLbut.setText("ON");
            sbc.setBackground(Color.white);
            sbc.setText("");
            setBg = false;
          }
        else if(setBg == false){
            switchHLbut.setText("OFF");
            sbc.setBackground(Color.yellow);
            sbc.setText("");
            setBg = true;
            }
    	}
});

but I want to call a method within the Action Listener

 
jbNew.addActionListener(new ActionListener(){
public void actionPerformed(ActionEvent e)
    {   
        System.out.println(" inside jbNew Button");
        switchjbNew();   //calling a new method
    }    
  }); //error:reached end of file while parsing
 
public void switchjbNew()   //error: illegal start of expression missing javadoc
{ 
    System.out.println("inside switchjbNew");
}

this was my attempt at calling a method from within the button ActionListener
Failed

Originally Posted by Norm

Are you trying to code that method inside another method? That is not allowed. Put the method outside of the {}s that define the method the jbNew.addActionListener(new ActionListener(){ statement is in.

I thought it was outside of any methods, It's below the button.
Thanks.
I have something to work with now.

--- Update ---

I got it, I put it right after the closing } of the constructor before

public static void main[]