編集履歴

回答編集履歴

enum Operation 定義名変更

2022/06/19 16:23

投稿

スコア12648

test CHANGED Viewed

@@ -109,8 +109,8 @@
 }
 enum Operation {
-  PLUS("+") { BigDecimal execute(BigDecimal a, BigDecimal b) { return a.add(b); } },
+  ADD("+") { BigDecimal execute(BigDecimal a, BigDecimal b) { return a.add(b); } },
-  MINUS("-") { BigDecimal execute(BigDecimal a, BigDecimal b) { return a.subtract(b); } },
+  SUB("-") { BigDecimal execute(BigDecimal a, BigDecimal b) { return a.subtract(b); } },
   MUL("*") { BigDecimal execute(BigDecimal a, BigDecimal b) { return a.multiply(b); } },
   DIV("/") { BigDecimal execute(BigDecimal a, BigDecimal b) { return a.divide(b); } }; //ゼロ割 無視中
@@ -164,12 +164,12 @@
     addButton("1", concatListener);
     addButton("2", concatListener);
     addButton("3", concatListener);
-    addButton(Operation.MINUS, calculator);
+    addButton(Operation.SUB, calculator);
     addButton("0", concatListener);
     addButton(".", concatListener);
     addButton("=", e -> calculator.calculation());
-    addButton(Operation.PLUS, calculator);
+    addButton(Operation.ADD, calculator);
   }
   private void addButton(String text, ActionListener listener) {

修正

2022/06/19 09:54

投稿

jimbe

スコア12648

test CHANGED Viewed

@@ -8,7 +8,7 @@
 >３.　演算子や小数点を連続で押せないように（押しても反応しないように？）したい。
 物理的(?)には、最後に押したボタンが何か(演算子かそれ以外か)を保存し、演算子を押した時にその直前に押したボタンも演算子だったら、今押されたボタンは無視する…ようにする感じでしょうか。
-ただ、演算子の連続入力は、演算子の押し間違いの修正の機能として活用できますので、修正の方向を検討するに値するかもしれません。
+ただ、演算子の連続入力は、演算子の押し間違いの修正の機能として活用できますので、修正の方向を検討する余地があるかもしれません。
 小数点は、現在の表示に既に小数点があったら無視するのが簡単に思います。
 >４．計算結果が〇.0の時に小数点を表示させないようにしたい。

追加

2022/06/19 09:51

投稿

jimbe

スコア12648

test CHANGED Viewed

@@ -20,6 +20,11 @@
 Swing にしたり BigDecimal にしたり enum とかしていたら大分変わってしまいました。
 役に立たないかも知れませんが、とりあえず動作を見るような参考程度ということで。
+確認の為、ディスプレイ右上に実行する演算子を表示するようにしています。
+演算子の連続入力時は最後の演算子を実行するようにしました。
+1+2+3+・・・= と連続出来ます。また、1+2= で 3 の表示後に +4= と入力で 7 と計算するようにしています。
+(簡単に確認しているだけですので、バグはあるかもしれません。)
 Calc.java
 ```java

コード追加

2022/06/19 09:27

投稿

jimbe

スコア12648

test CHANGED Viewed

@@ -15,3 +15,168 @@
 double でやるならば、文字列化して小数があったら右から一文字ずつ"0"以外が出るまで"0"を消すというような文字レベルの操作をすることになりそうです。
  BigDecimal を用いるなら stripTrailingZeros() というメソッドがやってくれます。
+---
+Swing にしたり BigDecimal にしたり enum とかしていたら大分変わってしまいました。
+役に立たないかも知れませんが、とりあえず動作を見るような参考程度ということで。
+Calc.java
+```java
+import java.awt.*;
+import java.awt.event.ActionListener;
+import java.math.BigDecimal;
+import javax.swing.*;
+public class Calc extends JFrame {
+  public static void main(String[] args) {
+    SwingUtilities.invokeLater(() -> new Calc().setVisible(true));
+  }
+  Calc() {
+    super("Calc");
+    setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
+    DisplayPanel display = new DisplayPanel();
+    add(display, BorderLayout.NORTH);
+    Calculator calculator = new Calculator(display);
+    KeyPanel keyPanel = new KeyPanel(display, calculator);
+    add(keyPanel, BorderLayout.CENTER);
+    pack();
+    setLocationRelativeTo(null);
+  }
+}
+class DisplayPanel extends JPanel {
+  private JLabel operationIndicator;
+  private JLabel mainDisplay;
+  private String input = "";
+  private Operation op;
+  DisplayPanel() {
+    super(new BorderLayout());
+    operationIndicator = new JLabel(" ");
+    operationIndicator.setHorizontalAlignment(SwingConstants.RIGHT);
+    operationIndicator.setFont(new Font("Courier", Font.BOLD, 10));
+    add(operationIndicator, BorderLayout.NORTH);
+    mainDisplay = new JLabel("0");
+    mainDisplay.setHorizontalAlignment(SwingConstants.RIGHT);
+    mainDisplay.setPreferredSize(new Dimension(50, 30));
+    mainDisplay.setFont(new Font("Courier", Font.BOLD, 15));
+    add(mainDisplay, BorderLayout.CENTER);
+  }
+  void setText(String value) {
+    mainDisplay.setText(value);
+    input = "";
+  }
+  void concat(String c) {
+    if(c.equals(".")) {
+      if(input.contains(".")) return; //多重小数点 ガード
+      if(input.isEmpty()) input = "0"; //"." だけの入力は "0." とする
+    } else if(input.equals("0")) {
+      input = ""; //ZERO ガード
+    }
+    input += c;
+    mainDisplay.setText(input);
+  }
+  String getText() {
+    return mainDisplay.getText();
+  }
+  void setOperation(Operation op) {
+    this.op = op;
+    operationIndicator.setText(op == null ? " " : op.text);
+    input = "";
+  }
+  Operation getOperation() {
+    return op;
+  }
+}
+enum Operation {
+  PLUS("+") { BigDecimal execute(BigDecimal a, BigDecimal b) { return a.add(b); } },
+  MINUS("-") { BigDecimal execute(BigDecimal a, BigDecimal b) { return a.subtract(b); } },
+  MUL("*") { BigDecimal execute(BigDecimal a, BigDecimal b) { return a.multiply(b); } },
+  DIV("/") { BigDecimal execute(BigDecimal a, BigDecimal b) { return a.divide(b); } }; //ゼロ割 無視中
+  final String text;
+  Operation(String text) {
+    this.text = text;
+  }
+  abstract BigDecimal execute(BigDecimal a, BigDecimal b);
+}
+class Calculator {
+  private DisplayPanel display;
+  private BigDecimal a;
+  Calculator(DisplayPanel display) {
+    this.display = display;
+  }
+  void setOperation(Operation operation) {
+    if(display.getOperation() != null) calculation(); //演算子の優先順位は無視中
+    a = new BigDecimal(display.getText());
+    display.setOperation(operation);
+  }
+  void calculation() {
+    BigDecimal b = new BigDecimal(display.getText());
+    Operation op = display.getOperation();
+    display.setText((op == null ? b : op.execute(a, b)).stripTrailingZeros().toPlainString()); //余計な "0" は削除
+    display.setOperation(null);
+  }
+}
+class KeyPanel extends JPanel {
+  KeyPanel(DisplayPanel display, Calculator calculator) {
+    super(new GridLayout(4, 4));
+    setFont(new Font("Courier", Font.BOLD, 20));
+    ActionListener concatListener = e -> display.concat(e.getActionCommand());
+    addButton("7", concatListener);
+    addButton("8", concatListener);
+    addButton("9", concatListener);
+    addButton(Operation.DIV, calculator);
+    addButton("4", concatListener);
+    addButton("5", concatListener);
+    addButton("6", concatListener);
+    addButton(Operation.MUL, calculator);
+    addButton("1", concatListener);
+    addButton("2", concatListener);
+    addButton("3", concatListener);
+    addButton(Operation.MINUS, calculator);
+    addButton("0", concatListener);
+    addButton(".", concatListener);
+    addButton("=", e -> calculator.calculation());
+    addButton(Operation.PLUS, calculator);
+  }
+  private void addButton(String text, ActionListener listener) {
+    JButton b = new JButton(text);
+    b.addActionListener(listener);
+    add(b);
+  }
+  private void addButton(Operation op, Calculator calculator) {
+    JButton b = new JButton(op.text);
+    b.addActionListener(e -> calculator.setOperation(op));
+    add(b);
+  }
+}
+```

追加

2022/06/19 08:14

投稿

jimbe

スコア12648

test CHANGED Viewed

@@ -8,6 +8,7 @@
 >３.　演算子や小数点を連続で押せないように（押しても反応しないように？）したい。
 物理的(?)には、最後に押したボタンが何か(演算子かそれ以外か)を保存し、演算子を押した時にその直前に押したボタンも演算子だったら、今押されたボタンは無視する…ようにする感じでしょうか。
+ただ、演算子の連続入力は、演算子の押し間違いの修正の機能として活用できますので、修正の方向を検討するに値するかもしれません。
 小数点は、現在の表示に既に小数点があったら無視するのが簡単に思います。
 >４．計算結果が〇.0の時に小数点を表示させないようにしたい。

追記

2022/06/19 07:46

投稿

jimbe

スコア12648

test CHANGED Viewed

@@ -1,3 +1,5 @@
+※ご提示のコードの動作を完全には把握しておりません。
 >１．最初に表示される０を、数字を入力すると消えるようにしたい。
 >２．最初に０を続けて入力出来ないようにしたい。

追加

2022/06/19 06:18

投稿

jimbe

スコア12648

test CHANGED Viewed

@@ -6,6 +6,7 @@
 >３.　演算子や小数点を連続で押せないように（押しても反応しないように？）したい。
 物理的(?)には、最後に押したボタンが何か(演算子かそれ以外か)を保存し、演算子を押した時にその直前に押したボタンも演算子だったら、今押されたボタンは無視する…ようにする感じでしょうか。
+小数点は、現在の表示に既に小数点があったら無視するのが簡単に思います。
 >４．計算結果が〇.0の時に小数点を表示させないようにしたい。