Discussion :

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ?

[Foobar1329]

Hello,

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
  /* fseek() inutile ici */
                        if ( fgets(msg, sizeof(msg), myFile) ) {

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 */
                          }

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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().

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 !!

A+

[Pragmateek]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Ou directement:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[kendras]

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