Problème Action Listener

Version imprimable

Bonjour à tous,

Je suis en pleine incompréhension. Je voudrais gérer les évènements d'une série de boutons de calculatrice (1, 2, 3) et écrire leur valeur, lorsqu'on clique dessus, dans un JPanel. Mais étonnamment, on ne rentre jamais dans la boucle if(s.charAt(0) == nombres[i]), même si manifestement, et après avoir intégré des System.out.println pour le vérifier, les valeurs coïncident toujours une fois. Le JPanel reste toujours désespérément vide, il n'affiche rien.

Pourriez-vous m'aider ?

Merci du temps que vous consacrerez à ce message :)
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 class ALG1 implements ActionListener { public void actionPerformed(ActionEvent event) { String s = ((JButton) event.getSource()).getText(); int[] nombres = {1, 2, 3, 4, 5, 6, 7, 8, 9, 0}; for(int i = 0; i < nombres.length; i++) { if(s.charAt(0) == nombres[i]) { if(vt == ".") { na = na + nombres[i]*Math.pow(10, compteur); compteur--; } else { na = na*10 + nombres[i]; System.out.println(na); compteur = -1; } écriture = String.valueOf(na); tab.repaint(); } } } }

30/06/2014, 00h53
thelvin
Citation:

Envoyé par coqalcoolique

Mais étonnamment, on ne rentre jamais dans la boucle if(s.charAt(0) == nombres[i]), même si manifestement, et après avoir intégré des System.out.println pour le vérifier, les valeurs coïncident toujours une fois.

Tu compares un char et un int.
Qu'est-ce que ça donne si tu fais :
Code:

1 2 3 4 int i = 'a'; System.out.println(i); char c = 64; System.out.println(c);
?

Explications : il n'y a rien à voir entre le nombre 0 et le caractère 'chiffre 0'. Notamment parce qu'un nombre, à quelques exceptions près, quand on l'écrit ça fait plus d'un caractère. Et parce que le nombre 'a' n'existe pas, par contre le caractère 'a' existe, et on en a besoin.
La comparaison char == int est autorisée, parce qu'en Java les char sont bel et bien considérés comme des nombres... Mais le char '0' n'est pas considéré comme étant le nombre 0.
La valeur nombre d'un char, est en fait le code Unicode de ce char.

Solution : comparer des poires avec des poires, ou des pommes avec des pommes.
Possibilité 1 : char[] nombres = {'1', '2', '3', '4', '5', '6', '7', '8', '9', '0'};.
Possibilité 2 : int nombreDeLaTouche = Integer.parseInt(((JButton) event.getSource()).getText());.

Vraie solution :
Ça sert à rien de boucler sur les dix chiffres pour savoir lequel est le bon. Le bon c'est celui obtenu avec parseInt() -_-°.

Merci beaucoup !

Merci beaucoup thelvin,

Maintenant mon code est plus beau, plus claire, et ça marche beaucoup mieux. Je joins mon code pour ceux qui rencontreraient un problème équivalent :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
class ALG1 implements ActionListener
	{
		public void actionPerformed(ActionEvent event)
		{
			String s = ((JButton) event.getSource()).getText();
			int nombre = Integer.parseInt(s);
 
			if(vt == ".")
			{
				na = na + nombre*Math.pow(10, compteur);
				compteur--;
			}
			else
			{
				na = na*10 + nombre;
				System.out.println(na);
				compteur = -1;
			}
			tab.setText(String.valueOf(na));
			tab.repaint();
 
		}	
	}