Как задать пул выполняемых task'ов для формы?

Question

[Unmaskedguy]

Вот я думаю можно создать какой нибудь список задач в который будут вписаны задачи которые необходимо выполнить до закрытия программы, а так же что-бы это не замораживало основной поток и не конфликтовало через раз само с собой?

Принцип задачи:
Существует Программа с классами:
Values - Значения
Attributes - Атрибуты
Changable - Изменения

Существует файл Excel'а с ячейками где хранятся все значения для считывания (около 100 000 Строк)

Считывание происходит как по мне крайне медленно. В принципе за секунду обрабатывается около 10-15 строк.
При этом чем дольше идет считывание тем медленнее работает программа.
Настроил 2 метода, что конфликтуют между собой во время работы программы. Оно и понятно обращаются они то к одному файлу Excel.

1. Считывание ячеек и создание объектов на их основе для списка вывода в listbox.
2. Запись в Excel файл строки для пропуска, или её полное удаление. И так же удалить объект из listbox'a.

ПС. Пока что не разобрался как тут выкидывать проекты, но если нужен будет весь исходник для решения вопроса по пулу исполняемых задач - подскажите как выложить и я выкину весь солюшн сюда.
ППС:
Как я запускаю Task:

await Task.Factory.StartNew(
                                             () => ReadForm(i),
                                             TaskCreationOptions.LongRunning);

Comments

[Пычев Анатолий]

Считывание происходит как по мне крайне медленно.

Пробуйте считывать не построчно, а массивом.

Может попробуйте построить алгоритм через VBA. У меня 70к-80к строк считываютсч за несколько секунд в массив, а дальше можно творить.

[Борис Животное]

Unmaskedguy, замедляется, похоже, потому что объект Changes имеет тип List, а искать каждый раз в пополняющемся списке всё дольше и дольше. Метод Find в классе List обходит во время поиска весь массив по новой каждый его вызов, пока не найдёт искомое значение.

[Борис Животное]

Unmaskedguy, так на вскидку, свойство Changes должно иметь такой вид:

public Dictionary<Guid, Changeable> Changes { get; private set; }

А Changeable (тип свойства с именем Guid не string, а Guid):

public class Changeable  
{
     public Guid Guid { get; set; }
}

[Борис Животное]

И ещё, правильно Changeable в вопросе у вас одна буква пропущена, а в коде другая. Писать правильно по-английски тоже важно.

[Борис Животное]

Ещё непонятно зачем вызывать метод ToString() у класса string, который возвращает this.

GUID = ws.Cells[i, 7].Text.ToString();

[Борис Животное]

Как запускается Task это последнее, что вообще может интересовать по данной задаче. Это совершенно неважно. Важен алгоритм.

[Борис Животное]

Код нужно писать не в "формах", а в отдельных классах, которые, возможно, генерируют события, которые обрабатывают другие классы обработчики, а всё это дело можно поместить уже хоть в ту же "форму" и то не факт, что стоит. Например, класс читатель, класс писатель. В двух словах не объяснишь, там нужно чётко понимать задачу для начала.

Answer 1

[Sergey В.]

Считывать и тут-же обрабатывать - не очень хорошая идея.
Попробуйте сначала считать всё в память, а потом многопоточно это всё обработать.

Comments

[Unmaskedguy]

В смысле вначале вначале сделать большой список из строк? Просто у меня выглядит вот так все:

private void ReadForm(int i)
        {
            //строка для проверки, что список не кончился (пока что проверку убрал)
            String a = ws.Cells[i, 1].Text.ToString();
            //строка считающая  что эту ячейку уже не надо проверять
            String b = ws.Cells[i, 10].Text.ToString();

            if (String.IsNullOrWhiteSpace(b))
            {
                //считываемые параметры
                String date, atribute, GUID, whomadeit, name, obozn, group;
                date = ws.Cells[i, 4].Text.ToString();
                GUID = ws.Cells[i, 7].Text.ToString();
                whomadeit = ws.Cells[i, 3].Text.ToString();
                atribute = ws.Cells[i, 5].Text.ToString();
                name = ws.Cells[i, 6].Text.ToString();
                obozn = ws.Cells[i, 9].Text.ToString();
                group = ws.Cells[i, 2].Text.ToString();
                //в строке atribute хранятся записи о наименовании атрибута и его измененном значении на этот момент или он создавался.
               
                if (atribute.Contains("Изменен"))
                {
                    // разбивание строки на атрибут и его значение исходя из записи
                    String[] matrix_atribute;
                    matrix_atribute = atribute.Split('\n');
                    foreach (String Attr_data in matrix_atribute)
                    {
                        String[] attr_value = Attr_data.Split('=');
                        if (Changes.Exists(Z => Z.GUID == GUID))
                        {
                            try
                            {
                                //Внесение записи об изменении в лист Changes из класса Form1 изменений объекта через метод UpdateString 
                                Changes.Find(U => U.GUID.Contains(GUID)).UpdateString(attr_value[0], attr_value[1], date, whomadeit, i);
                            }
                            catch
                            {
                                Changes.Find(U => U.GUID.Contains(GUID)).UpdateString("Ошибка чтения!", "ОШИБКА ЧТЕНИЯ!!!", date, whomadeit, i);
                            }
                        }
                        else
                        {
                             //создание нового объекта с его изменениями
                            Changes.Add(new Changeble(attr_value[0], attr_value[1], date, GUID, whomadeit, i, name, obozn, group));
                        }
                    }
                }
                // Если это не изменения значит это создание объекта - значит это перегруженный метод с записью о создании объекта и запись имени его создателя
                else
                {
                    GUID = ws.Cells[i, 7].Text.ToString();
                    if (Changes.Exists(Z => Z.GUID == GUID))
                    {
                        try
                        {
                            Changes.Find(U => U.GUID.Contains(GUID)).writeCreator(ws.Cells[i, 3].Text.ToString(), i);
                        }
                        catch
                        {
                            Changes.Find(U => U.GUID.Contains(GUID)).writeCreator("АВТОР НЕИЗВЕСТЕН", i);
                        }
                    }
                }
            }
        }

Конечно есть пробелы в логике записи, но их не спешу исправлять пока не пойму как можно было бы ускорить перепись всего файла Excel в программу.

[Борис Животное]

Где вы такой стиль оформления кода взяли? Всё не по канонам :).
Скиньте проект на bitbucket или github и дайте ссылку или залейте на какой-нибудь облачный сервис.
И скажите, вам точно нужна помощь, чтобы я впустую не тратил кучу времени, а вы не воспользовались кодом.

[Unmaskedguy]

Борис Животное, самоучка. Вечером смогу залить код часов в 7. Если сможете помочь, буду благодарен.