Javaで効率的にする方法

Question

私は以下の図のような簡単な電卓を作ろうとします。しかし、私が書きたい：

private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) 
{           
     number1 = Integer.parseInt(n1.getText()); 
     number2 = Integer.parseInt(n2.getText()); 
     JOptionPane.showMessageDialog(null,(number1+number2)); 

このコードは、すべてのボタンに、どのように私は共通の場所にいくつかの簡単なヘルパーメソッドについて

Answer 1

方法を宣言することによってそれを行うだろうか？

private int getNumber1() { 
    return Integer.parseInt(n1.getText()); 
} 
private int getNumber2() { 
    return Integer.parseInt(n2.getText()); 
} 

private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) 
{           
    JOptionPane.showMessageDialog(null, getNumber1() + getNumber2()); 
} 
private void jButton2ActionPerformed(java.awt.event.ActionEvent evt) 
{           
    JOptionPane.showMessageDialog(null, getNumber1() - getNumber2()); 
} 
private void jButton3ActionPerformed(java.awt.event.ActionEvent evt) 
{           
    JOptionPane.showMessageDialog(null, getNumber1() * getNumber2()); 
} 
private void jButton4ActionPerformed(java.awt.event.ActionEvent evt) 
{           
    JOptionPane.showMessageDialog(null, getNumber1()/getNumber2()); 
} 

Answer 2

操作の列挙を定義：

enum Operation { 
    PLUS { 
    @Override int operate(int a, int b) { return a + b; } 
    }, 
    MINUS { 
    @Override int operate(int a, int b) { return a - b; } 
    }, 
    // etc, for others. 
    ; 

    abstract int operate(int a, int b); 
} 

そしてパラメータとしてOperationかかりActionListenerサブクラス定義：次に、このクラスのインスタンスを追加

class MyActionListener implements ActionListener { 
    final Operation operation; 

    MyActionListener(Operation operation) { 
    this.operation = operation; 
    } 

    @Override public void actionPerformed(ActionEvent evt) { 
    int a = Integer.parseInt(n1.getText()); 
    int b = Integer.parseInt(n2.getText()); 
    int result = operation.operate(a, b); 
    JOptionPane.showMessageDialog(null, result); 
    } 
} 

を各ボタンへのイベントリスナー：

jButton1.addActionListener(new MyActionListener(Operation.PLUS)); 
jButton2.addActionListener(new MyActionListener(Operation.MINUS));