Probably doing something stupid

[Vicness]

I'm new to swing, having made the change from Netbeans to Eclipse recently. Up until now I had relied on Netbeans for the creation of everything GUI related, but am trying to write it by hand in eclipse now. I've so far created 2 text fields and a button. When I click the button, I'd like for the text in field 1 to be copied to field 2. However, it throws a:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at Pizza.MyPanel.actionPerformed(MyPanel.java:46)

Note: The text fields are named textPane as they started off life as textPanes, but I changed to fields to see if it made it work. I'm probably just doing something stupid, so any help would be appreciated.

public class MyPanel extends JPanel implements ActionListener{
private JButton button;
private String name;
private JTextField textPane;
private JTextField textPane2;




public MyPanel() {


JTextField textPane = new JTextField();
textPane.setText("Value");
add(textPane);


button=new JButton("Submit");
button.setVerticalTextPosition(AbstractButton.CENT ER);
button.setHorizontalTextPosition(AbstractButton.LE ADING);
button.setToolTipText("Click to submit");
button.addActionListener(this);
add(button);



JTextField textPane2 = new JTextField();
textPane2.setText("Value");
add(textPane2);

}



public void actionPerformed (ActionEvent e) {

System.out.println("clicked");
name=textPane.getText();

if (name != null){
textPane2.setText(name+" ");
}
else{
System.out.println("null");
}





}
}

Edit: I've imported stuff the necassary packages also

[JosAH]

Your local variables textPane and textPane2 have nothing to do with your member variables with the same name; your member variables are still equal to null after your constructor has finished; the cure is simple: don't use local variables in your constructor.

kind regards,

Jos

[Vicness]

Ah, I see why it's having an error now, thanks. How do I fix it though? ^^"

[JosAH]

As I wrote before: don't use a local variable with the same name as a member variable in your constructor; the first part of your constructor might look like this:

public MyPanel() {

   textPane = new JTextField(); // <--- see?
   textPane.setText("Value");
   add(textPane);
   ...

kind regards,

Jos

[Vicness]

Ah I see and feel stupid now ^^ Many thanks!