Discussion :

[stpaul04]

Bonjour a tous,

je suis debutant en Java et j'essaie de faire un petit programme qui doit afficher un texte quand on clique su bouton, et ca marche, le probleme maintenant je veux ajouter un deuxieme bouton qui affiche un autre texte quand on clique dessus, et la ca ne marche pas, pouvez-vous m'aider svp a corriger mon erreur.

voici mo programme:

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
import java.awt.*;
import java.awt.event.*;
import java.applet.*;
 
public class Bouton22 extends Applet implements ActionListener
{
 
String msg=" ";
 
Button click;
Button press;
 
public void init()
{
click=new Button("Click-me");
add(click);
click.addActionListener(this);
 
press=new Button("Pressez");
add(press);
press.addActionListener(this);
 
 
 
setBackground(Color.gray);
setForeground(Color.black);
}
public void actionPerformed(ActionEvent ae)
{
String str=ae.getActionCommand();
if(str.equals("CLick-me"));
{
msg="You pressed the button.";
}
 
else 
if(str.equals("Pressez"));
{
msg="Vous avez appuyez sur le bouton.";
}
 
repaint();
}
public void paint (Graphics g)
{
 
g.drawString(msg,80,100);
 
 
}
}

[Dnasty]

Alors déja dans tes blocs "if" tu mets des ";" après la condition... par conséquent le code que tu mets ensuite sera exécuté quoi qu'il arrive.

J'ai aussi remarqué que tu as mis un "L" à "Click-me"

Code :

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

1
2
 
if(str.equals("CLick-me"))

il faut un "l" minuscule.

[Bardack]

Pour + de propreté, dans ta methode actionPerformed, ne passe pas par :

Code :

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

1
2
3
4
5
 
String str=ae.getActionCommand();
if(str.equals("CLick-me")) {
    // Le code que tu veux faire
}

mais bien par :

Code :

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

1
2
3
4
5
6
7
 
    if (ae.getSource() == click) {
        // Le code associé au bouton click
    }
     if (ae.getSource() == press) {
        // Le code associé au bouton press
    }

Qui est tout de même plus beau.


Enfin pour ma part je ne bosse pas comme ca, pour + de clareté dans mon code je bosse de cette facon :

Code :

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

1
2
3
4
5
6
7
 
click = new JButton ("Click me");
click.addActionListener (new ActionListener() {
      public void actionPerformed (ActionEvent ae) {
             // Code associé au bouton click
      }
});

Voilà, si ce n est pas clair n'hesite pas a demander

[schniouf]

Exact,

1. Remplace "CLick-me" par "Click-me"
2. Enlève les ";" en fin de ligne des "if"

Pour Bardack : Je suis OK pour le e.getSource() == btn, mais attention à ta dernière solution : c'est bon pour les petits codes, quand tu n'as que quelques instructions ! Dans le cas de gros traitements, rien ne vaux un ActionListener, ou encore mieux une classe héritant de AbstractAction

++

[Bardack]

Tout à fait, c est bien de le préciser. Mais son code étant simple, cela me semblait approprié