Modification inattendue d'une variable

**Hepm** · 03/12/2007, 16h37

Bonjour à tous,

dans une CDialog, j'ai :
- une variable membre m_MaVariable de type int
- un bouton identifié IDC_MonBouton.

La valeur de m_MaVariable est MODIFIEE après l'instruction :

GetDlgItem(IDC_MonBouton)->ShowWindow(SW_HIDE);

J'ai vérifié en pas à pas et c'est ShowWindow qui provoque la modification, mais je ne vois pas pourquoi et comment...

Quelqu'un aurait-il une explication ou des conseils pour trouver l'erreur ?

**farscape** · 03/12/2007, 16h43

salut,
met un point d'arrêt dans DodataExchange pour voir si il y a un transfert de donnée..

**Hepm** · 03/12/2007, 16h59

Négatif, pas de passage par DoDataExchange au cours de la modification de la variable...

**PetitPapaNoël** · 03/12/2007, 17h25

Bonjour,

C'est quand même bizarre que la valeur soit modifiée par cette instruction. Tu es sûr que le débugueur est en phase avec ton code ? Pour être un peu plus sûr, tu devrais peut-être nettoyer et recompiler ta solution...

**farscape** · 03/12/2007, 17h50

met un point d'arrêt sur toutes les affectations directes de cette variable.
si c'est toujours négatif, il ne reste plus que le débordement mémoire, ou le mauvais diagnostic : tu ne regardes pas la bonne variable -> il te faut vérifier son adresse dans le debugger (avant modif et après ).

**Hepm** · 03/12/2007, 17h52

Quelques précisions :

Je fais dans cette CDialog des "choses" qui pourraient, peut-être, être la cause de la modification décrite plus haut.

Comme j'ai 50 boutons que je dois déplacer dynamiquement, j'ai déclaré le tableau :

Code :

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

1
2
 
CButton* m_pB [50];

Je l'initialise dans OnInitDialog :

Code :

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

1
2
3
4
5
 
m_pB[0] = (CButton*) GetDlgItem(IDC_Bouton0);
m_pB[1] = (CButton*) GetDlgItem(IDC_Bouton1);
...
m_pB[49] = (CButton*) GetDlgItem(IDC_Bouton49);

Pour déplacer mes boutons, j'ai le code :

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
 
//Extinction des boutons
for (int i = 0 ; i <= 49 ; i++)
   m_pB[i]->ShowWindow(SW_HIDE);
 
//Déplacement vertical de DeltaH des boutons
WINDOWPLACEMENT WndPl;
int DeltaH = 20;
for (int i = 0 ; i <= 49 ; i++)
{
   m_pB[i]->GetWindowPlacement(&WndPl);
   WndPl.rcNormalPosition.bottom += DeltaH;
   WndPl.rcNormalPosition.top += DeltaH;
   m_pB[i]->SetWindowPlacement(&WndPl);
}
 
//Allumage des boutons
for (int i = 0 ; i <= 49 ; i++)
   m_pB[i]->ShowWindow(SW_SHOW);

J'ai lu qu'un pointeur retourné par GetDlgItem est "temporaire" sans d'ailleurs savoir quelle est la durée de vie.
Est-ce que les opérations que j'effectue sont valides? sur des pointeurs qui restent valides (après les déplacements)?
Ou au contraire ces opérations, avec des pointeurs qui ne seraient plus valides, pourraient-elles altérer la mémoire et provoquer la modification dont j'ai parlé au début ?

**farscape** · 03/12/2007, 18h07

Effectivement ..,
il vaut mieux stocker les identifiants dans un tableau et faire le GetDlgItem à la demande pour chaque opération.
le retour de GetDlgItem cause un probléme si il n'y a pas de variables contrôle rattachée à l'identifiant..

**Hepm** · 03/12/2007, 18h23

Cela me semble une bonne solution de mettre tous mes identifiants dans un tableau.
Je n'aurai pas beaucoup plus de code et j'éviterai les soucis avec GetDlgItem.
Merci pour cette idée que je vais tester.