2012-03-28 3 views
1

私は電卓をJavaアプレットとして作っています。レイアウトを作成し、すべてのJButtonにactionListenerを登録しました。getActionCommand()と一致するString値を比較できないのはなぜですか?

String buttonClicked = e.getActionCommand(); 
// for number buttons 
for(int i = 0; i <= 9; i++) 
{ 
if(i == Integer.parseInt(buttonClicked)) 
answer.setText(answer.getText() + e.getActionCommand()); 
} 

...正しく動作:

ので、ActionListenerを実装して私のクラスでは、私は数字ボタンのすべてのために、このコードを持っています。だから、なぜこのコードは動作しません:

if(buttonClicked.equals("/")) 
{ 
answer.setText(answer.getText() + e.getActionCommand()); 
} 

... getActionCommand()Stringの内容とStringを比較することはできませんか?それとも、私は何かを取り入れるべきではないのでしょうか?あなたは "/" でのparseIntを行うにしようとしている

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

public class Part_C extends Applet 
{ 
    // answer field here (no panel; JTextField is 100% width without panel) 
    private JTextField answer = new JTextField(); 
    //answer.setHorizontalAlignment(JTextField.RIGHT); // align input to the right 

    // JButton array 
    private JButton[] buttons = 
    { 
     // (panel 1) backspace, clear entry, and clear buttons here 
     new JButton("backspace"), 
     new JButton("CE"), 
     new JButton("C"), 

     // (panel 2, row 1) numbers 7 - 9, divide, and sqare-root buttons here 
     new JButton("7"), 
     new JButton("8"), 
     new JButton("9"), 
     new JButton("/"), 
     new JButton("sqrt"), 

     // (panel 2, row 2) numbers 4 - 6, multiply, and percent buttons here 
     new JButton("4"), 
     new JButton("5"), 
     new JButton("6"), 
     new JButton("X"), 
     new JButton("%"), 

     // (panel 2, row 3) numbers 1 - 3, subtract, and inverse buttons here 
     new JButton("1"), 
     new JButton("2"), 
     new JButton("3"), 
     new JButton("-"), 
     new JButton("1/x"), 

     // (panel 2, row 4) number 0, positive/negative, decimal, addition, and equals buttons here 
     new JButton("0"), 
     new JButton("+/-"), 
     new JButton("."), 
     new JButton("+"), 
     new JButton("="), 

    }; // end of JButton array 

    // constructor for class Part_C 
    public Part_C() 
    { 
     // first panel for backspace, clear entry, and clear buttons 
     JPanel panel1 = new JPanel(); 
     panel1.setLayout(new GridLayout(1, 3)); 
     for(int i = 0; i < 3; i++) 
      panel1.add(buttons[i]); 

     // second panel for operators and operands 
     JPanel panel2 = new JPanel(); 
     panel2.setLayout(new GridLayout(4, 5)); 
     for(int i = 3; i < buttons.length; i++) 
      panel2.add(buttons[i]); 

     // third panel for BorderLayout of answer JTextField, and first 2 panels 
     JPanel panel3 = new JPanel(new BorderLayout()); 
     panel3.add(answer, BorderLayout.NORTH); 
     panel3.add(panel1, BorderLayout.CENTER); 
     panel3.add(panel2, BorderLayout.SOUTH); 
     add(panel3); // add panel3 to the JFrame 

     // action listener created here 
     ButtonListener listener = new ButtonListener(); 

     // action listener is added to all JButtons here 
     for (int i = 0; i < buttons.length; i++) 
      buttons[i].addActionListener(listener); 
    } 

    /* overriding the init() method of Applet; 
    * ...in an Applet, init() is used instead 
    * of the "main" method */ 
    public void init() 
    { 
     // create JFrame 
     Part_C frame = new Part_C(); 

     // the applet stops when the window/tab is closed 
     //frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
    } 

    // inner class ButtonListener 
    public class ButtonListener implements ActionListener 
    { 
     public void actionPerformed(ActionEvent e) 
     { 
      String buttonClicked = e.getActionCommand(); 

      // for number buttons 
      for(int i = 0; i <= 9; i++) 
      { 
       if(i == Integer.parseInt(buttonClicked)) 
        answer.setText(answer.getText() + e.getActionCommand()); 
      } 

      // for operators 
      if(buttonClicked.equals("/")) 
      { 
       answer.setText(answer.getText() + e.getActionCommand()); 
      } 
      else if(buttonClicked.equals("sqrt")) 
      { 
       answer.setText(answer.getText() + e.getActionCommand()); 
      } 
      else if(buttonClicked.equals("X")) 
      { 
       answer.setText(answer.getText() + e.getActionCommand()); 
      } 
      else if(buttonClicked.equals("%")) 
      { 
       answer.setText(e.getActionCommand()); 
      } 
      else if(buttonClicked.equals("-")) 
      { 
       answer.setText(answer.getText() + e.getActionCommand()); 
      } 
      else if(buttonClicked.equals("1/x")) 
      { 
       answer.setText(e.getActionCommand()); 
      } 
      else if(buttonClicked.equals("+/-")) 
      { 
       answer.setText(e.getActionCommand()); 
      } 
      else if(buttonClicked.equals(".")) 
      { 
       answer.setText(answer.getText() + e.getActionCommand()); 
      } 
      else if(buttonClicked.equals("+")) 
      { 
       answer.setText(e.getActionCommand()); 
      } 

      // for data-clearing operations 
      if(buttonClicked.equals("backspace")) 
      { 
       answer.setText(answer.getText().substring(0, answer.getText().length() - 1)); 
      } 
      else if(buttonClicked.equals("CE")) 
      { 

      } 
      else if(buttonClicked.equals("C")) 
      { 
       answer.setText(""); 
      } 

      // for the equals button 
      if(buttonClicked.equals("=")) 
      { 

      } 
     } 
    } // end of inner class buttonListener 
} // end of Part_C 
+0

実際にボタンにactionCommandを設定していますか?コードがなくても、私はすべて考えることができます。 *編集:スペル:P * – mcfinnigan

+0

"getActionCommand()"を正しく使用して設定していません...上記のコード全体を追加しました: –

+1

'..extends Applet .. // JButton array' Don ' SwingとAWTコンポーネントを混在させます。代わりに 'JApplet'を使用してください。または、もっと良い方法として、必要に応じて 'JFrame'を使用し、[Java Web Start](http://stackoverflow.com/tags/java-web-start/info)を使ってリンクから起動します。 –

答えて

1

: -

EDITは、ここに私の完全なコードです。 NumberFormatExceptionがスローされます。文字列を整数に変換できない場合、コードはforループを超えることはありません。

+0

いいえ、電卓のボタン0〜9で解析できる文字列を持つJButtonsのparseIntを行っています...そして、その部分は実際には動作しますが、getActionCommand( )は、動作していない整数に解析できないString値を持つJButtonを使用します(「/」などの場合、これらのいずれの場合でもparseIntを使用しない)。 –

+0

あなたのforループ 'for(int i = 0; i <= 9; i ++)は、どのボタンが押されたかにかかわらず、いつでも実行されます。そのforループの中には 'Integer.parseInt(buttonClicked)'があり、数値以外の何かをクリックするたびにNumberFormatExceptionが返されます。約束します。 – Mike

+0

ああ、そうです!それを指摘してくれてありがとう。 私はしばらくのうちにJavaをやっていない、ちょうどこの状況で例外処理をすることを思い出して、今すぐ動作させた。 そのセグメントのコードは次のとおりです。 文字列buttonClicked = ""; try { int number = Integer.parseInt(e.getActionCommand());数字ボタン ため//がため(INT i = 0; iが<= 9; I ++){ IF(I ==数) answer.setText(answer.getText()+ e.getActionCommand())。 } } catch(NumberFormatException err) { buttonClicked = e.getActionCommand(); } –

関連する問題