Discussion :

[b.speedy]

Bonjour,

Je débute dans Java et me familiarise avec le langage, d'où ma question surement simpliste.
Mon objectif est de gérer un menu déroulant.
J'ai 3 classes : MaFrame qui initialise une fenetre, MenuFenetre qui crée le menu et DispatchAction qui gère les évènements.
Lors du choix de menu, c'est DispatchAction qui réagit et je voudrai agir sur la variable "lbl" définie dans Maframe. Or celle-ci est nulle, pourquoi ?
Voici le code :

MaFrame

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
26
27
import java.awt.*;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;
 
public class MaFrame extends Frame {
 
	private static final long serialVersionUID = 1L;
	public Label lbl;
	public MaFrame m;
 
	public MaFrame() {
		super();
		setTitle(" Titre de la Fenetre ");
		setSize(300, 350);
		MenuFenetre mf = new MenuFenetre(this);
		setMenuBar(mf);
		setVisible(true); // affiche la fenetre
		addWindowListener(new WindowAdapter() {public void windowClosing(WindowEvent e){System.exit(0);}});
	}
 
	public static void main(String[] args) {
		MaFrame m =new MaFrame();
		Label lbl=new Label("Essai de texte");
		lbl.setName("lbl");
		m.add(lbl);
	}
}

MenuFenetre

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
import java.awt.*;
 
public class MenuFenetre extends java.awt.MenuBar {
 
	private static final long serialVersionUID = 1L;
	public MenuItem itemQuitter, itemNouveau, itemApropos;
	public MaFrame m;
 
	public MenuFenetre(MaFrame m) {
		Menu menuFichier = new Menu(" Fichier ");
		itemNouveau = new MenuItem(" Nouveau ");
		menuFichier.add(itemNouveau);
		add(menuFichier);
 
		itemNouveau.addActionListener(new DispatchAction(DispatchAction.NEW,m));	
	}
}

et DispatchAction

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
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
public class DispatchAction implements ActionListener{
 
	static final int NEW =1;
	protected MaFrame _obj;
	protected int _type;
 
	DispatchAction(int type,MaFrame obj){
		_obj=obj;
		_type=type;
	}
 
	public void actionPerformed(ActionEvent e) {
		switch (_type) {
		case NEW:
			System.out.print (_obj.lbl.toString());			// ERREUR 
			break;
		default:
			break;
		}
	}
}

Merci de votre aide.

[tchize_]

public Label lbl;

définis un champ lbl dans la class MaFrame, mais l'instruction dans ton main

Label lbl=new Label("Essai de texte");

concerne une variable locale à la méthode main(), ce qui n'a rien à voir avec le champ.

[b.speedy]

Je ne comprends pas la distinction.
Quand je déclare la variable (ou champ ?) "Label lbl" publique, ce n'est pas la même que celle créé dans main() ?
Il y en a une de locale et l'autre publique ?
Comment faut-il alors procéder ?

[tchize_]

Tout ce que tu déclare dans une méthode, ce sont des variable locale.

Pour accéder au lbl de la classe, il faut faire comme tu as faire dans tes autres classe:

Code :

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

1
2
3
4
 m =new MaFrame();
 m.lbl=new Label("Essai de texte");
 m.lbl.setName("lbl");
 m.add(m.lbl);

mais bon, c'est pas très propre, il vaut mieux créer un méthode d'instance dans MaFrame qui fait le travail pour pouvoir appeler

Code :

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

m.setLabel("Essai de texte");

et retirer ce code du main.

[b.speedy]

Ah super !
Ça fonctionne. J'avais pas bien saisi cette visibilité des variables.
Par contre je ne comprends pas la deuxième partie du message.
En quoi cette méthode n'est-elle pas propre ?
De plus "setLabel" est une méthode pour les boutons ou les cases à cocher, quelle relation ici ?

En tout cas merci bien pour le coup de main.

[tchize_]

Pas propre parce que tu fais dans le main du travail qui devrais revenir à ta classe Frame.