Problems with a result in a switch/case control statement

Question

I am having troubles with a switch/case statement in which I am trying to get a result returned from different operators. The problem is that the result always returns 0 no matter what I put in the driver class.
The class where the result needs to be returned looks like this:
public class Calculator
     private int num1;
     private int num2;
     private char operator;
     private int result;
     public Calculator(int num1, char operator, int num2)
          this.num1 = num1;
          this.operator = operator;
          this.num2 = num2;
     } // end constructor
     public int getNum1()
          return num1;
     public char getOperator()
          return operator;
     public int getNum2()
          return num2;
     public int getResult()
          return result;
     public int calculate()
          switch(operator)
               case '+':
                    int result = num1 + num2;
                    break;
               } // end case '+'
               case '-':
                    int result = num1 - num2;
                    break;
               } // end case '-'
               case '*':
                    int result = num1 * num2;
                    break;
               } // end case '*'
               case '/':
                    int result = num1 / num2;
                    break;
               } // end case '/'
               case '%':
                    int result = num1 % num2;
                    break;
               } // end case '%'
          } // end switch
          return result;
     public String toString()
          String message = (num1) + " " + (operator)
                              + " " + (num2) + " " + "=" + " " + result + "\n";
          return message;
} // end class
The driver class looks like this:
import javax.swing.JOptionPane;
import java.lang.StringBuffer;
public class CalculatorApp
     public static void main (String[] args) // begin main
          String choice = "";
          while (!(choice.equalsIgnoreCase("x"))) // begin while loop
               String number1 = JOptionPane.showInputDialog(
                              "Enter a positive integer: ");
               String operatorInput = JOptionPane.showInputDialog(
                              "Enter operator (+, -, *, /, %): ");
               String number2 = JOptionPane.showInputDialog(
                              "Enter another positive integer: ");
               int num1;
               char operator;
               int num2;
               num1 = Integer.parseInt(number1);
               operator = operatorInput.charAt(0);
               num2 = Integer.parseInt(number2);
               Calculator calculator = new Calculator(num1, operator, num2);
               String message = calculator.toString() + "\n"
                                   + "Press Enter to conitnue or 'x' to exit.";
               choice = JOptionPane.showInputDialog(message);
          } // end while loop
          System.exit(0);
     } // end main
any help would be wonderful! I don't know why the result returned is always 0 but it is. Thanks!
dragwit

Answer 1

The previous poster is correct. You never call the calculate method. If you don't call it, no one's going to call if for you. This stuff doesn't work by magic. I think you've misunderstood your prof's intent in this regard.
However, it still won't work with the changes previously suggested, because declaring an int result in the calculate method will hide the instance field of the same name, which is what your getResult() method retrieves. Don't delare result at all in your calculate method.
That said, it would be architecturally a lot better if the calculate method returned the result rather than storing it back into the instance.