Portée d'instance de classe.

**Mornor** · 08/12/2013, 17h19

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 :

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

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 :

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 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 :

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
 
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 ?

**tchize_** · 08/12/2013, 22h43

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

**Mornor** · 09/12/2013, 14h08

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

Code :

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

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 :

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

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 ...

**tchize_** · 09/12/2013, 14h47

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é

**Mornor** · 09/12/2013, 15h24

Ah sorry

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

Code :

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

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 :

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

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

**tchize_** · 09/12/2013, 15h46

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?

**Mornor** · 09/12/2013, 15h53

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

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
 
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 :

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
 
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 :

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

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 ...

**tchize_** · 09/12/2013, 16h08

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

Code :

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

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 :

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

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

**Mornor** · 09/12/2013, 16h14

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 ...

**Mornor** · 09/12/2013, 16h26

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

Code :

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

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 ...

**Mornor** · 09/12/2013, 16h43

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é

**tchize_** · 09/12/2013, 20h13

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

**Mornor** · 09/12/2013, 22h33

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

Tout grand merci chef

Portée d'instance de classe.

Java

Discussions similaires

Partager

Partager