Fichier et Fonction d'incrementation

J'ai ecrit une fonction qui va lire dans un fichier un nombre, puis l'incrémente et met à jour le fichier avec ce nouveau chiffre.
Clairement, ça me permet d'avoir une fonction d'incrémentation qui stok en dur le dernier nombre.

Voilà ce que ça donne :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 long Increment(char const * const filename) { FILE *myFile; long myCount = 0; char msg[512]; myFile = fopen(filename, "r"); if (myFile != NULL) { fseek( myFile, 0, SEEK_SET); fgets( msg, sizeof(msg), myFile); if(msg!=NULL) { // string to long sscanf(msg, "%d", &myCount); } fclose(myFile); } myCount++; // long to string sprintf(msg, "%d", myCount); myFile = fopen(filename, "w"); if (myFile != NULL) { fwrite(msg, strlen(msg), 1, myFile); fclose(myFile); } free(msg); return myCount; }
Avez-vous des suggestions pour simplifier le schmilblick ?

Hello,
Citation:
Envoyé par kendras
J'ai ecrit une fonction qui va lire dans un fichier un nombre, puis l'incrémente et met à jour le fichier avec ce nouveau chiffre.
Clairement, ça me permet d'avoir une fonction d'incrémentation qui stok en dur le dernier nombre.

Voilà ce que ça donne :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 long Increment(char const * const filename) { FILE *myFile; long myCount = 0; char msg[512]; myFile = fopen(filename, "r"); if (myFile != NULL) { fseek( myFile, 0, SEEK_SET); fgets( msg, sizeof(msg), myFile); if(msg!=NULL)
Attention, msg est le buffer que tu remplis, pas le retour de fgets(). Si fgets() renvoie NULL, msg sera toujours != NULL lui.
Code:

1 2 /* fseek() inutile ici */ if ( fgets(msg, sizeof(msg), myFile) ) {
Citation:
Envoyé par kendras
Code:

1 2 3 4 { // string to long sscanf(msg, "%d", &myCount);
strtol() sera mieux. Ca sied bien en plus à la variable myCount qui est un long. Pour aller vite et voir en mode debug, tu peux utiliser assert pour ce genre de verif. Et accessoirement verifier que myCount n'est pas une borne limite (LONG_MIN ou LONG_MAX, voir limits.h pour ça). Si tu utilies *scanf(), mets bien le bon specificateur de format pour le type attendu.
Code:

1 2 3 4 5 /* Use strtol() instead ;) */ /* Avec sscanf c'est %ld pour un long int*/ myCount = strtol(msg, NULL, 10); assert(errno != ERANGE); /* inclure assert.h et errno .h */ }
Citation:
Envoyé par kendras
Code:

1 2 3 4 5 6 7 8 9 fclose(myFile); } myCount++; // long to string sprintf(msg, "%d", myCount);
Attention si myCount vaut LONG_MAX, peu probable mais ne sait-on jamais ;) .

rebelote, %ld pour un long int avec sprintf().
Citation:
Envoyé par kendras
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 myFile = fopen(filename, "w"); if (myFile != NULL) { fwrite(msg, strlen(msg), 1, myFile); fclose(myFile); } free(msg); return myCount; }
Avez-vous des suggestions pour simplifier le schmilblick ?
Ro, tu fais un fwrite pour du fichier texte !? Ce n'est pas que c'est pas correct mais c'est pas très coherent avec un fichier ouvert en mode texte. Le pb c'est que tu ne l'utilises pas comme il faut.
fwrite(msg, sizeof*msg, strlen(msg) + 1, myFile);
Mais c'est tordu.
fprintf(myFile,"%s", msg); c'est mieux. Tant que tu ne fais pas l'inverse (utilisation fonctions ecriture/lecture formates sur fichiers ouverts en mode binaire...).

Que vois-je, un free sur un tableau automatique !! :fessee: :fessee:

A+

Code:

1
2
3
4
5
6
7
myFile = fopen(filename, "r");
...
fclose(myFile);
...
myFile = fopen(filename, "w");
...
fclose(myFile);

Ou directement:

Code:

1
2
3
myFile = fopen(filename, "r+");
...
fclose(myFile);

Merci pour tous ces précieux conseils que j'ai mis en application