[C] preblema di stack

[ggd94]

Salve a tutti! mi stavo cimentando in un esercizio C ne quale devo leggere da file degli ingredienti con la loro quantità e tipo (litri,grammi,unità) e sommare il peso in grammi di tutti gli ingredienti,ma all'avvio del programma mi rileva un errore di stack non riesco a capire perchè Vi ringrazio in anticipo.
questa è la main:

int main (){
    struct PesiSpecifici* pesi;
    int n_pesi=0;
    float  numero;
    pesi=(struct PesiSpecifici*)malloc(n_pesi*sizeof(struct PesiSpecifici));
    char* file="ex7-es9.txt";
    numero=PesoTotale(file,pesi,n_pesi);
    printf("il peso totale per il dolce è:%f",numero);
    free(pesi);
    return 0;
}

questo è il mio file.c

float PesoTotale(char nomefile[],struct PesiSpecifici pesi[],int n_pesi){
    FILE* file=fopen(nomefile,"r");
if(file==NULL){
printf("errore apertura file");
exit(1);
}
    char c1;
    float peso,peso1,totale=0;
    while(!feof(file)){
        n_pesi++;
        pesi=realloc(pesi,n_pesi*sizeof(struct PesiSpecifici));
        fscanf(file,"%s",&pesi[n_pesi-1].ingrediente);
        fscanf(file,"%c",&c1);
        fscanf(file,"%f",&peso);
        if(c1=='l'){
            peso1=peso*1000;
        }
        else if(c1=='u'){
            peso1=peso*60;
        }
        else{
            peso1=peso;
        }
        pesi[n_pesi-1].peso=peso;
        totale+=pesi[n_pesi-1].peso;
    }
    fclose(file);
    return totale;
}

e questo il file.h

Struct PesiSpecifici{
    char ingrediente[32];
    float peso;
};

float PesoTotale(char nomefile[],struct PesiSpecifici pesi[],int n_pesi);

[Scara95]

Scrivi esattamente che errore ottieni e magari specifica come è strutturato il tuo file. comunque quel codice è veramente mal organizzato e presenta alcuni problemi e alcune azioni inutili.

[Alex'87]

C'è un errore già all'inizio:

int n_pesi=0;
pesi=(struct PesiSpecifici*)malloc(n_pesi*sizeof(struct PesiSpecifici));

Non stai allocando lo spazio...

Il resto non l'ho guardato.

[Scara95]

C'è un errore già all'inizio:

int n_pesi=0;
pesi=(struct PesiSpecifici*)malloc(n_pesi*sizeof(struct PesiSpecifici));

Non stai allocando lo spazio...

Il resto non l'ho guardato.

C'è una realloc nella funzione per quello.

[Alex'87]

C'è una realloc nella funzione per quello.

ups, non ci avevo fatto caso (certo che fare una malloc a vuoto è abbastanza inutile )

[Scara95]

ups, non ci avevo fatto caso (certo che fare una malloc a vuoto è abbastanza inutile )

Va aggiunto che quel codice non tiene conto dei fatti che:

• le modifiche a pesi (realloc potrebbe copiare i dati in un nuovo spazio di memoria) e n_pesi non si riflettono nel main
• non è necessario memorizzare i dati
• malloc e realloc potrebbero restiruire NULL (vabbè, questo è spesso ignorato)

[ggd94]

grazie per avermi fatto notare alcune cose comunque per lo stack dovevo solamente mettere il free nel file.c e non nel main visto che non ho nessun puntatore nel main che punti a quelle zone di memoria

[oregon]

non ho nessun puntatore nel main che punti a quelle zone di memoria

Questo non è completamente vero dato che il puntatore iniziale è nel main e lo passi alla funzione.

A questo punto quel puntatore, la prima realloc e il suo passaggio è assolutamente inutile.

[Scara95]

grazie per avermi fatto notare alcune cose comunque per lo stack dovevo solamente mettere il free nel file.c e non nel main visto che non ho nessun puntatore nel main che punti a quelle zone di memoria

• le modifiche a pesi (realloc potrebbe copiare i dati in un nuovo spazio di memoria) e n_pesi non si riflettono nel main

Questo non è completamente vero dato che il puntatore iniziale è nel main e lo passi alla funzione.

A questo punto quel puntatore, la prima realloc e il suo passaggio è assolutamente inutile.

Ecco, appunto. Come dicevo quel codice è molto mal organizzato. Se devi presentarlo ad un docente è meglio che lo riscrivi sfruttando i suggerimenti ottenuti, altrimenti è meglio che lo riscrivi per capire come andrebbe strutturato correttamente.