ActionListener problems

[Amy Phillips]

Hi,
I have serveral classes, one of which lays out some buttons on the creen. I want these buttons to open new windows from other classes but I am hvaing trouble. The actionListener is working but how do I get a window containing a new class to open when the button is clicked?
The class with the buttons is

 
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.event.*;
public class MenuButtons extends JPanel
{
    public MenuButtons()  
    { 
        setLayout(new BoxLayout(this, BoxLayout.X_AXIS));

        ActionListener addListener = new ActionListener()
        {
            public void actionPerformed(ActionEvent event)
            {
                JDialog dialog = new CreateTest(TestFrame.this);
dialog.show();
            }
        };
        JButton addButton = new JButton("Add");    
        JButton editButton = new JButton("Edit"); 
JButton deleteButton = new JButton("Delete"); 
JButton searchButton = new JButton("Search"); 
addButton.addActionListener(addListener);
editButton.addActionListener(addListener);
deleteButton.addActionListener(addListener);
searchButton.addActionListener(addListener);
add(addButton);
add(editButton);
add(deleteButton);
add(searchButton);
    }
} 

and I want it to open up

 
import java.awt.*;
import java.awt.event.*;
import java.util.*;
import javax.swing.*;
import javax.swing.event.*;
public class CreateTest
{  
    public static void main( String args[] )  
    {    
        TestFrame tframe = new TestFrame();    
        tframe.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);    
        tframe.show();  
    }
}  
//class TestFrame extends JFrame
class TestFrame extends JDialog
{  
    private static final int WIDTH = 500; 
    private static final int HEIGHT = 400;      
    public TestFrame(JFrame owner)  
    {    
        setTitle("Enter New Test");    
        setSize(WIDTH, HEIGHT);        
        Container contentPane = getContentPane();    
        GridBagLayout layout = new GridBagLayout();    
        contentPane.setLayout (layout);       
        //create form objects    
        JLabel titleLabel = new JLabel("Create a New Test");    
        JLabel directLabel = new JLabel("Please complete the form and click 'Create Test'");    
        JLabel mandLabel = new JLabel("Fields marked with * are mandatory");    

        JLabel nameLabel = new JLabel("* Test Name: ");    
        JTextField nameField = new JTextField(20);    

        JLabel typeLabel = new JLabel(" Test Type: ");    
        JTextField typeField = new JTextField(20);       

        JLabel moduleLabel = new JLabel(" Project Module: ");    
        JTextField moduleField = new JTextField(20);   

        JLabel descripLabel = new JLabel(" Test Description: ");   
        JTextArea descripField = new JTextArea(20, 3);    
        JScrollPane descrip = new JScrollPane(descripField);    
        descripField.setLineWrap(true);    
        descripField.setWrapStyleWord(true); 

        JLabel exResultLabel = new JLabel("* Expected Test Result: ");    
        JTextArea exResultField = new JTextArea(20, 3);    
        JScrollPane exResult = new JScrollPane(exResultField);    
        exResultField.setLineWrap(true);    
        exResultField.setWrapStyleWord(true);      

        JLabel objectiveLabel = new JLabel("* Test Objective: ");   
        JTextArea objectiveField = new JTextArea(20, 3);   
        JScrollPane objective = new JScrollPane(objectiveField);    
        objectiveField.setLineWrap(true);    
        objectiveField.setWrapStyleWord(true);      

JPanel buttonPanel = new JPanel();
        BorderLayout buttonLayout = new BorderLayout();
        buttonPanel.setLayout(buttonLayout);

        JButton clearButton = new JButton("Clear Form");    
        JButton createButton = new JButton("Create Test");   

createButton.addActionListener(new ActionListener()
{
public void actionPerformed(ActionEvent evt)
{
setVisible(false);
}
});

buttonPanel.add(BorderLayout.EAST, clearButton);
buttonPanel.add(BorderLayout.WEST, createButton);
 
        //add components to the grid    
        GridBagConstraints constraints = new GridBagConstraints();        
        constraints.fill = GridBagConstraints.NONE;    
        constraints.anchor = GridBagConstraints.WEST;    
        constraints.weightx = 0;    
        constraints.weighty = 0;   
        constraints.insets = new Insets(0,0,5,0);    

        add(titleLabel, constraints, 1, 1, 1, 1);    
        add(directLabel, constraints, 1, 2, 1, 1);   
        add(mandLabel, constraints, 1, 3, 1, 1);    
        add(nameLabel, constraints, 0, 4, 1, 1);    
        add(typeLabel, constraints, 0, 5, 1, 1);    
        add(moduleLabel, constraints, 0, 6, 1, 1); 

        constraints.fill = GridBagConstraints.NONE;    
        constraints.anchor = GridBagConstraints.WEST;   
        constraints.weightx = 0;    
        constraints.weighty = 0.2;//        

        add(descripLabel, constraints, 0, 7, 1, 1);   
        add(exResultLabel, constraints, 0, 8, 1, 1);   
        add(objectiveLabel, constraints, 0, 9, 1, 1);   
        
constraints.fill = GridBagConstraints.HORIZONTAL;  
        constraints.anchor = GridBagConstraints.CENTER;   
        constraints.weightx = 0;   
        constraints.weighty = 0;//       
 
        add(nameField, constraints, 1, 4, 1, 1);    
        add(typeField, constraints, 1, 5, 1, 1);   
        add(moduleField, constraints, 1, 6, 1,1); 
       
       constraints.fill = GridBagConstraints.BOTH;// 
       constraints.anchor = GridBagConstraints.CENTER;  
       constraints.weightx = 0;  
       constraints.weighty = 0.2;     
   
       add(descrip, constraints, 1, 7, 1, 1);  
       add(exResult, constraints, 1, 8, 1, 1);  
       add(objective, constraints, 1, 9, 1, 1);  
      
       constraints.fill = GridBagConstraints.NONE; 
       constraints.anchor = GridBagConstraints.CENTER; 
       constraints.weightx = 0;  
       constraints.weighty = 0;     
   
       add(buttonPanel, constraints, 1,10, 1, 1);  
       //add(createButton, constraints, 1, 10, 1, 1);  
 }    
    //method to add a given component to the grid with the  
    public void add(Component c, GridBagConstraints constraints,  int x, int y, int w, int h)  
    {    
        constraints.gridx = x;   
        constraints.gridy = y;  
        constraints.gridwidth = w;
        constraints.gridheight = h;   
        getContentPane().add(c, constraints); 
    }
}  

Thanks very much

[Ernest Friedman-Hill]

I don't know what the purpose of CreateTest is; you actually want to open up a TestFrame window, right? In any event, CreateTest has no constructor, and definitely not one that takes a JFrame as a constructor argument, so I think there's some confusion here.
What I'd do is just move the few things that CreateTest.main() does either into TestFrame's constructor, or into the ActionListener, and have the ActionListener create a TestFrame directly.
If you really, really, want CreateTest to be involved for some reason, then as the only code in that class is in main(), the ActionListener should invoke CreateTest.main(), something like
CreateTest.main(new String[0]);

[Amy Phillips]

Thanks very much! I have created my forms seperatly and am not struggling to fit them into my applications. Stil I have followed your advice and everything is going well.