Discussion :

[on2101]

Bonjour,

Mon programme plante très souvent (pas tout le temps, cela a l air assez aleatoire) quand j essaie de traces des traits a l écran.

Dans le OnPaint(), je créé mon CClientDC que je passe (par adresse) a ma fonction d affichage pour tracer mes traits.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
CClientDC dc(this);
 
HPEN crayonRouge = CreatePen(PS_SOLID,1,RGB(255,0,0));
 
 
//Appel de la fonction d affichage
 
 
HPEN crayonNull = NULL;
dc.SelectObject(crayonNull);
dc.ReleaseAttribDC();
DeleteObject(crayonRouge);

Ma fonction d'affichage ressemble a ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
//Dans la fonction d affichage
 
dc -> SelectObject(crayon);
 
dc -> MoveTo(rect.left,rect.bottom);
 
for(int i=0;i<borne;i++)
{
    dc -> LineTo(rect.left + i,bas - (int) ( valeurs[i]*180/(3.14) ) );
}

Au debugger, les valeurs passées en parametre a la fonction MoveTo() sont corrects.

(Desole ppour les balises de code, je les ajouterais, je vais chercher comment faire autre part sur ce forum !)

Merci
pense à mettre la balise de code avec le bouton #, Merci Farscape

[TheGzD]

Quand tu parles de plantage, quelle erreur as-tu exactement ?
Quand se produit-elle : au lancement ? plus tard ?
Etats des pointeurs de ta fenêtre ainsi que des éléments qui la composent ? HWND ?
As-tu pensé à récupérer les objets courants (tels que le CPen) et les restauré après utilisation du tien ?

Tu vas devoir en dire plus, car comme tu peux le constater on manque un poil d'infos pour t'aider vraiment

[on2101]

Merci de vote reponse,

L erreur n as pas lieu au demarrage de l application mais après. En fait, je cherche a tracer des traits tout au long de l application (plusieurs fois par secondes) et l erreur se produit au bout d un certain temps, sans raison apparente.

Le message d erreur indique qu un probleme est survenu dans le fichier wingdi.cpp line 627 et la pile d appel me donne la ligne correspondant a mon MoveTo.

Dans le gestionnaire des ressources Windows, le nombre d objets GDI est de l ordre de 30 et reste constant.

Comme tu me l as fait remarqué, le problème peut venir de l objet CPaintDC lui meme. Lors du plantage, j inspecte les valeurs des champs de mon CClientDC :

CDC -> CObject ->rien de special
CDC -> m_hdc : unused, m_hAttribDC : unused
m_hWnd : unused

En effet, mon CClientDC n a pas l air tres en regles, il doit manquer quelques initialisations !

Comment initialiser ces attributs ?

Merci

[TheGzD]

Pourrais-tu nous montrer ton code stp ?

[Mat.M]

Comme tu me l as fait remarqué, le problème peut venir de l objet CPaintDC lui meme. Lors du plantage, j inspecte les valeurs des champs de mon CClientDC :

Tu sembles très mal utiliser le tracé avec CPen
Quand tu crées un CPen 1 il faut appeler CPen::Create puis SelectObject qui retourne un pointeur sur le CPen courant mettons CPen 0.
Une fois que le CPen 1 n'est plus utilisé il faut le détruire et restituer le CPen d'origine
Voilà un exemple du MSDN

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
 
void CMyView::OnDraw( CDC* pDC )
{
    CPen penBlack;  // Construct it, then initialize
    if( newPen.CreatePen( PS_SOLID, 2, RGB(0,0,0) ) )
    {
        // Select it into the device context
        // Save the old pen at the same time
        CPen* pOldPen = pDC->SelectObject( &penBlack );
 
        // Draw with the pen
        pDC->MoveTo(...);
        pDC->LineTo(...);
 
        // Restore the old pen to the device context
        pDC->SelectObject( pOldPen );
    }
    else
    {
        // Alert the user that resources are low
    }
}

tu vois que dans cet exemple on crée un pOldPen.
Il faut le restituer avec

Code :

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

pDC->SelectObject( pOldPen );

Or dans le code donné tu ne fais pas cela.

PS apparement il y a une erreur dans mon ( vieux ) MSDN ; c'est

Code :

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

1
2
 
if( penBlack .CreatePen( PS_SOLID, 2, RGB(0,0,0) ) )

et non

Code :

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

1
2
 
if( newPen.CreatePen( PS_SOLID, 2, RGB(0,0,0) ) )

[on2101]

Depuis que je récupère l'ancien crayon pour le remettre ensuite, je n'ai eu aucun nouveau plantage.

Merci, c'était très bête comme erreur

J'attends un peu avant de mettre RESOLU car le problème apparaissait de manière aléatoire !