13

Pregunta al usuario si quiere seguir jugando, toma la primera letra de la respuesta, y si es una "s" o una "S" el booleano da TRUE, pero por algún motivo si en vez de escribir el usuario SI o si escribe "seguir" devuelve FALSE y sale. La idea es que aunque responda algo con espacios, sólo tome la primera letra.

la idea es permitir: SI, si, seguir, Seguir, s (cualquier palabra que comience por la letra "S" o "s").

System.out.println("¿Volver a jugar?(si/no): ");
volverjugar = teclado.nextLine();
String primera=volverjugar.substring(0, 1);
boolean seguir;
seguir = (primera == "s" || primera == "S");
System.out.println(seguir);
4
  • 4
    que tal si haces seguir = primera.toUpperCase().equals("S") Commented el 28 nov. 2019 a las 12:13
  • En Java tienes que usar equals para comparar cadenas. Intenta así: seguir = ( primera.equals("s") || primera.equals("S") );
    – A. Cedano
    Commented el 28 nov. 2019 a las 12:13
  • 1
    cualquier palabra que comience por la letra "S" o "s" ¿Salmon?¿submarino?. Por que no comparas el String completo con .equalsIgnoreCase()? Relacionado: ¿.toUpperCase() o .equalsIgnoreCase() en variables tipo char?
    – x3k
    Commented el 28 nov. 2019 a las 12:25
  • Salmón y submarino también serían true. xD no pongo yo las condiciones.
    – ret
    Commented el 28 nov. 2019 a las 12:40

3 respuestas 3

16

En Java tienes que usar equals para comparar si el contenido de dos cadenas son iguales. El operador == sólo será verdadero si los objetos son iguales, o sea, si apuntan a la misma referencia.

De este modo, tu código debería funcionar:

seguir = ( primera.equals("s") || primera.equals("S") );

Como dice @JackNavaRow, puedes hacer una comparación única convirtiendo el dato a mayúscula.

O, si quieres simplificar, dado que sólo te interesa comparar la primera letra, puedes usar charAt() combinando con Character.toUpperCase, en ese cado, sí tienes que usar == porque sería una comparación de tipos primitivos (atención aquí al uso de comillas simples para 'S':

System.out.println("¿Volver a jugar?(si/no): ");
volverjugar = teclado.nextLine();
boolean seguir = Character.toUpperCase(volverjugar.charAt(0))=='S' ;
System.out.println(seguir);

O, como ha dicho @x3k_js, puedes recurrir a equalsIgnoreCase. En este ejemplo, usamos charAt para obtener sólo el primer carácter, lo convertimos a String para luego poder hacer una comparación basada en equalsIgnoreCase:

volverjugar = teclado.nextLine();
String primera = String.valueOf(volverjugar.charAt(0));
boolean seguir=(primera.equalsIgnoreCase("S"));
System.out.println(seguir);
2
  • gracias, pero me da en esa tercera linea del charAt "char cannot be converted to String". ¿Debe ser Char primera?
    – ret
    Commented el 28 nov. 2019 a las 12:33
  • 2
    @alderson puedes declararla del tipo Character y funcionará. Revisa el código editado.
    – A. Cedano
    Commented el 28 nov. 2019 a las 12:44
7
 System.out.println("¿Volver a jugar? (si/no)");
 String volverjugar = teclado.nextLine(); 

 boolean seguir = false;
 if(volverjugar.startsWith("S") || volverjugar.startsWith("s") ){ 
   seguir = true;
 }
4
  • la idea es permitir: SI, si, seguir, Seguir, s (cualquier palabra que comience por la letra "S" o "s").
    – ret
    Commented el 28 nov. 2019 a las 12:14
  • Ya lo tienes, solo necesitas saber si contiene la "s" o la "S"
    – EduBw
    Commented el 28 nov. 2019 a las 12:19
  • 1
    Con el contains compruebas que la palabra contenga la cadena, y esto no es lo que el OP quiere, en todo caso deberías de usar volverjugar.startsWith("S") para comprobar que la cadena comience con
    – x3k
    Commented el 28 nov. 2019 a las 13:42
  • Bien visto,gracias.
    – EduBw
    Commented el 28 nov. 2019 a las 15:03
5

lo que pasa es que estas comparando un String con otro y como dicen los colegas es con equals.

Si queres hacerlo con == podes comparar con char :

System.out.println("¿Volver a jugar?(si/no): ");
volverjugar = teclado.nextLine();
char primera=volverjugar.charAt(0); // charAt retorna el chat en la posicion n.
boolean seguir;
seguir = (primera == 's' || primera == 'S'); // Notese las comillas simples.
System.out.println(seguir);

Espero que te sirva , abrazo.

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.