Discussion :

[acryline]

Bonjour,

J'ai déjà utilisé WriteFile mais je n'ai jamais eu ce problème et je n'arrive pas à trouver l'origine du bogue:

J'ai fait une fonction pour enregistrer temporairement l'adresse d'un fichier dans un fichier texte. Voici la fonction:

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
void fichierTemporaireEcrire(char *fichier)
{
HANDLE hFichier=NULL;
DWORD nbcharRead= 0;

 hFichier = CreateFile( "~fichier.tmp", GENERIC_WRITE, 0,NULL,
                                            CREATE_ALWAYS,     FILE_ATTRIBUTE_NORMAL, NULL);

SetFilePointer( hFichier,0,0,FILE_BEGIN);
//La ligne qui suit est un simple test pour voir ce que contient fichier
 MessageBox(NULL,fichier,"",MB_OK);

 WriteFile(hFichier, &fichier,sizeof(char)*256, &nbcharRead, NULL);
 CloseHandle(hFichier);
}

Quand je teste fichier, l'adresse du fichier est correcte.
Mais dans ~fichier.tmp l'adresse est précédée de 16 caractères ... j'ai beau tourner le problème dans tous les sens je n'arrive pas à enlever ces 16 caractères... qq'un voit l'erreur ?

Merci !

Edité: je suis sur cette erreur depuis assez longtemps et il suffit que je pose la question ici pour que je trouve seule la réponse !!


Honte à moi j'ai mis & devant fichier
Avec ce code tout fonctionne .

Code :

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

1
2
 WriteFile(hFichier, fichier,sizeof(char)*256, &nbcharRead, NULL);

[Médinoc]

Sauf qu'on ne sait pas d'où sort le 256...
Et tu devrais renommer nbCharRead en NbCharWritten...

[acryline]

Sauf qu'on ne sait pas d'où sort le 256...
Et tu devrais renommer nbCharRead en NbCharWritten...

J'avais pensé écrire sizeof(fichier) c'est bon ?

Ok pour NbCharWritten , merci !

[Médinoc]

sizeof()ne marche pas sur un pointeur.

Le mieux, c'est de faire comme toutes les bonnes fonctions qui prennent en paramètre un pointeur non-const sur char: Passer également la taille du tableau (en caractères et non en octets, histoire de passer plus facilement en wchar_t si nécessaire) en paramètre...


Et pour la taille d'un tableau de caractères, ne pas hésiter à utiliser la macro ARRAYSIZE(), définie sous Visual.
Pour MinGW, une alternative: C'est la même chose que la version C de ARRAYSIZE() sous Visual:

Code :

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

1
2
3
#ifndef ARRAYSIZE
#define ARRAYSIZE(tab) (sizeof(tab)/sizeof(tab[0]))
#endif

(La version C++ de ARRAYSIZE() sous Visual est plus complexe, car elle vérifie à la compilation que c'est bien sur un tableau qu'on l'utilise et non sur un pointeur)

[acryline]

Ok merci !

[Médinoc]

Tiens, j'ai reconstruit la version C++ de mémoire, la voici:

Code C++ :

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

1
2
3
4
template< class T, size_t N >
char (* TArraySize(T (&tab)[N]) )[N];
 
#define ARRAYSIZE(arr) sizeof(*TArraySize(arr))

On a donc le code total:

Code C++ :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#ifndef ARRAYSIZE
	#ifdef __cplusplus
		//Version C++ de ARRAYSIZE() : 
		//Utilise un prototype de fonction template
		template< class T, size_t N >
		char (* TArraySize(T (&tab)[N]) )[N];
		#define ARRAYSIZE(arr) sizeof(*TArraySize(arr))
	#else
		//Version C de ARRAYSIZE() :
		//Utilise une bête division
		#define ARRAYSIZE(arr) (sizeof(arr)/sizeof(arr[0]))
	#endif
#endif

Et je rappelle l'avantage de la version C++ : Elle fait une erreur de compilation si on tente de l'essayer sur autre chose qu'un tableau.