JMenus and ActionListeners

Question

I'm making a MineSweeper clone and need some pointers on how to let people start a new game by either going to Game and choosing new or Ctrl_N.
Here is what I have so far:
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyEvent;
import javax.swing.*;
import javax.swing.event.*;
import java.awt.*;
import java.awt.event.*;
import java.awt.geom.*;
public class Menus extends JMenu
private static TimerDisplay TD=new TimerDisplay();
public static JMenuBar menus()
JMenuBar mb=new JMenuBar();
JMenu game=new JMenu("Game");
game.setMnemonic(KeyEvent.VK_G);
mb.add(game);
JMenuItem nw=new JMenuItem("New");
nw.setMnemonic(KeyEvent.VK_N);
nw.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_N, ActionEvent.CTRL_MASK));
nw.addActionListener(new ActionListener(){
public void actionPerformed(ActionEvent event)
// What could I put here.
JMenuItem exit=new JMenuItem("Exit");
exit.setMnemonic(KeyEvent.VK_E);
exit.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_F4, ActionEvent.ALT_MASK));
exit.addActionListener(new ActionListener(){
public void actionPerformed(ActionEvent event)
System.exit(0);
game.add(nw);
game.add(new JSeparator());
game.add(exit);
return mb;
}

Answer 1

I would probably opt for a different way of thinking about this. First off, I'd get rid of all of the statics there and make Menus a true OOP class. Next, I'd try to have it communicate in an OOP fashion with the class that displays and runs the game itself, here in my example called "MyGame". I'd give MyGame class an exit() method to allow it to decide how to exit, and a reset() method to allow it to decide how it wants to reset for a new game. This way the menus shouldn't have to and doesn't worry about these things. Something like so:
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyEvent;
import javax.swing.*;
public class Menus //extends JMenu
  private JMenuBar menubar = new JMenuBar();
  // a timerdisplay object may not be needed in the menu class
  //private TimerDisplay timerDisplay = new TimerDisplay();
  private MyGame myGame;
  Menus()
  //public void setGameDisplay(MyGame myGame, TimerDisplay timerDisplay)
  public void setGameDisplay(MyGame myGame)
    this.myGame = myGame;
    //this.timerDisplay = timerDisplay; // probably not needed
    initMenus();
  public JMenuBar getMenuBar()
    return menubar;
  public void initMenus()
    JMenu gameMenu = new JMenu("Game");
    gameMenu.setMnemonic(KeyEvent.VK_G);
    menubar.add(gameMenu);
    JMenuItem nw = new JMenuItem("New");
    nw.setMnemonic(KeyEvent.VK_N);
    nw.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_N,
        ActionEvent.CTRL_MASK));
    nw.addActionListener(new ActionListener()
      public void actionPerformed(ActionEvent event)
        myGame.reset();
    JMenuItem exit = new JMenuItem("Exit");
    exit.setMnemonic(KeyEvent.VK_E);
    exit.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_F4,
        ActionEvent.ALT_MASK));
    exit.addActionListener(new ActionListener()
      public void actionPerformed(ActionEvent event)
        //System.exit(0);
        myGame.exit();
    gameMenu.add(nw);
    gameMenu.add(new JSeparator());
    gameMenu.add(exit);
}Then it could all be tied together with a Controller or Main class that creates the MyGame class, creates the Menus class and adds one to the other.