Discussion :

[biyoudidou]

Salut, le code suivant me retourne un handle invalide sur le handle du fichier,
et le destructeur est appelé juste après l'assignation, avec hFile = le handle de fichier retourné par OpenFileEx dans le constructeur.

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
 
class MyFile
{
public:
    HANDLE hFile;
    MyFile(std::string filename)
    {
        hFile = CreateFile(filename.c_str(), GENERIC_READ, 0, NULL, OPEN_EXISTING, NULL, NULL);
    }
    MyFile()
    {
        hFile = NULL;
    }
    ~MyFile()
    {
        CloseHandle(hFile);
        // Breakpoint ici, il se produit juste après l'assignation et hFile est valide ici.
    }
};
 
MyFile fileA;
 
int _tmain(int argc, _TCHAR* argv[])
{
    fileA = MyFile("test.txt");
 
    char * buffer = new char[100];
    DWORD BytesRead = 0;
 
    ReadFile(fileA.hFile, buffer, 100, &BytesRead, NULL);
    int a = GetLastError();
    // Breakpoint ici, a = ERROR_INVALID_HANDLE
    delete[] buffer;
    return 0;
}

Donc la question est, est ce que mon compilo déc**** a plein tube, ou bien c'est moi qui n'ai rien compris ^^

Merci d'avance pour toutes les réponses.

[Joel F]

fileA = MyFile("test.txt");

tu construit un MyFile temporaire que tu affecte a fileA. Une fois fait, le temporaire est detruit.


Tout est normal.

[screetch]

regarde "constructeur de copie" et "operateur d'affectation" sur google

[biyoudidou]

Okay, je viens de saisir, la copie est bien faite mais l'objet global et celui créé lors de l'assignation partagent le meme handle, comme l'un est détruit, le handle de l'autre devient invalide.
Quelqu'un aurait une solution pour conserver le handle lors de la destruction d'une copie? Autre que passer par un new/delete ^^

[Joel F]

il te faut un operateur d'affectation si tu tiens a donner une semantique de valeur a ton objet.

[screetch]

Sinon, éviter la variable globale, interdire la copie avec un opérateur de copie/d'affectation privés, et passer ton objet par référence a ceux qui en ont besoin.

[3DArchi]

Histoire d'appeler les choses par leur nom : RAII

F.A.Q. : Le RAII
Tutoriel : Gérer ses ressources de manière robuste en C++ par Aurélien Regat-Barre

P.S. ça devrait devenir pavlovien:
: char * buffer = new char[100];
std::vector<unsigned char>buffer(100);

[biyoudidou]

réponse un peu lente désolé :S

la meilleure solution est souvent la plus simple :p
je pense que pour une classe aussi basique je vais donc rendre privé constructeur de copie et operateur d'assignation. Merci pour les réponses ^^