Ejercicio 4 Guía 8

[HoltzTomas]

Dos dudas puntuales sobre el uso de structs (Dejo el código completo abajo por contexto)

• Creo haber visto ejemplos donde se asignan los valores a un struct atributo por atributo, acá lo hice pasándole el struct directo y parece funcionar, pero no si es correcto hacerlo o puede fallar de alguna forma.

mazo[i] = (Carta){figuras[i % 13], palo};

• La segunda más que de Structs era de sí está bien planteado haber creado los vectores de strings figuras y palos de forma global como lo hice, no se me ocurría otra forma para dejar a figura y palo de cada Carta como un puntero a char.

Desde ya gracias, espero estén pasando un viernes por la noche tan divertido como el mío.

#define CARTAS 52
#define PALOS 4
#define SWAP_STEPS 52

char *figuras[] = {
    "A", // As
    "2",
    "3",
    "4",
    "5",
    "6",
    "7",
    "8",
    "9",
    "10",
    "J", // Jota
    "Q", // Reina
    "K"  // Rey
};
char *palos[] = {"Corazones", "Diamantes", "Tréboles", "Picas"};

typedef struct
{
    char *figura;
    char *palo;
} Carta;

typedef Carta Mazo[CARTAS];

int swap(int a, int b);
int randInt(int izq, int der);
void generarMazo(Mazo mazo);
void generarPalo(Mazo mazo, char *palo, unsigned indice_inicio);
void mezclarMazo(Mazo mazo);
void swapCartas(Carta *carta1, Carta *carta2);
void printCartas(Mazo mazo, unsigned cant);
void buscarConfigs(Mazo mazo, int configs[3]);

int main(void)
{

    srand(time(NULL));

    // Primero crear el mazo y mezclarlo
    Mazo mazo;
    mezclarMazo(mazo);

    // Primer posicion pares, segundo piernas, tercero poker
    int configuraciones[3] = {0};
    buscarConfigs(mazo, configuraciones);

    printf("PAR\tPIERNA\tPOKER\n");
    for ( int i = 0; i < 3; i++ ) {
        printf("%d\t", configuraciones[i]);
    }
    puts("");
}

void buscarConfigs(Mazo mazo, int configs[3]){
    int conteo[13] = {0}; // Arreglo para contar cuántas veces aparece cada figura

    for (int i = 0; i < 5; i++)
    {
        int cartaEncontrada = 0;
        for (int j = 0; j < 13 && !cartaEncontrada; j++)
        {
            if (strcmp(mazo[i].figura, figuras[j]) == 0)
            {
                conteo[j]++;
                cartaEncontrada++;
            }
        }
    }

    for (int i = 0; i < 13; i++)
    {
        switch (conteo[i])
        {
        case 2:
            configs[0]++;
            break;
        case 3:
            configs[1]++;
            break;
        case 4:
            configs[2]++;
            break;
        default:
            break;
        }
    }
}

void mezclarMazo(Mazo mazo)
{

    generarMazo(mazo);

    for (int i = 0; i < SWAP_STEPS; i++)
        swapCartas(&mazo[i], &mazo[randInt(0, CARTAS - 1)]);

    printCartas(mazo, 5);
}

void printCartas(Mazo mazo, unsigned cant)
{
    for (int i = 0; i < cant; i++)
        printf("%s\t%s\n", mazo[i].palo, mazo[i].figura);
    puts("");
}

void swapCartas(Carta *carta1, Carta *carta2)
{
    Carta aux = *carta1;
    *carta1 = *carta2;
    *carta2 = aux;
}

void generarMazo(Mazo mazo)
{
    generarPalo(mazo, palos[0], 0);
    generarPalo(mazo, palos[1], CARTAS / PALOS);
    generarPalo(mazo, palos[2], 2 * (CARTAS / PALOS));
    generarPalo(mazo, palos[3], 3 * (CARTAS / PALOS));
}

void generarPalo(Mazo mazo, char *palo, unsigned indice_inicio)
{
    for (int i = indice_inicio; i < indice_inicio + CARTAS / PALOS; i++)
        mazo[i] = (Carta){figuras[i % 13], palo};
}

[marcelogarberoglio]

Esta forma mazo[i] = (Carta){figuras[i % 13], palo}; está bien. S;olo hay que respetar el orden en que se definieron los campos. También se puede hacer como se mostró en la clase teórica

tipoPasaje pasaje;
...
pasaje = (tipoPasaje) {
       .aerolinea = "Flybondi",
       .fecha = "20240619",
       .vuelo = 905,
       .hora = "20:30"};

donde se indican los campos, entonces no importa en qué orden se definieron

En cuanto a la implementación, por el uso de vectores globales, se puede aceptar', ya que sólo son descriptores constantes. Lo que no aceptaríamos son variables globales, donde las mismas se modifiquen en distintas funciones. Una alternativa para no usar estos vectores globales es que en el struct sólo tengas enteros, por ejemplo un enum para los palos y el valor de la carta. Y los vectores de strings los definas como static únicamente en la función que se encargue de imprimir una carta. Esto es, en vez de guardar figuras[i%13] en el struct guardás i%13 (y si en vez de 13 como magic number usás un #define mejor aún)

Cada cual se divierte como quiere o puede.

[HoltzTomas]

Muy buena la de modificar el struct y guardar enteros.

Bien, esa forma de asignar un valor al pasaje queda bastante más clara también.

Gracias nuevamente, vuelvo a divertirme con lo que queda de la 8. Seguro algún issue más cae