Discussion :

[Sylvain__A_]

J'ai vraiment beaucoup de mal à comprendre le concept d'EDT.

Je n'arrive pas à placer correctement un écouteur qui customise l'action de fermer la fenêtre principale et quitter le programme. Quoique je fasse, j'ai cette exception :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
Exception in thread "AWT-EventQueue-0" java.lang.Error: Cannot call invokeAndWait from the event dispatcher thread

Je pensais que justement, appeler SwingUtilities.invokeAndWait ou SwingUtilities.invokeLater permettait d'obtenir des applications thread safe ...

Voici mon architecture :

DeviceManager est mon programme principal, c'est une classe statique, dans laquelle au sein du main, j'instancie ma classe WindowManager, et essaie de placer un écouteur sur l'évenement Closing, afin de pouvoir fermer facilement d'autres thread, dont j'ai les références ici, et que je n'aurais pas dans la classe windowManager.

Voici mon main :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
 
public static void main( String[] args )
{
	_window  = new WindowManager( "Les Tanukis : Java serial - socket daemon." );
 
	WindowListener wListener = new WindowAdapter()
	{
		public void windowClosing( WindowEvent e )
		{
			int confirm = JOptionPane.showOptionDialog( _window.get_mainFrame(), "Really Exit?", "Exit Confirmation",
					JOptionPane.YES_NO_OPTION, JOptionPane.QUESTION_MESSAGE, null, null, null );
			if ( confirm == 0 )
			{
				try {
					SwingUtilities.invokeAndWait(
						new Runnable(){
							public void run(){
								_window.get_mainFrame().dispose();
							}
						}
					);
				} catch ( InterruptedException ee) {
					ee.printStackTrace();
				} catch ( InvocationTargetException eee) {
					eee.printStackTrace();
				}
 
				if( DeviceManager.getSocket() != null )
					DeviceManager.getSocket().closeConnection();
				if( DeviceManager.getSerialListener() != null )
					DeviceManager.getSerialListener().close();
				if( DeviceManager.getPrinter() != null ){
					PrinterManager pm = DeviceManager.getPrinter();
					pm = null;
				}
				System.exit(0);
			}
		}
	};
	_window.get_mainFrame().addWindowListener( wListener );
 
	_printer = new PrinterManager( _window );
 
        ...

Ce code lance une exception après avoir ouvert le JOptionPAne :

Exception in thread "AWT-EventQueue-0" java.lang.Error: Cannot call invokeAndWait from the event dispatcher thread

Voici le code de WindowManager, le constructeur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
 
public WindowManager( String title ) throws HeadlessException
{
	try {
		javax.swing.SwingUtilities.invokeAndWait( 
			new Runnable() {
				public void run() {
			    	JFrame.setDefaultLookAndFeelDecorated(true);
					JDialog.setDefaultLookAndFeelDecorated(true);
 
					try {
						UIManager.setLookAndFeel(new SubstanceBusinessLookAndFeel() );
					}
					catch (UnsupportedLookAndFeelException e1){
						e1.printStackTrace();
					}
 
					_mainFrame.setDefaultCloseOperation( WindowConstants.DO_NOTHING_ON_CLOSE );
					_mainFrame.setSize(400, 300);
 
					try {
						set_icon( ImageIO.read( getClass().getResource("assets/1265388043_socket.png")));
					}
					catch ( IOException e ) {
						e.printStackTrace();
					}
 
					_mainFrame.setIconImage(get_icon());
					_mainFrame.setVisible(true);
 
					set_mainLayout( new GridBagLayout() );
 
					_mainFrame.setLayout( get_mainLayout() );
 
					set_mainTxtarea(new JTextArea());
 
					addComponent( _mainFrame, get_mainTxtarea(), 0, 0, 2, 3, GridBagConstraints.CENTER, GridBagConstraints.BOTH );
 
				}
			}
		);
	} catch (InterruptedException e1) {
		e1.printStackTrace();
	} catch (InvocationTargetException e1) {
		e1.printStackTrace();
	}
 
}

Quelqu'un pourrait il m'expliquer ce que je n'arrive pas à comprendre, c'est à dire à quoi sert exactement invokeAndWait, et pourquoi ce code n'est pas thread safe ...

[fr1man]

Les listeners sont gérés par l'EDT, donc tu n'as pas besoin de faire ton invokeAndWait pour exécuter ton code.

[Sylvain__A_]

Merci de ta réponse

Ce n'est pas "mon" invokeAndWait, je le voit dans beaucoup de doc, il semble que ça soit une bonne pratique ...

Par ex, je site "The Definitive Guide To Java Swing", 3ed, John Zukowski, p46 :

To demonstrate the proper way to create a Swing-based program, Listing 2-1 shows the source for a selectable button :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
public class ButtonSample {
    public static void main(String args[]) {
        Runnable runner = new Runnable() {
            public void run() {
                JFrame frame = new JFrame("Button Sample");
                frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                JButton button = new JButton("Select Me");
 
                // Define ActionListener
                ActionListener actionListener = new ActionListener() {
                    public void actionPerformed(ActionEvent actionEvent) {
                        System.out.println("I was selected.");
                }
            };
 
            // Attach listeners
            button.addActionListener(actionListener);
 
            frame.add(button, BorderLayout.SOUTH);
            frame.setSize(300, 100);
            frame.setVisible(true);
        }
    };
    EventQueue.invokeLater(runner);
    }
}

Ma question portait sur le comment organiser, écouter, correctement l'évenement windowClosing, sachant que ma fenetre est crée dans le constructeur de la classe WIndowManager, et que je voudrais, si c'est possible, écouter dans mon "Main", afin de disposer facilement de références vers d'autres thread de mon programme.

[fr1man]

La différence entre ce bloc de code et le tien, c'est que là, au début de ton main, on est pas encore dans l'EDT, donc on fait appel à invokeLater pour l'initialisation des composants.

Effectivement, invokeLater ou son cousin invokeAndWait font partie des bonnes pratiques, quand on sait s'en servir.

Pour en revenir à ton problème, tu vires donc l'appel à l'une de ces méthodes qui ne sert à rien.
Pour le reste, je n'ai toujours pas bien compris ce que tu cherches à faire.
Qu'est-ce qui t'empêche d'avoir des références vers d'autres threads ?

[bouye]

Quand on est dans l'EDT, invokeLater() oui , invokeAndWait() non !

Quand on est hors de l'EDT, invokeLater() oui , invokeAndWait() attention ca peut quand meme mener a des locks !

SwingUtilities contient une methode qui permet de tester si on est bien dans l'EDT et sinon c'est assez facile de tester la valeur retournee par Thread.currentThread(). Ca permet de faire des propagation d'evenement du style :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
public void setTruc(final Object value) {
  // Do stuff.
  if (SwingUtilities.isEventDispatchTread()) {
    [...]
  }
  // Start again at EDT.
  else {
     SwingUtilities.invokeLater(new Runnable() {
        @Override
        public void run() {
          setTruc(value)
        }
     });
  }
}

[fr1man]

Quand on est dans l'EDT, invokeLater() oui

Quel est l'intérêt d'appeler invokeLater si on est déjà dans l'EDT ?