0

El problema trata sobre realizar una funcion que reciba dos punteros char y dos valores char en los cuales los punteros representan a los jugadores y los valores char a el valor elejido lso cuales son: "p" para piedra, "h" para papel y "t" para tijera. La funcion debe decidir quien gana e infromarlos en los punteros a los jugadores con "w" para el que gane y "l" para el perdedro y por ultimo la funcion main debe de imprimir los resultados.

¿La pregunta es todo esto como puedo evitar que me salte el error de char to char asignarles los valores en main en la funcion?

Esto es lo que pude avanzar del codigo:

#include <stdio.h>


void juego(char *n1, char *n2, char nm1, char nm2)
{   
char win[] = "win";
char lost[] = "Lost";
 
if((nm1 == 'h') && (nm2 == 'p'))
{
    n1 = win;
    n2 = lost;  
    printf("\nJugador2 %c y jugador1 %c", *n2, *n1);    
} 
else if ((nm2 == 'h') && (nm1 == 'p'))
  {
    n1 = lost;
    n2 = win;   
    printf("\nJugador2 %c y jugador1 %c", *n2, *n1);
  } 
else if ((nm1 == 'p') && (nm2 == 't'))
  {
    n1 = win;
    n2 = lost;  
    printf("\nJugador2 %c y jugador1 %c", *n2, *n1);        
  }
else if ((nm2 == 'p') && (nm1 == 't'))
  {
    n1 = lost;
    n2 = win;   
    printf("\nJugador2 %c y jugador1 %c", *n2, *n1);        
  }
else if ((nm1 == 't') && (nm2 == 'h'))
  {
    n1 = win;
    n2 = lost;  
    printf("\nJugador2 %c y jugador1 %c", *n2, *n1);        
  }
else if ((nm2 == 't') && (nm1 == 'h'))
  {
    n1 = lost;
    n2 = win;   
    printf("\nJugador2 %c y jugador1 %c", *n2, *n1);        
  }
else if (nm2 == nm1)
  {
     printf("Empate");      
  }
 }

 int main()
 {   
 char *n1[5];
 char *n2[5];
 char nm1[1];
 char nm2[1];

 printf("Elija p = piedra, h = papel o t = tijera: ");
 scanf("%d", &nm1);

 printf("\nElija jugador2 p = piedra, h = papel o t = tijera: ");
 scanf("%d", &nm2);

 juego(n1, n2, nm1, nm2);
 }
4
  • ¿Cuál es la pregunta? el 16 jun. 2022 a las 14:04
  • 1
    Modifique el codigo y depaso le puse la pregunta.
    – Usuario
    el 16 jun. 2022 a las 14:16
  • ¿A qué te refieres con "el error de char to char"? el 16 jun. 2022 a las 14:18
  • Me refiero que en el prototipo de funcion lo tengo como puntero *char y en main lo tengo que definir devuelta para poder implementarlo en la funcion y lo defino como char en main el problema surge en que me salta el erorr de no se puede convertir un *char a char
    – Usuario
    el 16 jun. 2022 a las 14:27

1 respuesta 1

1

Nada más leer tu código se ven fallos evidentes:

  • Usa nombres auto-explicativos para saber qué hace cada variable. Puede que ahora mismo sepas lo que hace cada parte de tu código, pero si re-lees tu propio código pasado un tiempo te costará mucho entenderlo.

    En lugar de esto:

    char *n1[5];
    char *n2[5];
    char nm1[1];
    char nm2[1];
    

    Usa esto:

    char *jugador_1[5];
    char *jugador_2[5];
    char jugada_1[1];
    char jugada_2[1];
    
  • En las cadenas de caracteres se guardan en formaciones1 de caracteres, que son char formacion[tamaño]; en cambio tú has definido char *formacion[tamaño] que no es una formación de caracteres si no una formación de punteros a carácter.

    En lugar de esto:

    char *jugador_1[5];
    char *jugador_2[5];
    char jugada_1[1];
    char jugada_2[1];
    

    Usa esto:

    char jugador_1[5];
    char jugador_2[5];
    char jugada_1[1];
    char jugada_2[1];
    

    Ten en cuenta que una cadena de texto de cinco caracteres sólo puede almacenar hasta cuatro letras (pues se necesita un carácter para guardar la marca de final de cadena: '\0'), yo incrementaría el tamaño de las formaciones que guardan el nombre de cada jugador.

  • No necesitas una formación de un carácter para almacenar un carácter; usa directamente una variable de tipo char.

    En lugar de esto:

    char jugador_1[5];
    char jugador_2[5];
    char jugada_1[1];
    char jugada_2[1];
    

    Usa esto:

    char jugador_1[25];
    char jugador_2[25];
    char jugada_1;
    char jugada_2;
    
  • Para leer un carácter con la función de lectura desde consola scanf debes usar %c, si usas %d estarás leyendo números.

    En lugar de esto:

    char jugador_1[25];
    char jugador_2[25];
    char jugada_1[1];
    char jugada_2[1];
    
    printf("Elija p = piedra, h = papel o t = tijera: ");
    scanf("%d", &nm1);
    
    printf("\nElija jugador2 p = piedra, h = papel o t = tijera: ");
    scanf("%d", &nm2);
    

    Usa esto:

    char jugador_1[25];
    char jugador_2[25];
    char jugada_1;
    char jugada_2;
    
    printf("Jugador %s: Elija p = piedra, h = papel o t = tijera: ", jugador_1);
    scanf("%c", &jugada_1);
    
    printf("Jugador %s: Elija p = piedra, h = papel o t = tijera: ", jugador_2);
    scanf("%c", &jugada_2);
    
  • Defines la función juego para recibir dos punteros a carácter y dos caracteres, pero le provees dos punteros a puntero a carácter y dos punteros a carácter, esto provoca el error que describes.

    En lugar de esto:

    char *n1[5];
    char *n2[5];
    char nm1[1];
    char nm2[1];
    
    // ...
    
    juego(n1, n2, nm1, nm2);
    

    Usa esto:

    char jugador_1[25];
    char jugador_2[25];
    char jugada_1;
    char jugada_2;
    
    // ...
    
    juego(jugador_1, jugador_2, jugada_1, jugada_2);
    
  • En lugar de una larga cadena de if-else-if para definir la lógica de Piedra, Papel y Tijera, usa una tabla de resultados:

    // 0 empate, 1 ganar, -1 perder
    int resultado[3][3] = {
                    // Piedra | Papel | Tijera
    /* Piedra vs */ {    0,      -1,      1   },
    /* Papel  vs */ {    1,       0,     -1   },
    /* Tijera vs */ {   -1,       1,      0   } };
    

Siguiendo todos esos consejos tu código podría parecerse a:

#include <stdio.h>

void juego(char *jugador_1, char *jugador_2, char jugada_1, char jugada_2)
{
    // 0 empate, 1 ganar, -1 perder
    int resultado[3][3] = {
                        // Piedra | Papel | Tijera
        /* Piedra vs */ {    0,      -1,      1   },
        /* Papel  vs */ {    1,       0,     -1   },
        /* Tijera vs */ {   -1,       1,      0   } };

    int indice1 =
        (jugada_1 == 'p' ? 0 :
            (jugada_1 == 'h' ? 1 :
                (jugada_1 == 't' ? 2 : -1)));

    int indice2 =
        (jugada_2 == 'p' ? 0 :
            (jugada_2 == 'h' ? 1 :
                (jugada_2 == 't' ? 2 : -1)));

    if (indice1 == -1 || indice2 == -1)
    {
        printf("Jugada incorrecta");
        return;
    }

    switch (resultado[indice1][indice2])
    {
        case -1: printf("%s pierde\n", jugador_1); break;
        case 0: printf("%s y %s empatan\n", jugador_1, jugador_2); break;
        case 1: printf("%s pierde\n", jugador_2); break;
    }
}

int main()
{
    char jugador_1[] = "Sigerico";
    char jugador_2[] = "Chindasvinta";
    char jugada_1;
    char jugada_2;

    printf("Jugador %s: Elija p = piedra, h = papel o t = tijera: ", jugador_1);
    scanf("%c", &jugada_1);
    getchar();

    printf("Jugador %s: Elija p = piedra, h = papel o t = tijera: ", jugador_2);
    scanf("%c", &jugada_2);
    getchar();

    juego(jugador_1, jugador_2, jugada_1, jugada_2);

    return 0;
}

Puedes ver el código funcionando en Try it online!.


1También conocidas como arreglo o en inglés array.
2
  • Gracias por los conocimientos.
    – Usuario
    el 16 jun. 2022 a las 19:33
  • Lo mejor del primer punto es que creas código que se "explica solo". Por lo tanto necesitarás menos o incluso ningún comentario en tu código para que se entienda.
    – Mateo
    el 16 jun. 2022 a las 19:44

Tu Respuesta

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

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