Deleting a item from ComboBox..

[mpallavi]

Hello friends, I am saving telephone numbers in a combo box control. I want to delete the current selection in the combo box. DeleteString deletes item at specified index. How do i go about this? thank you pallavi

[ThatsAlok]

mpallavi wrote: I want to delete the current selection in the combo box. DeleteString deletes item at specified index. If you are using Single Selection combo Box :-

int nIndex = pmyComboBox.GetCurSel();
if ((nIndex != LB_ERR))
{
pmyComoBox.DeleteString(nIndex);
}

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

cheers, Alok Gupta

[Jose Lamas Rios]

mpallavi wrote: I want to delete the current selection in the combo box. DeleteString deletes item at specified index. How do i go about this?

int nCurrent = combo.GetCurSel();
if (nCurrent != CB_ERR)
combo.DeleteString(nCurrent);

-- jlr http://jlamas.blogspot.com/[^]

[Jack Puppy]

If you want to delete the item from the drop list: int nCurSel = m_myComboBox.GetCurSel(); if (nCurSel != CB_ERR) m_myComboBox.DeleteString(nCurSel); If you just want to clear the text (if combo is not a drop list): m_myComboBox.SetWindowText(_T(""));

:suss: Pssst. You see that little light on your monitor? That's actually a government installed spy camera. Smile and wave to big brother!

[mpallavi]

Thank you all.. Can anyone tell me how do i get the data associated with the current selection .. i mean the value of the current selection. thanx pallavi

[ThatsAlok]

mpallavi wrote: Can anyone tell me how do i get the data associated with the current selection .. See it this Help CComboBox::GetLBText Here is small code to help :-

int nIndex=m_combo.GetCurSel();
if(nIndex==LB_ERR)
{
CString str;
m_combo.GetLBText(nIndex,str);
}

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

cheers, Alok Gupta

[Jose Lamas Rios]

ThatsAlok wrote: if(nIndex==LB_ERR) I think you mean if(nIndex!=LB_ERR) Technically, in this case CB_ERR should be used instead of LB_ERR, but in practice both are defined as -1 :) Hey, this peer programming thing does work! :-) -- jlr http://jlamas.blogspot.com/[^]

[ThatsAlok]

Jose Lamas Rios wrote: I think you mean if(nIndex!=LB_ERR) Yeap, Sorry for Silly Mistake :) Jose Lamas Rios wrote: Hey, this peer programming thing does work! Yeap, thats why we are here :)

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

cheers, Alok Gupta

[mpallavi]

Thanx.. its working. I have still more problems with ComboBox.. I am saving all these telephone numbers in a text file.. If you remember, few days back i was coding for persistent combo box. Its working perfactly fine but the problem is when i delete a number. I am creating a temp file .. i want to write all the values in this file except for the current selection.. how do i go about this? pallavi

[ThatsAlok]

mpallavi wrote: I am creating a temp file .. i want to write all the values in this file except for the current selection.. how do i go about this? Let this variable m_arr of type CStringArray contain all the phone number present in combo box.

// first get Phone Number to delete
CString szPhoneNo;

int nIndex=m_combo.GetCurSel();
if(nIndex!=CB_ERR) //thanks to Mr. Rios :)
{
m_combo.GetLBText(nIndex,szPhoneNo);
}
else
// no string to delete from file
return;

// Open Temporary File
CString szTempPath;
GetTempPath(szTempPath.GetBuffer(MAX_PATH),MAX_PATH);

// release buffer get control of Path
szTempPath.ReleaseBuffer();

//Include our File Name in path
szTempPath+=CString("temp.txt");

// Now Open Text File
CStdioFile m_File;

if(m_File.Open(szTempPath,CFile::modeCreate|CFile:modeWrite))
{

for(int i=0;i<m_arr.GetUpperBound();i++)
{
//copy data
if(m_arr.GetAt(i)!=szPhoneNo)
m_File.WriteString(m_arr.GetAt(i));

}

m_File.Close();
}

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

cheers, Alok Gupta

[mpallavi]

ThatsAlok wrote: GetTempPath(szTempPath.GetBuffer(MAX_PATH),MAX_PATH); I am getting the following error at above line: 'GetTempPathA' : cannot convert parameter 1 from 'char *' to 'unsigned long' Please help.

[ThatsAlok]

mpallavi wrote: GetTempPath(szTempPath.GetBuffer(MAX_PATH),MAX_PATH); Sorry, Just Exchange Argument :) i.e.

GetTempPath(MAX_PATH,szTempPath.GetBuffer(MAX_PATH));

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

cheers, Alok Gupta

[mpallavi]

Hi Alok.. temp file is getting created but its empty.. i am trying to find what went wrong.. :( regards pallavi

[mpallavi]

oops.. My array of strings is empty.. I used m_arr.Add(m_strPhoneNum); to add phone numbers in the array. I think the syntax is wrong.. pallavi.

[David Crow]

mpallavi wrote: m_arr.Add(m_strPhoneNum); what is m_arr, and how are you iterating throught it when writing to the file?

---

"Ideas are a dime a dozen. People who put them into action are priceless." - Unknown

[ThatsAlok]

mpallavi wrote: to add phone numbers in the array. I think the syntax is wrong.. Hi Pal, First Add all the member of ComboBox to CStringArray or directly Enumerate them, When you are entering data in temporary File.

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

cheers, Alok Gupta