Thread: ActionListener help

I need help getting my action listener. I searched around and got a lot of help but I still can't quite get my issue resolved. It needs to perform an action when the user hits enter in a JTextField and I'm not even sure that I'm on the right track. I made notes where I am getting my errors and I believe that it is just a scope issue for the most part. It has been a while since I have wrote any code. Thanks in advance.

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
 
public class CurrencyConverter extends JFrame implements ActionListener{
	double dollars;
	double pounds;
 
	public static void main(String[] args){
		CurrencyConverter converter = new CurrencyConverter();
		final JTextField dollarsField = new JTextField("", 25);//create JTextFields
		dollarsField.addActionListener(GetDollarInputListener());//error at this line
		final JTextField poundsField = new JTextField("", 25);
		poundsField.addActionListener(GetPoundsInputListener());//error at this line
		JFrame frame = new JFrame();//create JFrame
		frame.setTitle("Currency Converter");//set frame properties
		frame.setDefaultCloseOperation(EXIT_ON_CLOSE);
		frame.setLocationRelativeTo(null);					
		frame.setVisible(true);				
		frame.add(converter.createGUI(dollarsField, poundsField));//call on createGUI to retrieve jpanel to create GUI and add it to the JFrame
		frame.pack();
		frame.setSize(350,110);
 
 
	}//end main method
 
 
	JPanel createGUI(JTextField dol, JTextField pou){
		JPanel text = new JPanel(new GridLayout(2,1));
		JPanel labels = new JPanel(new GridLayout(2,1));
		JPanel complete = new JPanel(new BorderLayout());
 
		JLabel dollars = new JLabel("Dollars:  ");
		JLabel pounds = new JLabel("Pounds:  ");
		labels.add(dollars);
		text.add(dol);
		labels.add(pounds);
		text.add(pou);
		complete.add(labels, BorderLayout.WEST);
		complete.add(text, BorderLayout.CENTER);
 
		return complete;		
	}//end createGUI method
 
	class GetDollarInputListener implements ActionListener{
		public void actionPerformed(ActionEvent e){
 
		}
	}//end GetDollarInputListener class
	class GetPoundsInputListener{
		public void actionPerformed(ActionEvent e){
 
			double poundsAmount = Double.parseDouble(converter.poundsField.getText());//error at this line
		}
 
	}//end GetPoundsInputListener class
}//end CurrencyConverter class

Thanks for the reply. However that did not work and the errors didn't go away.

what's the error?

The Method GetDollarsField is undefined for the type CurrencyConverter. The error down at the bottom in my action listener class is: converter can not be resolved. I also forgot that I have an error where I declare my class at the top asking for an action listener since it implements it.

I fixed as many errors as possible, but I still get the one when you add the action listener

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
 
public class CurrencyConverter extends JFrame {
    double dollars;
    double pounds;
 
    public static void main(String[] args){
        CurrencyConverter converter = new CurrencyConverter();
        JTextField dollarsField = new JTextField("", 25);//create JTextFields     
        dollarsField.addActionListener(new GetDollarInputListener());//error at this line
        JTextField poundsField = new JTextField("", 25);
        poundsField.addActionListener(new GetPoundsInputListener());//error at this line
        JFrame frame = new JFrame();//create JFrame
        frame.setTitle("Currency Converter");//set frame properties
        frame.setDefaultCloseOperation(EXIT_ON_CLOSE);
        frame.setLocationRelativeTo(null);                  
        frame.setVisible(true);             
        frame.add(converter.createGUI(dollarsField, poundsField));//call on createGUI to retrieve jpanel to create GUI and add it to the JFrame
        frame.pack();
        frame.setSize(350,110);
 
 
    }//end main method
 
 
    public JPanel createGUI(JTextField dol, JTextField pou){
        JPanel text = new JPanel(new GridLayout(2,1));
        JPanel labels = new JPanel(new GridLayout(2,1));
        JPanel complete = new JPanel(new BorderLayout());
 
        JLabel dollars = new JLabel("Dollars:  ");
        JLabel pounds = new JLabel("Pounds:  ");
        labels.add(dollars);
        text.add(dol);
        labels.add(pounds);
        text.add(pou);
        complete.add(labels, BorderLayout.WEST);
        complete.add(text, BorderLayout.CENTER);
 
        return complete;        
    }//end createGUI method
 
    private class GetDollarInputListener implements ActionListener{
        public void actionPerformed(ActionEvent e){
 
        }
    }//end GetDollarInputListener class
   private class GetPoundsInputListener implements ActionListener{
        public void actionPerformed(ActionEvent e){
 
            double poundsAmount = Double.parseDouble(converter.poundsField.getText());//error at this line
        }
 
    }//end GetPoundsInputListener class
}//end CurrencyConverter class

Awesome, thanks for all of the help. I will have to test it out a little later since I am in class but i will let you know if I have any more trouble. Thats a huge load off of my shoulders.

Everything worked great after another tweak or two so thanks again. Now I just have to get the action listener to respond to the enter key.

It worked fine for me without tweaking, and it responded to enter.

Originally Posted by Brt93yoda

I got it to work, you had to make the classes static.

No no no no no!

You have to create an instance of the class and call the (not static) methods on the instance.

Java is an object oriented language.

db

Originally Posted by Darryl.Burke

No no no no no!

You have to create an instance of the class and call the (not static) methods on the instance.

Java is an object oriented language.

db

oh? ok? My bad I guess.

Yea, i accidentally messed something up regarding the enter key but I got everything to work fine.

And thankyou for the advice darryl, I'll be sure to remember that and try it out next time.