How can I add a JMenuItem that closes my Java-program?

Question

Do I have to work with an ActionListener or AbstractAction?

EDITED BASED ON ANSWERS

So, the best way this is the best way to do it?

Action closeaction = new AbstractAction("Afsluiten"){

        @Override
        public void actionPerformed(ActionEvent ae) {
            System.exit(1);
        }

    };
    menuItem = new JMenuItem(closeaction);

*"Do I have to.."* Have to? That is an odd phrase for the common way to do it. — Andrew Thompson, Mar 23 '12 at 12:42
The general rule is to _always_ go for the highest abstraction available. Here that rule translates to _always_ use an Action. (so it's a clear no for your comment, @SirTroll :-) — kleopatra, Mar 23 '12 at 12:47
@kleopatra Oh (disappointed) I'll work on earning the title, just so I can give that reply. ;) — Andrew Thompson, Mar 23 '12 at 13:25

score 2 · Accepted Answer · edited May 23 '17 at 11:48

2

As @kleopatra comments, Action is the preferred abstraction, and AbstractAction is the right base class. In your handler, a non-zero status signifies an error condition. As an alternative, consider sending a WINDOW_CLOSING event, as shown here for JDialog.

Addendum: The WINDOW_CLOSING event is convenient if your application needs to take some action before terminating. Add a WindowListener to the example to see the effect:

this.addWindowListener(new WindowAdapter() {

    @Override
    public void windowClosing(WindowEvent e) {
        System.out.println(e);
    }
});

edited May 23 '17 at 11:48

Community

1
1

answered Mar 23 '12 at 14:58

trashgod

203,806
29
246
1,045

So, the System.exit(1) is no good way? I have to use WindowEvent.WINDOW_CLOSING? – Luc Peetersen Mar 23 '12 at 23:43
Well, use `System.exit(0)` for normal termination; it's just rather abrupt. More above. – trashgod Mar 24 '12 at 00:04

How can I add a JMenuItem that closes my Java-program?

1 Answers1