2

そのため、私の課題は、スタック データ構造を使用して加算、減算、乗算、および除算を実行する単純な GUI 電卓を設計することです。電卓ウィンドウには少なくとも 2 つのパネルが必要です。1 つは表示用で、もう 1 つはボタン (0 ～ 9、.、+、-、X、/、=、C) 用です。式が入力されると、その接尾辞または接頭辞が表示され、次に結果が表示されます。

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.util.*;
import java.util.Stack;

public class JCalculator implements ActionListener {
JFrame jfrm;
JFrame jfrm2;
JTextField txt;
JLabel results;
String str = "";
Stack operands = new Stack();
char[] a = new char[0];
int used = 0;



JCalculator(){
    jfrm = new JFrame("JCalc");
    jfrm.getContentPane().setLayout(new GridLayout(0,1));
    jfrm.setSize(210,210);

   results = new JLabel("",SwingConstants.RIGHT);
   jfrm.getContentPane().add(results);
   jfrm.setLocation(400,300);
   jfrm.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

   JButton[] calbut= new JButton[14];
   String []but = {"7","8","9","/","4","5","6","*","1","2","3","-","0","C"
   };
   JButton equal = new JButton("=");
   JButton add = new JButton("+");

   JPanel jbutton = new JPanel();
   jbutton.setLayout(new GridLayout(4,3));
   for(int i=0; i< but.length;i++){
       jbutton.add(calbut[i] = new JButton(but[i]));

       calbut[i].addActionListener(this);
   }
   jbutton.add(equal);
   jbutton.add(add);

   equal.addActionListener(this);
   add.addActionListener(this);

   jfrm.getContentPane().add(jbutton);


    jfrm.setVisible(true);
}
public void actionPerformed(ActionEvent ae){
    char[] a = new char[0];
    int used = 0;
    if(used == a.length){
        char[] newa= new char[a.length + 1];
        for(int i = 0; i <used; i++) newa[i]= a[i];
        a= newa;
    }

    if(ae.getActionCommand().equals("1")){
        str = results.getText();
        results.setText(str + "1");
        operands.push(1);
    }
    if(ae.getActionCommand().equals("2")){
        str = results.getText();
        results.setText(str + "2");
        operands.push(2);
    }
    if(ae.getActionCommand().equals("3")){
        str = results.getText();
        results.setText(str + "3");
        operands.push(3);
    }
    if(ae.getActionCommand().equals("4")){
        str = results.getText();
        results.setText(str + "4");
        operands.push(4);
    }
    if(ae.getActionCommand().equals("5")){
        str = results.getText();
        results.setText(str + "5");
        operands.push(5);
    }
    if(ae.getActionCommand().equals("6")){
        str = results.getText();
        results.setText(str + "6");
        operands.push(6);
    }
    if(ae.getActionCommand().equals("7")){
        str = results.getText();
        results.setText(str + "7");
        operands.push(7);
    }
    if(ae.getActionCommand().equals("8")){
        str = results.getText();
        results.setText(str + "8");
        operands.push(8);
    }
    if(ae.getActionCommand().equals("9")){
        str = results.getText();
        results.setText(str + "9");
        operands.push(9);
    }
    if (ae.getActionCommand().equals("0")) {
        str = results.getText();
        results.setText(str + "0");
        operands.push(0);
    }
    if(ae.getActionCommand().equals("+")){
        str = results.getText();
        double operand = Double.parseDouble(str);
        operands.push(operand);
        results.setText("");
        a[used]= '+';
        used++;

    }
    if(ae.getActionCommand().equals("-")){
        str = results.getText();
        double operand = Double.parseDouble(str);
        operands.push(operand);
        results.setText("");
        a[used]= '+';
        used++;
    }
    if(ae.getActionCommand().equals("/")){
        str = results.getText();
        double operand = Double.parseDouble(str);
        operands.push(operand);
        results.setText("");
        a[used]= '+';
        used++;
    }
    if(ae.getActionCommand().equals("*")){
        str = results.getText();
        double operand = Double.parseDouble(str);
        operands.push(operand);
        results.setText("");
        a[used]= '*';
        used++;
    }
    if(ae.getActionCommand().equals("=")){

        for(int i = 0; i< used; i++){
           performBinaryOp(a[i]);
        }

        str = String.valueOf(operands.pop());
        results.setText(str);
    }


}
 public void performBinaryOp(char nextOperation) {
     double leftOperand, rightOperand;
    Double result = new Double(0);
    rightOperand = (Double) operands.pop();
    leftOperand = (Double) operands.pop();
    switch (nextOperation) {
        case '+':
            result = new Double(leftOperand + rightOperand);
            break;
        case '-':
            result = new Double(leftOperand - rightOperand);
            break;
        case '*':
            result = new Double(leftOperand * rightOperand);
            break;
        case '/':
            result = new Double(leftOperand / rightOperand);
            break;
    }
    operands.push(result);
}

public static void main(String[] args)
{
    SwingUtilities.invokeLater(new Runnable() {
        public void run() {
            new JCalculator();                
        }            
    });        
}
}

私が遭遇するエラーは、等号ボタンを押すときです。たとえば、2 + 3 =を押すと、最初の値2.0が返され、次のようなエラーが表示されます。

Exception in thread "AWT-EventQueue-0" java.util.EmptyStackException
at java.util.Stack.peek(Unknown Source)
at java.util.Stack.pop(Unknown Source)
at JCalculator.actionPerformed(JCalculator.java:152)
at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source)
at java.awt.Component.processMouseEvent(Unknown Source)
at javax.swing.JComponent.processMouseEvent(Unknown Source)
at java.awt.Component.processEvent(Unknown Source)
at java.awt.Container.processEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$000(Unknown Source)
at java.awt.EventQueue$1.run(Unknown Source)
at java.awt.EventQueue$1.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.AccessControlContext$1.doIntersectionPrivilege(Unknown Source)
at java.security.AccessControlContext$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue$2.run(Unknown Source)
at java.awt.EventQueue$2.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.AccessControlContext$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

私の問題は空のスタックがあることだと確信していますが、コードのどこが間違っているのか、それを修正する方法がわからないので、どんな助けでも大歓迎です。

4

3 に答える 3

3

電卓が正しく機能するようになるまで、Swing と UI は脇に置いておくことをお勧めします。テキスト文字列を取り込んで評価できる別の Calculator クラスを作成します。それが機能したら、UI を追加します。

はい、例外が示すように、スタックに問題があります。

at JCalculator.actionPerformed(JCalculator.java:152)

JCalculator をテキスト エディタで開き、行番号表示をオンにして、152 行目に移動します。ここに問題があります。

あなたが投稿したコードを読み通す気はありませんが、デバッガーですばやくスピンすると、すぐに解決するはずです。

于 2012-04-10T01:32:31.933 に答える
1

あなたのactionPerformed()メソッドは、いくつかのクラス変数を隠しています。char [] aandを再宣言したint usedため、コードはほぼ確実に期待どおりに動作していないことに注意してください。私はこれを論理的な結論までたどっていません。

public class JCalculator implements ActionListener {
JFrame jfrm;
JFrame jfrm2;
JTextField txt;
JLabel results;
String str = "";
Stack operands = new Stack();
char[] a = new char[0];
int used = 0;
/* ... */
public void actionPerformed(ActionEvent ae){
    char[] a = new char[0];
    int used = 0;
    if(used == a.length){
        char[] newa= new char[a.length + 1];
        for(int i = 0; i <used; i++) newa[i]= a[i];
        a= newa;
    }

コードのこのセクションでは、2 桁の数字を受け入れていないと思います。

if(ae.getActionCommand().equals("1")){
    str = results.getText();
    results.setText(str + "1");
    operands.push(1);
}

operands.push(1)おそらく、最新の桁だけでなく、作成中の番号をプッシュする必要があります。(もちろん、1 桁の数字のみをサポートするつもりなら、これで問題ないようです。)

ifこれらの 10 個の小さなステートメントを 1 つの関数に置き換えることをお勧めします。ここで 1 つの関数を理解するのに時間をかける価値は十分にあります。これにより、将来このコードのメンテナンスが大幅に簡素化され、スキルが不可欠になります。JButton(初期化を早めに検討することをお勧めします。また、置換関数で配列を使用するようにしてください。最適ではないかもしれませんが、問題ないはずです。)

if(ae.getActionCommand().equals("/")){
    str = results.getText();
    double operand = Double.parseDouble(str);
    operands.push(operand);
    results.setText("");
    a[used]= '+';
    used++;
}

間違った記号a[used]が割り当てられていることに注意してください。変数のシャドウイングの問題を修正すると、これが問題になります。これらの 4 つのブロックを別の 1 つの関数に置き換える方法を考えてみてください。繰り返しますが、電卓に新しい演算子を追加するときにこのような間違いを犯しにくくなり、演算子を適用するコードのバグを修正するのがはるかに簡単になります。 、1 つの場所でバグを修正するだけでよいためです。if

コードを「最小の合理的な」関数に分割することがどれほど有用であるかを強調することはできません。小さな関数は大きな関数よりもはるかに簡単にテストでき、コードは将来的に読みやすくなります。

于 2012-04-10T01:37:43.773 に答える
-1

計算機のヘルプが必要な場合に備えて、これは Java での私の最初のプロジェクトの 1 つです。

https://github.com/orefalo/ExpressionEvaluator

于 2012-04-10T01:36:57.830 に答える