Discussion :

[Sylar44]

Bonjour tout le monde
j'ai un petit problème avec la gestion d'évenement sur des boutons.
J'ai crée un bouton comme tel

Code :

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

1
2
3
4
5
6
7
8
9
 
//creation du bouton jouer
Button jouer = new Button("Jouer");
jouer.setSize(50,25);
jouer.setLocation(190,75);
add(jouer);   // ajout "graphique" du composant
jouer.addActionListener(this);    
// ajout "événementiel" du composant
jouer.setVisible(true);

et je fais un test pour savoir ce su'il se passe

Code :

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

1
2
3
4
5
6
7
8
9
 
public void actionPerformed(ActionEvent evt) {
    Object source = evt.getSource(); 
    System.out.println(source);
    System.out.println(jouer);
    if (source == jouer) {nb_Coups.setNbCoups(); 
    System.out.println(txt.getText());
    } else System.out.println("a"+txt.getText());
}

Il me renvoit ceci quand j'appuie sur le bouton jouer.
Apparemment le bouton n'as pas été crée mais je vois pas le problème. J'espere que vous pourrez m'aider

java.awt.Button[button0,190,75,50x25,invalid,label=Jouer]
null
a9

[Pollux]

Ton bouton a bien été créé mais le problème c'est que sa portée est limitée au corps de ton constructeur ou de ta méthode. Modifie de cette manière:

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
public class Test
{
    private JButton jouer;
 
    public Test()
    {
         jouer = new JButton("jouer");
         //ton code...
    }
 
    public void actionPerformed(ActionEvent evt) {
        Object source = evt.getSource();
        System.out.println(source);
        System.out.println(jouer);
        if (source == jouer) {
            nb_Coups.setNbCoups();
            System.out.println(txt.getText());
        } else 
            System.out.println("a"+txt.getText());
    }
}

[Sylar44]

c'est ce que j'ai fait,
Je met mon code en entier, vous verrez plus ce que j'ai fait

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
 public class JeuDes extends Frame implements ActionListener{
     private Button jouer;
     public JeuDes(final String nom,final int i, final int j) {
 
    // titre de la fenêtre
    ...                
    //creation du la listbox
             ...                   
            //creation du bouton jouer
    Button jouer = new Button("Jouer");
             jouer.setSize(50,25);
             jouer.setLocation(190,75);
             add(jouer);   // ajout "graphique" du composant
             jouer.addActionListener(this);    // ajout "événementiel" du composant
             jouer.setVisible(true);
 
             public void actionPerformed(ActionEvent evt) {
        Object source = evt.getSource(); 
        //System.out.println(source);
        //System.out.println(jouer);
        if (evt.getActionCommand().equals("Jouer")) {
            nb_Coups.setNbCoups(txt.getText()); 
            //System.out.println(txt.getText()); 
        } 
    }
}

[Napalm51]

Le problème est que tu redéfini ton bouton dans ton constructeur. Il ne faut pas écrire

Code :

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

Button jouer = new Button("Jouer");

mais bien

Code :

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

jouer = new Button("Jouer");

comme dans le code qu'on t'a posté juste au dessus

Edit : essayes d'utiliser les balises CODE pour ton code (l'icône #)

[Sylar44]

désolé pour la présentation, je vais faire un effort
maintenant j'ai des messages d'erreurs

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
	at initial.JeuDes.actionPerformed(JeuDes.java:65)
	at java.awt.Button.processActionEvent(Unknown Source)
	at java.awt.Button.processEvent(Unknown Source)
	at java.awt.Component.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

ligne 65 : nb_Coups.setNbCoups(txt.getText());

ma classe ChampsnbCoups utilise cette méthode

Code :

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

1
2
3
4
5
6
7
8
public void setNbCoups(String nb) {
		try {
			int nb1=Integer.parseInt(this.getText());
			this.nbCoups=nb1;
		}catch (Exception ex) {
			this.setText("");
		}
	}

[Pollux]

C'est le même problème que pour ton bouton mais cette fois-ci pour nb_coups et/ou txt..

Vérifie l'initialisation et la portée de ces variables.

[kerinel]

Bonjour,
une petite nuance, c'est ce que tu penses faire mais probablement pas
lorsque tu écris
Button jouer = new Button("Jouer");

cette partie est probablement dans le constructeur de ta classe. Or en écrivant Button devant jouer, tu indiques que tu crées une variable dont la portée est limitée à l'intérieur du constructeur. Pour créer une variable dont la portée est étendue à toute la classe,l e code correct est :

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 class maClasse extends ActionListener{
 
private Button jouer; //déclaration en dehors du constructeur
 
public maClasse(){ //le constructeur
...
jouer = new Button("Jouer"); //instanciation dans le constructeur
//et non pas Button jouer= new Button("Jouer"); qui masque le bouton jouer pour le constructeur
 
jouer.addActionListener(this);
...
} 
 
public void actionPerformed(ActionEvent evt) {
Object source = evt.getSource();
//System.out.println(source);
//System.out.println(jouer);
if (evt.getActionCommand().equals("Jouer")) {
nb_Coups.setNbCoups(txt.getText());
//System.out.println(txt.getText());
} 
 
}

Bon code,
kerinel