Как разбить эту программу на несколько классов

Question

Я нашел эту реализацию простого калькулятора в Java, используя фреймворк Java Swing.

Исходный код здесь:

//Imports are listed in full to show what's being used 
//could just import javax.swing.* and java.awt.* etc.. 
import java.awt.EventQueue; 
import java.awt.GridLayout; 
import java.awt.BorderLayout; 
import java.awt.event.ActionListener; 
import java.awt.event.ActionEvent; 
import javax.swing.JFrame; 
import javax.swing.JPanel; 
import javax.swing.JTextField; 
import javax.swing.JButton; 
import java.awt.Container; 

public class SimpleCalc implements ActionListener{ 

    JFrame guiFrame; 
    JPanel buttonPanel; 
    JTextField numberCalc; 
    int calcOperation = 0; 
    int currentCalc; 

    //Note: Typically the main method will be in a 
    //separate class. As this is a simple one class 
    //example it's all in the one class. 
    public static void main(String[] args) { 

     //Use the event dispatch thread for Swing components 
     EventQueue.invokeLater(new Runnable() 
     { 

      @Override 
      public void run() 
      { 

       new SimpleCalc();   
      } 
     }); 

    } 

    public SimpleCalc() 
    { 
     guiFrame = new JFrame(); 

     //make sure the program exits when the frame closes 
     guiFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
     guiFrame.setTitle("Simple Calculator"); 
     guiFrame.setSize(300,300); 

     //This will center the JFrame in the middle of the screen 
     guiFrame.setLocationRelativeTo(null); 

     numberCalc = new JTextField(); 
     numberCalc.setHorizontalAlignment(JTextField.RIGHT); 
     numberCalc.setEditable(false); 

     guiFrame.add(numberCalc, BorderLayout.NORTH); 

     buttonPanel = new JPanel(); 

     //Make a Grid that has three rows and four columns 
     buttonPanel.setLayout(new GridLayout(4,3)); 
     guiFrame.add(buttonPanel, BorderLayout.CENTER); 

     //Add the number buttons 
     for (int i=1;i<10;i++) 
     { 
      addButton(buttonPanel, String.valueOf(i)); 
     } 

     JButton addButton = new JButton("+"); 
     addButton.setActionCommand("+"); 

     OperatorAction subAction = new OperatorAction(1); 
     addButton.addActionListener(subAction); 

     JButton subButton = new JButton("-"); 
     subButton.setActionCommand("-"); 

     OperatorAction addAction = new OperatorAction(2); 
     subButton.addActionListener(addAction); 

     JButton equalsButton = new JButton("="); 
     equalsButton.setActionCommand("="); 
     equalsButton.addActionListener(new ActionListener() 
     { 
      @Override 
      public void actionPerformed(ActionEvent event) 
      { 
       if (!numberCalc.getText().isEmpty()) 
       { 
        int number = Integer.parseInt(numberCalc.getText()); 
        if (calcOperation == 1) 
        { 
         int calculate = currentCalc + number; 
         numberCalc.setText(Integer.toString(calculate)); 
        } 
        else if (calcOperation == 2) 
        { 
         int calculate = currentCalc - number; 
         numberCalc.setText(Integer.toString(calculate)); 
        } 
       } 
      } 
     }); 

     buttonPanel.add(addButton); 
     buttonPanel.add(subButton); 
     buttonPanel.add(equalsButton); 
     guiFrame.setVisible(true); 
    } 

    //All the buttons are following the same pattern 
    //so create them all in one place. 
    private void addButton(Container parent, String name) 
    { 
     JButton but = new JButton(name); 
     but.setActionCommand(name); 
     but.addActionListener(this); 
     parent.add(but); 
    } 

    //As all the buttons are doing the same thing it's 
    //easier to make the class implement the ActionListener 
    //interface and control the button clicks from one place 
    @Override 
    public void actionPerformed(ActionEvent event) 
    { 
     //get the Action Command text from the button 
     String action = event.getActionCommand(); 

     //set the text using the Action Command text 
     numberCalc.setText(action);  
    } 

    private class OperatorAction implements ActionListener 
    { 
     private int operator; 

     public OperatorAction(int operation) 
     { 
      operator = operation; 
     } 

     public void actionPerformed(ActionEvent event) 
     { 
      currentCalc = Integer.parseInt(numberCalc.getText()); 
      calcOperation = operator; 
     } 
    } 
} 

Я хотел бы поставить класс OperatorAction в в отдельный класс, который я пытался, но тогда возникает проблема получения текста из JTextField, как недоступен в этом новом классе. Так как это сделать?

Answer 1

Все, что вам нужно, это еще один аргумент конструктора (и соответствующее поле экземпляра):

class OperatorAction implements ActionListener 
{ 
    private int operator; 
    private SimpleCalc calc; 

    public OperatorAction(SimpleCalc c, int operation) 
    { 
     calc = c; 
     operator = operation; 
    } 

    public void actionPerformed(ActionEvent event) { 
     calc.setCurrentCalc(Integer.parseInt( 
      ((JTextField)event.getSource()).getText())); 
     calcOperation = operator; 
    } 
} 

И обнажить SimpleCalc#currentCalc собственности через сеттер.

Это не очень чистый дизайн, хотя у вас все еще есть плотная связь между OperatorAction и SimpleCalc, но это может быть началом для вас.

Answer 2

Похоже, что вам нужно использовать шаблон контроллера модели (MVC)! MVC является неотъемлемой частью многих языков и систем, поэтому очень важно учиться! Что вы хотите сделать, это разбить программу на 3 слоя классов, * M * odels, * V * iews и * C * ontrollers (в этом случае проект, вероятно, достаточно прост, чтобы используйте только один из них.)

Модели собираются хранить ваши данные, это могут быть несколько классов или целая база данных в зависимости от размера вашего проекта.

Просмотров собираются отображать ваши данные.

Контролеры будут управлять остальными.

Так что у вас есть модель, которая просто хранит входные данные в калькулятор, контроллер, который принимает вход, выполняет фактическую математику и выталкивает вывод в представление.