Discussion :

[ciddu42]

Bonjour j'utilise visual studio 2022,

j'ai un problème avec l'ajout d'éléments à une combobox après sa création : si je tape

Code :

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

1
2
3
4
 
    TCHAR Planets[10] = L"Mercury";
    // Add string to combobox.
    SendMessage(hwnd_correct_shape_combobox, (UINT)CB_ADDSTRING, (WPARAM)0, (LPARAM)Planets);

dans la fonction:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
int WINAPI WinMain (HINSTANCE hinstance,
                    HINSTANCE hPrevInstance,
                    LPSTR     lpCmdLine, 
                    int       nCmdShow)
{
    HWND hwnd_correct_shape_combobox = CreateWindow(WC_COMBOBOX, TEXT(""),
        CBS_DROPDOWN | CBS_HASSTRINGS | WS_CHILD | WS_OVERLAPPED | WS_VISIBLE,
        xpos, ypos, nwidth, nheight, hwndParent, NULL, hinstance,
        NULL);
    TCHAR Planets[10] = L"Mercury";
    // Add string to combobox.
    SendMessage(hwnd_correct_shape_combobox, (UINT)CB_ADDSTRING, (WPARAM)0, (LPARAM)Planets);
}

la combobox ajoute bien le texte "Mercury",
mais si je tape le même code avec bien sur "hwnd_correct_shape_combobox" en variable globale :

Code :

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

1
2
3
4
 
    TCHAR Planets[10] = L"Mercury";
    // Add string to combobox.
    SendMessage(hwnd_correct_shape_combobox, (UINT)CB_ADDSTRING, (WPARAM)0, (LPARAM)Planets);

en dehors de la fonction :

Code :

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

1
2
3
4
5
 
int WINAPI WinMain (HINSTANCE hinstance,
                    HINSTANCE hPrevInstance,
                    LPSTR     lpCmdLine, 
                    int       nCmdShow)

la combobox reste vide....
Et j'aurais besoin que la combobox ajoute ce texte en dehors de WinMain

[Bousk]

T'es sûr que hwnd_correct_shape_combobox est initialisée quand tu l'utilises ?

[ciddu42]

oui je fait appel à sendmessage après la fonction winmain et createwindow aient été appelé

je l'utilise dans une

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
LRESULT CALLBACK MainWndProc(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
{
    HDC hdc;
    PAINTSTRUCT ps;
 
    switch (uMsg)
    {
    case WM_CREATE:
        Presenter_Create ();
        return 0;
 
    case WM_COMMAND:
        // Si le bouton "créer la forme" est pressé
        if (LOWORD(wParam) == IDC_Create_Shape_Button) 
        {
            Presenter_On_Create_Shape_Button (hwnd_create_shape_edit, hwnd_recognized_shape, hwnd_correct_shape_combobox);
        }
        break;
 
    case WM_PAINT:
        {
            // Ecrit les différents labels de la fenêtre principale
            hdc = BeginPaint(hwnd, &ps);
            TextOut(hdc, 12, 5, L"Drawn shape", 11);
            TextOut(hdc, 402, 5, L"Recognized shape", 16);
            TextOut(hdc, 230, 25, L"Shape name to create :", 22);
            TextOut(hdc, 230, 130, L"Shape name to correct :", 24);
            EndPaint(hwnd, &ps);
        }
		return 0;
    case WM_DESTROY:
        PostQuitMessage(0);
        return 0;
 
    default:
        return DefWindowProc(hwnd, uMsg, wParam, lParam);
    }

qui appele la fonction :

Code :

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

Presenter_On_Create_Shape_Button (hwnd_create_shape_edit, hwnd_recognized_shape, hwnd_correct_shape_combobox);

qui a son tour :

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
void 
Presenter_On_Create_Shape_Button (HWND Hwnd_Create_Shape_Button, HWND Hwnd_Recognized_Shape_Canvas, HWND hwnd_correct_shape_combobox_2)
{
	int Shape_Entries_Exist = 0;
	char Shape_Name_To_Create[20] = "";
 
	// On récupère le nom de la forme à créer
	GetWindowText (Hwnd_Create_Shape_Button, &Shape_Name_To_Create, 20);
 
	// Verifie si "l'Edit Text" n'est pas vide
	if (strcmp(Shape_Name_To_Create, "") != 0)
	{
		// Verifie si le nom donné à la forme existe déjà
		if (Model_Does_Shape_Name_Exist(Shape_Name_To_Create) == false)
		{
			// Verifie si la forme dessinée existe déjà et si le canvas n'est pas vide
			Shape_Entries_Exist = Model_Does_Shape_Entries_Exist ();
 
			if (Shape_Entries_Exist == SHAPE_DRAWN_DOESNT_EXIST)
			{
				Model_Create_Shape (Shape_Name_To_Create);
				View_Add_Shape_Name_To_Combobox(hwnd_correct_shape_combobox_2, Shape_Name_To_Create);
				Presenter_Draw_Recognized_Neuron(Hwnd_Recognized_Shape_Canvas);
			}
			else
			{
				if (Shape_Entries_Exist == SHAPE_DRAWN_EXIST)
				{
					MessageBox(NULL, L"La forme dessinée existe déjà", L"Erreur", MB_OK);
				}
				else
				{
					MessageBox(NULL, L"Dessinez une forme", L"Erreur", MB_OK);
				}
			}
		}
		else
		{
			MessageBox(NULL, L"Le nom de la forme existe deja", L"Erreur", MB_OK);
		}
	}
	else
	{
		MessageBox(NULL, L"Entrez un nom de forme", L"Erreur", MB_OK);
	}
}

et pour finir c'est la fonction :

Code :

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

1
2
3
4
5
6
7
void
View_Add_Shape_Name_To_Combobox(HWND hwnd_correct_shape_combobox_2, wchar_t Shape_Name_To_Create[20])
{
    TCHAR Planets[10] = L"Salut";
    // Add string to combobox.
    SendMessage(hwnd_correct_shape_combobox_2, (UINT)CB_ADDSTRING, (WPARAM)0, (LPARAM)Planets);
}

et la combobox ne se remplit pas

[Xelland]

si la Combobox ne se remplit pas, cela signifie que hwnd_correct_shape_combobox est différent de hwnd_correct_shape_combobox_2 dans ta fonction View_Add_Shape_Name_To_Comboboxtu peux vérifier dans View_Add_Shape_Name_To_Combobox en mettant (pour débugger)

Code :

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

if  (hwnd_correct_shape_combobox != hwnd_correct_shape_combobox_2) MessageBox (NULL, "erreur", "erreur", MB_OK);

d'ailleurs pourquoi passer hwnd_correct_shape_combobox en paramètre, si tu l'as défini en variable globale ? (sauf si tu appelles la méthode avec d'autres combobox)

[dalfab]

Quand tu écris

Code :

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

1
2
3
4
 
    TCHAR Planets[10] = L"Mercury";
    // Add string to combobox.
    SendMessage(hwnd_correct_shape_combobox, (UINT)CB_ADDSTRING, (WPARAM)0, (LPARAM)Planets);

tu transmets la valeur Planets qui est un pointeur sur une variable locale. Si la combo n'est pas créée avec le style CBS_HASSTRINGS, elle mémorise le pointeur pas la chaîne, la variable ayant disparue entre temps cela ne marche pas.

Un moyen rapide pour vérifier, utilise une chaîne littérale:

Code :

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

1
2
3
4
 
    const TCHAR* Planets = L"Mercury";
    // Add string to combobox.
    SendMessage(hwnd_correct_shape_combobox, (UINT)CB_ADDSTRING, (WPARAM)0, (LPARAM)Planets);

[ciddu42]

j'ai essayé d'utiliser la chaine littérale comme vous m'avez dit en l'appelant depuis la fonction :

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
 
{
 HWND hwnd_correct_shape_combobox = CreateWindow(WC_COMBOBOX, TEXT(""),
        CBS_DROPDOWN | CBS_HASSTRINGS | WS_CHILD | WS_OVERLAPPED | WS_VISIBLE,
        xpos, ypos, nwidth, nheight, hwndParent, NULL, hinstance,
        NULL);
View_Add_Shape_Name_To_Combobox(hwnd_correct_shape_combobox, "hey");
}
 
View_Add_Shape_Name_To_Combobox(HWND hwnd_correct_shape_combobox_2, wchar_t Shape_Name_To_Create[20])
{
    const TCHAR* Planets = L"Mercury";
    // Add string to combobox.
    SendMessage(hwnd_correct_shape_combobox, (UINT)CB_ADDSTRING, (WPARAM)0, (LPARAM)Planets);
}

ce qui ne fonctionne pas...., alors que si je colle l'intérieur de la fonction View_Add_Shape_Name_To_Combobox après la création de la combobox :

Code :

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

1
2
3
4
5
6
7
8
    HWND hwnd_correct_shape_combobox = CreateWindow(WC_COMBOBOX, TEXT(""),
        CBS_DROPDOWN | CBS_HASSTRINGS | WS_CHILD | WS_OVERLAPPED | WS_VISIBLE,
        xpos, ypos, nwidth, nheight, hwndParent, NULL, hinstance,
        NULL);
 
    const TCHAR* Planets = L"Mercury";
    // Add string to combobox.
    SendMessage(hwnd_correct_shape_combobox, (UINT)CB_ADDSTRING, (WPARAM)0, (LPARAM)Planets);

là ça fonctionne, il y a juste un appelle de fonction qui différencie les deux codes .... et le soucis c'est que j'ai besoin de l'appelle à la fonction View_Add_Shape_Name_To_Combobox pour faire un design pattern en modele-view-presenter.....

[henderson]

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
 
HWND Combo1;
//--------------------
HWND CreateComboBox(HINSTANCE Instance, HWND AOwner, int X, int Y)
{
HWND combo = CreateWindow("COMBOBOX", "", 
                CBS_DROPDOWN | CBS_HASSTRINGS | WS_CHILD | WS_VISIBLE, 
                X, 
                Y, 
                100, 
                200, // pour dérouler par défaut plusieurs items
                AOwner, NULL, Instance, NULL); 
return combo;
}
//--------------------
void AddString(HWND ACombo, const char *AString)
{
SendMessage(ACombo, CB_ADDSTRING, 0, (LPARAM)(LPCTSTR) AString);
}

Dans la WinMain après avoir la création de la fenêtre principale :

Code :

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

1
2
3
4
5
6
7
8
9
 
Combo1 = CreateComboBox(hInstance, hwnd, 10, 10);
if(Combo1 != NULL)
	{
	AddString(Combo1, "Hello !");
	AddString(Combo1, "Item2 !");
	AddString(Combo1, "What for ?");
	AddString(Combo1, "It works !");
	}

Ca fonctionne !

[henderson]

Pour donner vie, j'ai rajouté dans la WinProc :

Code :

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

1
2
3
4
5
6
7
 
case WM_LBUTTONDOWN:
	AddString(Combo1, "Ajouter A");
	break;
case WM_RBUTTONDOWN:
	AddString(Combo1, "Ajouter B");
	break;

Donc en cliquant dans la zone client de la fenêtre principale, les deux chaines sont bien rajoutées à Combo1.

[dalfab]

Il y a quelque chose qui me perturbe dans le code.
Les fenêtres s'attendent à recevoir un texte qui peut être ou pas UNICODE (à définir dans le projet).
La chaîne de caractère à passer est un const wchar_t* ou un const char*. Il faut utiliser le bon type.
Normalement le type TCHAR est justement fait pour s'adapter à chaque cas, et ne semble pas être char!

Mais il semble que la fonction marche avec const char*, à voir si c'est le type de la chaîne qui pose un problème.

[ciddu42]

Merci pour vos réponses !

la solution de henderson ne fonctionnait pas et ne créait pas de combobox du tout, il fallait faire ceci :

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
HWND Combo1;
//--------------------
HWND CreateComboBox(HINSTANCE Instance, HWND AOwner, int X, int Y)
{
HWND combo = CreateWindow(L"COMBOBOX", L"", 
                CBS_DROPDOWN | CBS_HASSTRINGS | WS_CHILD | WS_VISIBLE, 
                X, 
                Y, 
                100, 
                200, // pour dérouler par défaut plusieurs items
                AOwner, NULL, Instance, NULL); 
return combo;
}
//--------------------
void AddString(HWND ACombo, const char *AString)
{
SendMessage(ACombo, CB_ADDSTRING, 0, (LPARAM)(LPCTSTR) AString);
}

et celà pour avoir un texte correcte :

Code :

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

1
2
3
4
5
6
7
8
Combo1 = CreateComboBox(hInstance, hwnd, 10, 10);
if(Combo1 != NULL)
	{
	AddString(Combo1, L"Hello !");
	AddString(Combo1, L"Item2 !");
	AddString(Combo1, L"What for ?");
	AddString(Combo1, L"It works !");
	}

Mais je te remercie Henderson grâce à toi j'ai pu identifier le problème !