[c] Modifica file.. WTF!

[fenix-69]

Sono almeno 3 anni che combatto col C, ma è tutto il pomeriggio che non riesco a venire a capo di questo banalissimo algortmo di modifica di un file

void modifica(){
	studente stud;
	int i=0,j=0,risp;
	fp=fopen("PrimoAn.dat","rb");
	char tmp[10];
	if(fp!=NULL){
		printf("Inserisci il cognome dello studente che vuoi modificare: ");
		scanf("%s",tmp);
		printf("Elementi corrispondenti:\n\n");
		fread(&stud,sizeof(studente),1,fp);
		while(!feof(fp)){
			if(strcmp(tmp,stud.cognome)==0){
				printf("%d %s %s %s %s %d %s\n\n",stud.codice,stud.cognome,stud.nome,stud.voto,stud.lang,stud.classe,stud.sez);
				printf("Vuoi modificare questo elemento? 0=no 1=si\n");
				scanf("%d",&risp);
				if(risp==1){
					fclose(fp);
					fp=fopen("PrimoAn.dat","w+b");
					while(j<i){
						fread(&stud,sizeof(studente),1,fp);
						j++;
					}
					j=0;
					printf("Inserisci i dati dello studente separati da uno spazio:\n");
					printf("<Codice> <Cognome> <Nome> <Voto diploma> <Lingua> <Classe> <Sezione>\n");
					scanf("%d %s %s %s %s %d %s",&stud.codice,&stud.cognome,&stud.nome,&stud.voto,&stud.lang,&stud.classe,&stud.sez);
					fwrite(&stud,sizeof(studente),1,fp);
				}
			}
			fread(&stud,sizeof(studente),1,fp);
			i++;
		}
		fclose(fp);
	}
	else{
		printf("Archivio vuoto\n");
	}
}

Nel caso ci sia un riscontro tra il cognome immesso e quello nel file, dopo aver inserito i nuovi campi, mi va in loop il ciclo while come se il (!feof(fp)) non facesse il suo benedettissimo dovere
Il punto è che accade solo in questo caso, perche se inserisco un codice senza corrispondenze, il ciclo while non mi va in loop, quindi ipotizzo che il problema sia quella chiusura/riapertura del file nell'if.

Qualche anima pia mi sa dire cosa diamine accade?

Ecco il codice sorgente completo:

https://dl.dropboxusercontent.com/u/...ria_studenti.c

[fenix-69]

PS: nel caso non si fosse capito, ho necessità di utilizzare un approccio sequenziale su specifica richiesta della traccia

In altre occasioni ho anche optato per l'ausilio di un file temporaneo sul quale scrivere tutti i record, compreso quello corrispondente con i valori modificati ..
Per poi ripassare tutto nel primo file..
Ma questo algoritmo mi intrippa di più e mi piacerebbe davvero capire quale stupido errore sto commettendo
Perchè effettivamente non riesco a venirne a capo nemmeno col debug ..

[wingman87]

Facendo riferimento a questo:
http://www.cplusplus.com/reference/cstdio/fopen/
vedo 2 problemi:
- aprendo il file in modalità w+b cancelli il contenuto del file
- quando apri il file in modalità w+b, il valore di fp a cui fa riferimento il ciclo while cambia. In sostanza non stai più puntando a un file aperto per la lettura ma per la sola scrittura, quindi più che un loop mi sarei aspettato un qualche errore quando viene eseguita la fread o la feof, però evidentemente è successo qualcosa di inaspettato...

Se vuoi leggere e modificare il file mentre lo scorri, invece di aprire/chiudere il file diverse volte, dovresti aprirlo sin dal principio in modalità di lettura e scrittura... viene anche più semplice.

[fenix-69]

Originariamente inviato da wingman87

Facendo riferimento a questo:
http://www.cplusplus.com/reference/cstdio/fopen/
vedo 2 problemi:
- aprendo il file in modalità w+b cancelli il contenuto del file
- quando apri il file in modalità w+b, il valore di fp a cui fa riferimento il ciclo while cambia. In sostanza non stai più puntando a un file aperto per la lettura ma per la sola scrittura, quindi più che un loop mi sarei aspettato un qualche errore quando viene eseguita la fread o la feof, però evidentemente è successo qualcosa di inaspettato...

Se vuoi leggere e modificare il file mentre lo scorri, invece di aprire/chiudere il file diverse volte, dovresti aprirlo sin dal principio in modalità di lettura e scrittura... viene anche più semplice.

Eh ma il punto è che ho bisogno di chiuderlo e riaprirlo per riposizionare il puntatore nella posizione adeguata alla scrittura
In questo caso si sente la mancanza dell'fseek, ma purtroppo dovendolo risolvere in sequenziale me ne è pregiudicato l'utilizzo

PS: nel sorgente che c'era prima mancava un ciclo for che mi ero mangiato .. aggiornato il file su drop

[fenix-69]

Aggiornato il sorgente aprendo il file in r+b come da suggerimento ..
Wing vuoi farti 2 risate ?
Applicata modifica al volo su smartphone, compilato ed eseguito (c4droid) e la modifica fungeva alla perfezione ...
Festeggio allegramente
Torno a casa, incollo lo stesso identico codice soegente sia in dev che in visual c ed entrambi non mi scrivono in alcun modo su file.

Ho debuggato il programma con decine di break point e sono arrivato alla conclusione che tutto cicla e procede come dovrebbe anche su pc.
L'inghippo purtroppo scatta nell' fwrite che si trova poco prima della fine dell'if, che in pratica sembra rifiutarsi di scrivere su file il buffer stud appena scaneffato ..

Qualcuno sa spiegarmi dove sta l'errore ?
Qualcuno sa spiegarmi perchè su un compilatore android va e su uno desktop no?

[fenix-69]

Uppp

[wingman87]

Non ho capito questa parte:

Eh ma il punto è che ho bisogno di chiuderlo e riaprirlo per riposizionare il puntatore nella posizione adeguata alla scrittura
In questo caso si sente la mancanza dell'fseek, ma purtroppo dovendolo risolvere in sequenziale me ne è pregiudicato l'utilizzo

Comunque io intendevo una cosa di questo tipo (non posso testarlo ora):

void modifica(){
	studente stud;
	int i=0,j=0,risp;
	fp=fopen("PrimoAn.dat","r+b");
	fpos_t studentPos;
	char tmp[10];
	if(fp!=NULL){
		printf("Inserisci il cognome dello studente che vuoi modificare: ");
		scanf("%s",tmp);
		printf("Elementi corrispondenti:\n\n");
		fread(&stud,sizeof(studente),1,fp);
		while(!feof(fp)){
			fgetpos(fp, &studentPos);
			if(strcmp(tmp,stud.cognome)==0){
				printf("%d %s %s %s %s %d %s\n\n",stud.codice,stud.cognome,stud.nome,stud.voto,stud.lang,stud.classe,stud.sez);
				printf("Vuoi modificare questo elemento? 0=no 1=si\n");
				scanf("%d",&risp);
				if(risp==1){
					fsetpos(fp, &studentPos);
					printf("Inserisci i dati dello studente separati da uno spazio:\n");
					printf("<Codice> <Cognome> <Nome> <Voto diploma> <Lingua> <Classe> <Sezione>\n");
					scanf("%d %s %s %s %s %d %s",&stud.codice,&stud.cognome,&stud.nome,&stud.voto,&stud.lang,&stud.classe,&stud.sez);
					fwrite(&stud,sizeof(studente),1,fp);
				}
			}
			fread(&stud,sizeof(studente),1,fp);
			i++;
		}
		fclose(fp);
	}
	else{
		printf("Archivio vuoto\n");
	}
}

[Oceans11]

Originariamente inviato da fenix-69

L'inghippo purtroppo scatta nell' fwrite che si trova poco prima della fine dell'if, che in pratica sembra rifiutarsi di scrivere su file il buffer stud appena scaneffato ..

quindi se dopo la fwrite forzi la scrittura con fflush, funziona?

[lorenzo001]

Penso che abbia già risolto, vedi

http://forum.html.it/forum/showthrea...readid=1542404

[Oceans11]

ah grazie. ero curioso

[fenix-69]

Originariamente inviato da Oceans11

ah grazie. ero curioso

Purtroppo no, quel comando non risolve
La cosa strana però è che quell'fseek di switch tra lettura e scrittura non è necessarioin c4droid...
Vatti a capire il mistero ...