1

きちんとした小さな電卓プログラムを作成することにしましたが、今では、calculate を押して計算スクリプトを実行しても、答えを示すボックスが開きません。問題がわかりません。誰でも教えてもらえますか?

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

public class Gui extends JFrame{

    private JTextField text;
    private JTextField text2;
    private JRadioButton plus;
    private JRadioButton divide;
    private JRadioButton multiply;
    private JRadioButton subtract;
    private ButtonGroup group;
    private JButton button;
    private int str = 1;
    private String txt;
    private String txt2;
    private int num1;
    private int num2;
    private int ans;

    public Gui(){

        super("Calculator");
        setLayout(new FlowLayout());

        text = new JTextField(10);
        text2 = new JTextField(10);
        add(text);
        add(text2);
        plus = new JRadioButton("Add", true);
        divide = new JRadioButton("Divide", false);
        multiply = new JRadioButton("Multiply", false);
        subtract = new JRadioButton("Subtract", false);
        group = new ButtonGroup();
        group.add(plus);
        group.add(divide);
        group.add(multiply);
        group.add(subtract);
        add(plus);
        add(divide);
        add(multiply);
        add(divide);
        add(subtract);
        button = new JButton("Calculate");
        add(button);

        plus.addItemListener(
                new ItemListener(){
                    public void itemStateChanged(ItemEvent event){
                        str = 1;
                        System.out.println(str);
                    }
                }
            );
        divide.addItemListener(
                new ItemListener(){
                    public void itemStateChanged(ItemEvent event){
                        str = 2;
                        System.out.println(str);
                    }
                }
            );
        multiply.addItemListener(
                new ItemListener(){
                    public void itemStateChanged(ItemEvent event){
                        str = 3;
                        System.out.println(str);
                    }
                }
            );
        subtract.addItemListener(
                new ItemListener(){
                    public void itemStateChanged(ItemEvent event){
                        str = 4;
                        System.out.println(str);
                    }
                }
            );
        HandlerClass handler = new HandlerClass();
        button.addActionListener(handler);

    }

    private class HandlerClass implements ActionListener{
        public void ActionPerformed(ActionEvent event){
                txt = text.getText();
                txt2 = text2.getText();
                num1 = Integer.parseInt(txt);
                num2 = Integer.parseInt(txt2);
                if(str==1){
                    ans=num1+num2;
                }
                if(str==2){
                    ans=num1/num2;
                }
                if(str==3){
                    ans=num1*num2;
                }
                if(str==4){
                    ans=num1-num2;
                }
                JOptionPane.showMessageDialog(null, String.format("Answer: %s", ans));

            }
        }
    }

必要に応じて、これを実行するスクリプトを次に示します。

import javax.swing.JFrame;

public class apples {
    public static void main(String[] args){
        Gui bucky = new Gui();
        bucky.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        bucky.setSize(300,150);
        bucky.setVisible(true);
    }
}

「計算」をクリックしたときのエラーコードは次のとおりです。

スレッド「AWT-EventQueue-0」の例外 java.lang.Error: 未解決のコンパイルの問題: タイプ Gui.HandlerClass は、継承された抽象メソッド ActionListener.actionPerformed(ActionEvent) を実装する必要があります

at Gui$HandlerClass.actionPerformed(Gui.java:85)
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$3.run(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$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

2 に答える 2

2

コンパイルできないコードを実行しようとしています-これを行わないでください!

代わりに、コンパイル時にすべてのコンパイルの問題を修正し、コードが正常にコンパイルされたときにのみ実行します。

actionPerformed(...)まず、正しく大文字にする必要があります。

次へ:JRadioButtonにリスナーを設定しないでくださいむしろ、メイン計算JButtonのActionListenerでのみ押されているボタンを確認してください。その理由は、ユーザーが1つのJRadioButtonを選択し、後者が気が変わって別のJRadioButtonを選択するかどうかは気にせず、JButtonの計算が押されたときにどのJRadioButtonが選択されたかだけを気にするからです。

于 2012-06-02T18:41:33.240 に答える
2

問題はリスナーにある可能性があります。すべてのラジオ ボタンはItemListeners を使用していますが、これは問題ありませんが、JButton も同様です。使用する標準リスナーJButtonActionListener. あなたItemListenerActionListener(そしてイベントをActionEvent)に変更すると、うまくいくはずです。

item イベントが発生しない理由は、が呼び出されたJButton場合にのみイベントが発生するためsetSelectedです。

UPDATE : 新しいコードのエラーは、メソッド名のActionPerformed代わりに配置することです。actionPerformed大文字と小文字が重要です。

更新 2 : すべての変数を宣言する場所に、次の変数を追加します。

private ArrayList<JRadioButton> radioButtons = new ArrayList<JRadioButton>

java.util.ArrayListまだインポートしていない場合は、インポートする必要がある場合があります。

次に、行の下

plus = new JRadioButton("Add", true);
divide = new JRadioButton("Divide", false);
multiply = new JRadioButton("Multiply", false);
subtract = new JRadioButton("Subtract", false);

以下を追加します。

radioButtons.add(plus);
radioButtons.add(divide);
radioButtons.add(multiply);
radioButtons.add(subtract);

どのボタンが選択されているかを確認する必要がある場合は、次のコードを使用します。

for (JRadioButton button : radioButtons) { // Iterate over each button in the list
    if (button.isSelected()) { // If the button is selected...
        // do something
    }
}

他のコードは問題ないように見えますが、+-/* 操作を管理するために文字列定数 (またはさらに良いのは列挙型) を使用することをお勧めします。つまり、str = "add"(et. al.) の代わりに、次の列挙型を宣言する必要があります。

private enum OperationType {
    ADD, SUBTRACT, MULTIPLY, DIVIDE;
}

変更する

private String str = "add"

private OperationType op = OperationType.ADD;

この結果、入力エラー (tpyo または aCcidental 大文字を含む) が発生すると、文字列が無効になり、奇妙な動作が発生します。

また、ステートメントを使用できることも意味しますswitch。多くのステートメントを使用する代わりに、次のようにしてif .. elseください。

switch (op):
    case ADD:
        // blah
        break;
    case SUBTRACT:
        // blah
        break;
    case MULTIPLY:
        // blah
        break;
    case DIVIDE:
        // blah
        break;
    default:
        System.out.println("Error: unsupported operation");
        // or whatever
}

これは一般に、よりクリーンなコードと見なされており、見栄えが良く、保守が容易です。

幸運を!

于 2012-06-02T18:36:17.690 に答える