Printable View
i hava a combo box in my application. now this is declared as int. and there are 2 values in the combo box. when i am clicking the value the index is returned. that is known. but what should i do get the original value? i cant map the combo box to CString. that is giving me some errors because this int value which is declared is used in other files too. so it has to an int.
i'm not sure i understand what you mean.
If you want to get list of string from index, you can use GetLBText() function,
or if you want to get data inside combobox, then use GetItemData().
Henky
i have a combo box which holds 2 values. 100 and 200. if i click on 100 then the index of the value is returned. i need the value not the index. i have declared the combo box as int. and there are lot of places which uses the same value. so cant change the data type. is there any way to get the value. ?
create a new member variable of type control for the combobox.
Then call GetItemData on the control.
Laitinen
i am still getting the retuen value as 0.
yes ,i think you are right.
need a solution for this.. i know its right. that is why i send a message like that.
It might be better to attach your code. So, we'll know what your problem is.Quote:
Originally Posted by Chandru080
As i wrote:
Quote:
<CODE> int value =m_combo1.GetCurSel();
CString str;
str.Format("%d",value);
m_combo1.GetLBText(value,str);
int value1 = atoi(str);</CODE>
m_combo1 is mapped with CCombobox.
now int value1 needs to be passed to another file. how do i hold this value and pass it to another file?
Check it step by step.Quote:
Originally Posted by Chandru080
Does value have the current selection?
I don't know what the str.Format is all about. You don't need it.
What's in str after the call to GetLBText()?
What's in value1?
Use the square brackets for code tags []
yes value does have the corrent selection. str.Format is to convert the interger that is selected from the combo box to string. and the value of str is a number that is there in the combo box. value1 will have the interger that is converted to string. i need to pass this value to a file. how do i do it?
Okay, but you need to answer the questions. We're trying to find out which line fails.Quote:
Originally Posted by Chandru080
str.Format does nothing useful for you here. You're converting the index of the selection to a string - why I don't know, then overwriting it with GetLBText().
You say value has the correct selection.
What's in str after you call GetLBText()? Is it right?
You can use small arrays of int and CString to do this:
I hope that helps a bit. The idea is to sychronize the two separate arrays which helps eliminate the use of atoi() function.....and can be dealt with easily.Code://------------------------------------------------------------------------------------------
// In the class header...
CArray <int, int> value;
CStringArray str_value;
CComboBox m_Combo1;
//------------------------------------------------------------------------------------------
//------------------------------------------------------------------------------------------
// In the class implementation....
//------------------------------------------------------------------------------------------
// in OnInitialUpdate() OR OnInitDialog()....
for (int i = 0; i < count /* number of elements in combo-box */; i++)
{
this->value.Add(0);
this->str_value.Add(_T(""));
}
// Set your values...if you did not while adding elements....
this->value.SetAt(0, number1);
this->value.SetAt(1, number2);
....
....
// Set the equivalent string values...
for(int k = 0; k < value.GetCount(); k++)
{
int num = this->value.GetAt(k);
str.Format(_T("%i"), num);
this->str_value.SetAt(k, str);
str.Empty();
}
// Or you can just set the numbers in the Add function as well...
// Fill up the combo-box...
int num = 0;
CString str;
for (int j = 0; j < count; j++)
{
// num = this->value.GetAt(j);
// str.Format(_T("%i"), num);
// Instead of the above two statements, use the following....
str = this->str_value.GetAt(j);
this->m_Combo1.AddString(str);
str.Empty();
}
//---------------------------------------------------------------------------------------------
// Handle the CBN_SELCHANGE event...
const int i = m_combo1.GetCurSel();
// CString str;
// m_combo1.GetLBText(value,str);
int value1 = this->str_value.GetAt(i);
//---------------------------------------------------------------------------------------------
// Now do what ever you want with value1.....
P.S.: My code may not be flawless. Regardless of that you would get a clue about how to approach your problem....
Regards,
Bhushan.
How is that better or easier? If you're going to do something like that a map would be better, but until the OP explains why what should be simple code isn't working, it's a bit premature to propose a kludgey work around, IMHO of course.Quote:
Originally Posted by bhushan1980
There are two relatively easy ways to solve your problem.
First, once you get the index from the ComboBox, use the GetLBText to retrieve the text for that item (e.g. "100"). Then convert that to an int using atoi or _ttoi. That int will be the value you want.
The second method involves changing the way you add data to the ComboBox. I assume you're using something like:You could change that toCode:myCombo.AddString(_T("100"));
Next, to retrieve the value 100 as an integer instead of the index, you can useCode:int iIndex = -1;
iIndex = myCombo.AddString(_T("100"));
myCombo.SetItemData(iIndex, 100); // this sets the "ItemData" to the integer value of 100
Hope that helps.Code:int myActualValue = 0;
myActualValue = myCombo.GetItemData(myCombo.GetCurSel());
Hi. You are probably right. My code may not be a better one at all. But I suppose this would be easy for me to understand atleast. Because, all I have done is broken down the task into smaller ones and then sychronizing them. That way I am trying to stay strictly with MFC and not going into atoi() which is a standard C/C++ function. Not that it is important, its just a way of doing things. Also, I found out from the MSDN Collect sample that we can easily synchronize MFC collection classes using combo-boxes and that would be with or without using CMap and its derivatives. Having said that, it is always better to use mapping. I did not use them just because of the fact that I have not used them frequently. Further, I did not see any reason to include them in the stated situation. Also, since I am working on a project involving multiple combo-boxes and MFC-collections, I just decided to share whatever I have. I, however, completely agree, to your mention about waiting on OP's clarification of his objective. It was a bit premature, but I don't think that I have matured enough to help any poster with all the versatile problems he/she may have. I just share whatever I have and also appreciate the suggestions made by members like you.
Thankyou,
Bhushan.