Discussion :

[dede92]

Bonjour,
Dans mon appli je fais appel à une boite de dialogue dans laquelle je suis amené à rechercher des élément via une seconde boite de dialogue.
Pour passer l'information j'utilise "SetWindowLong ()" mais au retour une fois récupéré par "GetWindowLong()", l'application plante.
http://www.photosreflex.net/temp/erreur_visualc.jpg
Pourtant j'affiche bien le bon resultat par MessageBox() ...
Il a sûrement quelque chose ...
Comment passer une structure d'un dialogue à un autre ...
Si quelqu'un a une idée
Merci d'avance
DialogBox appelante :

Code :

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

1
2
3
4
5
6
7
case BN_CLICKED:
            DialogBox(hInst, MAKEINTRESOURCE(Libpredef), hDlg, LibpredefDlgProc);
            lbpdef = (LIBPREDEF *)GetWindowLong(hDlg, DWL_USER);
			strcpy(szBuffer, lbpdef->LibPredef);
			MessageBox (hDlg, szBuffer, szAppName, MB_OK | MB_ICONEXCLAMATION);
            SetDlgItemText (hDlg, IDC_OBJET, szBuffer, strlen(szBuffer));
            break;

DialogBox appelée :

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
BOOL CALLBACK LibpredefDlgProc (HWND hDlg, UINT message, UINT wParam, LONG lParam)
{
HGLOBAL hglb;
LIBPREDEF *lbpdef;

     switch (message)
	 {
       case WM_INITDIALOG:
            if (hglb = GlobalAlloc(GPTR,sizeof(LIBPREDEF))) {
		       if (lbpdef = (LIBPREDEF *)GlobalLock(hglb)) {
                 SetWindowLong(GetParent(hDlg),DWL_USER,(LONG)lbpdef);
				 lbpdef->hglb = hglb;
			     strcpy(lbpdef->LibPredef, "Test transition de données");
                 return(0L);
			   }
			   GlobalFree(hglb);
			 }
		    return TRUE;

       case WM_COMMAND:
           switch (wParam)
		   {
             case IDOK :
             case IDCANCEL :
				   
                  EndDialog (hDlg, 0) ;
                  return TRUE ;
		   }
           break ;
	 }
     return FALSE ;
}

[rigobert]

Je ne sais pas si ça peut avoir un rapport avec ton problème, mais ta façon d'allouer de la mémoire est un peu douteuse :

- GlobalAlloc (GPTR, ...) alloue un bloc de mémoire physique et l'initialise à 0 ; le retour est donc directement un pointeur sur la mémoire allouée et non un handle ;
- pour la même raison GlobalLock() est inutile (et peut peut-être même mettre la zone, surtout s'il n'y a pas de GlobalUnlock() associé, mais je ne suis pas sûr) ;
- les fonctions Globalxxx() ne sont pas tout à fait obsolètes dans Win32 ... mais presque :

-> pour la gestion standard de la mémoire dynamique mieux vaut utiliser malloc()/free() en C, et new/delete en C++.
-> pour une gestion plus spécifique et avancée du/des tas (heaps), des fonctions adaptées à Win32 existent, plus pratiques & beaucoup plus rapides : Heapxxx().
-> toutefois, héritage technologique oblige, les fonctions Globalxxx() sont parfois nécessaires dans les domaines suivants : DDE, clipboard & OLE. Mais c'est vraiment tout.

[dede92]

J'ai modifié l'allocation memoire que j'ai remplacé par "malloc()", mais le bug est toujours là, et toujours dans le dalogue 1 au retour du dialogue 2.

Code :

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

1
2
3
4
5
6
7
8
BOOL CALLBACK LibpredefDlgProc (HWND hDlg, UINT message, UINT wParam, LONG lParam)
{

     switch (message)
	 {
       case WM_INITDIALOG:
lbpdef = malloc (sizeof(LIBPREDEF));
strcpy(lbpdef->LibPredef, "Test transition de données");

Code :

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

1
2
3
4
5
case BN_CLICKED:
            DialogBox(hInst, MAKEINTRESOURCE(Libpredef), hDlg, LibpredefDlgProc);
            lbpdef = (LIBPREDEF *)GetWindowLong(hDlg, DWL_USER);
            SetDlgItemText (hDlg, IDC_OBJET, "xxx", 3);
            return FALSE;

Le bug se produit au moment du SetDlgItemText ()

[rigobert]

Code :

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

SetDlgItemText (hDlg, IDC_OBJET, "xxx", 3);

3 questions :

1/ c'est quoi ce 4ème argument (3) ? SetDlgItemText() n'en prend que 3 il me semble.

2/ est-ce que tu as vérifié que hDlg est valide ?

3/ est-ce que tu as vérifié que IDC_OBJET correspondait bien à un controle fils valide ?

4/ est-ce que tu as vérifié qu'il n'y avait pas un problème de réentrance dans le dialogue fils (en mettant un point d'arrêt dans WM_INITDIALOG par ex) ?

Si tu as vérifié tout ça, que tout est OK et que ça plante toujours avec le même message d'erreur, c'est qu'à priori la pile est corrompue quelque part en amont.

[dede92]

Code :

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

SetDlgItemText (hDlg, IDC_OBJET, "xxx", 3);

3 questions :

1/ c'est quoi ce 4ème argument (3) ? SetDlgItemText() n'en prend que 3 il me semble.

2/ est-ce que tu as vérifié que hDlg est valide ?

3/ est-ce que tu as vérifié que IDC_OBJET correspondait bien à un controle fils valide ?

4/ est-ce que tu as vérifié qu'il n'y avait pas un problème de réentrance dans le dialogue fils (en mettant un point d'arrêt dans WM_INITDIALOG par ex) ?

Si tu as vérifié tout ça, que tout est OK et que ça plante toujours avec le même message d'erreur, c'est qu'à priori la pile est corrompue quelque part en amont.

C'est vraiment la honte !!!
C'est effectivement le 4 eme argument, que j'ai été chercher je ne sais où (enfin si, de la fonction inverse, le Get) de SetDlgItemText()
Merci encore.