Working with threads C#

Question

I need some help trying to figure what I'm doing wrong. I'm trying to get a collection of items from the system log on a separate thread to keep the form from being frozen during the collection process. I can get the background worker to grab them all, but I am having some issues add them to the ListBox on the form.

private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
{

  foreach (System.Diagnostics.EventLogEntry entry in eventLog1.Entries)
  {
     listBox1.Items.Add(
        entry.EntryType.ToString() + " - " + 
        entry.TimeWritten + "     - " + 
        entry.Source);
  }
}

Obviously this doesn't work as expected, since there are 2 separate threads, and you can't change objects on different threads, as I have found out. So, If someone could guide me in the right direction, I would be thankful.

Take a look on this answer: stackoverflow.com/questions/1136399/… — Klinger
– Klinger, Commented Jun 12, 2011 at 7:25

Alex Aza · Accepted Answer · 2011-06-12 07:28:25Z

You should not access UI elements from non-UI thread. Run ReportProgress, which will be synced with UI thread.

private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
{
    foreach (System.Diagnostics.EventLogEntry entry in eventLog1.Entries)
    {
        var newEntry = entry.EntryType + " - " + entry.TimeWritten + "     - " + entry.Source;
        backgroundWorker1.ReportProgress(0, newEntry);
    }
}

void backgroundWorker1_ProgressChanged(object sender, ProgressChangedEventArgs e)
{
    var newEntry = (string)e.UserState;
    listBox1.Items.Add(newEntry);
}

Make sure you enable WorkerReportsProgress.

backgroundWorker1.WorkerReportsProgress = true;

and subscribed to ProgressChanged

backgroundWorker1.ProgressChanged += backgroundWorker1_ProgressChanged;

Another approach is to call Control.Invoke inside

private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
{
    foreach (System.Diagnostics.EventLogEntry entry in eventLog1.Entries)
    {
        var newEntry = entry.EntryType.ToString() + " - " + entry.TimeWritten + "     - " + entry.Source;
        Action action = () => listBox1.Items.Add(newEntry);
        Invoke(action);
    }
}

But with this approach you don't need BackgroundWorker as whole point of it is to use ProgressChanged and RunWorkerCompleted event handler which are synced with the UI thread.

Chen Kinnrot · Accepted Answer · 2011-06-12 07:20:42Z

1

You need to use the report progress option of the background worker. google this

answered Jun 12, 2011 at 7:20

Chen Kinnrot

21.1k18 gold badges82 silver badges142 bronze badges

Comments

Rob · Accepted Answer · 2011-06-12 07:39:26Z

1

Try this, very simple way of invoking an action on the Control's thread:

private void Form1_Load(object sender, EventArgs e)
{
    var bw = new BackgroundWorker();
    bw.DoWork += DoWork;
    bw.RunWorkerAsync();
}
private void DoWork(object sender, DoWorkEventArgs e)
{
    var itemList = new List<int> {1, 22, 3, 4};
    var func = new Action<int>(itemToAdd => listBox1.Items.Add(itemToAdd));
    foreach (var item in itemList)
    {
        listBox1.Invoke(func, item);
    }
}

answered Jun 12, 2011 at 7:39

Rob♦

27.5k16 gold badges89 silver badges103 bronze badges

Comments

crypted · Accepted Answer · 2011-06-12 07:48:08Z

There should not be any issues as you are using BackgroundWorker. All the call to the callback method runs on the same UI context.

EDIT:

if you want to report progress, you need to store SynchronizationContext.Current to preferably startup. or you can Use IsInvokeRequired pattern. Here is how I use SynchronizationContext

 private SynchronizationContext uiContext;
        public Form1()
        {
            uiContext = SynchronizationContext.Current;
            InitializeComponent();
            FillItem();
        }

I have following code , and it is working like charm.

    public void FillItem()
            {
                BackgroundWorker worker = new BackgroundWorker();
                worker.WorkerReportsProgress = true;
                worker.DoWork += (a, b) =>
                                     {
                                         int i = 0; //Percentage complete, roll your own logic.
                                         foreach (var eventLog in EventLog.GetEventLogs())
                                         {
                                             foreach (EventLogEntry entry in eventLog.Entries)
                                             {
                                                 this.listBox1.Items.Add(entry.Message);
 uiContext.Post(z=>worker.ReportProgress(i++),null);

                                             }
                                         }


                                     };
                worker.RunWorkerAsync();
                worker.ProgressChanged += (a, b) => this.progressBar1.Value = b.ProgressPercentage;


            }

jdehaan · Accepted Answer · 2011-06-12 07:24:28Z

0

Only the GUI thread is allowed to modify GUI elements. If you do not respect this rule, you will get an exception. You may:

use a MethodInvoker to trigger a call from within the GUI thread
use the report progress feature (which does actually 1. For you)
store the objects in another data structure (lock) and use a timer in the GUI thread to pull the objects and display them.

answered Jun 12, 2011 at 7:24

jdehaan

20k6 gold badges65 silver badges99 bronze badges

Collectives™ on Stack Overflow

Working with threads C#

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related