VPF::DataGridView и многопоточность - Форум программистов

Resident

Дата 4.4.2010, 16:21 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 75
Регистрация: 25.9.2006

Репутация: нет
Всего: 0

Есть функция, которая заполняет DataGridView данными. При обычном вызове этой функции программа зависает, пока функция не выполнится. Решил вызвать ее в отдельном потоке и все бы хорошо, если б не скролл в DataGridView. После выполнения потока, он "замораживается", да и во время выполнения потока, появляется по кускам как то.

Вот код функции

Код


private: System::Void checkBids() {
                 this->button1->Enabled  = false;
                 this->button2->Enabled  = true;
                 this->button3->Enabled  = false;
                 this->textBox1->Enabled = false;
                 this->textBox2->Enabled = false;
                 this->textBox3->Enabled = false;
                 this->textBox4->Enabled = false;

                 this->dataGridView1->Rows->Clear();

                 array<double>^ bids;

                 for ( int i = 0; i < this->textBox1->Lines->Length; i++) {
                     System::String^ umaxXmlUrl = L"http://xml.umaxfeed.com/xmlfeed.php?&q=" + this->textBox1->Lines[i]->Trim() + L"&aid=" + this->textBox2->Text->Trim() + L"&auth=" + this->textBox3->Text->Trim() + L"&ip=" + this->textBox4->Text->Trim();

                     System::Xml::XmlDocument doc;
                     
                     doc.Load(umaxXmlUrl);

                     System::Xml::XmlNodeList^ list = doc.GetElementsByTagName(L"bid");
                     
                     bids = gcnew array<double>(list->Count);

                     for ( int j = 0; j < list->Count; j++ ) {
                         bids[j] = System::Math::Round( System::Convert::ToDouble( list->Item(j)->InnerText->Trim()->Replace(L".", L",") ), 2);
                     }

                     this->dataGridView1->Rows->Add(i+1,
                         this->textBox1->Lines[i]->Trim(),
                         System::Math::Round( System::Linq::Enumerable::Max(bids), 2 ),
                         System::Math::Round( System::Linq::Enumerable::Average(bids), 2 ),
                         System::Math::Round( System::Linq::Enumerable::Min(bids), 2 ),
                         System::Math::Round( System::Linq::Enumerable::Sum(bids), 2) );
                 }

                 MessageBox::Show(L"Finish", L"", MessageBoxButtons::OK, MessageBoxIcon::Information);

                 this->button1->Enabled  = true;
                 this->button2->Enabled  = false;
                 this->button3->Enabled  = true;
                 this->textBox1->Enabled = true;
                 this->textBox2->Enabled = true;
                 this->textBox3->Enabled = true;
                 this->textBox4->Enabled = true;
             }

Поток создаю так

Код


ThreadStart^ threadDelegate = gcnew ThreadStart( this, &Form1::checkBids );
this->checkBidsThread = gcnew Thread( threadDelegate );

this->checkBidsThread->Priority = ThreadPriority::Highest;
this->checkBidsThread->Start();
this->checkBidsThread->Join();

С многопоточностью никогда не работал, поэтому даже предположить не могу в чем может быть проблема, т.к. ошибок нет ни на этапе компиляции, ни на этапе выполнения программы.

jonie

Дата 4.4.2010, 16:36 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 5613
Регистрация: 21.8.2005
Где: Владимир

Репутация: 5
Всего: 118

ну вот тут у тебя замораживается основной поток:

Код


this->checkBidsThread->Join();

фактически у тебя в коде есть два потока, но работают они как один: первый создает второй и ждет его окончания в блокировке.

Тебе надо: в потоковой функции сделать

Код


while(true) { 
  вот тут получать данные;
  вот тут сравнивать с предыдущими данными (зачем весь грид перезаполнять?)
  вот тут потокобезопасно (через BeginInvoke()) обновлять данные, на которые забинден грид *
}

*) я предпологаю что ты не будешь первокурсником и прочитаешь что такое биндинг 8) в общих чертах тебе надо будет сделать класс - олицетворяющий строку твоей таблицы, унаследовать его от NotifyPropertyChanged, в коде программы использовать BindingList<ТвойТипСтроки> , и именно его использовать как DataSource твоего грида.
...

--------------------

Что-то не поняли? -> Напейтесь до зеленых человечков... эта сверхцивилизация Вам поможет...

Resident

Дата 4.4.2010, 18:06 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 75
Регистрация: 25.9.2006

Репутация: нет
Всего: 0

А без создания дополнительного класса никак не обойтись? Чем плох dataGridView1->Rows->Add() в данном случае? Попробовал так сделать

Код


array<double>^ bids = gcnew array<double>(3);

for ( int i = 0; i < bids->Length; i++ ) {
    bids[i] = i*2;
}

this->dataGridView1->DataSource = bids;

В гриде появилось 3 строоки, но пустые.

jonie

Дата 4.4.2010, 19:31 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 5613
Регистрация: 21.8.2005
Где: Владимир

Репутация: 5
Всего: 118

Цитата

Чем плох dataGridView1->Rows->Add() в данном случае?

тем что это г-код. Есть паттерны программирования вроде MVC, стоит озаботиться чтением...

--------------------

Что-то не поняли? -> Напейтесь до зеленых человечков... эта сверхцивилизация Вам поможет...

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| Разработка Windows Forms \| Следующая тема »