Handling event problem

Question

Hi boys and girls.
I've been working in a multiframe GUI. I have a main window (a custom class extending JFrame) with a menu.
My menu items events are beeing handled by my custom class that implements ActionListener. This cutom class, through actionPerformed(ActionEvent e) method, handles all my button events and the menu items events. To manage focus i also implemented in this class the WindowListener interface, listening to all other frames of my application, so that they can behave more or less like a modal JDialog does.
One of my menu items is a 'Quit' option. It simply exits the application if there is no changed data on a JList or asks the user if he wants to save changes first. As i said before this action is handled by actionPerformed(ActionEvent e) in my custom class.
Once the WindowListener interface is already implemented i thought i could make the frame respond the same way when the user clicks the 'X' icon just by implementing on the windowClosing(WindowEvent we) method a redirection to actionPerformed li ke this:
public void windowClosing(WindowEvent e){
          this.dispacthEvent(new ActionEvent(quitItem, ActionEvent.ACTION_PERFORMED, quitItem.getActionCommand()));
being quitItem my JMenuItem. My interpretation is that, before exiting the system, it would first launch the code that i've established within actionPerformed for the quit option.
Am i completely wrong or am i doing at least something correctly?
Please help me. Thank you

Answer 1

Instead of that do this:
        mainWindow.setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE); //This is handled by windowClosing.
        mainWindow.addWindowListener(new QuitListener());
        menuItem = new JMenuItem("Quit", KeyEvent.VK_Q);
        menuItem.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_F4, ActionEvent.ALT_MASK));
        menuItem.getAccessibleContext().setAccessibleDescription("Exit the program");
        menuItem.addActionListener(new QuitListener());
        menu.add(menuItem);
class QuitListener implements ActionListener, WindowListener{
        public void actionPerformed(ActionEvent event) {
                checkForSaveAndClose(); //Or System.exit(0) if you don't want to do something on close.
        public void windowClosing(WindowEvent e) {
                checkForSaveAndClose();
        public void windowClosed(WindowEvent e) { }
        public void windowOpened(WindowEvent e) { }
        public void windowIconified(WindowEvent e) { }
        public void windowDeiconified(WindowEvent e) { }
        public void windowActivated(WindowEvent e) { }
        public void windowDeactivated(WindowEvent e) { }
        public void windowGainedFocus(WindowEvent e) { }
        public void windowLostFocus(WindowEvent e) { }
        public void windowStateChanged(WindowEvent e) { }
}Or, at least, that's what I'm doing. As I understand it this is the more correct way. Feel free to correct me if I'm wrong.