Combobox contains only item Message box pop up

[faraz321]

I am trying to show a Messagebox to come up if the combobox contains only one item and then terminate the program execution when the user dismisses the message box. This is done in Winform.

private void btn_add_Click(object sender, EventArgs e)
        {
            if (combobox.SelectedIndex != -1)
                ListBox.Items.Add(combobox.SelectedItem);
            combobox.Items.RemoveAt(combobox.SelectedIndex);
 
 
            else if(combobox.SelectionLength = null) 
 
            MessageBox.Show("ComboBox must have one item left", " Warning", MessageBoxButtons.OKCancel);
        }

[GaryTexmo]

I think you need to tell us what the problem is; however, I will mention that you need to put braces around your if statements.

private void btn_add_Click(object sender, EventArgs e)
{
  if (combobox.SelectedIndex != -1)
  {
    ListBox.Items.Add(combobox.SelectedItem);
    combobox.Items.RemoveAt(combobox.SelectedIndex);
  }
  else if(combobox.SelectionLength = null)
  {
    MessageBox.Show("ComboBox must have one item left", " Warning", MessageBoxButtons.OKCancel);
  }
}

[tlhintoq]

TIP: When you are writing your question, there is a button on the tool bar that wraps the [code] tags around your copy/pasted code. It helps a bunch. Its the button with a '#' on it. More on tags. They're cool. Check'em out.

[tlhintoq]

I have xxx, then do yyy then zzz happens. Here's my code ...

Ok. You know what you have.
What you don't have is a question. Nobody here knows why you posted this since you haven't asked anything. What is it you are looking for?
I recommend you read the FAQ about How to ask a good question so the volunteers will be able to help you.

[faraz321]

Thanks Gary for correcting me. I did however, put the braces in there but somehow did not show up here. My question is still unanswered.
Question is I want my program to ensure that the ComboBox contains at least one item. If it does not print a message , using a messagebox, then terminate program execution when the user dismisses the message box. Thanks for all your help guys.

private void btn_add_Click(object sender, EventArgs e) 
{ 
  if (combobox.SelectedIndex != -1) 
  { 
    ListBox.Items.Add(combobox.SelectedItem); 
    combobox.Items.RemoveAt(combobox.SelectedIndex); 
  } 
  else if(combobox.SelectionLength = null) 
  { 
    MessageBox.Show("ComboBox must have one item left", " Warning", MessageBoxButtons.OKCancel); 
  } 
} 

[tlhintoq]

Question is I want my program to ensure that the ComboBox contains at least one item. If it does not print a message , using a messagebox, then terminate program execution when the user dismisses the message box.

What Gary is saying is : That isn't a question. Its a list of wanted behavior. There is no mention of what you have already coded to accomplish there. There is no mention of errors or exceptions.

The way it reads now, it sounds like you are saying "Please write my code / fix my program for me" and I'm sure that isn't your intent.

[faraz321]

I have already accomplish adding and removing an item from the combobox and adding it to the listbox. However, I have failed to get a messagebox to work properly when the combobox contains at least one item. Probably my choice of selectionlength = null is not appropriate. I am new to C sharp.

[tlhintoq]

Ok... Let me see if I have this correct.
You want to produce the MessageBox if and only if the combo box text entry area is blank. Is that right?
Can you simply change the combobox style from DropDown to DropDownList?
This way there is *no* text area: The user is forced to select one of the .Items from the combo box.

I generally feel it is better to avoid a problem then try to compensate for one.

[faraz321]

Combobox must have one last item and a messagebox should pop up indicating to terminate the program if one tries to remove the last item. I wish it would be simple to just use the dropdownlist as you sugggested but its not as simple as that. Thanks for your suggestions

[tlhintoq]

If there must be at least 1 item, then you should be checking the count of items, not the selection.

Items.Count > 0