GUI cross thread operations

See more:

C#

I have winForm project that has a dataGridView that's bound to a dataTable that's updated from a workerThread. When I update my dataTable on the workerThread I use dataGridView.BeginInvoke(). I do this to handle any cross threading issues and to prevent blocking. I also have a winForm timerControl that fires every second and does some clean operations on my dataTable.

My question is, by my dataTable being updated with data from and on the workerThread and cleaned up on the mainThread with the timerControl am I having threading issues here?

Example:

class Form1: Form
{
   MyDataCls myDatCls;
   DataGridView grid;
   DataTable dataTbl;
   Timer tmr;

   public Form1()
   {
      grid = new DataGridView();
      
      dataTbl = new DataTable();

      DataColumn col;
      col = new DataColumn();
      col.ColumnName = "Time"
      dataTbl.Columns.Add(col);

      col = new DataColumn();
      col.ColumnName = "Name"
      dataTbl.Columns.Add(col); 

      col = new DataColumn();
      col.ColumnName = "Addrs"
      dataTbl.Columns.Add(col);

      grid.DataSource = dataTbl;

      myDataCls = new MyDataCls();
      myDataCls.PushData += new PushDataEvntHndlr(myDataCls_PushData);
      
      tmr = new Timer();
      tmr += new Timer(tmr_Tick);
      tmr.Interval = 1000;
      tmr.Start();
   }

   // this is the handler I use to receive data that
   // comes from a workerThread
   private void myDataCls_PushData(string[] someData)
   {       
       if(grid.InvokeRequired)
       {
          DataRow row = dataTbl.NewRow();

          grid.BeginInvoke(new MethodInvoker(delegate()
          {
             row["Time"] = someData[0];
             row["Name"] = someData[1];
             row["Addrs"] = someData[2];
             dataTbl.Rows.Add(row);
          }));
       }
       else
       {
          row["Time"] = someData[0];
          row["Name"] = someData[1];
          row["Addrs"] = someData[2];
          dataTbl.Rows.Add(row);
       }
   }

   // this is the timer handler that does the cleanup
   private void tmr_Tick(object sender, EventArgs e)
   {
      if (dataTbl.Rows.Count > 0)
      {
         foreach (DataRow row in dataTbl.Select())
         {
             if (row["Name"] == "Joe Doe")
                 row["Name"].Delete();
         }
      }
   }
}