Discussion :

[St-42-FCM-57]

Bonjour,

J'aimerais changer le label d'une certaine classe, à partir d'une autre classe. Je ne sais pas si cela est possible, mais j'ai fait ceci, sans que ça ne marche :

Dans ma classe A :

Code :

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

1
2
3
4
5
6
7
 
 public final Label label1;
....
 public void setLabel1(String msg)
	 {
		 label1.setText(msg);
	 }

Dans ma classe B :

Code :

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

1
2
 
classA.setLabel1("Nouveau Message");

Merci d'avance pour votre aide.
Bonne journée

[joel.drigo]

Salut,

Pour qu'une instance communique avec une autre instance, quelque soit ce qu'on fait, changer le texte d'un JLabel, ou d'une valeur d'attribut, ou n'importe quoi d'autre, il faut effectivement une méthode sur l'instance que l'autre instance va invoquer (ou éventuellement un attribut public).

C'est exactement le même concept quand on fait :

Code :

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

classA.setLabel1("Nouveau Message");

ou

Code :

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

label1.setText(msg);

Aussi faut-il que cela soit la bonne instance. Diffiicile de te répondre plus que ça vu le peu de code que tu montres.

[St-42-FCM-57]

En fait, j'ai un jeu de Monopoly, et quand j'utilise la fonction acheter appartenant à la classe Joueur, je voudrais que ça me modifie un label dans ma fenêtre principal.

Dans la classe joueur j'ai fait ceci :

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
 
private FenetrePrincipale fenetre;
...
...
 
public void acheter(int argent)
 
	{
		if(getArgent()-argent < 0 )
		{
			//Pas assez d'argent
			fenetre.setMessageIndication("Vous n'avez pas assez d'argent pour acheter ce bien !"); // Pour afficher sur le label
		}
		else
		{
			setArgent(getArgent()-loyer);
			fenetre.setMessageIndication("Vous venez d'acheter cette case, bravo !");
		}
	}

Et dans la classe fenêtre principale :

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
 
 
public final Label label1;
...
 
public String messageIndication="";
 
	 public void setMessageIndication(String msg)
	 {
		 messageIndication=msg;
	 }
 
...
 
private class ValiderReponse implements EventHandler<ActionEvent> {
		public void handle(ActionEvent event) {
 
              ...
              ...
              joueur.get(indiceJoueur).acheter(case));
              label1.setText(messageIndication);
              ...
        }
}

Et quand je lance ceci, j'ai un PointerNullException qui se lance, et vient de ça : fenetre.setMessageIndication("..."); (dans la classe Joueur).

Merci

[joel.drigo]

Et quand je lance ceci, j'ai un PointerNullException qui se lance, et vient de ça : fenetre.setMessageIndication("..."); (dans la classe Joueur).

Bah, c'est ce que je disais : il faut que la variable fenetre contienne la référence à la bonne instance de FenetrePrincipale, donc implicitement à une instance de FenetrePrincipale. Ou alors il faut utiliser un modèle et passer par le modèle, mais ça ne change rien question variable et instance.

[St-42-FCM-57]

Désolé, mais je ne comprends pas trop ce qu'il faut que je rajoute ?
Ca ne suffit pas dans joueur de mettre " FenetrePrincipale fenêtre " ?

[joel.drigo]

Non, ça ne suffit pas de déclarer une variable pour qu'elle est une valeur.

Il faut qu'il y ait quelque part fenetre = <quelque chose (et le bon quelque chose)>;, et au bon moment évidemment.

Code :

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

1
2
3
4
5
6
7
public class Machin {
 
    public void faitCeci() {
    }
 
 
}

Code :

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

1
2
3
4
5
6
7
8
public class Truc {
   private Machin machin;
 
   public void faitCeciAvecMachin() {
         machin.faitCeci();
   }
 
}

Donc soit :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public class Truc {
   private Machin machin;
 
   public Truc(Machin machin) {
        this.machin=machin;
   }
 
   public void faitCeciAvecMachin() {
         machin.faitCeci();
   }
 
 
}

soit

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public class Truc {
   private Machin machin;
 
   public void setMachin(Machin machin) {
        this.machin=machin;
   }
 
 public void faitCeciAvecMachin() {
         machin.faitCeci();
   }
 
}

Soit encore :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public class Truc {
   private Machin machin;
 
   public Truc() {
        this.machin=new Machin();
   }
 
   public void faitCeciAvecMachin() {
         machin.faitCeci();
   }
 
 
}

il faut une affectation quelque part

[St-42-FCM-57]

Je n'ai jamais vu cette syntaxe, je suis plutôt débutant en java. A la place du "quelque chose" on doit y mettre quoi ? Un champ ? Une adresse ?

[joel.drigo]

Je n'ai jamais vu cette syntaxe, je suis plutôt débutant en java. A la place du "quelque chose" on doit y mettre quoi ? Un champ ? Une adresse ?

La référence à l'instance de FenetrePrincipale !!! Tu as bien dans ton code quelque part quelque chose comme new FenetrePrincipale().
Donc si tu fais :

Code :

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

FenetrePrincipale fenetrePrincipale = new FenetrePrincipale();

La valeur de la variable fenetrePrincipale c'est la référence à l'instance de FenetrePrincipale créée par le new. Valeur que tu peux affecter à la variable fenetre, soit pas passage de paramètre, soit par le constructeur. Tu as également la possibilité si l'instance de joeur est créée dans la classe FenetrePrincipale, de passer this en paramètre (this représentant l'instance de FenetrePrincipale elle-même).

[St-42-FCM-57]

Ah oui je vois. J'avais pas compris.
Seulement, mon constructeur de fenetrePrincipale contient un Stage, donc quand j'essaie de faire fenetre=new fenetrePrincipale() dans joueur, Ecplise attend un Stage entre les parenthèses...

Edit :
désolé, je connaissais ce dont tu m'as parlé, j'étais juste "perdu" à cause des < >, mais c'était une façon de mettre ton pseudo texte...

[joel.drigo]

Ah oui je vois. J'avais pas compris.
Seulement, mon constructeur de fenetrePrincipale contient un Stage, donc quand j'essaie de faire fenetre=new fenetrePrincipale() dans joueur, Ecplise attend un Stage entre les parenthèses...

Oui mais ça il ne faut pas faire, parce que tu vas avoir la référence d'une AUTRE instance de FenetrePrincipale dans la variable de ton joueur. Quand j'ai dit la "bonne" instance, ça excluait implicitement une "autre" instance que la "bonne".

Elle est bien créée quelque part cette FenetrePrincipale ? Ça à l'air d'être du JavaFX ton truc (si tu avais directement mis ta question dans le forum JavaFX, ça aurait été plus évident). Je ne connais pas trop JavaFX, et le système de fxml. Si ça fonctionne sans qu'on ait de new FenetrePrincipale explicite ou pas. Mais tu dois avoir un moyen de récupérer la référence de la fenêtre (ou d'autre chose, comme un modèle). Ou alors, il faut utiliser un système évenementiel pour créer tes joueurs et le reste, et au déclenchement de l'évenement, passer la référence qui va bien.

[St-42-FCM-57]

Oui c'est du javaFX.
En fait elle est juste crée dans une classe Main comme suit :

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
 
public class Main extends Application {
	public Main(){};
	@Override
  	public void start(Stage primaryStage) {
	try {
		new FenetrePrincipale(primaryStage);	
 
	} catch(Exception e) {
		e.printStackTrace();
	}
}
 
public static void main(String[] args) {
	launch(args);
}
}

[joel.drigo]

Et les instances de Joueur sont créés où ?

[St-42-FCM-57]

Elles sont crées dans le constructeur de FenetrePrincipale(Stage primaryStage).

[St-42-FCM-57]

En gros dans fenetrePrincipale j'ai ça (en un peu plus complexe car je gère la création de plusieurs joueurs) :

Code :

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

1
2
3
4
5
6
7
 
private Joueur joueur;
public fenetrePrincipale(Stage primaryStage)
{
    joueur = new Joueur(nomJoueur);
    ....
}

[joel.drigo]

Donc tu peux mettre FenetrePrincipale en paramètre du constructeur de Joueur :

Code :

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

1
2
3
4
public Joueur(String nomJoueur, FenetrePrincipale fenetrePrincipale) {
      /*...*/
      fenetre=fenetrePrincipale;
}

Et dans le code du constructeur de FenetrePrincipale, tu auras new Joueur(nomJoueur,this);, this dans le constructeur de FenetrePrincipale étant la référence de l'instance en cours de création de FenetrePrincipale.

[St-42-FCM-57]

Oui c'est parfait, ça marche !
Avant de poster le message sur le forum, j'avais essayé quasiment la même chose, sauf que quelque part (je ne sais plus où), je faisais un "new", et du coup ca me faisait une boucle infinie pour la création de mes joueurs.

Un grand merci en tout cas pour ton aide !
Bonne journée !