Issues With Manual Event Handlers WPF C#

Question

I am programmatically creating menu items and adding event handlers to them, and for some reason I can't seem to get each menu item's event to register its own unique information. For example:

            for (int i = 0; i < this.Slideshow.Speeds.GetLength(0); i++) 
            {

                MenuItem m = new MenuItem() { Header = this.Slideshow.Speeds[i, 0] };

                //Set initial checked MenuItem
                if (i == 3)
                {
                    m.IsChecked = true;
                }


                m.Click += (s, e) => {

                    foreach (MenuItem m_ in this.slideshowSpeedMenuItem.Items)
                    {
                        m_.IsChecked = false;
                    }

                    m.IsChecked = true;

                    this.Slideshow.CurrentSpeed = (int)this.Slideshow.Speeds[i, 1];

                    //i is always the same. i SHOULD be getting a different entry in the array
                    MessageBox.Show(i.ToString());

                };

                this.slideshowSpeedMenuItem.Items.Add(m);

            }

Why is i always the same for each menu item's click event? The idea is to have each menu item do some stuff with a different given speed, as indicated by i.

You're capturing `i` in the lambda expression, and then incrementing it. You need a *copy* of `i`. I'll find a good question that this is a duplicate of... — Jon Skeet, May 23 '14 at 14:15
Humbug - the dupe I logged is for `foreach` instead of `for`. Finding a better one now... — Jon Skeet, May 23 '14 at 14:16
http://stackoverflow.com/questions/7867906/ is probably closer. — Jon Skeet, May 23 '14 at 14:17

Issues With Manual Event Handlers WPF C#

0 Answers0