killing me comboboxs

[WaterNut]

Ok, first I apologize for not being able to follow the multiple hits on the searches found on this site and the msdn example I found.

I am having an issue trying to assign data to a combobox. I want to assign the data dynamically, not assign it before I compile it, specifically I want to assign via a text file. I figure if I can learn from this example from msdn:

	// The pointer to my combo box.
	CComboBox* pIDC_COMBO1;

	// Add 20 items to the combo box.
	CString str;

	for (int i=0;i < 20;i++)
	{
	   str.Format(_T("item string%d"), i);
	   pIDC_COMBO1->AddString( str );
	}

I added my combobox name as the pointer and though it compiles fine it does not add anything? I am confused. Can anyone help explain this example?

[novacain]

Nothing is added because the pointer to the combo box has not been set to 'point' at an ACTUAL combo box.

(The MSDN example uses a pointer defined as external ie global)


Try adding a member var to the dialogs class with the class wizard.

Or send in the HWND of the dialog (or combo) to the function. Then use something like

//get the combos HWND using parent dlgs hwnd and its ID#
HWND hCombo = GetDlgItem(hDialog,  IDC_YOUR_COMBO );
//add the string
SendMessage(hCombo, CB_ADDSTRING, (WPARAM) 0 , (LPARAM) sString);

[WaterNut]

Thanks novacain, your right on the money. That fixed it. I apologize for the late response, but I really appreciate the help. thanks-

[WaterNut]

Well, I thought I had it, I am successfully pointing to the variable, but now I get a memory violation error when it tries to assert the value to the variable I have assigned to the combobox.

// The pointer to variable assigned to my combo box.
	CComboBox* pComboString;

	// Add 20 items to the combo box.
	CString str;

	for (int i=0;i < 20;i++)
	{
	   str.Format(_T("item string%d"), i);
	   pComboString->AddString( str );
	} 

	pComboString->AddString( str );

[cyreon]

I would suggest trying novacain's suggestion and instead using your method. However, I still don't see any code that actually points to the combobox.

CComboBox* pComboString; // still no value set

This could be your problem. I don't program C++, but to me that seems to be your problem. I would still go with the SendMessage method, however.