Calculadora Java

import javax.swing.JOptionPane;
import java.lang.Math;

public class sum {


public static void main(String args []) {

String num1,num2,opcion;
int op;/
doublen1,n2,rpta=0;

opcion=JOptionPane.showInputDialog(null,"1. Suma\n2. Resta.\n3. Multiplicacion.\n4. Division.\n5. Potencia.\n6. Raiz Cuadrada.\n7. Factorial","MI MENU",JOptionPane.QUESTION_MESSAGE);op=Integer.parseInt(opcion);

if(op==1 || op==2 || op==3 || op==4 || op==5){//2
num1=JOptionPane.showInputDialog("Digita el Primer Valor");num2=JOptionPane.showInputDialog("Digita el Primer Valor");
n1=Double.parseDouble(num1);
n2=Double.parseDouble(num2);

switch(op){//1

case 1: rpta=n1+n2; break;
case 2: rpta=n1-n2; break;
case 3:rpta=n1*n2; break;
case 4: if(n2!=0) //ahora como la division no es posible se el dividendo es cero la condicionamos para q eso no pase
rpta=n1/n2;
elseJOptionPane.showMessageDialog(null,"Error","Division Entre Cero",JOptionPane.ERROR_MESSAGE);
break;
default: rpta=Math.pow(n1,n2);break;

}//cierre 1
}//cierre 2
elseif(op==6 || op==7){//para factorizacion 3

num1=JOptionPane.showInputDialog("Digita el Numero para la Raiz Cuadrada o Factorial");//Pide solo un valor para evaluar la raiz,potencia y elfactorial
n1=Double.parseDouble(num1);

if(op==6)
rpta=Math.sqrt(n1);
else
{
rpta=1;// necesariamente se cambia su inicializacion a 1 ya que semultiplicara con este valor
for(double i=n1 ; i>0 ; i--){
rpta=rpta*i;
}

}
}

else
JOptionPane.showMessageDialog(null,"Opcion Fuera de Rango","Error",JOptionPane.ERROR_MESSAGE);


if(rpta!=0)//esta ultima condicion evalua que en la variable que se guardan las repuestas tenga un contenido diferente del inicial q es cero....