Java - Tengo un problema / bug con un juegecito...

   
Vista:

Tengo un problema / bug con un juegecito...

Publicado por Ismael (14 intervenciones) el 23/11/2015 12:18:17
Estimados amigos programadores:
Tengo un pequeño problema ya desde hace unas semanas... La verdad es que estoy haciendo un juego a lo ahorcado y me sale todo el rato un fallo o un bug cuando el programa pregunta, diga una letra para adivinar y yo pongo por ejemplo de la palabra DISCO pongo la D y me muestra D----, e incluso imprime un guion mas nose porque pero lo imprime, y si tiene mas de una letra de esas pondria ejemplo: de MONITOR si pongo la O deberia poner -O---O- pero el problema está en que siempre saca un error que pone: :huh: :huh: :huh:
Exception in thread "main" java.lang.StringIndexOutOfBoundsException: String index out of range: -1
at java.lang.String.substring(Unknown Source)
at ej01.ahorcado.main(ahorcado.java:72)
Y me jode porque estaba funcionando (mas o menos) y sale este error tan extraño
lo que me gustaría que me ayudasen a detectar el problema, soy novato en programación y no me vendría nada mal un empujoncito vuestro... Lo que quiero es que me vaya guardando las letras en la cadenavacia y a medida que me vaya preguntando pueda ir completando la palabra hasta el final... Si fuesen ustedes tan amables de echarme una mano lo estaré muy agradecido... Aqui les dejo el codigo... Gracias por todo, anticipadamente ;D

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
package ej01;
 
import java.util.*;
 
public class ahorcado {
 
 public static void main(String[] args) {
               int intentos,dato1,c=0,i,t,longit,longitud,l,cont1=0;
  char letra;
  String palabra="",letrapulsada,cadenavacia2="",cadenavacia="";
  String lista="PERRO LAPIZ ORDENADOR TECLADO PROCESADOR SISTEMA INTERFAZ UNIX PROGRAMA DISCO CONECTOR CABLEADO ALMOHADILLA VENTANA JERSEY CAMISETA PANTALON RECREO MONITOR BOCADILLO";
  Scanner teclado = new Scanner(System.in);
  Random azar =new Random();
     dato1=azar.nextInt(20)+1; //genera numero al azar de 1 a 20 para contar las 20 palabras que hay
 
     for(i=0; i<lista.length(); i++){ //recorremos la cadena para buscar nuestra palabra escogida por el numero al azar dado
 
           letra = lista.charAt(i); //Vamos contando letra a letra y guardandolo en la variable letra
 
            if(letra == ' ') { //si se encuentra con un espacio que sume 1 al contador 
                c++; //especifica porqué palabra del ciclo va
            }
            if(c == dato1) { //y cuando el contador coincida con el numero generado al azar que especifica la palabra que ha elegido
 
                palabra=palabra+lista.charAt(i+1); // que coja el carácter por el que va (+1 para que no coja el espacio), que es el que coincide con la palabra elegida y con el ciclo que se complete dicha palabra guardandolo en la variable palabra
            }
        }
 
 
        System.out.println(palabra);
        do{
         for(intentos=1;intentos<=10;intentos++){
          cadenavacia="";
             cadenavacia2="";
        System.out.println("Introduce una letra para adivinar la palabra: ");
        letrapulsada=teclado.nextLine();
        letrapulsada=letrapulsada.toUpperCase();
 
 
        if(palabra.indexOf(letrapulsada) >=0){ //si la letra que ha pulsado coincide con la letra pulsada por el usuario
 
         for(t=0;t<palabra.length();t++){ //para ir contando las letras
 
 
          if(!letrapulsada.equals(palabra.charAt(t))){ //que ponga la cadena en guiones
           cadenavacia=cadenavacia+"-";
 
           }
 
         }
 
        }
 
 
         longit=palabra.indexOf(letrapulsada); //posicion de esa letra
         cadenavacia=cadenavacia.substring(0,longit)+letrapulsada+cadenavacia.substring(longit,cadenavacia.length()-1);
      //Lo que hacemos es concatenar desde el inicio hasta la longitud de esa palabra y alli ponemos la letra, a continuacion hasta el final de la palabra 
         System.out.println(cadenavacia);
 
 
          longit=palabra.indexOf(letrapulsada);
       longitud=palabra.indexOf(letrapulsada, longit+1);
             cadenavacia2=cadenavacia.substring(0,longitud)+letrapulsada+cadenavacia.substring(longitud,cadenavacia.length()-1);
             System.out.println(cadenavacia2);
             System.out.println("Estas en el intento "+intentos);
 
         }
        }while(intentos>=10);
 
 }
}
Valora esta pregunta
Me gusta: Está pregunta es útil y esta claraNo me gusta: Está pregunta no esta clara o no es útil
0
Responder