java calculation

Question

import java.awt.*;
import java.awt.event.*;
import javax.swing.JTextField;

Comments

[jimbe]

とりあえず以下はお読みいただいたほうが宜しいかと思います。
https://teratail.com/help/question-tips#questionTips31

[退会済みユーザー]

ありがとうございます！
しっかりありがちな失敗していましたね。
タイトル変更します！

[jimbe]

変更ありがとうございます。
さらにご質問と関係無いことなのですが…AWT と Swing のコンポーネントを混ぜて使うのは不具合の原因になります。( JTexrField は Swing です。)
Swing も古いですが AWT はもっと古いですので、どちらかを使うなら Swing のコンポーネントで統一されたほう良いように思います。

[退会済みユーザー]

ありがとうございます。

[Zuishin]

意図的に内容が抹消された質問として低評価しました。
そのようにするのがこのサイトの仕組みです。

Answer 1

※ご提示のコードの動作を完全には把握しておりません。

１．最初に表示される０を、数字を入力すると消えるようにしたい。
２．最初に０を続けて入力出来ないようにしたい。

数字を concat している個所で、現在の表示が "0" だったら追加せず置き換える(もしくは消して追加する)ようにすれば良いように思います。

３.　演算子や小数点を連続で押せないように（押しても反応しないように？）したい。

物理的(?)には、最後に押したボタンが何か(演算子かそれ以外か)を保存し、演算子を押した時にその直前に押したボタンも演算子だったら、今押されたボタンは無視する…ようにする感じでしょうか。
ただ、演算子の連続入力は、演算子の押し間違いの修正の機能として活用できますので、修正の方向を検討する余地があるかもしれません。
小数点は、現在の表示に既に小数点があったら無視するのが簡単に思います。

４．計算結果が〇.0の時に小数点を表示させないようにしたい。

double でやるならば、文字列化して小数があったら右から一文字ずつ"0"以外が出るまで"0"を消すというような文字レベルの操作をすることになりそうです。
BigDecimal を用いるなら stripTrailingZeros() というメソッドがやってくれます。

---

Swing にしたり BigDecimal にしたり enum とかしていたら大分変わってしまいました。
役に立たないかも知れませんが、とりあえず動作を見るような参考程度ということで。

確認の為、ディスプレイ右上に実行する演算子を表示するようにしています。
演算子の連続入力時は最後の演算子を実行するようにしました。
1+2+3+・・・= と連続出来ます。また、1+2= で 3 の表示後に +4= と入力で 7 と計算するようにしています。
(簡単に確認しているだけですので、バグはあるかもしれません。)

Calc.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 {
  ADD("+") { BigDecimal execute(BigDecimal a, BigDecimal b) { return a.add(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); } }; //ゼロ割 無視中

  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.SUB, calculator);

    addButton("0", concatListener);
    addButton(".", concatListener);
    addButton("=", e -> calculator.calculation());
    addButton(Operation.ADD, 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);
  }
}

Comments

[退会済みユーザー]

ありがとうございます。難しいですね。
今丁度連続計算出来なくて困ってたところです。

[jimbe]

画面への表示と操作のつじつまを合わせつつどのようなデータをいつどこに設定するか・クリアするか・利用するかを既存の電卓の動作から考えるのが大変ですね。
テキトウにやると if だらけフラグだらけになりそうです。

回答のコードの場合、キー入力は DisplayPanel.input に溜めて表示する一方、計算に使う値は DisplayPanel.mainDisplay から得ているのがミソかなと思います。

Answer 2

一部だけ。

１．最初に表示される０を、数字を入力すると消えるようにしたい。
２．最初に０を続けて入力出来ないようにしたい。

@@ -6,6 +6,7 @@
 import java.awt.event.ActionListener;
 import java.awt.event.WindowAdapter;
 import java.awt.event.WindowEvent;
+import java.math.BigDecimal;
 
 class Key extends Panel {
   /**
@@ -59,30 +60,34 @@
   JTextField t;
   Key key;
   int il;
+  
+  private void appendtText(final String text) {
+    t.setText(new BigDecimal(t.getText().concat(text)).toString());
+  }
 
   public void actionPerformed(ActionEvent e) {
     if (e.getActionCommand() == "0") {
-      t.setText(t.getText().concat("0"));
+      appendtText("0");
     } else if (e.getActionCommand() == "1") {
-      t.setText(t.getText().concat("1"));
+      appendtText("1");
     } else if (e.getActionCommand() == "2") {
-      t.setText(t.getText().concat("2"));
+      appendtText("2");
     } else if (e.getActionCommand() == "3") {
-      t.setText(t.getText().concat("3"));
+      appendtText("3");
     } else if (e.getActionCommand() == "4") {
-      t.setText(t.getText().concat("4"));
+      appendtText("4");
     } else if (e.getActionCommand() == "5") {
-      t.setText(t.getText().concat("5"));
+      appendtText("5");
     } else if (e.getActionCommand() == "6") {
-      t.setText(t.getText().concat("6"));
+      appendtText("6");
     } else if (e.getActionCommand() == "7") {
-      t.setText(t.getText().concat("7"));
+      appendtText("7");
     } else if (e.getActionCommand() == "8") {
-      t.setText(t.getText().concat("8"));
+      appendtText("8");
     } else if (e.getActionCommand() == "9") {
-      t.setText(t.getText().concat("9"));
+      appendtText("9");
     } else if (e.getActionCommand() == ".") {
-      t.setText(t.getText().concat("."));
+      appendtText(".");
     }
 
     if (e.getActionCommand() == "+") {
@@ -132,6 +137,7 @@
     t.setHorizontalAlignment(JTextField.RIGHT);
     t.setPreferredSize(new Dimension(50, 30));
     t.setFont(new Font("Courier", Font.BOLD, 15));
+    t.setEditable(false);
 
     add(t, "North");
     add(key, "Center");

Answer 3

１．最初に表示される０を、数字を入力すると消えるようにしたい。
２．最初に０を続けて入力出来ないようにしたい。

数値は文字列で持つのではなく、整数で持つようにすればこの心配はなくなります
０をいくら続けて入力しても０にしかなりません

Comments

[Zuishin]

0. と押した場合、数値では 0 と区別がつけられません。
一つの問題が改善するとともに、別の問題が発生することを考慮に入れていない机上の空論です。