Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > Разработка Windows Forms > DataGridView и многопоточность


Автор: Resident 4.4.2010, 16:21
Есть функция, которая заполняет 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
ну вот тут у тебя замораживается основной поток:
Код

this->checkBidsThread->Join();
фактически у тебя в коде есть два потока, но работают они как один: первый создает второй и ждет его окончания в блокировке.

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

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

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

Автор: Resident 4.4.2010, 18:06
А без создания дополнительного класса никак не обойтись? Чем плох 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
Цитата

Чем плох dataGridView1->Rows->Add() в данном случае?
тем что это г-код. Есть паттерны программирования вроде MVC, стоит озаботиться чтением...

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)