Discussion :

[____22]

Bonjour

j'ai un bug qui ne se produisait pas avant et que j'arrive pas à resoudre

je pense que ça vient du onDraw de la scrollview

merci beaucoup si vou savez des idées sur ce bug

[PetitPapaNoël]

Bonjour,

Si tu n'en dis pas plus, ou si tu ne nous montres pas de code, on ne va pas pouvoir t'aider. Pourquoi penses-tu que cela pourrait venir du OnDraw ?

[____22]

il y a beaucoup de fichiers sources

je peux plus joindre de fichiers apparemment

c'est un CDialog qui transfere des infos (pView->UpdateTag(m_ref,m_blind,m_refCnt,m_blindCnt) à une dérivant de CScrollView MyView.cpp

je sais que c'est pas conseillé de mélanger view et dialog mais ça marchait bien jusquà recemment j'ai fait des petites modif et là ça plante systematiquement

MyView.cpp

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
// MyVw.cpp : implementation of the CMyVw class
//
 
#include "stdafx.h"
#include "LEMonitor.h"
 
#include "MyVw.h"
 
#ifdef _DEBUG
#define new DEBUG_NEW
#undef THIS_FILE
static char THIS_FILE[] = __FILE__;
#endif
 
/////////////////////////////////////////////////////////////////////////////
// CMyVw
 
IMPLEMENT_DYNCREATE(CMyVw, CScrollView)
 
BEGIN_MESSAGE_MAP(CMyVw, CScrollView)
	//{{AFX_MSG_MAP(CMyVw)
	ON_WM_LBUTTONDOWN()
	ON_WM_RBUTTONDOWN()
	//}}AFX_MSG_MAP
END_MESSAGE_MAP()
 
/////////////////////////////////////////////////////////////////////////////
// CMyVw construction/destruction
 
CMyVw::CMyVw()
{
	// TODO: add construction code here
	m_zoomFactor=25;
	m_refCnt=0;
	m_blindCnt=0;
	SetScrollSizes(MM_TEXT,CSize(1000,1000));
}
 
CMyVw::~CMyVw()
{
}
 
void CMyVw::DoDataExchange(CDataExchange* pDX)
{
	CScrollView::DoDataExchange(pDX);
	//{{AFX_DATA_MAP(CMyVw)
		// NOTE: the ClassWizard will add DDX and DDV calls here
	//}}AFX_DATA_MAP
}
 
BOOL CMyVw::PreCreateWindow(CREATESTRUCT& cs)
{
	// TODO: Modify the Window class or styles here by modifying
	//  the CREATESTRUCT cs
 
	return CScrollView::PreCreateWindow(cs);
}
 
/////////////////////////////////////////////////////////////////////////////
// CMyVw drawing
 
void CMyVw::OnDraw(CDC* pDC)
{
	SetMapMode(pDC);
SetScrollSizes(MM_TEXT,CSize(50*m_zoomFactor,50*m_zoomFactor));
 
	//DrawGrid(pDC);
	//pDC->BitBlt(0,0,m_nBmpWidth,m_nBmpHeight,&m_MemDC,0,0,SRCCOPY);
 
	//char lo_id[4]="12"; 
	CPen p;
 
	if (m_refCnt){
		//p.CreatePen(PS_SOLID ,1,RGB(150, 150, 150));
		//CPen *pen=pDC->SelectObject(&p);
		int i;
		for (i=0;i<m_refCnt;i++){
			//pDC->MoveTo(m_ref[i].x,m_ref[i].y-2);pDC->LineTo(m_ref[i].x,m_ref[i].y+2);
			//pDC->MoveTo(m_ref[i].x-2,m_ref[i].y);pDC->LineTo(m_ref[i].x+2,m_ref[i].y);
			//sprintf(lo_id,"%d",m_ref[i].id % 256);
			pDC->TextOut(m_ref[i].x+2,m_ref[i].y,"ok");
		}
		//pDC->SelectObject(pen);
	}
 
	if (m_blindCnt){
		//p.CreatePen(PS_SOLID ,1,RGB(255, 25, 9));
		//CPen *pen=pDC->SelectObject(&p);
		int j;
		for (j=0;j<m_blindCnt;j++){
			//pDC->MoveTo(m_blind[j].x,m_blind[j].y-2);pDC->LineTo(m_blind[j].x,m_blind[j].y+2);
			//pDC->MoveTo(m_blind[j].x-2,m_blind[j].y);pDC->LineTo(m_blind[j].x+2,m_blind[j].y);
			//sprintf(lo_id,"%d",m_blind[j].id % 256);
 			pDC->TextOut(m_blind[j].x+2,m_blind[j].y,"b");
		}
		//pDC->SelectObject(pen);
	}
}
 
 
 
BOOL CMyVw::OnEraseBkgnd(CDC* pDC) 
{
    return TRUE;
}  
 
int CMyVw::SetMapMode(CDC* pDC)
{
	int previousMode = pDC->SetMapMode(MM_ISOTROPIC);
	pDC->SetWindowExt(100,100);
	//pDC->SetViewportOrg(CPoint(0,0));
	pDC->SetViewportExt(10*m_zoomFactor,10*m_zoomFactor);
 
	return previousMode;
}
 
void CMyVw::DrawGrid(CDC* pDC)
{
 
	CPen p;
	p.CreatePen(PS_SOLID ,0,RGB(200,200,200));
	CPen *pen=pDC->SelectObject(&p);
 
	for (int j=0;j<20;j++){
		pDC->MoveTo(4*j,0);
		pDC->LineTo(4*j,80);
		pDC->MoveTo(0,4*j);
		pDC->LineTo(80,4*j);
	}
	pDC->SelectObject(pen);
	for (int i=0;i<7;i++){
		pDC->MoveTo(40*i,0);
		pDC->LineTo(40*i,64*4);
		pDC->MoveTo(0,40*i);
		pDC->LineTo(64*4,40*i);
	}
	pDC->TextOut(40,0,"10 m");
	//pDC->RestoreDC(-1);
 
}
 
/*void CMyVw::UpdateRefTags(TAG_LIST *tag,int tagCnt)
{
	m_ref=tag;
	m_refCnt=tagCnt;
	Invalidate(TRUE);
}
void CMyVw::UpdateBlindTags(TAG_LIST *tag,int tagCnt)
{
	m_blind=tag;
	m_blindCnt=tagCnt;
	Invalidate(TRUE);
}*/
void CMyVw::UpdateTag(TAG_LIST *rtag,TAG_LIST *btag,int rtagCnt,int btagCnt)
{
	m_ref=rtag;
	m_refCnt=rtagCnt;
	m_blind=btag;
	m_blindCnt=btagCnt;
	Invalidate(TRUE);
}
 
 
 
 
/////////////////////////////////////////////////////////////////////////////
// CMyVw diagnostics
/*
#ifdef _DEBUG
void CMyVw::AssertValid() const
{
	CScrollView::AssertValid();
}
 
void CMyVw::Dump(CDumpContext& dc) const
{
	CScrollView::Dump(dc);
}
 
 
#endif*/ //_DEBUG
 
/////////////////////////////////////////////////////////////////////////////
// CMyVw message handlers
 
 
void CMyVw::OnLButtonDown(UINT nFlags, CPoint point) 
{
	// TODO: Add your message handler code here and/or call default
 
	m_zoomFactor+=5;
	//SetScrollSizes(MM_TEXT,CSize(50*m_zoomFactor,50*m_zoomFactor));
	Invalidate(TRUE);
}
 
 
void CMyVw::OnRButtonDown(UINT nFlags, CPoint point) 
{
	// TODO: Add your message handler code here and/or call default
 
	m_zoomFactor-=5;
	//SetScrollSizes(MM_TEXT,CSize(50*m_zoomFactor,50*m_zoomFactor));
	Invalidate(TRUE);
}

[PetitPapaNoël]

Quel est le bug ? Quand se produit-il ?

[____22]

c'est quand je transfere de nouvelle données vers la view

j'ai

[PetitPapaNoël]

Ok. Pardon pour les questions précédentes : je suis derrière un pare-feu qui m'empêche de voir les images jointes (mais j'ai quand même pu les voir).
Il faudrait étudier ça à tête reposée, mais je n'ai pas le temps maintenant...

[bacelar]

Ton debugger te donne la réponse :
le paramètre "h" est différent de "pObject->m_hObject".

Maintenant il ne te reste qu'à savoir pourquoi, et là, c'est le visualisateur de variables et de pile d'appel de ton debugger qui seront le plus à même de te répondre.

Il ne faut pas s'arrêter au fait que cela marchait avant et penses que tu as de la chance de pouvoir le reproduire systématiquement.

[____22]

à côté de hObject il y a "CPen" ça veut dire qu'il y a un probleme sur les CPen?

[bacelar]

C'est juste que le Debuggeur (WinDbg ??, ce n’est pas un dévermineur pour les mauviettes), a cherché à trouver le type de l'objet pointé par pObject.

Généralement, si on ne fout pas trop trop le bordel avec des pointeurs merdeux, cette détermination du type des objets marche très bien.

Donc, c'est juste pour indiquer que pObject pointe sur un CPen.

Le problème n'est pas sur "LES" CPen mais vraisemblablement dans un CPen.
C'est juste que le membre "m_hObject" de l'objet CPen trouvé dans la map des objets GDI gérés par la bibliothèque MFC n'a pas la même valeur que le paramètre "h" de la méthode FromHandle.

Là, on commence à être dans les entrailles des MFC et si tu ne les maîtrises pas et que t’as ce type d’erreur, c’est que tu as fais une petite boulette dans les API de « hauts niveaux ». Je regarderais du coté d’un objet GDI créé dans pour une fenêtre et utilisé pour une autre etc…

Il nous faudrait un code compilable minimal capable de reproduire le problème.

[Mat.M]

Bonjour,

1-lorsqu'on touche aux MFC on utilise obligatoirement le MSDN, as-tu jeté un coup d'oeil à cette aide ?
2-si tu consultes le MSDN et les examples tu verras que quand tu crées un CPen tu initialises un autre CPen qui est le CPen d'origine.
Lorsque tu as fini de dessiner tu dois restorer le premier CPen d'origine avec SelectObject :

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
    }
}

Est-ce que tu fais cette opération ?
Au vu du code tout laisse à penser le contraire..
si tu ne le fais pas le système sera engorgé d'objets GDI...