Discussion :

[Robin_]

Bonjour à tous. Je suis en train de créer une application du style de Motus en Java, et j'ai un souci avec les threads.

Ma grille de mots est une matrice de [6][8] JTextField qui contiennent chacun une lettre.
Lorsque j'écris un mot dans la grille et le valide, j'utilise la boucle suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
for(int i = 0 ; i < 8 ; i++) // On affiche chaque lettre
		{
                        // On espace chaque affichage de lettre de 2 dixièmes de seconde
			try { Thread.sleep(200); } catch (InterruptedException e) {}
 
			if(lettre au bon endroit)
			{
				grille[motActuel.getListe().size()][i].setBackground(Color.RED);
                        }
			else
			{
				grille[motActuel.getListe().size()][i].setBackground(Color.ORANGE);
			}
		}

Le souci est le suivant : les 8 coupures de temps fonctionnent bien, mais les changements d'affichage ne sont effectifs qu'à la fin de la boucle. D'où cela peut-il venir ?

[wax78]

La on ne vois presque rien de ton code.

Cependant je pense que ton problèmes vient du fait que tu ne crée pas de thread (pour l'animation si je puis dire). Et dans ce thread utiliser des SwingWorker.invokeLater au moment de manipuler les object graphique.

D'ou tu as un "callage" de l'interface pendant le temps de ta boucle puis après tu vois seulement les changements.

Tu pourrais faire par exemple quelque chose comme :

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
Thread animationLettre = new Thread()
		{
			public void run()
			{
				for(int i = 0 ; i < 8 ; i++) // On affiche chaque lettre
				{
		            // On espace chaque affichage de lettre de 2 dixièmes de seconde
					try { Thread.sleep(200); } catch (InterruptedException e) {}
		 			SwingUtilities.invokeLater(new Runnable()
					{
						public void run()
						{
							if(lettre au bon endroit)
								grille[motActuel.getListe().size()][i].setBackground(Color.RED);
							else
				      			grille[motActuel.getListe().size()][i].setBackground(Color.ORANGE);
 
 
						}
					});
				}
			}
		};
		animationLettre.start();

Ou regarder http://rom.developpez.com/java-swingworker/

[Robin_]

Mais comment je peux accéder à mes données précédentes (grille, i etc) en utilisant une classe interne ?

[wax78]

Tu fais comme ceci :

Code :

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

Taclasseprincipale.this.tavariable

(nom de la classe principale, this pour indiquer qu'il s'agit de l'instance de la dite classe, suivi de la variable voulue).

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
public class MaClasseMere
{
	String message = "";
	public void fonction()
	{
		Thread t = new Thread()
		{
			public void run()
			{
				MaClasseMere.this.message="dudul";
			}
		};
		t.start();
	}
}

[Robin_]

Il reste la variable i de la boucle qui évolue et donc ne peut être mise "final" ?

[wax78]

Relis mon code, la variable I est dans le Thread pas en dehors de celui ci donc pas de problèmes (ou alors je ne te comprends pas)

Edit : Ha oui ok tu parlais surement de celui qui est dans le swingworker.

Une solution rapide et peut être pas adéquate. (tu auras sans doute le meme probleme avec grille)

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
Thread animationLettre = new Thread()
		{
			public void run()
			{
				for(int i = 0 ; i < 8 ; i++) // On affiche chaque lettre
				{
 final int j = i;
		            // On espace chaque affichage de lettre de 2 dixièmes de seconde
					try { Thread.sleep(200); } catch (InterruptedException e) {}
		 			SwingUtilities.invokeLater(new Runnable()
					{
						public void run()
						{
							if(lettre au bon endroit)
								grille[motActuel.getListe().size()][j].setBackground(Color.RED);
							else
				      			grille[motActuel.getListe().size()][j].setBackground(Color.ORANGE);
 
 
						}
					});
				}
			}
		};
		animationLettre.start();