destructeur appelé n'importe quand

Version imprimable

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:

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.

07/08/2011, 08h23
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.
07/08/2011, 10h02
screetch

regarde "constructeur de copie" et "operateur d'affectation" sur google
07/08/2011, 15h10
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 ^^
07/08/2011, 15h36
Joel F

il te faut un operateur d'affectation si tu tiens a donner une semantique de valeur a ton objet.
07/08/2011, 16h24
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.
07/08/2011, 21h55
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];:fessee:
std::vector<unsigned char>buffer(100);:chin:
12/08/2011, 03h02
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 ^^