Discussion :

[smag]

Bonjour concernant le code ci-dessus j'ai les questions suivantes:

1°) Si je tape "2", je vais aller dans InsertAction, afficher "2" et start sera à false.Ensuite si je tape "/" je vais aller dans CommandAction, mettre "/" dans lastCommand et pour moi le résult sera 0, car la comande est result /= 2 et result est intialisé à 0.Pourquoi ai-je donc "2" ( en l'occureence 2.0 suite conversion) qui s'affiche?


2°)Pourquoi la méthode addButton est privée et la méthode calculate est public?

Merci pour vos réponses et bonne journée:

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
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
 
public class Calculator
{
   public static void main(String[] args)
   {  
      CalculatorFrame frame = new CalculatorFrame();
      frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      frame.setVisible(true);
   }
}
 
/**
   A frame with a calculator panel.
*/
class CalculatorFrame extends JFrame
{
   public CalculatorFrame()
   {
      setTitle("Calculator");
      CalculatorPanel panel = new CalculatorPanel();
      add(panel);
      pack();
   }
}
 
/**
   A panel with calculator buttons and a result display.
*/
class CalculatorPanel extends JPanel
{  
   public CalculatorPanel()
   {  
      setLayout(new BorderLayout());
 
      result = 0;
      lastCommand = "=";
      start = true;
 
      // add the display
 
      display = new JButton("0");
      display.setEnabled(false);
      add(display, BorderLayout.NORTH);
 
      ActionListener insert = new InsertAction();
      ActionListener command = new CommandAction();
 
      // add the buttons in a 4 x 4 grid
 
      panel = new JPanel();
      panel.setLayout(new GridLayout(4, 4));
 
      addButton("7", insert);
      addButton("8", insert);
      addButton("9", insert);
      addButton("/", command);
 
      addButton("4", insert);
      addButton("5", insert);
      addButton("6", insert);
      addButton("*", command);
 
      addButton("1", insert);
      addButton("2", insert);
      addButton("3", insert);
      addButton("-", command);
 
      addButton("0", insert);
      addButton(".", insert);
      addButton("=", command);
      addButton("+", command);
 
      add(panel, BorderLayout.CENTER);
   }
 
   /**
      Adds a button to the center panel.
      @param label the button label
      @param listener the button listener
   */
   private void addButton(String label, ActionListener listener)
   {  
      JButton button = new JButton(label);
      button.addActionListener(listener);
      panel.add(button);
   }
 
   /**
      This action inserts the button action string to the
      end of the display text.
   */
   private class InsertAction implements ActionListener
   {
      public void actionPerformed(ActionEvent event)
      {
         String input = event.getActionCommand();
         if (start) 
         {
            display.setText("");
            start = false;
         }
         display.setText(display.getText() + input);
      }
   }
 
   /**
      This action executes the command that the button
      action string denotes.
   */
   private class CommandAction implements ActionListener
   {
      public void actionPerformed(ActionEvent event)
      {  
         String command = event.getActionCommand();
 
         if (start)
         {  
            if (command.equals("-")) 
            { 
               display.setText(command); 
               start = false; 
            }
            else 
               lastCommand = command;
         }
         else
         {  
            calculate(Double.parseDouble(display.getText()));
            lastCommand = command;
            start = true;
         }
      }
   }
 
   /**
      Carries out the pending calculation. 
      @param x the value to be accumulated with the prior result.
   */
   public void calculate(double x)
   {
      if (lastCommand.equals("+")) result += x;
      else if (lastCommand.equals("-")) result -= x;
      else if (lastCommand.equals("*")) result *= x;
      else if (lastCommand.equals("/")) result /= x;
      else if (lastCommand.equals("=")) result = x;
      display.setText("" + result);
   }
 
   private JButton display;
   private JPanel panel;
   private double result;
   private String lastCommand;
   private boolean start;
}

[pierre.zelb]

2°)Pourquoi la méthode addButton est privée et la méthode calculate est public?

addButton n'est utilisée que dans le constructeur pour "creer" la calculatrice, il n'y a donc aucune raison que les autres classes voient cette méthode.

calculate (public) peut etre appelé de l'extérieur et c'est plutot logique, c'est une fonctionnalité "haute" on va dire

[yann2]

Bonjour,

Essaie

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
 
{  
         String command = event.getActionCommand(); 
 
         if (start) 
         {  
            if (command.equals("-")) 
            { 
               display.setText(command); 
               start = false; 
            } 
            else 
               lastCommand = command; 
         } 
         else 
         {  
            lastCommand = command;   // CHANGEMENT !!! (inversion de lignes)
            calculate(Double.parseDouble(display.getText())); 
            start = true; 
         } 
      } 
   }

à la place de

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
 
{  
         String command = event.getActionCommand(); 
 
         if (start) 
         {  
            if (command.equals("-")) 
            { 
               display.setText(command); 
               start = false; 
            } 
            else 
               lastCommand = command; 
         } 
         else 
         { 
 
            calculate(Double.parseDouble(display.getText())); 
            lastCommand = command; 
            start = true; 
         } 
      } 
   }

bon courage.

[smag]

merci