Un soucis pour capturer le clic sur un bouton.

Version imprimable

Bonjour tout le monde,

Avant tout, ceci est mon premier message sur ce forum que je parcours depuis quelques années maintenant, c'est une vrai mine d'or et un puit sans fond de connaissances, donc merci d'exister ;) .

je suis en train d'ajouter quelques fonctionnalités à un micrologiciel que j'ai eu l'occasion d'utiliser l'année dernière en cours, c'est un assembleur très basique pour faire découvrir le code bas niveau aux étudiants de mon BTS : http://sylvain.cherrier.free.fr/LT.html

j'essaye d'implanter un bouton de sauvegarde et un de chargement vers ou à partir d'un fichier texte, mais je bute sur un problème pour lier le clic à mes fonctions. un code vaut
mille mots donc voici l'extrait:
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 [...] public class Memoire extends JFrame implements ActionListener{ private JTable laMemoire; TableMemoire model; public Memoire() { setTitle("la Mémoire"); setSize(150,400); setLocation(580,20); model=new TableMemoire(); laMemoire= new JTable(model); laMemoire.getColumnModel().getColumn(0).setPreferredWidth(12); Container c=getContentPane(); c.add(new JScrollPane(laMemoire),"Center"); // ajout d'un boutton pour sauvergarder la memoire dans un fichier texte dans le repertoire courant ou charger depuis ce meme fichier texte. JPanel boutons=new JPanel(); boutons.setLayout(new BorderLayout()); JButton sauvegarder = new JButton("enregistrer"); sauvegarder.addActionListener(this); boutons.add(sauvegarder,BorderLayout.NORTH); JButton charger = new JButton("charger"); charger.addActionListener(this); boutons.add(charger); setVisible(true); c.add(boutons,BorderLayout.WEST); show(); } public void actionPerformed(ActionEvent e) { Object source = e.getSource(); if(source == sauvegarder){ createSave(); System.out.println("enregistrer"); } if(source == charger){ loadSave(); System.out.println("charger"); } } [...]
je n'arrive pas à intercepter le clic des deux boutons, sauvegarder et charger. Je dois mal m'y prendre dans la réécriture de la méthode actionPerformed mais je sèche pour l'instant... un petit coup de main serait le bienvenu, à votre bon cœur !

Salut,

Lorsqu'on regarde la méthode actionPerformed, on voit que tu associes chaque action à chaque bouton par comparaison avec une variable :

Code:

1
2
3
4
5
6
7
8
public void actionPerformed(ActionEvent e) {
		Object source = e.getSource();
 
		if(source == sauvegarder){
			createSave();
			System.out.println("enregistrer");
		}
   /* ... */

Par exemple, ici, sauvegarder : cette variable n'est déclarée nulle part, ni localement à la méthode, ni comme membre de la classe.

Dans le constructeur de la classe Memoire, tu déclares bien une variable sauvegarder :

Code:

1
2
3
4
/* ... */
JButton sauvegarder = new JButton("enregistrer");
sauvegarder.addActionListener(this);
/* ... */

Elle est locale au constructeur, donc inacessible depuis la méthode actionPerformed. Déclare-là par exemple comme membre de la classe :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
public class Memoire extends JFrame implements ActionListener{
	private JTable laMemoire;
	TableMemoire model;
        private final JButton sauvergarder;
 
	public Memoire() {
 
            /* ... */
	    sauvegarder = new JButton("enregistrer");
            sauvegarder.addActionListener(this);
            /* ... */
        }

Autre solution, utiliser les commandes :

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
public class Memoire extends JFrame implements ActionListener{
        private static final String COMMAND_SAVE="save";
	private JTable laMemoire;
	TableMemoire model;
 
	public Memoire() {
 
            /* ... */
	    JButton sauvegarder = new JButton("enregistrer");
            sauvegarder.setActionCommand(COMMAND_SAVE);
            sauvegarder.addActionListener(this);
            /* ... */
       }
 
	public void actionPerformed(ActionEvent e) {
                switch( e.getActionCommand() ) {
                      case COMMAND_SAVE:
			createSave();
			System.out.println("enregistrer");
                        break;
                      /* ... */
                }
	}
}

Autre solution, utiliser un ActionListener par bouton, ce qui évite d'ajouter une méthode public à ta classe :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
public class Memoire extends JFrame {
	private JTable laMemoire;
	TableMemoire model;
 
	public Memoire() {
 
            /* ... */
	    JButton sauvegarder = new JButton("enregistrer");
            sauvegarder.addActionListener(new ActionListener() {
                    	public void actionPerformed(ActionEvent e) {
			createSave();
			System.out.println("enregistrer");
                }
             });
            /* ... */
       }
 
}

et qu'on peut aussi écrire plus simplement avec une expression lambda en Java 8:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
public class Memoire extends JFrame {
	private JTable laMemoire;
	TableMemoire model;
 
	public Memoire() {
 
            /* ... */
	    JButton sauvegarder = new JButton("enregistrer");
            sauvegarder.addActionListener(e-> createSave() );
            /* ... */
       }
 
}

01/09/2015, 20h30
lankoande

Heuh ya plus rien à dire :roll: !
Joël, c'est une réponse complète j'aurais clické deux sur j'aime mais hélas.
Merçi de la réponse !:king:
02/09/2015, 07h52
gregoire sauvaigo

merci pour cette réponse

vraiment excellent cette réponse si rapide en plus... moins d'une heure, merci joel.drigo !

que demande le peuple je vous le demande !!!

c'était bien un problème de portée de variables en effet, la déclarer en membre de la classe a résolu le problème, c'est l'erreur du débutant quoi ^_^, je passe pour un guignol ou ça va ? merci pour tout, je pense que l'on peut passer le topic en résolu !