Modification d'un JLabel depuis une autre classe

Version imprimable

04/11/2010, 12h26
bigram

Modification d'un JLabel depuis une autre classe

Bonjour à tous,

J'ai cherché sur le net mais n'ai pas trouvé de réponses applicables à mon problème(simple, je suis sûr).

J'explique :

J'ai une JFrame avec des Horizontalbox et verticalBox dans lesquelles j'ai des composants JLabel.
Dans ma JFrame, j'ajoute un JPanel.

Dans mon autre classe, la classe Panneau est un extend de JPanel.
Dans Panneau, je modifie de manière fréquente un string que je dois placer dans le JLabel qui se trouve dans ma classe Main.
COmment dois je faire?J'ai essayé de placer le string en paramètre dans le constructeur de Panneau, mais ca ne sera exécuté qu'une fois or, les modif sont fréquentes.:cry:

Merci de votre aide.
04/11/2010, 12h53
kaoualeo

En fait tu souhaites que ton JLabel se mette automatiquement à jour à chaque changement de valeur de ta String?

La question est: comment saisis-tu ta nouvelle String? est-ce une saisie manuelle par une interface graphique?
04/11/2010, 13h13
bigram

Oui, je voudrais qu'a chaque fois que la variable score se met à jour, la valeur soit transmise automatiquement vers mon JFrame qui se trouve dans la classe Main.
Et oui, c'est mon interface graphique qui modifie ma variable score.

Merci.
04/11/2010, 13h27
kaoualeo

Il y a plusieurs façons de procéder, plus ou moins adaptées selon la façon dont tu crée ton appli.

Pour ma part celle que je préfère c'est d'utiliser de créer un objet qui contiendra une propriété score par exemple, de type String et un PropertyChangeSupport.

Voir cette discussion à ce propos

Quand tu modifies le champs score de l'interface graphique (ton Panneau), tu fais en sorte de modifier la variable score de l'objet.

Pour se faire, met en écoute ton interface graphique par un keyListener par exemple, et quand elle est modifiée tu fait tonObjet.setScore(uneString).

Ta méthode setScore(String) doit faire un fire de ton PropertyChangeSupport.

Ensuite, tu fait en sorte que ton JLabel écoute ton objet score (score.addPropertyChangeListener()

C'est peut etre pas trés clair... dis moi si c'est le cas.
04/11/2010, 13h54
bigram

Pour le keyListener, j'ai plus ou moins compris que ca pourrait etre utile pour savoir quand changer le texte du label.
Mais j'ai pas compris les explications pour le propertyChangeSUpport.

Score est une variable dans ma classe Panneau, n'est il pas simplement possible de transferer le contenu de la variable vers ma classe Main qui contient un JFrame et le JLabel en question?

Si, tu peux aussi.

Si tu ecoutes avec le keyListener, le texte se mettra à jour a chaque saisie de lettre, alors que sur un actionListener, ce sera lors de l'appuie sur la touche entrée.

A toi de voir ce que tu préfère.

Concrètement, met ton panneau en écoute sur le champs de saisie du score:
champsScore.addKeyListener(this), ici "this" représente le panneau.

Lors de la déclaration de ta classe Panneau, indique:

Code:

public class Panneau implements KeyListener

, puis ajoute les méthodes:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 @Override public void keyTyped( KeyEvent e ) { // TODO Auto-generated method stub } @Override public void keyPressed( KeyEvent e ) { // TODO Auto-generated method stub } @Override public void keyReleased( KeyEvent e ) { // TODO Auto-generated method stub }
Ensuite va sur l'une des méthodes que tu viens de créer (par exemple, keyReleased()).

Cette méthode sera appelée à chaque saisie de caractère dans ton champs score. C'est là qu'il faut mettre à jour ton JLabel.
Débrouille-toi pour acceder à ton JLabel depuis cette méthode, et fais lui un

Code:

setText(champsScore.getText());

Citation:

Envoyé par bigram

Score est une variable dans ma classe Panneau, n'est il pas simplement possible de transferer le contenu de la variable vers ma classe Main qui contient un JFrame et le JLabel en question?

Oui c'est parfaitement possible si tu disposes d'une méthode de mise à jour dans la JFrame, après c'est un choix de conception avec la solution par notification proposée :
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 class Main extends JFrame { private JLabel leJLabelQuiAfficheLeScore = new JLabel("0"); public void scoreChanged(int newScore) { leJLabelQuiAfficheLeScore.setText(String.valueOf(newScore)); } } class Panneau extends JPanel { private Main mainFrame; private int score; public Panneau(Main mainFrame) { this.mainFrame = mainFrame; } private void updateScore() { score += ...; mainFrame.scoreChanged(score); } }
A noter qu'il n'est pas très sain que la classe Panneau connaisse la classe Main (la dépendance est dans le mauvais sens). Tu devrais créer une classe/une interface permettant de transmettre les modifications à d'autres classes... Exemple :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 interface Notifier { void scoreChanged(int newScore); } class Main extends JFrame implements Notifier { ... } class Panneau extends JPanel { private Notifier notifier; public Panneau(Notifier notifier) { this.notifier= notifier; } ...

Avec un exemple, ca sera plus clair :)
VOilà ma classe Main
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 26 public static void main(String[] args) { // TODO Auto-generated method stub JFrame fen = new JFrame(); fen.setSize(1024,700); JPanel pan = new Panneau(); JLabel lblBriques = new JLabel("Briques restantes : "); JLabel lblNbBriques = new JLabel(); Box hBoxBrick = Box.createHorizontalBox(); hBoxBrick.setPreferredSize(new Dimension (50,50)); hBoxBrick.add(lblBriques); hBoxBrick.add(lblNbBriques); JLabel lblVie = new JLabel("Vies restantes : "); JLabel lblNbVie = new JLabel(); Box hBoxVie = Box.createHorizontalBox(); hBoxVie.setPreferredSize(new Dimension(50,50)); hBoxVie.add(lblVie); hBoxVie.add(lblNbVie); JLabel lblBonus = new JLabel("Bonus : "); JLabel lblTypBonus = new JLabel(); Box hBoxBonus = Box.createHorizontalBox(); hBoxBonus.setPreferredSize(new Dimension(50,50)); hBoxBonus.add(lblBonus); hBoxBonus.add(lblTypBonus); JLabel lblScore = new JLabel("Score : "); JLabel lblPoints = new JLabel();
la classe contient un label lblPoints qui doit etre modifié.

ma classe panneau dont l'objet pan a été créé dans ma classe Main se charge de modifier automatiquement le score quand une action est réussie.
Mais de a partir de ma classe panneau, je ne sais pas accèder a mon JLabel qui se trouve dans ma classe Main.

Je ne peux appliquer de keyListener car le score ne change pas en fonction d'une touche du clavier, c'est en fonction d'un évenement graphique qui se produit dans ma classe panneau.

Mes explications ne sont peut être pas très claires mais ma classe panneau est trop grande à afficher.

Merci encore

Ce n'est pas beau du tout... mais voila une solution pour ton problème:

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
 
 
import java.awt.BorderLayout;
import java.awt.Dimension;
 
import javax.swing.Box;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextField;
 
public class Test1 extends JFrame {
	private JLabel lblScore = new JLabel( "Score : " );
 
	public static void main( String[] args ) {
		Test1 t = new Test1();
		t.pack();
		t.setVisible( true );
	}
 
	public void setScore( String tonScore ) {
		lblScore.setText( tonScore );
	}
 
	public Test1() {
		JFrame fen = new JFrame();
		fen.setSize( 1024, 700 );
		JPanel pan = new Panneau( this );
		JLabel lblBriques = new JLabel( "Briques restantes : " );
		JLabel lblNbBriques = new JLabel();
		Box hBoxBrick = Box.createHorizontalBox();
		hBoxBrick.setPreferredSize( new Dimension( 50, 50 ) );
		hBoxBrick.add( lblBriques );
		hBoxBrick.add( lblNbBriques );
		JLabel lblVie = new JLabel( "Vies restantes : " );
		JLabel lblNbVie = new JLabel();
		Box hBoxVie = Box.createHorizontalBox();
		hBoxVie.setPreferredSize( new Dimension( 50, 50 ) );
		hBoxVie.add( lblVie );
		hBoxVie.add( lblNbVie );
		JLabel lblBonus = new JLabel( "Bonus : " );
		JLabel lblTypBonus = new JLabel();
		Box hBoxBonus = Box.createHorizontalBox();
		hBoxBonus.setPreferredSize( new Dimension( 50, 50 ) );
		hBoxBonus.add( lblBonus );
		hBoxBonus.add( lblTypBonus );
		JLabel lblPoints = new JLabel();
		getContentPane().add( pan );
	}
 
	public class Panneau extends JPanel {
		JTextField score = new JTextField();
		Test1 t = null;
 
		public Panneau( Test1 t ) {
			this.t = t;
			setLayout( new BorderLayout() );
			add( score, BorderLayout.CENTER );
		}
 
		public void methodeAAppeler() {
			t.setScore( score.getText() );
		}
	}
}

Appèle la méthode de ton Panneau "methodeAAppeler()" lors de ton évènement graphique.

05/11/2010, 11h03
bigram

Bonjour,
j'ai essayé le code proposé mais ce n'est pas concluant.
Je suis novice en Java mais je ne pensais quand même pas qu'il est si difficile en JAVA de transmettre une string d'une classe vers un label d'une autre classe...:(

Comment aurais je dû m'y prendre pour que cela fonctionne correctement?

Créer une classe Score? (j'en vois pas trop l'utilité)
avec un interface, j'y arrive pas

Je suis perduuuuuu :)

Je ne doute pas de l'implication des membres à trouver une solution, j'en suis reconnaissant mais je ne doute pas non plus que mes connaissances en Java sont nulles et c'est pourquoi des explications me seraient TRES utiles.

Merci encore.
05/11/2010, 11h09
herve91

Le code que je t'ai donné plus haut ne te permet pas d'avancer ?
05/11/2010, 11h13
kaoualeo

Grosso modo, tu n'utilises pas vraiment les possibilités Objet de Java.
Travailler comme tu le fais dans le main te bloque...

Si tu regardes le bout de code que je t'ai passé, tu notera que le main ne fait plu grand chose. Ce que j'ai fais n'est pas la panacée mais devrait t'aiguiller...

Si tu as la possibilité, passe nous ton code complet, se sera plus simple.

Herve91

j'ai une erreur quand je teste ton code

Code:

1
2
3
4
5
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
	at Panneau.updateScore(Panneau.java:69)
	at Panneau.animation(Panneau.java:190)
	at Panneau.access$0(Panneau.java:95)
	at Panneau$1.actionPerformed(Panneau.java:250)

voilà ou ca bloque :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
public Panneau(Main m){
 
		this.setSize(700, 700);
		this.setBackground(Color.black);
		rac = new Racket();
		bal = new Balle();
		tab = new Brique [maxCol][maxLigne];
		brick = new Brique();
		addMouseMotionListener(this);
		addMouseListener(this);
		addKeyListener(this);
		//this.notifier = notifier;
		this.m = m;
		init();
		createLevel();
		partie();		
	}

et dans ma classe Main

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
import java.awt.Color;
import java.awt.Container;
import java.awt.Dimension;
 
import javax.swing.Box;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
 
 
public class Main extends JFrame{
 
	private JLabel lblPoints = new JLabel("0");
	static Main m = new Main();
 
 
	public void scoreChanged(int newScore) {
	      lblPoints.setText(String.valueOf(newScore));
	}
 
	   public static void main(String[] args) {
			// TODO Auto-generated method stub
 
			m.pack();
			m.setVisible(true);
		}
	/**
         * @param args
         */
	public Main(){
 
		JFrame fen = new JFrame();
		fen.setSize(1024,700);
		JPanel pan = new Panneau(m);
		JLabel lblBriques = new JLabel("Briques restantes : ");
		JLabel lblNbBriques = new JLabel();
		Box hBoxBrick = Box.createHorizontalBox();
		hBoxBrick.setPreferredSize(new Dimension (50,50));
		hBoxBrick.add(lblBriques);
		hBoxBrick.add(lblNbBriques);
		JLabel lblVie = new JLabel("Vies restantes : ");
		JLabel lblNbVie = new JLabel();
		Box hBoxVie = Box.createHorizontalBox();
		hBoxVie.setPreferredSize(new Dimension(50,50));
                ...

Si tu peux m'aider, ce serait magnifique.

Citation:
Envoyé par bigram
Code:

1 2 3 4 5 Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException at Panneau.updateScore(Panneau.java:69) at Panneau.animation(Panneau.java:190) at Panneau.access$0(Panneau.java:95) at Panneau$1.actionPerformed(Panneau.java:250)
Si tu peux m'aider, ce serait magnifique.
Merci de fournir le code de la méthode updateScore pour déterminer la cause de l'exception

05/11/2010, 11h38
bigram

kaoualeo,

merci de ton aide, oui comme je l'ai dit, je débute en java donc il fort possible que je m'y prenne mal.
j'essaie de créer un jeu de casse brique, certes, c'est un peu compliqué pour un débutant, mais je me dit aussi que dans un jeu, on fait appel à toutes sortes d'instructions, ce qui peut m'aider dans mon apprentissage...
A aprtir de ce que j'ai posté plus haut, s'il y a des remarques ou critiques, n'hésite pas :)

voilà le code updateScore()

Code:

1
2
3
4
5
private void updateScore() {
	      score += 10;
 
	      m.scoreChanged(score);
	   }

05/11/2010, 11h42
kaoualeo

Pas de soucis.
Pour ton problème, comme te l'a dit herve91, il faut le code de la méthode updateScore et plus particulierement la ligne 69 de ta classe Panneau.

En gros, ton erreur signifie que en ligne 69 de ta classe Panneau, dans ta méthode updateScore, tu fais demande des choses à un objet null.
Reste à determiner quel est cet objet.
(tu as posté ta réponse entre temps... ici, c'est m qui est null. tu ne dois pas l'initialiser)

Si tu met static ton main alors fais ceci:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
 
public class Main extends JFrame{
 
	private JLabel lblPoints = new JLabel("0");
	static Main m = new Main();
 
 
	public static void scoreChanged(int newScore) {
	      m.scoreChanged(newScore);
	}
 
	public void scoreChanged(int newScore) {
	      lblPoints.setText(String.valueOf(newScore));
	}

Code:

1
2
3
4
5
6
 
private void updateScore() {
	      score += 10;
 
	      Main.scoreChanged(score);
	   }

Non, quelle idée de mettre 'm' static !!

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
26
27
28
29
30
31
32
33
34
public class Main extends JFrame{
 
	private JLabel lblPoints = new JLabel("0");
	// a supprimer tu es déjà dans Main ! static Main m = new Main();	
	
	public void scoreChanged(int newScore) {
	      lblPoints.setText(String.valueOf(newScore));
	}
   
	   public static void main(String[] args) {
			// TODO Auto-generated method stub
			Main m = new Main();
			m.pack();
			m.setVisible(true);
		}
	/**
	 * @param args
	 */
	public Main(){
		
		JFrame fen = new JFrame();
		fen.setSize(1024,700);
		JPanel pan = new Panneau(m);
		JLabel lblBriques = new JLabel("Briques restantes : ");
		JLabel lblNbBriques = new JLabel();
		Box hBoxBrick = Box.createHorizontalBox();
		hBoxBrick.setPreferredSize(new Dimension (50,50));
		hBoxBrick.add(lblBriques);
		hBoxBrick.add(lblNbBriques);
		JLabel lblVie = new JLabel("Vies restantes : ");
		JLabel lblNbVie = new JLabel();
		Box hBoxVie = Box.createHorizontalBox();
		hBoxVie.setPreferredSize(new Dimension(50,50));
                ...

05/11/2010, 12h05
bigram

Merci Kaoualeo,

maintenant ca fonctionne bien:ccool:, le score augmente en effet quand la brique se casse.

Maintenant, il me reste plus qu'a essayer de comprendre ce que tu as fait pour que ca fonctionne:)
Et surtout de voir comment j'aurais pu faire autrement plus facilement.
05/11/2010, 12h08
herve91

Citation:

Envoyé par bigram

Merci Kaoualeo,

maintenant ca fonctionne bien:ccool:, le score augmente en effet quand la brique se casse.

Maintenant, il me reste plus qu'a essayer de comprendre ce que tu as fait pour que ca fonctionne:)
Et surtout de voir comment j'aurais pu faire autrement plus facilement.

Ne mets pas 'm' en static :roll:

herve91,

si je fais ce que tu dis, j'ai une erreur
Code:

1 2 3 4 5 Exception in thread "main" java.lang.Error: Unresolved compilation problem: m cannot be resolved to a variable at Main.<init>(Main.java:37) at Main.main(Main.java:25)
c'est dû a quoi?

Merci.

05/11/2010, 12h10
herve91

Code:

JPanel pan = new Panneau(this);
05/11/2010, 12h59
kaoualeo
Citation:

Envoyé par herve91

Non, quelle idée de mettre 'm' static !!

Ce n'est pas la solution avec un grand "L" je l'admet... disons que ca fontionne.

Mais puisque le main est unique, pourquoi ne pas appliquer le principe du singleton?
Code:

1 2 3 4 5 6 7 8 9 10 11 private static Main m = null; private Main(){ } public static Main getInstance(){ if(m==null){ m=new Main(); } return m; }
Ca te donne accès au main de n'importe ou en faisant Main.getInstance();
05/11/2010, 14h06
herve91

Je ne vois pas l'intérêt de mettre le Main en singleton, ce n'est pas un gestionnaire de ressources, et qui plus est les autres classes ne devraient même pas le connaître (c'est une classe de "haut" niveau). La classe Main est instanciée dans la méthode main(String[]), donc pas de souci. Il est toujours préférable de se passer de variables statiques, et accessoirement de singletons.
05/11/2010, 14h12
bigram

mon code suscite beaucoup de commentaires à ce que je vois :)
Merci à tous les 2 pour votre aide,étant donné que la classe Main est une classe de haut niveau et que les autres classes ne devraient pas la "voir", comment est ce vous auriez fait si vous aviez un label dans la classe main et qu'il faille le changer depuis un autre endroit.

Ou alors est ce que vous n'auriez même pas proceder de la sorte?
05/11/2010, 14h26
kaoualeo

Le mieux est de faire que ce soit l'objet à mettre à jour qui écoute le changement de score et le répercute.

En effet, si tu changes ton JLabel et que tu le met dans une autre classe demain, tu va devoir revoir ta structure (il ne faudra plu passer le Main en paramètre de ton Panneau par exemple).
05/11/2010, 14h27
Caalador

Bonjour à tous,

Personnellement, j'opterai pour une class model représentant le score, et l'utilisation du pattern Observer permettant à d'autres class de suivre la progression de ce score.
Là, on parle juste de l'affichage de ce score dans une fenêtre, mais lorsque le score va évoluer, il y aura peut-être d'autres choses à suivre : vie supplémentaires, augmentation du niveau de difficulté, etc.

Après, pour débuter et avoir une solution pas trop complexe, la class main en singleton n'est pas si mal je trouve. Ca permet de débuter sur un pattern relativement simple. Si il ouvre cette fenêtre via un bouton, cela lui permettra aussi de n'avoir qu'une fenêtre au lieu d'en avoir une à chaque clic.
05/11/2010, 14h36
herve91

Je persiste à dire que la classe Main n'a rien à faire avec un singleton ! C'est la classe principale de l'application, il n'y en a qu'une seule instance, créée par la méthode "main"
Le pattern singleton est généralement réservé pour contrôler/gérer l'accès à une ressource.
05/11/2010, 14h50
bigram

Avec tous ces commentaires, je suis un peu perdu :)

je sais pas pourquoi mais j'aime bien l'idée de Caalador d'utiliser un pattern Observer (meme si je ne sais pas exactement ce que c'est), mais c'est vrai que d'autres labels devront etre modifiés, donc c'est peut etre pas une mauvaise idée.

Qu'en penses tu?
05/11/2010, 15h50
herve91

Bonne idée, c'était esquissé dans le post 5570211
avec l'alternative d'une classe Notifier permettant de transmettre les modifications à d'autres classes... mais le pattern Observateur est beaucoup plus généraliste et adaptable.

Fuseau horaire GMT +2. Il est actuellement 12h03.