Programa Ahorcado - Un error

[AlmaVagabunda]

Hola, soy nuevo por aqui como se daran cuenta.

Tengo que entregar un trabajo de programacion en C, lo tengo que hacer con el Qt.

El caso, es que llevo 3 dias volviendome loco (aun estoy aprendiendo, hace dos meses no sabia nada). El programa compila y ejecuta, pero me salen 3 errores. Y ademas, una instruccion que deberia hacer, no consigo que la haga.

En concreto es cuando un jugador acierta la palabra, deberia imprimir en pantalla que ha ganado y que si se juega otra partida, para que el otro jugador elija otra, pero no funciona, la palabra queda completada, pero sigue pidiendo letra... solo sale de ahi cuando fallas todos los intentos.

Ese es mi mayor problema.
Despues otro, que no me lo han pedido, pero me gustaria saber como ponerlo, es limitar al usuario que solo ponga letras a la hora de elegir la palabra a adivinar, y si pone numeros que avise de que no puede ser. Pero si hay que cambiar demasiado codigo, lo descarto, es simplemente algo que me hacia ilusion hacer.

Y el ultimo, que he dicho al principio, me salen 3 pequeños errores, y no consigo saber de que son.

He decidido publicar aqui mis dudas por que he visto que el foro es muy bueno, ya he solucionado algunas dudas haciendo visitas sin registrarme, pero bueno, ya me toco a mi ser el pregunton jajaja.

Aqui les dejo el codigo que tengo, espero que no sea mucha cosa, no me gusta molestar.
Un saludo, y muchas gracias a todos los que ayudan en el foro ;)

Código c:

//INCLUIMOS LAS LIBRERIAS NECESARIAS PARA QUE EL CODIGO SE EJECUTE CORRECTAMENTE.
 
#include <stdlib.h>
#include <stdio.h>
#include <conio.h>
#include <string.h>
 
int main(void) {
 
    //DECLARACION VARIABLES A UTILIZAR EN EL PROGRAMA.
    int i=5, j=0, puntosJug1 = 0, puntosJug2 = 0, turno = 1, medidaCad, centinela;
    char palabra[50], cadena[51];
    char jugar, letra;
 
    do {
        //CONTROLAMOS LOS TURNOS DE CADA JUGADOR MEDIANTE ESTE IF-ELSE.
        if (turno%2!=0) {
            //CONTROLAMOS EL DATO QUE INTRODUCE EL USUARIO.
            do {
                printf("Introduce la palabra a adivinar Jugador 1: ");
                scanf("%s", palabra);
                fflush(stdin);
                system("cls");
            }
 
            while (strlen(palabra) > 50);
            medidaCad = strlen(palabra);
            strcpy(cadena, "--------------------------------------------------");
            cadena[medidaCad] = '\0';
 
            do {
                //INICIALIZAMOS LA VARIABLE CENTINELA.
                centinela = 0;
                printf("Palabra: %s\n", cadena);
                printf("Intentos restantes: %d\n\n", i);
 
                do {
                    printf("Introduce una letra minuscula (Jug.2): ");
                    scanf("%c", &letra);
                    fflush(stdin);
                }
 
                while (letra < 'a' || letra > 'z');
 
                for (j=0;j<strlen(palabra);j++){
 
                    if (palabra[j] == letra)
                    {
                        cadena[j] = letra;
                        centinela = 1;
                    }
                }
 
                if (centinela != 1)i--;
                printf("\n\n");
            }
 
            while(i != 0 || cadena==palabra);
      //AQUI ES DONDE ESTA EL CODIGO QUE DEBERIA EJECUTARSE SI SE ACIERTA LA PALABRA, PARANDO , IMPRMIENDO LA FRASE Y SUMANDO 1 PUNTO AL JUGADOR. PERO NO CONSIGO QUE FUNCIONE
            if (cadena==palabra) {
                printf("Has acertado la palabra!!");
                puntosJug2++;
            }
 
            else puntosJug1++;
        }
 
        else{
            //INICIALIZAMOS LA VARIABLE CENTINELA.
            centinela = 0;
 
            //CONTROLAMOS EL DATO QUE INTRODUCE EL USUARIO.
            do {
                printf("Introduce la palabra a adivinar Jugador 2: ");
                scanf("%s", palabra);
                fflush(stdin);
                system("cls");
            }
 
            while (strlen(palabra) > 50);
            medidaCad=strlen(palabra);
            strcpy(cadena, "--------------------------------------------------");
            cadena[medidaCad]='\0';
 
            do{
                //INICIALIZAMOS LA VARIABLE CENTINELA.
                centinela = 0;
                printf("Palabra: %s\n", cadena);
                printf("Intentos restantes: %d\n\n", i);
 
                do{
                    printf("Introduce una letra minuscula (Jug.1): ");
                    scanf("%c", &letra);
                    fflush(stdin);
                }
 
                while (letra < 'a' || letra > 'z');
 
                for (j=0;j<strlen(palabra);j++) {
 
                    if (palabra[j]==letra) {
                        cadena[j]=letra;
                        centinela=1;
                    }
                }
 
                if (centinela != 1)i--;
                printf("\n\n");
            }
 
            while(i!=0 || cadena==palabra);
 
            if (cadena==palabra) {
                printf("Has acertado la palabra!!");
                puntosJug1++;
            }
 
            else puntosJug2++;
        }
 
        do{
            printf("Quereis jugar mas partidas? <S o N>: ");
            scanf("%c", &jugar);
            fflush(stdin);
        }
 
        while (jugar != 'S' && jugar != 's' && jugar != 'N' && jugar != 'n');
        //INCREMENTAMOS LA VARIABLE TURNO PARA QUE PREGUNTE AL OTRO JUGADOR.
        turno++;
        system("cls");
        //INICIALIZAMOS LOS VALORES QUE NECESITAMOS PONER EN SU ESTADO INICIAL.
        i=5;
        j=0;
    }
 
    while (jugar == 'S' || jugar == 's');
    printf("Puntuacion final: (Jug.1) %d - %d (Jug.2)\n", puntosJug1, puntosJug2);
 
    //FUNCIÓN QUE NOS PERMITE CONTROLAR LA SALIDA DEL PROGRAMA, DANDOLE A 1 TECLA.
    system ("pause");
    return(0);
}
 

[sam90]

Priemro de todo deberias cambiar : while(i != 0 || cadena==palabra);
por while(i!=0 && strcmp(cadena,palabra)!= 0);

Para comparar cadenas de caracteres hay que usar una funcion. Ya que el == usado asi solo compara direcciones de memoria. strcmp devuelve 0 si la camdena es igual.

Ademas necesitas usar el AND (&&). para chequer las dos condiciones.

Lo de los numeros ya esta chequeado, en este DO

Código C++:

do {
                    printf("Introduce una letra minuscula (Jug.2): ");
                    scanf("%c", &letra);
                    fflush(stdin);
}while (letra < 'a' || letra > 'z');
 

Por ultimo una sugerencia asi aprender mas, te recomendaria leer mas sobre arrays, asi evitas escribir dos veces el codigo, uno para cada jugador.

Saludos

[AlmaVagabunda]

Recien probado, y funciona correctamente. Muchisimas gracias.
Estaba mirando justo esa funcion para comparar, pero no me salia.
Gracias de nuevo!!
En los proximos dias que termine unos examenes, mirare sobre arrays.

Un gran abrazo! ;)

[AlmaVagabunda]

Perdon, se me olvido.

Lo de la letra se que ya lo tenia (ya me costo mi rato entenderlo jajaja)
Pero lo que necesito es que cuando se inicia el juego, que hay que escribir una palabra, varias letras, hacer que no acepte ningun numero o caracter especial aunque este mezclado con letras, solo un conjunto de letras.

No se si me explique, estoy euforico por haber arreglado esa tonteria :)

Es posible hacerlo sin mucha complicacion? tendre que aprender mas? jajaja

Un saludo

Edit:
Por cierto, los dos errores que me salen, son de los dos for que tengo en el codigo, y dice lo siguiente:
comparison between signed and unsigned integer expressions

¿Que significa?

Edit2:
Ya no me sale el error, no se por que, pero ya no sale :)
Edit3:
Debo de ser menos inocente... volvio el error grrr

[sam90]

Código C:

int sololetras(char * cadena)
{
     int i;
     for (i=0 ; i < strlen(cadena); i++ ) 
           if (cadena[i] < 'a' || cadena[i] > 'z' ) return 0;
     return 1;
}
 

y ahora modificas este linea while (strlen(palabra) > 50); por :

Código C:

while (strlen(palabra) > 50 && !sololetras(palabra) );
 

saludos

[AlmaVagabunda]

No consigo que funcione.

el:

Código C:

int sololetras(char * cadena)
 

Es la cabecera de la funcion, lo que seria el main, no?

Y el:

Código C:

{
     int i;
     for (i=0 ; i < strlen(cadena); i++ )
           if (cadena[i] < 'a' || cadena[i] > 'z' ) return 0;
     return 1;
}
 

Donde lo pongo? He probado justo antes del while, tambien justo despues del scanf, y no consigo que funcione, que estoy haciendo mal?

[sam90]

Asi no se declaran las funciones...no las tenes que cortar en dos.

Código C++:

int sololetras(char * cadena)
    {
         int i;
         for (i=0 ; i < strlen(cadena); i++ )
               if (cadena[i] < 'a' || cadena[i] > 'z' ) return 0;
         return 1;
    }
 
 
int main()
{
// resto del codigo
}
 

[AlmaVagabunda]

Soy muy pesado, lo se.
Pero no consigo que funcione, compila perfectamente, pero cuando meto numeros solos, o con letras, sigue ejecutandose en vez de volver a solicitar.

No se bien trabajar con modulos, y me pierdo un poco.
Se supone que la funcion sololetras, coje la cadena palabra, y mira caracter a caracter si hay numero, si lo hay devuelve un 0, no?
Y luego en el while

Código C++:

while (strlen(palabra) > 50 && !sololetras(palabra) );
 

lo de !sololetras(palabra) ¿esta bien?

Soy un poco terco jejeje
Muchas gracias por preocuparte sam90 ;)

Un abrazo!

[sam90]

Proba asi:

while (strlen(palabra) > 50 || 0==sololetras(palabra) );

El error fue mio.

[AlmaVagabunda]

Ahora si!! funciona perfecto.

Muchas gracias por todo! pense que nunca iba a acabarlo :)

Un abrazo