0

У меня есть задача - создать табличку из текстбоксов, ширина TableWidth, высота TableHeigth. Для больших значений (50*50) табличка создается довольно долго, поэтому решил сделать параллельную отрисовку по строчкам. Но данный код полностью весит программу. Подскажите пожалуйста, что может быть не так? Есть предположения что где-то выполняется потоконебезопасная операция, но где именно...

Parallel.For(1, table.TableWidth + 1, x =>
{
    var i = x;
    for (int y = 1; y <= table.TableHeight; y++)
    {
        var j = y;
        TextBox textbox;
        if (!textBoxes.ContainsKey(new Point(i, j)))
        {
            textbox = new TextBox();
            textBoxes.Add(new Point(i, j), textbox);
            textbox.BorderStyle = BorderStyle.Fixed3D;
            Controls.Add(textbox);
        }

        textbox = textBoxes[new Point(i, j)];
        textbox.Location = new Point(ColumnsCoords[i], RowsCoords[j]);
        textbox.Width = table.ColumnsWidth[i];
        textbox.Height = table.RowsHeight[j];
        textbox.Text = table[i, j].Data;

        textbox.GotFocus += (s, a) =>
        {
            focusedCellCoords.Text = new Point(i, j).ToString();
            currentTextBox = textbox;
            focusedCell.Text = textbox.Text;
        };
        textbox.TextChanged += (s, a) =>
        {
            PushData(new Point(i, j), textbox.Text);
            currentTextBox = textbox;
            focusedCell.Text = textbox.Text;
        };
    }

});
8
  • 2
    Если мне не изменяет память весь UI должен быть в одном потоке и разнести его по разным потокам требует не только бубен, но и опытного шамана. Ну и традиционный вопрос - WPF или WinForms?
    – rdorn
    5 июн 2016 в 17:45
  • В данном случае использую WinForms 5 июн 2016 в 17:51
  • А что означает «долго»? И я не вижу в вашем коде серьёзных вычислений, так что сильно сомневаюсь, что разбивка на потоки поможет. И да, все контролы должны создаваться в UI-потоке, так что вынести в фоновый поток можно разве что длинные вычисления.
    – VladD
    5 июн 2016 в 18:07
  • 1
    Слишком много вопросов по вашему коду. Что такое table? Коллекция textBoxes потокобезопасная? Планируется ли в дальнейшем удаление этих текстбоксов? События в виде лямбд не дадут отписаться от них - возможна утечка памяти. 5 июн 2016 в 23:36
  • И вообще, не проще ли взять компонент типа DataGridView? Или, например, TableLayoutPanel для размещения текстбоксов? 5 июн 2016 в 23:37

2 ответа 2

4

Даже если бы приведенный вами код не вешал программу - он бы не смог работать. Дело в том, что любой контрол слушает события в том потоке, в котором был создан. Но цикл обработки сообщений крутится только в главном потоке приложения, потоки пула не могут получить оконных сообщений. Поэтому, ваши текстбоксы не будут реагировать на события (в них даже нельзя ввести текст, ввод текста тоже требует активного цикла обработки сообщений).

Единственное возможное решение - выкинуть Parallel.For и заменить его на обычный цикл for.


Что же до "подвешивания" программы - рискну предположить, что происходит взаимоблокировка в строчке Controls.Add(textbox); - наверняка этому методу надо отправить оконное сообщение в главный поток UI, а он как раз занят циклом for. Но это гадание, если нужна точность - запустите отладчик и проверьте где оно висит точно.

Еще одна ваша возможная ошибка - судя по коду, textBoxes - это обычный словарь (Dictionary<Point,TextBox>). Но со словарем нельзя работать из нескольких потоком одновременно! Если бы код вообще имел смысл, вам следовало бы использовать ConcurrentDictionary<,> для той же цели.

-1

Есть смысл реализовать в виде обычного цикла, как и написали выше, но обернуть его в async/await. Тогда долгий цикл не будет вешать GUI.

Можно посмотреть тут https://stephenhaunts.com/2014/10/14/using-async-and-await-to-update-the-ui-thread/

1
  • Не поможет. В цикле создаются контролы (его нельзя вытащить в другой поток) и нет долгих операций. 9 июн 2016 в 9:18

Ваш ответ

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

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