[coastal5]

Hello i'm new to windows form programming in c#. Here's my problem: I have 2 listbox that are associated and I want to remove elements from both listbox with the remove button. When i try to remove elements from both listbox i get "InvalidArgument=Value of '2' is not valid for 'SelectedIndex'".

 private void Team1_SelectedIndexChanged(object sender, EventArgs e)
        {
            Team2.SelectedIndex = Team1.SelectedIndex;
        }

        private void Team2_SelectedIndexChanged(object sender, EventArgs e)
        {
            Team1.SelectedIndex = Team2.SelectedIndex;
        }

       

        private void btnRemove_Click(object sender, EventArgs e)
        {
            Team1.Items.Remove(Team2.SelectedItem);
            Team2.Items.Remove(Team1.SelectedItem);
        }

[mouse88]

Which bit of code are you getting the error on?

For the first part of your code you should check the value of the SelectedIndex first as it will change once when it leaves the current item and again when it selects the new item.

[coastal5]

The error is coming from the SelectedIndexChanged event. I'm not sure how to correct it.

[jayfella]

at a glance i think its because when you remove the item, the index changes and fires the IndexChanged event. Try:

private void Team2_SelectedIndexChanged(object sender, EventArgs e)
{
    Team1.SelectedIndex = (Team1.Items.Count >= Team2.SelectedIndex) ? Team2.SelectedIndex : -1;
}

which is short-hand for:

if (Team1.Items.Count >= Team2.SelectedIndex)
    Team1.SelectedIndex = Team2.SelectedIndex;
else
    Team1.SelectedIndex = -1;

This post has been edited by jayfella: 20 February 2011 - 03:08 PM

[tlhIn`toq]

Nearly all collections in .NET are zero indexed.

If you have two items they are indexes 0 and 1.
If you have five items they are indexes 0, 1, 2, 3, 4

Also take a look at line 15
As soon as you remove the selected item it means that some other item is therefore suddenly selected, triggering your .SelectedItemChanged event handler. Once that selection happens you no longer have that many items in both listboxes because you've deleted one.

In other words, as soon as line 15 executes BUT BEFORE LINE 16 EXECUTES the selected index changed handler gets triggered. So between lines 15 and 16 your two listboxes are not the same.

Breakpoints. Breakpoints are your friend. If you were to walk through this and look at the listbox.items collection you would see this happen.

[coastal5]

jayfella, I still get the same errors

[coastal5]

I understand whats going on but I'm not to sure on how to fix it?

tlhIn`toq, on 20 February 2011 - 03:10 PM, said:

Nearly all collections in .NET are zero indexed.

If you have two items they are indexes 0 and 1.
If you have five items they are indexes 0, 1, 2, 3, 4

Also take a look at line 15
As soon as you remove the selected item it means that some other item is therefore suddenly selected, triggering your .SelectedItemChanged event handler. Once that selection happens you no longer have that many items in both listboxes because you've deleted one.

In other words, as soon as line 15 executes BUT BEFORE LINE 16 EXECUTES the selected index changed handler gets triggered. So between lines 15 and 16 your two listboxes are not the same.

Breakpoints. Breakpoints are your friend. If you were to walk through this and look at the listbox.items collection you would see this happen.

[tlhIn`toq]

coastal5, on 20 February 2011 - 04:31 PM, said:

I understand whats going on but I'm not to sure on how to fix it?

Very straightforward logic here:
If the problem is that you are reacting to the SelectedIndexChanged event while you are still deleting items...
then stop listening for the SelectedIndedChanged event while you are deleting items.

• At the start of your delete method unsubscribe from the SelectedIndexChanged events.
  
• Do your deletes
  
• Re-subscribe to the events.

private void Team1_SelectedIndexChanged(object sender, EventArgs e)
       {
           Team2.SelectedIndex = Team1.SelectedIndex;
       }

       private void Team2_SelectedIndexChanged(object sender, EventArgs e)
       {
           Team1.SelectedIndex = Team2.SelectedIndex;
       }

      

       private void btnRemove_Click(object sender, EventArgs e)
       {
           Team1.SelectedIndexChanged -= SelectedIndexChanged;
           Team2.SelectedIndexChanged -= SelectedIndexChanged;


           Team1.Items.Remove(Team2.SelectedItem);
           Team2.Items.Remove(Team1.SelectedItem);

           Team1.SelectedIndexChanged += SelectedIndex;
           Team2.SelectedIndexChanged += SelectedIndexChanged;
       }

[coastal5]

doesn"t seem to work either?

[tlhIn`toq]

Quote

Breakpoints. Breakpoints are your friend. If you were to walk through this and look at the listbox.items collection you would see this happen.

Have you put breakpoints in the deletion method?
When you look at the items collection of each listbox what are the contents?

Write it out on paper if you need to.
Make a list of Team1.Items
Make a list of Team2.Items

Breakpoint at line 15 and and walk through it. Look at the collections to see what is happening.
Feel free to post screen shots along the way if you need to so we can see what you see and help guide you to it. But we don't have your project so we can't exactly duplicate your results.

[coastal5]

tlhIn`toq, on 20 February 2011 - 07:17 PM, said:

Quote

Breakpoints. Breakpoints are your friend. If you were to walk through this and look at the listbox.items collection you would see this happen.

Have you put breakpoints in the deletion method?
When you look at the items collection of each listbox what are the contents?

Write it out on paper if you need to.
Make a list of Team1.Items
Make a list of Team2.Items

Breakpoint at line 15 and and walk through it. Look at the collections to see what is happening.
Feel free to post screen shots along the way if you need to so we can see what you see and help guide you to it. But we don't have your project so we can't exactly duplicate your results.

I just have 2 listbox with 4 random numbers inside each listbox and a remove button. For some reason i cannot remove both selected numbers from each listbox?

[tlhIn`toq]

I'll say it again, but more directly this time.

• Put a breakpoint at line 15.
  
• Open the Locals and Autos pallet.
  
• Have them set so you can see all the values of each listbox.list collection.
  
• Take a screen shot at the start of each line from 15 to the end of the method. That should amount to about 5 screen shots.
  
• Post them to this thread so we can see what you see.