자바를 사용하여 GUI 계산기를 만들었습니다. 변수가 최신 상태로 유지되지 않고 0으로 유지됩니다.

Question

이 프로그램의 기능을 향상시킬 수있는 더 나은 방법이 있음을 알고 있습니다.하지만 할 수 없었습니다. 이 프로그램이하는 정확히 어떻게, 어떤 제안이 도움이됩니다 않는 더 나은 알고리즘을 마련

package calc; 

import java.awt.FlowLayout; 
import java.awt.event.ActionEvent; 
import java.awt.event.ActionListener; 
import javax.swing.JButton; 
import javax.swing.JFrame; 
import javax.swing.JTextField; 

/** 
* 
* @author Ben 
*/ 
public class GUI extends JFrame { 

int response, count=0;  
double num1, num2, num3, num5, num6, num7, num8, total=0; 
String operation, answer, num, testnum; 
private JButton one, two, three, four, five, six, seven, eight, 
     nine, zero, multiply, divide, subtract, add, equals, clear; 
private JTextField display, fakedisplay; 

public GUI(){ 
    super("Calculator"); 
    setLayout (new FlowLayout()); 

    fakedisplay = new JTextField(10); 
    display = new JTextField(10); 
    display.setEditable(false);   
    add(display);     
    one = new JButton("1");  
    add(one); 
    two = new JButton("2"); 
    add(two); 
    three = new JButton("3"); 
    add(three); 
    four = new JButton("4"); 
    add(four); 
    five = new JButton("5"); 
    add(five); 
    six = new JButton("6"); 
    add(six); 
    seven = new JButton("7"); 
    add(seven); 
    eight = new JButton("8"); 
    add(eight); 
    nine = new JButton("9"); 
    add(nine); 
    zero = new JButton("0"); 
    add(zero);    
    multiply = new JButton("*"); 
    add(multiply);   
    divide = new JButton("/"); 
    add(divide); 
    subtract = new JButton("-"); 
    add(subtract); 
    add = new JButton("+"); 
    add(add); 
    equals = new JButton("="); 
    add(equals); 
    clear = new JButton("Clear"); 
    add(clear); 


    handler handle = new handler(); 

    one.addActionListener(handle); 
    two.addActionListener(handle); 
    three.addActionListener(handle); 
    four.addActionListener(handle); 
    five.addActionListener(handle); 
    six.addActionListener(handle); 
    seven.addActionListener(handle); 
    eight.addActionListener(handle); 
    nine.addActionListener(handle); 
    zero.addActionListener(handle); 
    multiply.addActionListener(handle); 
    divide.addActionListener(handle); 
    subtract.addActionListener(handle); 
    add.addActionListener(handle); 
    equals.addActionListener(handle); 
    clear.addActionListener(handle); 

} 
private class handler implements ActionListener{ 

    @Override 
    public void actionPerformed(ActionEvent e){ 

     if(e.getSource()==one){ 
      response = 1;    
      display.setText(display.getText() + response); 
      fakedisplay.setText(fakedisplay.getText() + response); 
     } 
     else if (e.getSource()==two){ 
      response = 2; 
      display.setText(display.getText() + response);  
      fakedisplay.setText(fakedisplay.getText() + response); 
     } 
     else if (e.getSource()==three){ 
       response = 3; 
       display.setText(display.getText() + response); 
       fakedisplay.setText(fakedisplay.getText() + response); 
     } 
     else if (e.getSource()==four){ 
       response = 4; 
       display.setText(display.getText() + response); 
       fakedisplay.setText(fakedisplay.getText() + response); 
     } 
     else if (e.getSource()==five){ 
       response = 5; 
       display.setText(display.getText() + response); 
       fakedisplay.setText(fakedisplay.getText() + response); 
     } 
     else if (e.getSource()==six){ 
       response = 6; 
       display.setText(display.getText() + response); 
       fakedisplay.setText(fakedisplay.getText() + response); 
     } 
     else if (e.getSource()==seven){ 
       response = 7; 
       display.setText(display.getText() + response); 
       fakedisplay.setText(fakedisplay.getText() + response); 
     } 
     else if (e.getSource()==eight){ 
       response = 8; 
       display.setText(display.getText() + response); 
       fakedisplay.setText(fakedisplay.getText() + response); 
     } 
     else if (e.getSource()==nine){ 
       response = 9; 
       display.setText(display.getText() + response); 
       fakedisplay.setText(fakedisplay.getText() + response); 
     } 
     else if(e.getSource()==zero){ 
      response = 0; 
      display.setText(display.getText() + response); 
      fakedisplay.setText(fakedisplay.getText() + response); 
     } 
     else if (e.getSource()==multiply){ 
      if(count == 0){ 
       num1 = Double.parseDouble(display.getText()); 
       count++; 
      }     
      else if(count == 1){ 
       num2 = Double.parseDouble(fakedisplay.getText()); 
       total = num1*num2; 
       count++; 
      } 
      else if(count == 2){ 
       num3 = Double.parseDouble(fakedisplay.getText()); 
       total = total*num3; 
       count++; 
      } 
      operation = "*"; 
      display.setText(display.getText() + operation); 
      fakedisplay.setText(""); 

     } 
     else if (e.getSource()==divide){ 
      if(count == 0){ 
       num1 = Double.parseDouble(display.getText()); 
       count++; 
      } 
      else if(count == 1){ 
       num2 = Double.parseDouble(fakedisplay.getText()); 
       total = num1/num2+num1%num2; 
       count++; 
      } 
      else if(count == 2){ 
       num3 = Double.parseDouble(fakedisplay.getText()); 
       total = total/num3+total%num3; 
       count++; 
      } 
       operation = "/";      
       display.setText(display.getText() + operation); 
       fakedisplay.setText(""); 
     } 
     else if (e.getSource()==add){ 
      if(count == 0){ 
       num1 = Double.parseDouble(display.getText()); 
       count++; 
      } 
      else if(count == 1){ 
       num2 = Double.parseDouble(fakedisplay.getText()); 
       total = num1+num2; 
       count++; 
      } 
      else if(count == 2){ 
       num3 = Double.parseDouble(fakedisplay.getText()); 
       total = total+num3; 
       count++; 
      } 
       operation = "+"; 
       display.setText(display.getText() + operation); 
       fakedisplay.setText(""); 
     } 
     else if (e.getSource()==subtract){ 
      if(count == 0){ 
       num1 = Double.parseDouble(display.getText()); 
       count++; 
      } 
      else if (count == 1) { 
       num2 = Double.parseDouble(fakedisplay.getText()); 
       total = num1-num2; 
       count++; 
      } 
      else if(count == 2){ 
       num3 = Double.parseDouble(fakedisplay.getText()); 
       total = total-num3; 
       count++; 
      } 
       operation = "-"; 
       display.setText(display.getText() + operation); 
       fakedisplay.setText(""); 
     } 
     else if (e.getSource()==equals){     
      operation = "="; 
      display.setText(display.getText() + operation + total);       
     } 
     else if (e.getSource()==clear){ 
      display.setText(""); 
     } 


} 



} 
} 

Answer 1

이 내가 볼 문제의 몇 가지가 있지만, 가장 먼저 필요한 것은 함께 당신을 도울 :

등호 연산을 수행하는 위치는 어디입니까? 간단한 산술 함수를 하 :

A op B = ? 

(A 및 B 시험 번호 및 영업 + 중 하나이고 -, *, /)

가 num1에 슬롯 번호를되지만 op 수행되지 작업 count 동일 자신을 돕기 위해 1

, 마지막 else if 블록 뒤에 다음을 추가하지 않습니다 때문에 :

System.out.println("num1: " + num1 + " num2: " + num2 + " num3: " + num3 
    + "\ndisplay: " + display.getText() + " fakedisplay: " + fakedisplay.getText() 
    + "\nresponse: " + response + "\ncount: " + count + "\ntotal: " + total); 

이것은 각 키/버튼을 누른 후에 방금 일어난 일에 대한 힌트를 제공합니다. 당신이해야 할 당신이 논리의 단계에 대해 생각하는 경우

은 다음과 같습니다

 
Create fields for leftValue, operator, rightValue. 
Handle the input just entered 
    if input is a number and leftValue is null 
     leftValue = input 
    if input is a number and leftValue (and operator) are not null 
     leftValue = leftValue OPERATION input 
    if input is an operator and leftValue is not null 
     operator = input