I'm new and confused

Question

I don't want to annoy anyone but if someone could help me I would appreciate it greatly. What I'm trying to do is bring up a window that is used to get the player names for a game and when they click the button that window goes away and another game window shows up. I decided to put the code for my class in here and hopefully someone can help point me in the right direction to get what I want to happen. Also any comments regarding better/more efficient coding is certainly welcome. Thanks.
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.lang.Object.*;
import java.io.*;
import java.util.*;
public class wheelGUI extends JFrame implements ActionListener
public wheelGame newGame;
public JFrame gameWindow;
public JFrame playerSetupWindow;
//components for enter players frame
private JTextField enterName1 = new JTextField("Player 1",15);
private JTextField enterName2 = new JTextField("Player 2",15);
private JLabel P1 = new JLabel("Player 1:");
private JLabel P2 = new JLabel("Player 2:");
private JButton okButton = new JButton ("OK");
//components for game frame
private JMenuBar menuBar = new JMenuBar();
private JMenu fileMenu;
private JMenu helpMenu;
private JMenuItem createNewGame = new JMenuItem ("Start a New Game");
private JMenuItem viewHighScores = new JMenuItem ("View High Scores");
private JMenuItem quit = new JMenuItem ("Quit");
public JLabel player1;
public JLabel player2;
public JLabel scoreP1 = new JLabel("0");
public JLabel scoreP2 = new JLabel("0");
public JLabel clueCategory;
public JLabel currentValue = new JLabel ("Current Value:");
public JLabel generatedValue;
private JButton guessButton = new JButton ("Guess a Letter!");
private JTextField guessBox;
public String [] clueCategoryArray;
public String [] clueArray;
public JTextArea [] letterBoxes;
private JTextArea blackBox;
public String currentGuess;
public wheelGUI(wheelGame newGame)
this.newGame = newGame;
setDefaultCloseOperation(EXIT_ON_CLOSE);
playerSetupWindow = new JFrame("Enter Players");
Container contentPane = playerSetupWindow.getContentPane();
GridBagLayout gridbag = new GridBagLayout();
GridBagConstraints c = new GridBagConstraints();
contentPane.setLayout(gridbag);
c.fill = GridBagConstraints.BOTH;
c.gridx = 0; c.gridy = 0; c.ipadx = 10; c.ipady = 10; c.insets = new Insets(0,10,3,10); //top,left,bottom,right
gridbag.setConstraints(P1 , c);
contentPane.add(P1);
c.gridx = 1; c.gridy = 0; c.ipadx = 10; c.ipady = 10;
gridbag.setConstraints(enterName1 , c);
contentPane.add(enterName1);
enterName1.setEditable(true);
c.gridx = 0; c.gridy = 1; c.ipadx = 10; c.ipady = 10; c.insets = new Insets(0,10,0,10);
gridbag.setConstraints(P2 , c);
contentPane.add(P2);
c.gridx = 1; c.gridy = 1; c.ipadx = 10; c.ipady = 10;
gridbag.setConstraints(enterName2 , c);
contentPane.add(enterName2);
enterName2.setEditable(true);
c.gridx = 0; c.gridy = 2; c.gridwidth = 2; c.insets = new Insets(5,40,0,40);
gridbag.setConstraints(okButton, c);
contentPane.add(okButton);
okButton.addActionListener(this);
playerSetupWindow.setVisible(true);
pack();
public void drawGameWindow ()
//draw main frame
gameWindow = new JFrame("Fortune Wheel");
gameWindow.setJMenuBar(menuBar);
     menuBar.add(fileMenu);
fileMenu.add(createNewGame);
fileMenu.add(viewHighScores);
fileMenu.add(quit);
menuBar.add(helpMenu);
createNewGame.addActionListener(this);
viewHighScores.addActionListener(this);
quit.addActionListener(this);
Container contentPane2 = gameWindow.getContentPane();
GridBagLayout gridbag2 = new GridBagLayout();
GridBagConstraints c2 = new GridBagConstraints();
contentPane2.setLayout(gridbag2);
c2.fill = GridBagConstraints.BOTH;
c2.gridx = 0; c2.gridy = 0; c2.ipadx = 5; c2.ipady = 5;
gridbag2.setConstraints(player1 , c2);
contentPane2.add(player1);
c2.gridx = 1; c2.gridy = 0; c2.ipadx = 5; c2.ipady = 5;
gridbag2.setConstraints(scoreP1 , c2);
contentPane2.add(scoreP1);
c2.gridx = 2; c2.gridy = 0; c2.ipadx = 5; c2.ipady = 5;
gridbag2.setConstraints(player2 , c2);
contentPane2.add(player2);
c2.gridx = 3; c2.gridy = 0; c2.ipadx = 5; c2.ipady = 5;
gridbag2.setConstraints(scoreP2 , c2);
contentPane2.add(scoreP2);
c2.gridx = 0; c2.gridy = 1; c2.gridwidth = 4;
gridbag2.setConstraints(clueCategory, c2);
contentPane2.add(clueCategory);
//Box to hold boxes creaeted for display
Box letterBoxesBox = new Box(BoxLayout.X_AXIS);
for(int i = 0 ; i < clueArray.length ; i++)
if(letterBoxes.getText() == " ")
//draw black rectangel
blackBox.setSize(1,1);
blackBox.setBackground(Color.black);
letterBoxesBox.add(blackBox);
letterBoxesBox.add(Box.createHorizontalStrut(5));
else
letterBoxesBox.add(letterBoxes[i]);
letterBoxesBox.add(Box.createHorizontalStrut(5));
c2.gridx = 0; c2.gridy = 2; c2.gridwidth = 4;
gridbag2.setConstraints(letterBoxesBox, c2);
contentPane2.add(letterBoxesBox);
c2.gridx = 0; c2.gridy = 3; c2.ipadx = 5;
gridbag2.setConstraints(currentValue, c2);
contentPane2.add(currentValue);
c2.gridx = 1; c2.gridy = 3; c2.ipadx = 5;
gridbag2.setConstraints(generatedValue, c2);
contentPane2.add(generatedValue);
guessBox.setSize(1,1);
c2.gridx = 0; c2.gridy = 4; c2.ipadx = 5; c2.ipady = 20; c2.anchor = GridBagConstraints.SOUTH;
gridbag2.setConstraints(guessBox, c2);
contentPane2.add(guessBox);
c2.gridx = 1; c2.gridy = 4; c2.ipadx = 5; c2.ipady = 20; c2.anchor = GridBagConstraints.SOUTH;
gridbag2.setConstraints(guessButton, c2);
contentPane2.add(guessButton);
guessButton.addActionListener(this);
gameWindow.setVisible(false);
pack();
public void actionPerformed (ActionEvent e)
Object source = e.getSource();
if(source == okButton)
player1.setText(enterName1.getText());
player2.setText(enterName2.getText());
clueCategory.setText(newGame.getClueCategory());
generatedValue.setText(newGame.getGeneratedValue());
for(int i = 0 ; i < newGame.getLengthOfLetterArray() ; i ++)
letterBoxes[i].setText(newGame.getLetters(i));
playerSetupWindow.setVisible(false);
drawGameWindow();
gameWindow.setVisible(true);
gameWindow.repaint();
else if(source == guessButton)
//deal with game action
currentGuess = guessBox.getText();
newGame.guessCheck(currentGuess);
repaint();
else if(source == this.createNewGame)
//new game selected from menu bar
newGame.clearGame();
else if(source == this.viewHighScores)
//Dialog Box with High Scores displayed
else if(source == this.quit)
System.exit(0);
public static void main(String[] args)
if (args.length != 1)
System.err.println("Usage: java WheelGUI <filename>");
System.exit(0);
wheelGame newGame = new wheelGame();
wheelGUI myGUI = new wheelGUI(newGame);
myGUI.show();
try{ newGame.readDataFromFile(args[0]); }
catch (IllegalArgumentException e) {
System.err.println(e.getMessage());
newGame.readDataFromFile(args[0]);

Answer 1

As I see it, it'd be best to have two classes, NameWindow and GameWindow. The window in which you enter your name would have a button, to which you can add an action listener. The code for that listener would be something like:public void actionPerformed (ActionEvent event) {
    String[] names = ...;
    hide ();
    new GameWindow (names);
}Some side remarks:
1. You don't have to import the java.lang package (by the way, there's not a java.lang.Object package, just a class named that way).
2. Use [code][/code] tags around your code next time: it will become easier to read.
3. If you're using a variable in just one method, don't declare it as a member variable (e.g. all your labels).
Kind regards,
Levi