Question

In My application have time consuming process.There fore i try to do that operation in separate thread.Even i Stared it separate thread my Main UI still freezes during the time of long running process.But still i couldn’t figure out the reason for that?Some thing wrong in my code?

My Event Hander Code

private void BtnloadClick(object sender, EventArgs e)
{
   if (null != cmbSource.SelectedItem)
   {
     string selectedITem = ((FeedSource) cmbSource.SelectedItem).Url;
     if (!string.IsNullOrEmpty(selectedITem))
     {                    
        Thread starter = new Thread(() => BindDataUI(selectedITem));
        starter.IsBackground = true;
        starter.Start();
     }            
}

private void BindDataUI(string url)
{
   if (feedGridView1.InvokeRequired)
   {
      BeginInvoke(new Action(() => BindDataGrid(url)));
   }
   else
     BindDataGrid(ss);
}


 private void BindDataGrid(string selectedItem)
  {
    for (int i = 0; i < 10; i++)
    {
      //Time consuming Process
    }
 }

Answer 1

Your thread is completely useless 🙂

In your thread you are executing BindDataUI which marshals the execution back to the UI thread using Invoke.

Your complete code is equivalent to this:

private void BtnloadClick(object sender, EventArgs e)
{
   if (null != cmbSource.SelectedItem)
   {
     string selectedITem = ((FeedSource) cmbSource.SelectedItem).Url;
     if (!string.IsNullOrEmpty(selectedITem))
     {     
        BindDataGrid(selectedITem);
     }            
}

private void BindDataGrid(string selectedItem)
{
    for (int i = 0; i < 10; i++)
    {
      //Time consuming Process
    }
}

It would be better to only marshal these parts of BindDataGrid to the UI thread that really need to run on this thread because they need to update the UI.