Java GUI calculator

[Yukka]

Hello! I just can't figure out how can i do few operations in one transaction (ex: 5-4+1) i would appreciate if somebody could point on my mistakes. Here is my code. for some reason after pressing second operator it would replace the first one....any ideas?

class Operators implements ActionListener
{        
int num1=0;
int num2=0;
int op=0;
int a=1;
 
 
public void actionPerformed(ActionEvent e)
{
 
Object src = e.getSource();
String valuestr=n1Txt.getText();
int valueint= Integer.parseInt(valuestr);
 
if(src==bmin)
{op=1;//-
if(a==1){
a=2;
num1=valueint;
}
else if(a==2){
a=3;
num2=valueint;
num1=num1-num2;}
else if(a==3){
a=2;
num2=valueint;
num1=num1-num2;    
}                        
}
 
if(src==bpl)
{op=2;//+
if(a==1){
a=2;
num1=valueint;
}
else if(a==2){
a=3;
num2=valueint;
num1=num1+num2;}
else if(a==3){
a=2;
num2=valueint;
num1=num1+num2;    
}                        
}
 
if(src==bmult)
{op=3;//*
if(a==1){
a=2;
num1=valueint;}
else if(a==2){
a=3;
num2=valueint;
num1=num1*num2;}
else if(a==3){
a=2;
num2=valueint;
num1=num1*num2;    
}                        
}
 
if(src==bdev)
{op=4;// /
if(a==1){
a=2;
num1=valueint;}
else if(a==2){
a=3;
num2=valueint;
num1=num1/num2;}
else if(a==3){
a=2;
num2=valueint;
num1=num1/num2;    
}                        
}
 
n1Txt.setText(""); // where n1Txt is the display field
if(src==beq){
switch (op) {
    case 1 :num2=valueint;
            num1=num1-num2;
    break;
    case 2 :num2=valueint;
            num1=num1+num2;
    break;
    case 3 :num2=valueint;
            num1=num1*num2;
    break;
    case 4 :num2=valueint;
            num1=num1/num2;
    break;
 
}
String result = Integer.toString(num1);
n1Txt.setText(result);
}}

[r035198x]

Better format your code properly and post it wrapped in code tags.

[Yukka]

Hello! I just can't figure out how can i do few operations in one transaction (ex: 5-4+1) i would appreciate if somebody could point on my mistakes. Here is my code. for some reason after pressing second operator it would replace the first one....any ideas?

 int num1=0; 
    int num2=0; 
    int op=0; 
    int a=1; 
 
 
    public void actionPerformed(ActionEvent e) 
    { 
 
    Object src = e.getSource(); 
    String valuestr=n1Txt.getText(); 
    int valueint= Integer.parseInt(valuestr)
    if(src==bmin) 
    {op=1;  //- 
        if(a==1){ 
         a=2; 
                    num1=valueint; } 
 
        else if(a==2) 
        {a=3; 
        num2=valueint; 
        num1=num1-num2;} 
 
        else if(a==3) 
        {a=2; 
        num2=valueint; 
        num1=num1-num2;}         
    } 
 
    if(src==bpl) 
    {op=2;//+ 
        if(a==1) 
         {a=2; 
         num1=valueint;} 
 
         else if(a==2) 
        {a=3; 
        num2=valueint; 
        num1=num1+num2;} 
 
        else if(a==3) 
        {a=2; 
        num2=valueint; 
        num1=num1+num2;}  
     }     
 
    if(src==bmult) 
    {op=3;//* 
        If(a==1) 
        {a=2; 
        num1=valueint;} 
 
        else if(a==2) 
        {a=3; 
        num2=valueint; 
        num1=num1*num2;} 
 
        else if(a==3) 
        {a=2; 
        num2=valueint; 
        num1=num1*num2;}
    } 
 
    if(src==bdev) 
    {op=4;// / 
        if(a==1) 
        { a=2; 
        num1=valueint;} 
 
        else if(a==2)  
        { a=3; 
        num2=valueint; 
        num1=num1/num2;} 
 
        else if(a=3) 
        {a=2; 
        num2=valueint; 
        num1=num1/num2;} 
    }     
 
n1Txt.setText(""); // where n1Txt is the display field 
    if(src==beq){ 
    switch (op) { 
        case 1 :num2=valueint; 
                    num1=num1-num2; 
            break; 
            case 2 :num2=valueint; 
                    num1=num1+num2; 
            break; 
            case 3 :num2=valueint; 
                    num1=num1*num2; 
            break; 
            case 4 :num2=valueint; 
                    num1=num1/num2; 
            break; 
 
    } 
 
String result = Integer.toString(num1); 
n1Txt.setText(result); 
                } 
} 
 

[quote=Yukka;3491355]