0

Estoy desarrollando un sencillo juego del 3 en línea.

Tengo un problemilla: cuando se consumen todos los movimientos y no hay un ganador, el programa no finaliza (como sería lo esperado), sino que continúa su ejecución.

No se como minimizar mas el código y hacer reproducible el problema.

#include <stdio.h>
int main()
{
    char t1 = '1',t2 = '2',t3 = '3',t4 = '4',t5 = '5',t6 = '6',t7 = '7',t8 = '8',t9 = '9';
    char turno = 'X';
    char casilla = ' ';
    int fin = 0;
    int error = 0;
    int contador = 0;

    printf(".-----+-----+-----.\n");
    printf("|     |     |     |\n");
    printf("|  %c  |  %c  |  %c  |\n",t7,t8,t9);
    printf("|     |     |     |\n");
    printf("|-----+-----+-----|\n");
    printf("|     |     |     |\n");
    printf("|  %c  |  %c  |  %c  |\n",t4,t5,t6);
    printf("|     |     |     |\n");
    printf("|-----+-----+-----|\n");
    printf("|     |     |     |\n");
    printf("|  %c  |  %c  |  %c  |\n",t1,t2,t3);
    printf("|     |     |     |\n");
    printf(".-----+-----+-----.\n");

    do{
        printf("Juega %c [1-9] " , turno);
        scanf("%s", &casilla);

        if(casilla=='1' || casilla=='2'|| casilla=='3'|| casilla=='4'
           || casilla=='5'|| casilla=='6'|| casilla=='7'|| casilla=='8'|| casilla=='9'){

            if(casilla == t1){
                t1 = turno;
            }else if (casilla == t2){
                t2 = turno;
            }else if (casilla == t3){
                t3 = turno;
            }else if (casilla == t4){
                t4 = turno;
            }else if (casilla == t5){
                t5= turno;
            }else if (casilla == t6){
                t6 = turno;
            }else if (casilla == t7){
                t7 = turno;
            }else if (casilla == t8){
                t8 = turno;
            }else if (casilla == t9){
                t9 = turno;
            }else{
                error = 1;
            }
        } else{
            error = 1;
        }

        if (error == 1){
            printf("ERROR!!!\n");
            error = 0;
        }else{
            if((t1==t2 && t2==t3)||(t4==t5 && t5==t6)||(t7==t8 && t8==t9)
                || (t1==t4 && t4==t7)||(t2==t5 && t5==t8)||(t3==t6 && t6==t9)
                ||( t1==t5 && t5==t9)||(t3==t5 && t5==t7)) {
                fin = 1;
            }else{
                if(turno == 'X'){
                    turno = 'O';
                }else{
                    turno = 'X';
                }
            }
        }
        printf(".-----+-----+-----.\n");
        printf("|     |     |     |\n");
        printf("|  %c  |  %c  |  %c  |\n",t7,t8,t9);
        printf("|     |     |     |\n");
        printf("|-----+-----+-----|\n");
        printf("|     |     |     |\n");
        printf("|  %c  |  %c  |  %c  |\n",t4,t5,t6);
        printf("|     |     |     |\n");
        printf("|-----+-----+-----|\n");
        printf("|     |     |     |\n");
        printf("|  %c  |  %c  |  %c  |\n",t1,t2,t3);
        printf("|     |     |     |\n");
        printf(".-----+-----+-----.\n");

        contador++;
    }
    while((fin != 1) && (contador <= 8));
    {
        if(fin==1)printf("GANADOR!!! %c\n" , turno);
            else printf("EMPATE!!! \n");
    }


}
11
  • 4
    Es buena práctica explicar un poco que hace tu código y además, poner comentarios en el código que ayuden al programador y a la gente que lea tu código a entender qué es lo que hace.
    – Héctor
    el 14 jun. 2017 a las 8:53
  • 2
    contador nunca se modifica por lo que la unica opcion de salir del bucle es que fin sea 1 por lo que siempre saldra GANADOR
    – Lithorell
    el 14 jun. 2017 a las 8:53
  • @Lithorell tal vez lo estoy mirando mal, pero yo veo que contador se modifica al final del do-while. el 14 jun. 2017 a las 8:58
  • @ROcas ¿Estás confundiendo el do-while, que es una sola estructura de control, con dos estructuras de control independientes, una do y una while (que por separado no existen en c++)? el 14 jun. 2017 a las 8:59
  • 2
    ¿Cómo que no puede haber un empate? Puede haberlo exactamente igual que existen las tablas en ajedrez. El juego llega a un estado en el que o bien no se pueden realizar más jugadas o bien resulta imposible que un jugador consiga ganar. Realmente no entiendo porque se ha cerrado la pregunta: el problema me parece claro, de hecho yo lo estuve mirando ayer y no encontré porqué no funcionaba. el 15 jun. 2017 a las 8:59

1 respuesta 1

3

Tienes 1 problema: Declaras

char casilla = ' ';

pero luego haces

scanf("%s", &casilla);

Es decir, internamente usas un char, pero lees una cadena. Toda cadena, como mínimo, te ocupará 2 bytes: para el carácter, y para el 0 final.

En cada lectura, te sales fuera del espacio reservado, pisando el valor de otras variables; ello produce comportamientos curiosos.

Para solucionarlo, basta con hacer

scanf("%c", &casilla);

con lo cual ya lees 1 solo byte. Ahora bien, esto presenta un problema extra: el ' \n' que pulsas se queda en el buffer de entrada, por lo que tenemos que eliminarlo. Una posible opción es cambiar tu formato de entrada por " %c", colocando un espacio delante de %c.

Así pues, la solución final: cambia

scanf("%s", &casilla);

por

scanf(" %c", &casilla);

Con ese sencillo cambio, todo funciona como debiera.

Tu Respuesta

Al pulsar en “Publica tu respuesta”, muestras tu consentimiento a nuestros términos de servicio, política de privacidad y política de cookies

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