Portée d'instance de classe.

Version imprimable

Bonsoir tout le monde !

Je suis sur un petit projet et j'ai quelques questions pour lesquelles je n'arrive pas à trouver la solution ...

J'ai une classe StrategoBackground dans laquelle je fais plusieurs réglages de fenêtre, de composants etc ...
Via cette classe, je peut créer une nouvelle fenêtre NewGameFrame proposant un nouveau jeu au joueur.
Cette classe (NewGameFrame) appelle une méthode randomizePiecesPositions d'une autre classe (PlacePieces), forcément, cette méthode intervient sur les attributs de la classe StrategoBackground pour y place des pions.

Lors de l'appel de NewGameFrame dans Stratego Background :
Code:

1 2 3 4 5 6 public class StrategoBackground extends JFrame{ private void newGameActionPerformed(ActionEvent e){ NewGameFrame newGameFrame = new NewGameFrame(this); } }
Je passe l'instance courante afin de garder tout les attributs.

Dans la classe NewGameFrame :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 public class NewGameFrame extends JFrame{ StrategoBackground stb; public NewGameFrame(StrategoBackground stb){ this.stb = stb; initComponents(this.stb); } private void playActionPerformed(ActionEvent e){ // TODO : Methode de random des pions. PlacePieces pp = new PlacePieces(this.stb); if(randomize.isSelected()) pp.randomizePiecesPositions(); } }
Je garde donc l'instance courante de StrategoBackground.

Et enfin, dans PlacePieces :
Code:

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 public class PlacePieces { StrategoBackground stb; public PlacePieces(StrategoBackground stb){ this.stb = stb; } public void randomizePiecesPositions(){ JLabel battleBackground = stb.getBattleBackground(); Pions tabPions[][] = new Pions[10][10]; for(int l = 0 ; l < 10 ; l++) { for(int c = 0 ; c < 10 ; c++) { /*Creation du tableau de Pions*/ } } //stb.setTabButtons(tabPions); stb.setBattleBackground(battleBackground); System.out.println("ICI : " +stb.getTest()); } }
Je n'arrive pas au résultat escompter.

D'ou viens la perte de mes informations de l'instance courante de StrategoBackground ?

A noter que si j'appele la méthode randomizePiecesPosition *depuis* StrategoBackground, tout fonctionne ...

Des idées ?

08/12/2013, 22h43
tchize_

Il n'y a pas de perte d'information. Es-tu sur que newGameActionPerformed et playActionPerformed sont bien appelés?

Oui, toutes les deux sont appelées via un actionListener :

Code:

1
2
3
4
5
6
7
8
9
10
11
 
public class NewGameFrame extends JFrame
{
   /*Bla bla bla */
                      play.addActionListener(new ActionListener(){			
			@Override
			public void actionPerformed(ActionEvent e) {
				playActionPerformed(e); 				
			}
		});
}

Et :

Code:

1
2
3
4
5
6
7
8
9
10
 
public class StrategoBackground extends JFrame
{
                        newGame.addActionListener(new ActionListener() {			
			@Override
			public void actionPerformed(ActionEvent e) {
				newGameActionPerformed(e);
			}
		});
}

Je dois dire que je suis un peu perdu ...

09/12/2013, 14h47
tchize_

je demande la confirmation qu'on est rentré dedans (System.out.println par exemple). Je peux mettre beaucoup de chose dans des actionListeners qui ne seront jamais appelé :)

Ah sorry :D

Oui, j'entre bien dedans en mettant à chaque fois un sysout (dans les deux):

Code:

1
2
3
4
5
6
7
8
 
 play.addActionListener(new ActionListener(){			
			@Override
			public void actionPerformed(ActionEvent e) {
				playActionPerformed(e); 
				System.out.println("Play Action Performed"); 
			}
		});

Ainsi que dans la méthode elle-même (dans les deux aussi) :

Code:

1
2
3
4
5
6
7
 
private void newGameActionPerformed(ActionEvent e)
	{
		// TODO 
		NewGameFrame newGameFrame = new NewGameFrame(this); 
		System.out.println("New Game"); 
	}

09/12/2013, 15h46
tchize_

Ok, a chaque niveau de tes appels (création des listeners, enregistrement des listener, appel des listener, appel des méthodes dans par les listeners), tu peux afficher la valeur de StrategoBackground ? Histoire qu'on découvre à quel endroit tu mixe les deux.

Es-tu sur de ne pas faire un new StrategoBackground quelque part?

Afficher la valeur de Strategobackgroung = un simple sysout ?
Edit : En simple sysout :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
public class StrategoBackground extends JFrame
{
newGame.addActionListener(new ActionListener() {			
			@Override
			public void actionPerformed(ActionEvent e) {
				newGameActionPerformed(e);
				System.out.println("Appel 0 : " +this);
			}
		});
 
        private void newGameActionPerformed(ActionEvent e)
	{
		// TODO 
		System.out.println("Appel 1 : " +this);
		NewGameFrame newGameFrame = new NewGameFrame(this); 
		this.player1Name = newGameFrame.getPlayer1Name(); 
		this.player2Name = newGameFrame.getPlayer2Name(); 
	}
}

Et dans NewGameFrame :

Code:

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
 
private void playActionPerformed(ActionEvent e)
{
StrategoBackground stb; 	
 
	public NewGameFrame(StrategoBackground stb){		
		this.stb = stb; 
		initComponents(this.stb); 
	}
		// Todo : Quitter quand on a fini avec cette fenetre. 
		this.player1Name = (String) player1Tf.getText(); 
		this.player2Name = (String) player2Tf.getText();
 
 
		// TODO : Methode de random des pions. 
		PlacePieces pp = new PlacePieces(this.stb); 
		if(randomize.isSelected())
			pp.randomizePiecesPositions();
		System.out.println("Appel 2 : " +this.stb);
 
 
		//else
		//	pp.placePiecesFromPlayerChoice(); 
 
	}

Resultat console :

Code:

1
2
3
4
5
 
Appel 1 : be.stratego.gui.StrategoBackground[frame0,617,345,1206x788,invalid,layout=java.awt.BorderLayout,title=Stratego,normal,defaultCloseOperation=EXIT_ON_CLOSE,rootPane=javax.swing.JRootPane[,3,25,1200x760,invalid,layout=javax.swing.JRootPane$RootLayout,alignmentX=0.0,alignmentY=0.0,border=,flags=16777673,maximumSize=,minimumSize=,preferredSize=],rootPaneCheckingEnabled=true]
Appel 0 : be.stratego.gui.StrategoBackground$2@5e32007f
ICI : LALA
Appel 2 : be.stratego.gui.StrategoBackground[frame0,617,345,1206x788,invalid,layout=java.awt.BorderLayout,title=Stratego,normal,defaultCloseOperation=EXIT_ON_CLOSE,rootPane=javax.swing.JRootPane[,3,25,1200x760,invalid,layout=javax.swing.JRootPane$RootLayout,alignmentX=0.0,alignmentY=0.0,border=,flags=16777673,maximumSize=,minimumSize=,preferredSize=],rootPaneCheckingEnabled=true]

Donc, c'est apparemment la même ...

Si tu n'y a pas surchargé toString, oui, ça donnera un truc du style

Code:

1
2
3
4
5
6
7
8
9
10
 
System.out.println("avant play.addActionListener(new ActionListener(): "+this.stb);
                      play.addActionListener(new ActionListener(){			
			@Override
			public void actionPerformed(ActionEvent e) {
				System.out.println("Avant Play Action Performed: "+stb); 
				playActionPerformed(e); 		
				System.out.println("Après Play Action Performed: "+stb); 		
			}
		});

ca donnera des truc du style:

Code:

avant play.addActionListener(new ActionListener(): StrategoBackground@12AB34

09/12/2013, 16h14
Mornor

Remarque :

Quand je fais l'appel dans la classe PlacePieces, les appels se font correctement, mais pas du tout dans le bon ordre ...

J'ai :
Appel 1 - Appel 0 - Appel 3 - Appel 2

Edit : C'était de ma faute, c'est régler.

Mais néanmoins, les StrategoBackground renvoyés sont les mêmes ...

Mais je n'ai pas d'appel 5, dans StrategoBackgound :

Code:

1
2
3
4
5
6
7
8
9
10
private void newGameActionPerformed(ActionEvent e)
	{
		// TODO 
		System.out.println("Appel 1 : " +this);
		NewGameFrame newGameFrame = new NewGameFrame(this); 
		System.out.println("Appel 5 : "+this);
		this.player1Name = newGameFrame.getPlayer1Name(); 
		this.player2Name = newGameFrame.getPlayer2Name(); 
	}

Edit : Si en fait ...

09/12/2013, 16h43
Mornor

Tout con ...

Il suffisait de rajouter un biesse .pack() dans randomizePositions ...

Quoi qu'il en soit, un tout tout grand merci du temps que tu m'as consacré ;)
09/12/2013, 20h13
tchize_

le pack() n'est pas indispensable. Je pensais pas que tu modifiais des composant swing directement. Quand to modifie un conteneur visible (ajout/suppression d'éléments dedans), il faut appeler revalidate() dessus après ;)
09/12/2013, 22h33
Mornor

Tiens, je ne connaissais pas ça ! Bon à savoir ;)

Tout grand merci chef :)