boolean à false et ensuite à true

**s2a07** · 07/05/2018, 09h55

Bonjour,
je suis bloqué. Je ne comprends pas ce code. Pourquoi, pour créer une calculatrice, dans le egallistner, le booleen disant si on a cliqué doit passer à false et dans PlusListener il passe à true ?
merci d'avance.

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
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
 
private void calcul(){
		if(operateur.equals("+")){
			chiffre1 = chiffre1 + 
					Double.valueOf(ecran.getText()).doubleValue();
			ecran.setText(String.valueOf(chiffre1));
		}
		if(operateur.equals("-")){
			chiffre1 = chiffre1 - 
					Double.valueOf(ecran.getText()).doubleValue();
			ecran.setText(String.valueOf(chiffre1));
		}          
		if(operateur.equals("*")){
			chiffre1 = chiffre1 * 
					Double.valueOf(ecran.getText()).doubleValue();
			ecran.setText(String.valueOf(chiffre1));
		}     
		if(operateur.equals("/")){
			try{
				chiffre1 = chiffre1 / 
						Double.valueOf(ecran.getText()).doubleValue();
				ecran.setText(String.valueOf(chiffre1));
			} catch(ArithmeticException e) {
				ecran.setText("0");
			}
		}
	}
 
	//Listener utilisé pour les chiffres
	//Permet de stocker les chiffres et de les afficher
	class ChiffreListener implements ActionListener {
		public void actionPerformed(ActionEvent e){
			//On affiche le chiffre additionnel dans le label
			String str = ((JButton)e.getSource()).getText();
			if(update){
				update = false;
			}
			else{
				if(!ecran.getText().equals("0"))
					str = ecran.getText() + str;
			}
			ecran.setText(str);
		}
	}
 
	//Listener affecté au bouton =
	class EgalListener implements ActionListener {
		public void actionPerformed(ActionEvent arg0){
			calcul();
			update = true;
			clicOperateur = false;
		}
	}
 
	//Listener affecté au bouton +
	class PlusListener implements ActionListener {
		public void actionPerformed(ActionEvent arg0){
			if(clicOperateur){
				calcul();
				ecran.setText(String.valueOf(chiffre1));
			}
			else{
				chiffre1 = Double.valueOf(ecran.getText()).doubleValue();
				clicOperateur = true;
			}
			operateur = "+";
			update = true;
		}
	}

**johnlobs** · 07/05/2018, 11h45

Bonjour,

Ton code n'est malheureusement pas complet pour qu'on puisse te donner une réponse très précise car on ne sait pas quand est ce que tes listeners sont sollicités.
Bon ,vu que c'est une calculatrice ,je suppose que tes listeners s'exectutent au click sur les bouton plus(Pluslistener) et egal(Egallistener).
Le comportement (vu le code) est correct car le plus est sollicité avant le bouton egal donc le plusListener sera appelé avant;dans Pluslistener,il y a une clause (if/else) puis en sortant du listener,le Boolean update est setté a true.
pourquoi?
Pour faire la différence entre le premier chiffre et le deuxième chiffre entré: la deuxième fois que l'utilisateur entre un chiffre ,le Boolean est a true donc dans la fonction chiffrelistener il n'y a pas le même traitement.

Le PlusListener sera le même que le MoinsListener etc il vaut donc mieux implementer un OperateurListener qui fera le travail pour tous les operateurs en utilisant un switch() et affecter l'operateur qu'il faut à Operateur.

Ce qui me parait par contre bizar c'est pourquoi mettre update à true apres le EgalListener ??
il doit y avoir une fonction qui reinitialise l'ecran a 0 et update à false pour un autre calcul... à part si la calculatrice autorise un autre calcul apres le egal et une reinitialisation a l'appuie du bouton "C"

De toutes les facons tout le code est à revoir et à optimiser...

Pour le Boolean operateur ,il passe a false dans le EgalListener et il passe a true au click sur un operateur;ce qui est normal...

Il faut essayer d'etre plus clair dans ta question car on ne sait pas de quel boolean tu parles.Bref...

Bon courage...

**s2a07** · 07/05/2018, 14h31

merci beaucoup c'est un code qui m'as traumatise ou il y a aucune explication de la conception de l'algorithme
cordialement

**johnlobs** · 07/05/2018, 15h52

Tu as eu les explications que tu cherchais ou bien il y a encore des questionnements.
Si c'est le cas tu peux le passer a resolu!!

merci à toi...
Bon courage

**s2a07** · 09/05/2018, 10h06

bonjour juste une dèrniere question dans quel cas le clicOperateurdu du PlusListener est a true ?
merci d'avance

**s2a07** · 09/05/2018, 10h43

en fait je n'arrive pas a comprendre quand es ce que l'on rentre dans le if de ce code help me

Code :

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

1
2
3
4
5
6
class MoinsListener implements ActionListener {
		public void actionPerformed(ActionEvent arg0){
			if(clicOperateur){
				calcul();
				ecran.setText(String.valueOf(chiffre1));
			}

merci d'avance

**Robin56** · 09/05/2018, 10h56

Même réponse que johnlobs en début de discussion, il manque trop de code pour pouvoir répondre correctement à ta demande. On ne sait pas où la variable clicOperateur est déclarée, initialisée, manipulée, etc. Il peut être intéressant de ne pas montrer l’entièreté du code pour faciliter la lecture. Cependant, dans ton cas, il est préférable d'en montrer bien plus et cela se voit aisément..

Envoyé par s2a07

en fait je n'arrive pas a comprendre quand es ce que l'on rentre dans le if de ce code help me

En l'état, la réponse courte a cette question est : lorsque clicOperateur est à true.

**s2a07** · 09/05/2018, 11h11

voici le code en fait je pense avoir compris c'est a la deuxieme fois que l'on appui sur le bouton plus que l'on rentre dans le if
merci pour la confirmation

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
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
package calculatrice;
 
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Font;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener; 
import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
 
public class Calculatrice extends JFrame {
	private JPanel container = new JPanel();
	//Tableau stockant les éléments à afficher dans la calculatrice
	String[] tab_string = {"1", "2", "3", "4", "5", "6", "7", "8", "9", "0", ".", "=", "C", "+", "-", "*", "/"};
	//Un bouton par élément à afficher
	JButton[] tab_button = new JButton[tab_string.length];
	private JLabel ecran = new JLabel();
	private Dimension dim = new Dimension(50, 40);
	private Dimension dim2 = new Dimension(50, 31);
	private double chiffre1;
	private boolean clicOperateur = false, update = false;
	private String operateur = "";
 
	public Calculatrice(){
		this.setSize(240, 260);
		this.setTitle("Calculette");
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.setLocationRelativeTo(null);
		this.setResizable(false);
		//On initialise le conteneur avec tous les composants
		initComposant();
		//On ajoute le conteneur
		this.setContentPane(container);
		this.setVisible(true);
	}
 
	private void initComposant(){
		//On définit la police d'écriture à utiliser
		Font police = new Font("Arial", Font.BOLD, 20);
		ecran = new JLabel("0");
		ecran.setFont(police);
		//On aligne les informations à droite dans le JLabel
		ecran.setHorizontalAlignment(JLabel.RIGHT);
		ecran.setPreferredSize(new Dimension(220, 20));
		JPanel operateur = new JPanel();      
		operateur.setPreferredSize(new Dimension(55, 225));
		JPanel chiffre = new JPanel();
		chiffre.setPreferredSize(new Dimension(165, 225));
		JPanel panEcran = new JPanel();
		panEcran.setPreferredSize(new Dimension(220, 30));
 
		//On parcourt le tableau initialisé
		//afin de créer nos boutons
		for(int i = 0; i < tab_string.length; i++){
			tab_button[i] = new JButton(tab_string[i]);
			tab_button[i].setPreferredSize(dim);
			switch(i){
			//Pour chaque élément situé à la fin du tableau
			//et qui n'est pas un chiffre
			//on définit le comportement à avoir grâce à un listener
			case 11 :
				tab_button[i].addActionListener(new EgalListener());
				chiffre.add(tab_button[i]);
				break;
			case 12 :
				tab_button[i].setForeground(Color.red);
				tab_button[i].addActionListener(new ResetListener());
				operateur.add(tab_button[i]);
				break;
			case 13 :
				tab_button[i].addActionListener(new PlusListener());
				tab_button[i].setPreferredSize(dim2);
				operateur.add(tab_button[i]);
				break;
			case 14 :
				tab_button[i].addActionListener(new MoinsListener());
				tab_button[i].setPreferredSize(dim2);
				operateur.add(tab_button[i]);
				break;	
			case 15 :	
				tab_button[i].addActionListener(new MultiListener());
				tab_button[i].setPreferredSize(dim2);
				operateur.add(tab_button[i]);
				break;
			case 16 :
				tab_button[i].addActionListener(new DivListener());
				tab_button[i].setPreferredSize(dim2);
				operateur.add(tab_button[i]);
				break;
			default :
				//Par défaut, ce sont les premiers éléments du tableau
				//donc des chiffres, on affecte alors le bon listener
				chiffre.add(tab_button[i]);
				tab_button[i].addActionListener(new ChiffreListener());
				break;
			}
		}
		panEcran.add(ecran);
		panEcran.setBorder(BorderFactory.createLineBorder(Color.black));
		container.add(panEcran, BorderLayout.NORTH);
		container.add(chiffre, BorderLayout.CENTER);
		container.add(operateur, BorderLayout.EAST);
	}
 
	//Méthode permettant d'effectuer un calcul selon l'opérateur sélectionné
	private void calcul(){
		if(operateur.equals("+")){
			chiffre1 = chiffre1 + 
					Double.valueOf(ecran.getText()).doubleValue();
			ecran.setText(String.valueOf(chiffre1));
		}
		if(operateur.equals("-")){
			chiffre1 = chiffre1 - 
					Double.valueOf(ecran.getText()).doubleValue();
			ecran.setText(String.valueOf(chiffre1));
		}          
		if(operateur.equals("*")){
			chiffre1 = chiffre1 * 
					Double.valueOf(ecran.getText()).doubleValue();
			ecran.setText(String.valueOf(chiffre1));
		}     
		if(operateur.equals("/")){
			try{
				chiffre1 = chiffre1 / 
						Double.valueOf(ecran.getText()).doubleValue();
				ecran.setText(String.valueOf(chiffre1));
			} catch(ArithmeticException e) {
				ecran.setText("0");
			}
		}
	}
 
	//Listener utilisé pour les chiffres
	//Permet de stocker les chiffres et de les afficher
	class ChiffreListener implements ActionListener {
		public void actionPerformed(ActionEvent e){
			//On affiche le chiffre additionnel dans le label
			String str = ((JButton)e.getSource()).getText();
			if(update){
				update = false;
			}
			else{
				if(!ecran.getText().equals("0"))
					str = ecran.getText() + str;
			}
			ecran.setText(str);
		}
	}
 
	//Listener affecté au bouton =
	class EgalListener implements ActionListener {
		public void actionPerformed(ActionEvent arg0){
			calcul();
			update = true;
			clicOperateur = false;
		}
	}
 
	//Listener affecté au bouton +
	class PlusListener implements ActionListener {
		public void actionPerformed(ActionEvent arg0){
			if(clicOperateur){
				calcul();
				ecran.setText(String.valueOf(chiffre1));
			}
			else{
				chiffre1 = Double.valueOf(ecran.getText()).doubleValue();
				clicOperateur = true;
			}
			operateur = "+";
			update = true;
		}
	}
 
	//Listener affecté au bouton -
	class MoinsListener implements ActionListener {
		public void actionPerformed(ActionEvent arg0){
			if(clicOperateur){
				calcul();
				ecran.setText(String.valueOf(chiffre1));
			}
			else{
				chiffre1 = Double.valueOf(ecran.getText()).doubleValue();
				clicOperateur = true;
			}
			operateur = "-";
			update = true;
		}
	}
 
	//Listener affecté au bouton *
	class MultiListener implements ActionListener {
		public void actionPerformed(ActionEvent arg0){
			if(clicOperateur){
				calcul();
				ecran.setText(String.valueOf(chiffre1));
			}
			else{
				chiffre1 = Double.valueOf(ecran.getText()).doubleValue();
				clicOperateur = true;
			}
			operateur = "*";
			update = true;
		}
	}
 
	//Listener affecté au bouton /
	class DivListener implements ActionListener {
		public void actionPerformed(ActionEvent arg0){
			if(clicOperateur){
				calcul();
				ecran.setText(String.valueOf(chiffre1));
			}
			else{
				chiffre1 = Double.valueOf(ecran.getText()).doubleValue();
				clicOperateur = true;
			}
			operateur = "/";
			update = true;
		}
	}
 
	//Listener affecté au bouton de remise à zéro
	class ResetListener implements ActionListener {
		public void actionPerformed(ActionEvent arg0){
			clicOperateur = false;
			update = true;
			chiffre1 = 0;
			operateur = "";
			ecran.setText("");
		}
	}      
}

**Robin56** · 09/05/2018, 11h23

Envoyé par s2a07

voici le code en fait je pense avoir compris c'est a la deuxieme fois que l'on appui sur le bouton plus que l'on rentre dans le if

Oui c'est cela. Quand on clique la première fois, on passe dans la clause else. Et si le bouton "Reset" n'a pas été sélectionné entre temps, la fois suivante, on entrera dans le if.

**s2a07** · 09/05/2018, 11h49

je tiens vivement a vous remercier car je n'ai jamais eu d'aussi bonne explication
je remercie aussi l'humanite des personnes qui donne de leur temps pour faire evouler les developpeurs comme moi
Avec tout mes respects pour votre travail chapeau et merci

**Robin56** · 09/05/2018, 15h14

Merci, je suppose donc que ton problème est résolu. Si c'est le cas, tu peux clore le sujet en cliquant sur

**johnlobs** · 09/05/2018, 17h27

Bonjour,

Je suis désolé pour le retard de ma réponse mais j’étais en surcharge donc pas de connexion au site...
Je remercie Robin56 d'avoir pris le relais...

bonjour juste une dernière question dans quel cas le clicOperateurdu du PlusListener est a true ?
merci d'avance

A chaque Click sur le bouton PLUS il y a une clause if/else et ensuite clicOperateur est sétté a true.

La réponse à la deuxième question est correcte on accède au if à partir du deuxième appel (sauf en cas de reset).

il serait sympas d'optimiser le code lorsque tu l'auras compris.
Un exemple simple pourquoi écraser la même valeur (true) de clickOperateur si elle est déjà a true....?
Pourquoi un listener par operateur?
Gérer la division par 0 et afficher un message correct et pas une exception console...
etc...

Comme t'as dis Robin56 si c'est OK tu peux passer la discussion à resolue.

Bon courage

boolean à false et ensuite à true

Java

Discussions similaires

Partager

Partager