Tu programa necesita algunas correcciones rápidas para funcionar:
Basicamente es aplicar el principio responsabilidad única de SOLID al método main, este principio consiste en dejar que un método tenga un único trabajo, en el caso del main es el control del flujo, asíque hay variables que no necesitas tener allí, de esa manera si tienes problemas con la suma o la resta puedes ir al método en aprticular y si tienes problemas con la opción ya sabes qne parte del código buscar. ESto te hará mucho más fácil al depuración y la modificación de tu aplicación, además de que sin notarlo estarás optimizando la ejecucuión, pues las variables no usadas al salir de un método serán marcadas para una recolección de basura que te liberará esos espacios de memoria automáticamente.
para obtener un caracter puedes aplicar el truco que sugiere ccarlos pero no es lo ideal, te explico una vez veas la corrección rápida del código.
import java.util.Scanner;
public class Ayuda {
public static void main(String[] args) {
Scanner teclado = new Scanner(System.in);
int opcion;
do {
System.out.println(" TE PUEDO AYUDAR A "
+ " 1. SUMAR "
+ " 2. RESTAR "
+ " 3. FIN "
+ " ESCOGER OPCION ");
opcion = teclado.nextInt();
switch (opcion) {
case 1: {
ayudaSumar(teclado);
break;
}
case 2: {
ayudaRestar(teclado);
break;
}
default:
break;
}
} while (opcion != 3);
}
public static void ayudaSumar(Scanner teclado) {
char desea = 'S';
int resuMaq;
int num1;
int num2;
int resuNi;
do {
System.out.print("Ingrese numero 1: ");
num1 = teclado.nextInt();
System.out.print("Ingrese numero 2: ");
num2 = teclado.nextInt();
System.out.print("Ingrese rsultado: ");
resuNi = teclado.nextInt();
resuMaq = num1 + num2;
if (resuMaq == resuNi) {
System.out.print("La suma esta correcta");
} else {
System.out.print("La suma esta incorrecta");
}
System.out.print("¿Desea revisar otra suma (S/N)");
desea = teclado.next().charAt(0);
} while (desea == 'S');
}
public static void ayudaRestar(Scanner teclado) {
char desea = 'S';
int resuMaq;
int num1;
int num2;
int resuNi;
do {
System.out.print("Ingrese numero 1: ");
num1 = teclado.nextInt();
System.out.print("Ingrese numero 2: ");
num2 = teclado.nextInt();
System.out.print("Ingrese rsultado: ");
resuNi = teclado.nextInt();
resuMaq = num1 - num2;
if (resuMaq == resuNi) {
System.out.print("La resta esta correcta");
} else {
System.out.print("La resta esta incorrecta");
}
System.out.print("¿Desea revisar otra resta (S/N)");
desea = teclado.next().charAt(0);
} while (desea == 'S');
}
}
Ahora la explicación del caracter..... primero con un chiste en inglés:
Como verás lo ideal es hacer tu código a prueba de usuarios, pueden agregar espacios.
por ello un primer intento de corregir eso podría ser
desea = teclado.nextLine().trim().charAt(0);
pero eso nos lleva a otro posible error..
por ejemplo para la corrida:
TE PUEDO AYUDAR A 1. SUMAR 2. RESTAR 3. FIN ESCOGER OPCION
1
Ingrese numero 1: 2
Ingrese numero 2: 2
Ingrese rsultado:
3
esto produciría un
java.lang.StringIndexOutOfBoundsException: String index out of range: 0
porque el caracter sería un \r\n
o un \n
Así que primeor validamos el tamaño y probamos de nuevo:
String linea = teclado.nextLine().trim(); desea = linea.length() > 0 ? linea.charAt(0) : 'N';
Ahora otro escenario que no hemos pensaod es que el usuario pueda apretar más de una tecla... por ejemplo poner bn
,as
, o incluso algo loco como sn
en tu caso, por ejemplo en la corrida:
TE PUEDO AYUDAR A 1. SUMAR 2. RESTAR 3. FIN ESCOGER OPCION 1 Ingrese numero 1: 2 Ingrese numero 2: 2 Ingrese rsultado: 3 La suma esta incorrecta¿Desea revisar otra suma (S/N) TE PUEDO AYUDAR A 1. SUMAR 2. RESTAR 3. FIN ESCOGER OPCION bn
Exception in thread "main" java.util.InputMismatchException
así que mejor olvídate de comparar con un caractér, comapremos cadenas.