0

Tengo estas 2 estructuras declaradas:

typedef struct {
    char *name;
    float quantity;
} tIngredient;

typedef struct {
    char *name;
    int id;
    float price;
    tIngredient *ingredients;
    int numIng;
} tBurguer;

Y luego en el main:

tBurguer b;
tIngredient i;

b.id = 0;
strcpy(b.name, "special");
b.ingredients=NULL;
b.numIng=0;
b.price=10.5;

strcpy(i.name, "lettuce");
i.quantity = 100;
add_ingredient (&b, i);

El código de add_ingredient() es el siguiente:

void add_ingredient (tBurguer *b, tIngredient i){
    
    if (strcmp(b->ingredients->name,i.name)!=0){
        b->numIng+=1;
        b->ingredients = (tIngredient*)realloc(b->ingredients, (b->numIng)*sizeof(tIngredient));
        strcpy (b->ingredients->name, i.name);
        b->ingredients->quantity = i.quantity;      
    } else {
        b->ingredients->quantity += i.quantity;
    }
}

Lo que intento hacer es comparar primero si el ingrediente que se encuentra en i.name es igual al que se encuentre en b.ingredients.name, para añadirlo si no existe o aumentar b.ingredients.quantity con lo que guarde i.quantity en caso de que ya exista. Y con el realloc quiero redimensionar el array manteniendo lo anterior para indicar que estamos añadiendo un nuevo ingrediente (en caso de que este no exista).

Mi duda es:

  1. El malloc dónde se haría, ya que ese es el main que me dan, y no tengo posibilidad de añadir ni borrar cosas. Si lo hago en la función, me va como a dimensionar cada vez que la llame, pero no creo que eso es lo que deba hacer siempre.

  2. Cómo podría acceder a un puntero a estructura dentro de otra. Creo que mi fallo viene por todo esto.

CORRECCIÓN Hice lo siguiente gracias a la ayuda de los comentarios:

  1. Los array de tipo char * de los structs los cambié a tamaño estático de 50, ya que interpreté mal el enunciado.

  2. En la función, reservé la memoria de esta forma:

    void add_ingredient (tBurguer *b, tIngredient i){

     if (b->numIng==0){
         b->ingredients = (tIngredient*)malloc(b->numIng*sizeof(tIngredient));
     } else if (strcmp(b->ingredients->name,i.name)!=0){
         b->numIng+=1;
         b->ingredients = (tIngredient*)realloc(b->ingredients, (b->numIng)*sizeof(tIngredient));
         strcpy (b->ingredients->name, i.name);
         b->ingredients->quantity = i.quantity;      
     } else {
         b->ingredients->quantity += i.quantity;
     }
    

    }

De esta forma, reservo memoria dinámica siempre y cuando el número de ingredientes sea 0, es decir, solo la primera vez, y las siguientes veces que llame a la función ya entrará en el else if puesto que ya será mayor de 0, por lo que uso realloc para que me mantenga la información anterior y me la amplíe en 1.

10
  • 1
    Se entiende que el campo tIngredient *ingredients; dentro de tBurger ha de ser un puntero a un array de ingredientes y no solo a un ingrediente. Cuando ese puntero es NULL es que no hay aún array, así que deberías crear uno con malloc() y darle el tamaño de un ingrediente. El acceso a un ingrediente particular sería con b->ingredients[0], b->ingredients[1], etc. Cada uno de esos sería ya un struct y no un puntero a struct, por lo que podrías hacer ` b->ingredients[0].name` por ejemplo. El realloc se haría sobre el puntero a array, para añadirle otro struct para otro ingrediente
    – abulafia
    el 28 feb. 2021 a las 20:31
  • 1
    Por otro lado, dentro del ingrediente, el nombre es un char*, es decir, otro puntero que debes inicializar antes de intentar copiar a él un nombre con strcpy(), o de lo contrario romperá. Puedes usar strdup() como alterativa a malloc()+strcpy(), ya que strdup() te crea la memoria necesaria a la vez que copia la cadena, y te devuelve el puntero a la copia.
    – abulafia
    el 28 feb. 2021 a las 20:33
  • Entonces, según he entendido, debería hacer malloc según número de ingredientes (como en el primer momento b->numIng es 0, reservará solo 1 espacio de memoria) y luego ya ir ampliando con realloc? De esta forma: el 1 mar. 2021 a las 1:13
  • void add_ingredient (tBurguer b, tIngredient i){ if (b->numIng==0){ b->ingredients = (tIngredient)malloc(b->numIngsizeof(tIngredient)); } else if (strcmp(b->ingredients->name,i.name)!=0){ b->ingredients = (tIngredient)realloc(b->ingredients, b->numIng*sizeof(tIngredient)); y ya abajo guardar los valores de las demás variables el 1 mar. 2021 a las 1:15
  • 1
    Veo en tu implementación que, si no entendí mal, añades un nuevo ingrediente cada vez que se invoca add_ingredient() si su nombre no coincide con el del primer ingrediente. Creí entender que el ingrediente se añade sólo si no estaba ya. Por tanto tendrías que recorrer el array de ingredientes para comparar con cada nombre, y sólo si no es igual a ninguno de ellos añadirlo. No estás usando sintaxis de array
    – abulafia
    el 1 mar. 2021 a las 20:06

0

Tu Respuesta

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

Examina otras preguntas con la etiqueta o formula tu propia pregunta.