[C] Lettura di un file binario

[toni00c]

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

struct persona {
	int numero ;
	char *nome ;
	char cognome [6] ;
} ;

typedef struct persona persona ;

int main ()


{
   FILE *Jtr ;

   persona persona ;
    
	if ( (Jtr = fopen ("persona.dat" , "wb" ) ) != NULL )

	{

        /* inizializzo la struttura */

	persona.numero = 1 ;
	persona.nome = "tony" ;
	gets (persona.cognome) ;


        fwrite (&persona  , sizeof (struct persona  ) , 1 ,  Jtr ) ;
	fprintf (stdout , "%s\n" , persona.nome ) ;
	}
  


	fclose (Jtr ) ;
	system ("PAUSE") ;

	return 0 ;
}

quando vado per leggerla con questo codice :

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

struct persona {
	int numero ;
	char *nome ;
	char cognome [6] ;
} ;

typedef struct persona persona ;

int main ()


{
	
   FILE *Jtr ;

    persona persona ;


    
	if ( (Jtr = fopen ("persona.dat" , "rb" ) ) != NULL )
	{

	fread (&persona  , sizeof (struct persona  ) , 1 ,  Jtr ) ;

	fprintf (stdout , "%d%s\n" , persona.numero , persona .cognome   ) ;
        fprintf (stdout , "%s\n", persona .nome   ) ; /* questa e la riga che mi crea problemi */
	

	}
  


	fclose (Jtr ) ;
	system ("PAUSE") ;

	return 0 ;
}

mi da errore di allocazione della memoria ;

qualcuno sa spiegarmi il perchè ?
grazie

[lolide]

struct persona {
int numero ;
char *nome ;
char cognome [6] ;
} ;

Questa struttura ha un problema.
Il campo nome non è un array statico di char, ma è un puntatore, quindi lo devi inizializzare con malloc o crearne uno statico con char nome[dimensione]

persona.nome = "tony" ;

Nella lettura t'è andata bene, perchè ogni stringa che scrivi, viene allocata nella string table dell'eseguibile, quindi hai copiato l'indirizzo della stringa in quello di nome, ma nel momento in cui passi un array non allocato a fread ti da errore di allocazione.

[LeleFT]

Prendi visione del Regolamento interno, per conoscere le norme di partecipazione al forum.

In particolare, il codice va postato indentato e all'interno degli appositi tag CODE.

Sistemo io.


Ciao.

[toni00c]

ciao ti ringrazio molto , ma ancora non capisco del tutto il problema


ho provato a modificare così

                .............................................
                fread (&persona  , sizeof (struct persona  ) , 1 ,  Jtr ) ;
		persona.nome = (char*) malloc ( sizeof (char) * 6 ) ;
		 
		fprintf (stdout , "%s\n" , persona .cognome   ) ;
                fprintf (stdout , "%s\n", persona .nome   ) ;
.................................

ma mi visualizza a schermo tutto fuorchè la stringa che avevo creato
dov'è che sbaglio?
grazie

[lolide]

Originariamente inviato da toni00c
ciao ti ringrazio molto , ma ancora non capisco del tutto il problema


ho provato a modificare così

                .............................................
                fread (&persona  , sizeof (struct persona  ) , 1 ,  Jtr ) ;
		persona.nome = (char*) malloc ( sizeof (char) * 6 ) ;
		 
		fprintf (stdout , "%s\n" , persona .cognome   ) ;
                fprintf (stdout , "%s\n", persona .nome   ) ;
.................................

ma mi visualizza a schermo tutto fuorchè la stringa che avevo creato
dov'è che sbaglio?
grazie

l'allocazione la devi fare prima.

o cambi la struttura così

struct persona { int numero ; char nome[10] ; char cognome [6] ; } ;

oppure prima di fread fai malloc

[toni00c]

ciao ti ringrazio molto per il chiarimento , ma nel mio programma c'è qualcosa che non va
nella prima parte ho dichiarato e inizializzato nella struct un puntatore a una stringa
e creato un file di nome Persona.txt

persona.nome = "tony" ;

in un programma separato di lettura ho inizializzato la memoria prima di fread , ma
il risultato è il medesimo errore ;

if ( (Jtr = fopen ("persona.txt" , "rb" ) ) != NULL )
	{

		

	        persona.nome = (char*) malloc ( sizeof (char) * 10 ) ;
		fread (&persona  , sizeof (struct persona  ) , 1 ,  Jtr ) ;
		
		
	
		
		 
		fprintf (stdout , "%s\n" , persona .cognome   ) ;
        fprintf (stdout , "%s\n", persona .nome   ) ;
	
}
  .......................................

"Unhandled exception at 0x6514984f (msvcr90d.dll) in lettura.c.exe: 0xC0000005: Access violation reading location 0x00f55758"

ma dove può essere l'errore?

[shodan]

L'errore è qui:

struct persona {
	int numero ;
	char *nome ;
	char cognome [6] ;
} ;

cioè che usi un puntatore e poi serializzi la struttura.
Quando vai a scrivere, non scrivi il contenuto di nome, ma il valore del puntatore.
Allo stesso modo quando vai a leggere, assegni a quel puntatore 4 bytes presi nel file che non sono quelli che vorresti. E tutto questo è indipendente da una allocazione tramite malloc.
Se vuoi serializzare la struttura con:

fwrite (&persona  , sizeof (struct persona  ) , 1 ,  Jtr ) ;
fread (&persona  , sizeof (struct persona  ) , 1 ,  Jtr ) ;

quel campo deve avere dimensione fissa.

struct persona {
	int numero ;
	char nome[10] ;
	char cognome [6] ;
} ;

[toni00c]

ciao ti ringrazio molto , finalmente ho capito !

una chiarezza estrema

ma ho un ulteriore domanda che penso possa essere interessante :

struct titolo {
	int codTitolo ;
	float ap ;
	float ch ;
	float valore ;
} ;

typedef struct titolo titolo ;
typedef titolo *titoloPtr ;

int main ()

{

	FILE *Ptr ;

	titoloPtr result ;
	int codTitoli [10] = { 22 , 12 , 10 , 10 , 22 , 12 , 15  , 30 ,    30 ,  15 } ;
	float valori [10]  = { 100 , 202.2 , 300.2 , 123.4 , 21.22 , 124.66 , 938.34 , 42.3 , 333.4 , 1.25 } ;
	int dim = 10 ;
	int i ;
	int j ;
	int b ;
	int resultDim  = 0 ;
	int key   ;



	result = (titolo*)  malloc (sizeof (struct titolo ) * (dim/2) ) ;



............................................................


if ((Ptr =fopen ("titolo.txt" , "wb") ) != NULL )

{
	for ( b = 0 ; b < resultDim ; b++)
		fwrite ( &result , sizeof ( struct titolo ) , 1 , Ptr ) ;
}

.......................

in questo caso scrivo un vettore di dimensione ( dim/2) dinamico sul file "titolo.txt" ;

presumo che andrei incontro allo stesso identico problema non è vero?

Insomma chiariamo una volta per tutte cosicchè anche altre persone possano trarre giovamente da questo topic :

posso scrivere "dati di dimensione dinamica" per poi successivamente leggerli ?

[ramy89]

Alla fine puoi scrivere qualsiasi tipo di dato,purchè dopo tu sappia come leggerli.
Un metodo è quello di utilizzare un carattere di riconoscimento per separare tutti gli elementi.
Altrimenti devi sapere la dimensione.
Comunque è quello che fai tutte le volte che usi funzioni come scanf o printf,in fondo queste funzioni leggono finchè non trovano il carattere newline,che inserisci premendo invio.
Alla fine almeno per scelta personale io non mi preoccuperei tanto di come scrivere i dati sul file,tanto in qualsiasi tipo di dato,che sia un float o un carattere non inconterai mai difficoltà se ha cifre variabili,basta che sai quando smettere di leggere,cioè quando incontri il carattere separatore.
Le funzioni come fwrite e fread le uso solo per leggere una gran quantità di elementi più velocemente,per il resto ho la funzione apposita per leggere da file e allocare dinamicamente l' array.

[lolide]

La faccenda è questa:

size_t fread ( void * ptr, size_t size, size_t count, FILE * stream );

fread non conosce il formato del buffer che gli passi. Lui vede quel buffer solamente come un insieme di size bytes.

La funzione legge tot byte dal file e copia byte-byte il buffer letto nel buffer che gli passi.

Se tu passi una struttura de genere:

struct anag {
    char* nome;
    char cognome[10],
};

anche se poi allochi nome quel char* nome è sempre una variabile long (ovvero mantiene in se un indirizzo di memoria di 4 byte) e quella struttura è quindi di 14 byte.

Se tu scrivi sul file questa stringa "Dante Alighieri", quando l'andrai a leggere con fread in quella struttura, la funzione scriverà byte per byte, quindi avrai una struttura danneggiata che conterrà al posto di char* nome char nome[4], metterà "e " nei primi due posti di cognome e poi troncherà "Alighieri".

quando vuoi leggere 20 byte da un file binario, devi passargli precisamente una struttura di 20 byte.