2

Есть поток, в нём цикл . В comeBox выводиться почему то 5 раз цифра 6. Что я делаю не так?

private void button11_Click(object sender, EventArgs e)
        {
                System.Threading.Thread myThread = new System.Threading.Thread(get_lists);
                myThread.Start();
        }
        void get_lists()
        {
        for (int i = 1; i <= 5; i++)
        {
        BeginInvoke(new MethodInvoker(() => comboBox1.Items.Add(i)));
        }
        }

1 ответ 1

2

Делайте так:

for (int i = 1; i <= 5; i++)
{
    int closed_i = i;
    BeginInvoke(new MethodInvoker(() => comboBox1.Items.Add(closed_i)));
}

Проблема в том, что вы делаете closure на переменную цикла, а не на её значение. А она меняется! Код comboBox1.Items.Add(i) начинает выполняться не немедленно во время вызова BeginInvoke, а через какое-то время, в будущем. В тот момент, когда он начнёт выполняться, цикл скорее всего уже закончится, и переменная i примет финальное значение 6.

Классическая статья Эрика по теме.

5
  • Да уж. С веб-программирования переходить на прикладное трудно =(
    – MadKeks
    20 июл 2014 в 22:53
  • @MadKeks: в javascript'е та же фигня :-(
    – VladD
    20 июл 2014 в 22:56
  • Мне повезло и с циклами в JS не приходилось работать. Только в PHP
    – MadKeks
    20 июл 2014 в 22:57
  • @MadKeks: Удачи вам в изучении C#. Мой совет — сразу бросайте устаревшие WinForms и садитесь на WPF.
    – VladD
    20 июл 2014 в 23:00
  • Времени у меня много, успею. Спасибо вам за помощь.
    – MadKeks
    20 июл 2014 в 23:06

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.