Unable to detect when user closes (Xs) JFrame

Question

For moderators: Please see the comments of this answer: stackoverflow.com/a/16388697/1574975

I'm trying to detect when a user is done using a client program with a GUI so that when they close the JFrame for example by Xing it, I can then close the socket connections.

But I'm unable to make anything happen when user closes JFrame. I was informed that I need to use WindowListener with my JFrame but it doesn't seem to work. My class extends JFrame and here is part of the constructor setting up the JFrame:

        this.setContentPane(pane);
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        this.pack();
        this.setVisible(true);
        final JFrame thisFrame = this;
        thisFrame.addWindowListener(new WindowAdapter() {

            @Override
            public void windowClosed(WindowEvent e) {
                System.out.println("User has closed window");
            }
        });
    }

The println inside doesn't get printed which is why I think it doesn't work. Am I doing it right or do I need to do something additional?

score 3 · Accepted Answer · answered May 06 '13 at 03:07

3

windowClosed won't be called because the JVM has been closed in response to the setDefaultCloseOperation been set to JFrame.EXIT_ON_CLOSE.

Use windowClosing instead or change the defaultCloseOperation

answered May 06 '13 at 03:07

MadProgrammer

343,457
22
230
366

1

This answer was already given to him today. To be believed why he's not following it. – Hovercraft Full Of Eels May 06 '13 at 03:08
1

@HovercraftFullOfEels Actually, I think it was my comment that altered the accepted answer, saying just the same thing...:P – MadProgrammer May 06 '13 at 03:08
@HovercraftFullOfEels, please read the comments of this answer: http://stackoverflow.com/a/16388697/1574975 – Faahmed May 06 '13 at 03:14
@MadProgrammer, I didn't understand it at first. But now that it was explained in a bit more detail, I finally got it and it's fixed. Thanks alot. I wish my question didn't get voted down though. – Faahmed May 06 '13 at 03:15
@HovercraftFullOfEels Some times, we need these things beaten into us ;) – MadProgrammer May 06 '13 at 03:20

Unable to detect when user closes (Xs) JFrame

1 Answers1