[C] Passaggio puntatore e realloc in funzione

[salvodel]

Salve a tutti
avrei un piccolo problema nel realizzare una funzione. Il mio obiettivo è leggere un file di dimensioni non note. Per fare questo volevo realizzare una funzione che riceve in ingresso un puntatore e all'interno della funzione viene cambiata la dimensione del puntatore per memorizzare tutti i dati. Il problema è che facendo in questo modo all'esterno della funzione non riesco a vedere i dati corretti. La forma del funzione che vorrei è questa:

int lettura(int *numeri)
{
    ....
    numbers = (int*) realloc (numbers, count * sizeof(int));
    ...
}

int main()
{
    int * numbers = NULL;
    ...
    leggi(numbers)
    ....
}

In un thread precedente avevano provato ad illuminarmi ma non mi è chiaro come si aggiorna il puntatore. Qualcuno di buona fede potrebbe spiegarmelo in parole semplici?
Grazie

[wingman87]

Devi usare un puntatore a puntatore:

int lettura(int **numeri)
{
    ....
    *numeri = (int*) realloc (*numeri, count * sizeof(int));
    ...
}

int main()
{
    int **numbers = (int**) malloc (sizeof(int*));
    ...
    leggi(numbers)
    ....
}

Spero di aver messo giusti tutti gli asterischi... prova!

[khelidan1980]

ma guarda che se leggi con fopen ed fread mica devi sapere la dimensione del file!

Non ho mica capito che devi fare mi sa......

[salvodel]

Provo a postare quello che ho scritto con i suggerimenti:

int legge_turb(float **time, float **dati, float tMAX, float dati_medio)
{
	FILE *fdati;
	int count;

	count=0;
	if((fdati=fopen("turb.dat","r"))==NULL)
		printf("Impossibile aprire il file");
	else
	{
		printf("Lettura dati in corso\n");
		if(count==0)
		{
			/*Leggo la prima riga*/
			fscanf(fdati,"%f",&tMAX);
			fscanf(fdati,"%f",&dati_medio);
		}
		while(fscanf(fdati,"%f",&time[count])>0)qui è sicuramente sbagliato
		{
			fscanf(fdati,"%f",&dati[count]);
			count++;
			*time = (float*) realloc (*time, (count+1)*sizeof(float));
			*dati = (float*) realloc (*dati, (count+1)*sizeof(float));
		}
	}
	return 1;
}

Fuori dalla funzione come l'aggiorno?
Grazie

[wingman87]

Potresti spiegare cosa deve fare questa funzione? E come è strutturato il file che stai leggendo?

int legge_turb(float **time, float **dati, float tMAX, float dati_medio)
{
	FILE *fdati;
	int count;

	count=0;
	if((fdati=fopen("turb.dat","r"))==NULL)
		printf("Impossibile aprire il file");
	else
	{
		printf("Lettura dati in corso\n");
		if(count==0) //count sarà sicuramente uguale a zero, perché l'if?
		{
			/*Leggo la prima riga*/
			fscanf(fdati,"%f",&tMAX);
			fscanf(fdati,"%f",&dati_medio);
		}
		while(fscanf(fdati,"%f",&time[count])>0) //Senza & dovrebbe compilare
		{
			fscanf(fdati,"%f",&dati[count]);//Anche qui devi togliere l'&
			count++;
			*time = (float*) realloc (*time, (count+1)*sizeof(float));
			*dati = (float*) realloc (*dati, (count+1)*sizeof(float));
		}
	}
	return 1; //Ritorni 1 sia se abbia avuto successo che non, sicuro che sia corretto?
}

Queste sono correzioni al volo ma non so neanche cosa debba fare la funzione, quindi non ti so dire se questo basta.

[salvodel]

Originariamente inviato da wingman87

Potresti spiegare cosa deve fare questa funzione? E come è strutturato il file che stai leggendo?

Queste sono correzioni al volo ma non so neanche cosa debba fare la funzione, quindi non ti so dire se questo basta.

Prima di tutto grazie mille per l'aiuto.
La funzione deve leggere un file .dat di due colonne per un numero di righe non noto. Puoi trascurare quell'if perché è ridondante in effetti poiché cosi facendo legge sicuramente la prima riga. Quello che mi importa è che in time e dati metta la prima e la seconda colonna di dati. Questo funziona se faccio tutto nel main ma se passo i puntatori alla funzione e poi rialloco no. Da quello che ho capito dovrei tenere aggiornati i puntatori ma non mi è chiaro come. Cionci aveva provato a spiegarmelo nel thread che ho linkato ma non mi è chiaro. Ho solo un manuale di C in biblio e non è che ci sia molto quindi se riuscite a darmi una mano ve ne sarei grato.
Cmq ho provato a togliere quegli & di tropo e non va. Ottengo:
Unhandled exception at 0x00412519 in programma.exe: 0xC0000005: Access violation reading location 0x00000000.
in

while(fscanf(fdati,"%f",time[count])>0)

Grazie

[wingman87]

Perche' stai tentando di accedere (tra l'altro in scrittura) ad un'area di memoria che non hai ancora allocato. Il primo realloc lo dovresti fare prima del while

[salvodel]

Originariamente inviato da wingman87

Perche' stai tentando di accedere (tra l'altro in scrittura) ad un'area di memoria che non hai ancora allocato. Il primo realloc lo dovresti fare prima del while

Il problema è che io faccio un malloc fuori dalla funzione e poi passo quello. Sto provando tutte le cose piu stane che mi passano per la testa:
nel main

time =(float **)malloc(1*sizeof(float*));
dati =(float **)malloc(1*sizeof(float*));

int legge_turb(float **time, float **dati, float tMAX, float dati_medio)
{
	......
	while(fscanf(fdati,"%f",time[count])>0)
	{
	fscanf(fdati,"%f",dati[count]);
	count++;
	*time = (float*) realloc (*time, (count+1)*sizeof(float));
	*dati = (float*) realloc (*dati, (count+1)*sizeof(float));
	}
}

Sto cercando su internet esempi per capire meglio il funzionamento...ma niente. Aspetto suggerimenti. Grazie.

[wingman87]

Originariamente inviato da salvodel

Il problema è che io faccio un malloc fuori dalla funzione e poi passo quello. Sto provando tutte le cose piu stane che mi passano per la testa:
nel main

time =(float **)malloc(1*sizeof(float*));
dati =(float **)malloc(1*sizeof(float*));

Allora, con questo hai allocato lo spazio per due puntatori a float (o ad un array di float), e va bene, tuttavia non è stato ancora allocato alcun array, io ti sconsiglio di farlo nel main, è meglio che a questo ci pensi la funzione che legge il file.

Mi sono anche accorto che volendoti spiegare nel dettaglio come funzionano le cose ti ho fatto scrivere un codice un po' confusionario, almeno nel main, riprendendo il primo esempio che ti ho fatto, avevo scritto:

int lettura(int **numeri)
{
    ....
    *numeri = (int*) realloc (*numeri, count * sizeof(int));
    ...
}

int main()
{
    int **numbers = (int**) malloc (sizeof(int*));
    ...
    leggi(numbers)
    ....
}

Ma questo è equivalente a:

int lettura(int **numeri)
{
    ....
    *numeri = (int*) realloc (*numeri, count * sizeof(int));
    ...
}

int main()
{
    int *numbers;
    ...
    leggi(&numbers)
    ....
}

Io preferisco questa, se non altro perché alla fine non devo fare una free per liberare il puntatore.

int legge_turb(float **time, float **dati, float tMAX, float dati_medio)
{
	......
	while(fscanf(fdati,"%f",time[count])>0)
	{
	fscanf(fdati,"%f",dati[count]);
	count++;
	*time = (float*) realloc (*time, (count+1)*sizeof(float));
	*dati = (float*) realloc (*dati, (count+1)*sizeof(float));
	}
}

Sto cercando su internet esempi per capire meglio il funzionamento...ma niente. Aspetto suggerimenti. Grazie.

Dunque questo codice diventa:

int legge_turb(float **time, float **dati, float tMAX, float dati_medio)
{
	......
	while(!feof(fdati))
	{
	   *time = (float*) realloc (*time, (count+1)*sizeof(float));
	   *dati = (float*) realloc (*dati, (count+1)*sizeof(float));
	   fscanf(fdati,"%f",time[count])
	   fscanf(fdati,"%f",dati[count]);
	   count++;
	}
}

[salvodel]

Qui ho riassunto un po quello che mi hai detto. Spero di non aver fatto un mix di !!

#include <stdlib.h>
#include <stdio.h>

int legge(float **dati);

int main()
{
	int i;
	float *dati;

	legge(&dati);
	
	for(i=0;i<20;i++)
		printf("%f\n",dati[i]);
		
	return 1;
}
	

int legge(float **dati)
{
	FILE *fdati;
	int count=0;
	fdati=fopen("dati.dat","r");
	while(!feof(fdati))
	{
	   *dati = (float*) realloc (*dati, (count+1)*sizeof(float));
	   fscanf(fdati,"%f",dati[count]);
	   count++;
	}
}

Se non sei al mare e il cervello non si è sciolto come il mio con l'afa, potresti dargli un'occhaita?
Grazie

[wingman87]

Purtroppo sono a casa e mi sto sciogliendo lo stesso...
Abbiamo fatto un po' di errori tutti e due (scusa), ti posto il sorgente corretto:

#include <stdlib.h>
#include <stdio.h>

int legge(float **dati);

int main()
{
	int i;
	float *dati=NULL; //Va inizializzato a NULL per non mandare in tilt la realloc (in alternativa puoi fare una malloc prima delle realloc), errore mio

	legge(&dati);

	for(i=0;i<20;i++)
		printf("%f\n",dati[i]);
	free(dati); //Ricordati di liberare la memoria quando hai finito

	return 1;
}


int legge(float **dati)
{
	FILE *fdati;
	float *temp;
	int count=0;
	fdati=fopen("dati.dat","r");
	if(!fdati){  //Controlliamo l'esito dell'apertura
            printf("Errore nell'apertura del file\n");
            return 0;
        }
	while(!feof(fdati))
	{
	   temp = (float*) realloc (*dati, (count+1)*sizeof(float));//Errore mio, bisognerebbe sempre controllare l'esito delle operazioni (per questo ho aggiunto la variabile temp)
	   if(!temp){
	        printf("Errore nella riallocazione\n");
            return 0;
       }
	   *dati=temp;
	   fscanf(fdati,"%f",*dati+count); //Errore mio
	   count++;
	}
	fclose(fdati); //Ricordati di chiudere il file
	return 1;
}

Ora dovrebbe essere tutto ok

[salvodel]

Originariamente inviato da wingman87

Purtroppo sono a casa e mi sto sciogliendo lo stesso...
Abbiamo fatto un po' di errori tutti e due (scusa), ti posto il sorgente corretto:
Ora dovrebbe essere tutto ok

Perfetto! Grazie anche per i vari suppellettili! Non li avevo messi semplicemente per tagliare corto sul passaggio del puntatore.
Di nuovo grazie mille. Ora me lo confeziono per bene per tutti i programmini in cui mi serve aprire un file dati. Spero di non avere ulteriori problemi.
Ciao