Code de calculatrice

**linconnue55** · 19/06/2011, 12h15

salut,

j'essaye de programmer une calculatrice et je me suis planté au niveau de la méthode ActionListener ( lors de +, -, * et /).
voilà mon code -->

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
import javax.swing.*;
 
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
public class calcul extends JFrame implements ActionListener{
 
 
	String[] tabCar = {"1", "2", "3", "4", "5", "6", "7", "8", "9", "0", ".", "=", "C", "+", "-", "*", "/"};
    JButton[] tabBut = new JButton[tabCar.length];
	private JPanel panOrg = new JPanel();
	private JLabel ecran = new JLabel("0");
 
	private Dimension dimChf = new Dimension(50, 40);
    private Dimension dimOpr = new Dimension(50, 30);
	public String caractere="";
	public double chiffre1;
	public double chiffre;
 
	public calcul(){
 
		this.setAlwaysOnTop(true);
		this.setSize(240, 250);
		this.setLocationRelativeTo(null);
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.setResizable(false);
 
		JPanel chiffre = new JPanel();
		chiffre.setPreferredSize(new Dimension(165, 225));
 
		JPanel operateur = new JPanel();        
        operateur.setPreferredSize(new Dimension(55, 225));
 
        JPanel panEcran = new JPanel();
        panEcran.setPreferredSize(new Dimension(220, 30));
 
        ecran.setHorizontalAlignment(JLabel.RIGHT);
        ecran.setPreferredSize(new Dimension(220, 20));
 
 
		for (int i=0;i<tabCar.length;i++){
 
        	tabBut[i]=new JButton(tabCar[i]);
        	tabBut[i].setPreferredSize(dimChf);
 
        	switch(i){
 
        	case 11:
        		tabBut[i].addActionListener(this);
        		chiffre.add(tabBut[i]);
        		break;
 
        	case 12:
        		tabBut[i].addActionListener(this);
        		tabBut[i].setPreferredSize(dimOpr);
        		operateur.add(tabBut[i]);
        		break;
 
        	case 13:
        		tabBut[i].addActionListener(this);
        		tabBut[i].setPreferredSize(dimOpr);
        		operateur.add(tabBut[i]);
        		break;
 
        	case 14:
        		tabBut[i].addActionListener(this);
        		tabBut[i].setPreferredSize(dimOpr);
        		operateur.add(tabBut[i]);
        		break;
 
        	case 15:
        		tabBut[i].addActionListener(this);
        		tabBut[i].setPreferredSize(dimOpr);
        		operateur.add(tabBut[i]);
        		break;
 
        	case 16:
        		tabBut[i].addActionListener(this);
        		tabBut[i].setPreferredSize(dimOpr);
        		operateur.add(tabBut[i]);
        		break;
 
        	default :
        		tabBut[i].addActionListener(this);
        		tabBut[i].setPreferredSize(dimChf);
        		chiffre.add(tabBut[i]);	
        		break;
        	}
        }
 
        panEcran.add(ecran);
        panEcran.setBorder(BorderFactory.createLineBorder(Color.black));
        panOrg.add(panEcran, BorderLayout.NORTH);
        panOrg.add(chiffre, BorderLayout.CENTER);
        panOrg.add(operateur, BorderLayout.EAST);
 
        this.setContentPane(panOrg);
		this.setVisible(true);
 
    }
 
	private void operation(){
		if (caractere.equals("+")){
 
			chiffre1 = chiffre1 + Double.valueOf(ecran.getText()).doubleValue();
			ecran.setText(String.valueOf(chiffre1));
		}
 
		else if (caractere.equals("-")){
			chiffre1 = chiffre1 - Double.valueOf(ecran.getText()).doubleValue();
			ecran.setText(String.valueOf(chiffre1));
		}
 
		else if (caractere.equals("*")){
			chiffre1 = chiffre1 * Double.valueOf(ecran.getText()).doubleValue();
			ecran.setText(String.valueOf(chiffre1));
		}
 
		else if (caractere.equals("/")){
			try{
				chiffre1 = chiffre1 / Double.valueOf(ecran.getText()).doubleValue();
				ecran.setText(String.valueOf(chiffre1));
			}catch(ArithmeticException e){
				ecran.setText("Division impossible");
			}
 
		}
 
 
	}
	public void actionPerformed(ActionEvent arg0) {
		if(arg0.getSource()==tabBut[12]){
			/* reset */
			ecran.setText("0");
		}
		else if(arg0.getSource()==tabBut[13]){
			chiffre = Double.valueOf(ecran.getText()).doubleValue();
			caractere="+";
			operation();
 
		}
 
		else if(arg0.getSource()==tabBut[14]){
			caractere="-";
 
		}
 
		else if(arg0.getSource()==tabBut[15]){
			caractere="*";
 
		}
 
		else if(arg0.getSource()==tabBut[16]){
			caractere="/";
 
		}
		else if(arg0.getSource()==tabBut[11]){
			ecran.setText(String.valueOf(chiffre1));
		}
		else{
			String str =((JButton)arg0.getSource()).getText();
			if (ecran.getText()=="0"){
				ecran.setText(null);
			}
 
			else{
				str=ecran.getText()+str;
			}
			ecran.setText(str);
		}
	}
 
}

mon problème c'est que lorsque je tape un nombre puis j’appuie sur par exemple + et ensuite je dois entrer le deuxième nombre à additionner. Là, il s'affiche concaténer avec le précédent. En plus, les résultats sont incompréhensibles

SVP, j'ai besoin d'une idée

Merci

**Jimmy_** · 20/06/2011, 10h08

Bonjour,

C'est ta séquence qui n'est pas bonne,

Si tu as une opération renseigné, il faut effacer l’écran avant d'afficher la nouvelle valeur.

Ta méthode actionPerformed doit gérer un mini-automate d'état, la elle est simplement sans état. Tu dois savoir ce qui s'est passé avant.

Sinon au niveau des best practices en java, les noms d'objet commence par une majuscule (Calcul et non calcul) , et on n'importe que ce dont on a besoin : pas de import java.swing.* ça n'a pas de sens.

Code de calculatrice

Java

Discussions similaires

Partager

Partager