Discussion :

[aperdemak]

Bonjour, j'ai fait un classe Screen qui charge un bitmap de l'image à l'écran, mais lorsque je fait une boucle contenant le constructeur et destructeur de cette classe, j'ai la ram qui se remplie. J'ai essayé de delete(scnBmpData) et delete(scnOct) mais ça fait cracher mon programme. Je suis débutant en programmation et ça fait 1 semaine que mon programme est bloqué à cause de ça. Si quelqu'un pouvait m'aider j'en serais reconaissant. Merci d'avance.

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
screen::screen()
{  
  // Donne les dimensions de l'écran
  int x1, y1, x2, y2;
  x1  = GetSystemMetrics(SM_XVIRTUALSCREEN);
  y1  = GetSystemMetrics(SM_YVIRTUALSCREEN);
  x2  = GetSystemMetrics(SM_CXVIRTUALSCREEN);
  y2  = GetSystemMetrics(SM_CYVIRTUALSCREEN);
  this->w   = x2 - x1;
  this->h   = y2 - y1;
 
  // copie l'écran dans le bitmap
  this->hScreen = GetDC(NULL);
  this->hDC     = CreateCompatibleDC(hScreen);
  this->hBitmap = CreateCompatibleBitmap(hScreen, w, h);
  HGDIOBJ old_obj = SelectObject(hDC, hBitmap);
  BOOL    bRet    = BitBlt(hDC, 0, 0, w, h, hScreen, x1, y1, SRCCOPY);
 
  //Charge le bitmap dans le buffer
  this->scnBmp = new Gdiplus::Bitmap(hBitmap,NULL);
  Gdiplus::Rect rectScnBmp(0,0, w, h);
  this->scnBmpData = new Gdiplus::BitmapData;
  scnBmp->LockBits(&rectScnBmp, Gdiplus::ImageLockModeRead,
		   PixelFormat32bppARGB, scnBmpData);
  this ->scnOct = (byte*)(void*)scnBmpData->Scan0;
}
screen::~screen()
{
  scnBmp->UnlockBits(scnBmpData);
  DeleteDC(hDC);
  ReleaseDC(NULL, hScreen);
  DeleteObject(hBitmap);
  delete(scnBmpData);
}

//ma boucle qui fait monter la ram, mais qui marche quand même.

Code :

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

1
2
3
4
5
6
7
8
 while(!pos.x && !pos.y)
  {
    screen monEcran;
    //pos = monEcran.findBmp(monbmp);
    monEcran.~screen();
    //if(pos.x && pos.y)
    //  cout << "trouvé en: (" << pos.x << ","<< pos.y << ")"<< endl;
  }

edit: j'ai trouvé (en fait non)

[Bousk]

Je vois au moins un new sans delete, sur scnBmp

[aperdemak]

j'ai déplacé la partie buffer dans une autre partie du programme et j'ai delete le plus possible de choses mais ça n'a pas l'air de marcher, en fait dès que je lance le programme le programme atteint les 3go de ram très vite et il y reste. Et même avec un sleep(1000) il finit par les atteindre.

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
screen::screen()
{  
  // get screen dimensions
  int x1, y1, x2, y2;
  x1  = GetSystemMetrics(SM_XVIRTUALSCREEN);
  y1  = GetSystemMetrics(SM_YVIRTUALSCREEN);
  x2  = GetSystemMetrics(SM_CXVIRTUALSCREEN);
  y2  = GetSystemMetrics(SM_CYVIRTUALSCREEN);
  this->w   = x2 - x1;
  this->h   = y2 - y1;
 
  // copy screen to bitmap
  this->hScreen = GetDC(NULL);
  this->hDC     = CreateCompatibleDC(hScreen);
  this->hBitmap = CreateCompatibleBitmap(hScreen, w, h);
  HGDIOBJ old_obj = SelectObject(hDC, hBitmap);
  BOOL    bRet    = BitBlt(hDC, 0, 0, w, h, hScreen, x1, y1, SRCCOPY);
  this->scnBmp = new Gdiplus::Bitmap(hBitmap,NULL);
 
  DeleteObject(old_obj);
 }
screen::~screen()
{
  DeleteDC(hDC);
  ReleaseDC(NULL, hScreen);
  DeleteObject(hScreen);
  DeleteObject(hBitmap);  
  DeleteObject(scnBmp);
}

[jo_link_noir]

Pour quelle raison fais-tu un appel explicite au destructeur ?

L'objet est détruit et le destructeur appeler tout seul à la fin de sa vie/portée.

[aperdemak]

je savais pas ça merci, mais le problème vient pas de là.

edit: c'est à partir du bitblt que ça merde

[jo_link_noir]

Il manque le delete sur scnBmp, comme l'a indiqué Bousk.

[aperdemak]

j'ai déjà essayé ça fait cracher le programme, je pense le bitmap n'est pas libéré correctement car ça fuit de 9mo par passage dans la boucle.

[foetus]

Pas sûr à 100% mais je pense que c'est la variable old_obj qui n'est pas libérée.

Dans les codes Internet, cette variable est sauvegardée et après l'opération, elle est "remise en place" SelectObject(XXXDC, old_obj); afin d'avoir "le même état" avant et après.

Dans ton cas, il faut peut être juste faire un DeleteObject(old_obj);.


Édit: dans ta deuxième version tu le fais déjà

[aperdemak]

Oui je viens de voir ça, par contre ça change rien. La seule chose efficace pour le moment c'est de commenter l'initialisation du bitmap.

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
screen::screen()
{  
  // get screen dimensions
  int x1, y1, x2, y2;
  x1  = GetSystemMetrics(SM_XVIRTUALSCREEN);
  y1  = GetSystemMetrics(SM_YVIRTUALSCREEN);
  x2  = GetSystemMetrics(SM_CXVIRTUALSCREEN);
  y2  = GetSystemMetrics(SM_CYVIRTUALSCREEN);
  this->w   = x2 - x1;
  this->h   = y2 - y1;
 
  // copy screen to bitmap
  this->hScreen = GetDC(NULL);
  this->hDC     = CreateCompatibleDC(hScreen);
  this->hBitmap = CreateCompatibleBitmap(hScreen, w, h);
  this->old_obj = SelectObject(hDC, hBitmap);
  BOOL    bRet    = BitBlt(hDC, 0, 0, w, h, hScreen, x1, y1, SRCCOPY);
  this->scnBmp = new Gdiplus::Bitmap(hBitmap,NULL); // si je commente= => plus de fuite
 }
screen::~screen()
{
  ReleaseDC(NULL, hScreen);
  DeleteDC(hScreen);
  SelectObject(hDC, old_obj);
  DeleteObject(hBitmap);
  DeleteObject(scnBmp);
  DeleteObject(old_obj);
  DeleteDC(hDC);
}

edit: le Bitmap se libère en faisant un delete apparemment, donc je comprends pas pourquoi ça crache, peut être qu'il se libère à la fin du constructeur mais je ne vois pas pourquoi

[aperdemak]

J'ai trouvé vos réponses font sens maintenant
En fait il fallait juste faire un delete scnBmp et supprimer l'appel du destructeur.
Merci bien !

Mon programme consomme do,c que 9mb et 1% cpu pour trouver un bmp à l'écran en temps réel, le lockbits est définivement meilleur que getpixel

[Bousk]

http://cpp.developpez.com/faq/cpp/?p...un-destructeur
http://cpp.developpez.com/faq/cpp/?p...r-d-une-classe
http://cpp.developpez.com/faq/cpp/?p...un-destructeur
http://cpp.developpez.com/faq/cpp/?p...objets-membres