JApplet/Application Menu Help

[Ryan10]

I had my program running well and i just added an "exit" button to my menu and it comes up fine but you can't see the rest of the program at all. Can someone tell me what i've done wrong. I believe the problem has to do with the ActionListener part of the applet.

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.border.LineBorder;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JOptionPane;
import java.awt.Color;

public class TicTacToe1 extends JApplet implements ActionListener{

private char whoseTurn = 'X';
private Cell[][] cells = new Cell[3][3];
private JLabel jlblStatus = new JLabel("X's turn to play");
private JFrame window = new JFrame("Tic-Tac-Toe");


/** Initialize UI */


public TicTacToe1() {

JMenuBar menuBar = new JMenuBar();
  setJMenuBar(menuBar);
    JMenu fileMenu = new JMenu("File");
       menuBar.add(fileMenu);

JMenuItem exitAction = new JMenuItem("Exit");
  fileMenu.add(exitAction);
    exitAction.setActionCommand("Exit");
      exitAction.addActionListener(this);

}

  public void actionPerformed(ActionEvent e){
           String arg = e.getActionCommand();{
		  if (arg.equals("Exit")){
			  System.exit(0);
       }
{

JPanel p = new JPanel(new GridLayout(3, 3, 0, 0));
    for (int i = 0; i < 3; i++)
    for (int j = 0; j < 3; j++)
        p.add(cells[i][j] = new Cell());


p.setBorder(new LineBorder(Color.red, 1));
  jlblStatus.setBorder(new LineBorder(Color.black, 1));
     add(p, BorderLayout.CENTER);
     add(jlblStatus, BorderLayout.SOUTH);

}
   }
      }


public boolean isFull() {
     for (int i = 0; i < 3; i++)
     for (int j = 0; j < 3; j++)
        if (cells[i][j].getToken() == ' ')
  return false;
  return true;
}


public boolean isWon(char token) {
    for (int i = 0; i < 3; i++)
        if ((cells[i][0].getToken() == token)
             && (cells[i][1].getToken() == token)
             && (cells[i][2].getToken() == token)) {
return true;
}

      for (int j = 0; j < 3; j++)
          if ((cells[0][j].getToken() == token)
           && (cells[1][j].getToken() == token)
           && (cells[2][j].getToken() == token)) {
return true;
}

         if ((cells[0][0].getToken() == token)
           && (cells[1][1].getToken() == token)
           && (cells[2][2].getToken() == token)) {
return true;
}

        if ((cells[0][2].getToken() == token)
           && (cells[1][1].getToken() == token)
           && (cells[2][0].getToken() == token)) {
return true;
}

return false;
}


public class Cell extends JPanel {
             private char token = ' ';

public Cell() {
           setBorder(new LineBorder(Color.black, 1)); // Set cell's border
           addMouseListener(new MouseListener()); // Register listener
}


public char getToken() {
           return token;
}

public void setToken(char c) {
            token = c;
            repaint();
}

protected void paintComponent(Graphics g) {
super.paintComponent(g);

if (token == 'X') {
               g.drawLine(10, 10, getWidth() - 10, getHeight() - 10);
               g.drawLine(getWidth() - 10, 10, 10, getHeight() - 10);
}
else if (token == 'O') {
               g.drawOval(10, 10, getWidth() - 20, getHeight() - 20);
     }
}

private class MouseListener extends MouseAdapter {
public void mouseClicked(MouseEvent e) {

if (token == ' ' && whoseTurn != ' ') {
           setToken(whoseTurn); // Set token in the cell


if (isWon(whoseTurn)) {
           jlblStatus.setText(whoseTurn + " won! The game is over");
           whoseTurn = ' '; // Game is over
}
else if (isFull()) {
           jlblStatus.setText("Draw! The game is over");
           whoseTurn = ' '; // Game is over
}
else {
          whoseTurn = (whoseTurn == 'X') ? 'O': 'X'; // Change the turn
          jlblStatus.setText(whoseTurn + "'s turn"); // Display whose turn

               }
           }
       }
   }
}

/** This main method enables the applet to run as an application */
public static void main(String[] args) {
         JFrame frame = new JFrame("TicTacToe");
            TicTacToe1 applet = new TicTacToe1();
                 frame.add(applet, BorderLayout.CENTER);

frame.setSize(300, 300);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.setVisible(true);
    }
}

[Ryan10]

can anyone figure this out? i cant

[Fubarable]

Again as has been said to you many times before if you want help quicker you really must fix your code formatting. It really is very bad, and your code is very hard to read.

[Ryan10]

There i re-edited my post take a look now

[Iron Lion]

public void actionPerformed(ActionEvent e){

	  String arg = e.getActionCommand();{
		  if (arg.equals("Exit")){
			  System.exit(0);

       }
}

Does this even compile? What happens if you delete the opening brace after e.getActionCommand();?

[Ryan10]

i did have it working but i messed around with it..with the code i gave you i got a few errors


F:\TicTacToe\TicTacToe1.java:38: incompatible types
found : javax.swing.JMenuItem
required: java.awt.MenuItem
MenuItem exitAction = new JMenuItem("Exit");
^
F:\TicTacToe\TicTacToe1.java:39: cannot find symbol
symbol : method add(java.awt.MenuItem)
location: class javax.swing.JMenu
fileMenu.add(exitAction);
^

[Iron Lion]

Read the compiler messages; they don't lie to you. The first one, and maybe the second as well, can be fixed by changing MenuItem to JMenuItem.

[Ryan10]

ok now there is no more errors..the application comes up with the menu and the exit feature works. Now where is all my other stuff? its just a blank page with the menu bar

[Iron Lion]

JPanel p = new JPanel(new GridLayout(3, 3, 0, 0));
    for (int i = 0; i < 3; i++)
    for (int j = 0; j < 3; j++)
        p.add(cells[i][j] = new Cell());


p.setBorder(new LineBorder(Color.red, 1));
  jlblStatus.setBorder(new LineBorder(Color.black, 1));
     add(p, BorderLayout.CENTER);
     add(jlblStatus, BorderLayout.SOUTH);

You need to cut this from its current location and paste it into the constructor.

[Ryan10]

where is that? sorry im very new

[Fubarable]

You can surely search your code and find this block, right?

[Ryan10]

yes i did thanks everything works now i gave all of you higher rep.