Problem with ActionPerformed

[Juhan Voolaid]

Hi
I have this wierd problem in my program (I'm a nube so it must be a basic stuff). It is a GUI problem where are 3 classes +main class - buttonPanel, infoPanel and Stats.
This buttonPanel has buttons and ActionPerformed method. Infopanel has a basic constructor and methods that should show stuff on infoPanel. And this Stats class has just a constructor.
What my goal right now is, is to make a program so that when i press on buttonPanel - Stats class appears to infoPanel.
Problem is when everything should be ok it doesen't work - it shows a blank panel on infopanel + to the console on the background it gives me stuff about action performed things EventDispatchThread - don't know what to do wiht that.
code:

 
import javax.swing.*;
import javax.swing.border.*;
import javax.swing.event.*;
import java.awt.event.*;
public class hero extends JFrame{

buttonPanel b;
infoPanel i;


public static void main(String[] args){
hero h = new hero();
}


public hero(){
i=new infoPanel();

JFrame f=new JFrame("Hero ... early development");
f.setSize(640,480);
f.setDefaultCloseOperation(EXIT_ON_CLOSE);
f.setResizable(false);
f.setLocation(100, 100);
Container container=f.getContentPane();

GridBagLayout GB = new GridBagLayout();
container.setLayout(GB);
GridBagConstraints c= new GridBagConstraints();
c.weightx = 1;//fill x
c.weighty = 1;//fill y
c.fill=GridBagConstraints.BOTH;

c.gridheight=3;
c.gridwidth=3;
c.gridx=0;
c.gridy=0;
Button b1=new Button("Action");
GB.setConstraints(b1, c);
container.add(b1);

c.gridheight=1;
c.gridwidth=1;
c.weightx = 0.1; 
c.gridx=1;
c.gridy=0;

c.gridx=3;
c.gridy=0;
//s=new startPanel();
GB.setConstraints(b1, c);
container.add(b1);

c.gridx=3;
c.gridy=1;
c.weighty = 0.5;
b=new buttonPanel(i);//here i give pass to infoPanel so i can call methods inside iP 
GB.setConstraints(b, c);
container.add(b);

c.gridx=3;
c.gridy=2;
c.weighty = 2;
GB.setConstraints(i, c);
container.add(i);

f.setVisible(true);



}
private class buttonPanel extends JPanel implements ActionListener{
JPanel center;
JLabel north;
JButton stats, health, inv, spells, notes, score;
infoPanel i;

public buttonPanel(infoPanel i){
this.i=i;
setBorder(new EtchedBorder(EtchedBorder.LOWERED));
BorderLayout BL = new BorderLayout();
setLayout(BL);

north=new JLabel("Menu:", SwingConstants.CENTER);
center=new JPanel();
add(north, BL.NORTH);

GridBagLayout GB=new GridBagLayout();
GridBagConstraints c=new GridBagConstraints();

stats=new JButton("Stats");
stats.addActionListener(this);
health=new JButton("Health");
health.addActionListener(this);
inv=new JButton("Inv.");
inv.addActionListener(this);
spells=new JButton("Spells");
spells.addActionListener(this);
notes=new JButton("Notes");
notes.addActionListener(this);
score=new JButton("Score");
score.addActionListener(this);

center.setLayout(GB);
c.fill=GridBagConstraints.BOTH;
c.gridx=0;
c.gridy=0;
GB.setConstraints(stats, c);
center.add(stats);

c.gridx=0;
c.gridy=1;
GB.setConstraints(health, c);
center.add(health);

c.gridx=1;
c.gridy=0;
GB.setConstraints(inv, c);
center.add(inv);

c.gridx=1;
c.gridy=1;
GB.setConstraints(spells, c);
center.add(spells);

c.gridx=0;
c.gridy=2;
GB.setConstraints(notes, c);
center.add(notes);

c.gridx=1;
c.gridy=2;
GB.setConstraints(score, c);
center.add(score);

center.setBorder(new EtchedBorder(EtchedBorder.LOWERED));
add(center, BL.CENTER);

}


public void actionPerformed(ActionEvent e){

JButton button = (JButton)e.getSource();

if(button == stats)  
i.showMessage();

 
}

}


private class infoPanel extends JPanel{

Stats stats;

public infoPanel(){
stats=new Stats();
Border etch = BorderFactory.createEtchedBorder(  );    
Border titled=BorderFactory.createTitledBorder(etch, "Title");
    setBorder(titled);
    // if i call showMessage() from here it works but if by actionPerformed - it doesen't

}

public void showMessage() {

add(stats); //it doesent work
//System.out.println(test);  --it works
}



}

private  class Stats extends JPanel{
JLabel heading;
public Stats(){
JLabel heading=new JLabel("Stats");
add(heading);
}

}

}
 

[ February 18, 2004: Message edited by: Juhan Voolaid ]

[Lalit K Kumar]

As far as I know ,of all the components you have used, you can add "actionlistener" only to the jButton. In the snapshot of your code I can'nt see you attaching a listener to any of your code. Add the listener ("addactionlistener") to your respective button (in the constructor) and let me know in case of any confusion.

[Juhan Voolaid]

Eee... didn't quite get that. I am adding actionlistener to my buttons in my code. Tell me what should i change in my code in your point of view.

[Jose Botella]

You should invoke validate() after add(stats). This is so because you modified the content of the JPanel after it was shown.
Button b1 was added twice to container!
hero class does not have to extend Frame because the one you are using is created in hero constructor.
The class stats is not needed. Make the variable stats in infoPannel of type JLabel instead.
The Java Tutorial is very useful to learn GUI stuff.
The convention of using Capital names for Types is a wise thing to do.
[ February 19, 2004: Message edited by: Jose Botella ]

[Juhan Voolaid]

Thank you. I'm still doing my first steps in java and all the help is very welcome. And yes - i itended to add button b1 2 times.
[ February 21, 2004: Message edited by: Juhan Voolaid ]