Making sure OnPropertyChanged() is called on UI thread in MVVM WPF app

前端 未结 5 2056
执念已碎
执念已碎 2020-11-28 07:15

In a WPF app that I\'m writing using the MVVM pattern, I have a background process that doing it\'s thing, but need to get status updates from it out to the UI.

I\'m

相关标签:
5条回答
  • 2020-11-28 07:33

    I REALLY like Jeremy's answer: Dispatching In Silverlight

    Summary:

    • Placing Dispatcher in ViewModel seems inelegant

    • Creating an Action<Action> property, set it to just run the action in the VM constructor

    • When using the VM from the V, set the Action property to invoke the Dispatcher
    0 讨论(0)
  • 2020-11-28 07:43

    I had a similar scenario just this week (MVVM here too). I had a separate class doing its thing, reporting back status on an event handler. The event handler was being called as expected, and I could see the results coming back right on time with Debug.WriteLine's.

    But with WPF, no matter what I did, the UI would not update until the process was complete. As soon as the process finished, the UI would update as expected. It was as if it was getting PropertyChanged, but waiting for the thread to complete before doing the UI updates all at once.

    (Much to my dismay, the same code in Windows.Forms with a DoEvents and .Refresh() worked like a charm.)

    So far, I've resolved this by starting the process on its own thread:

    //hook up event handler
    myProcess.MyEvent += new EventHandler<MyEventArgs>(MyEventHandler); 
    
    //start it on a thread ...
    ThreadStart threadStart = new ThreadStart(myProcess.Start);
    
    Thread thread = new Thread(threadStart);
    
    thread.Start();
    

    and then in the event handler:

    private void MyEventHandler(object sender, MyEventArgs e) { 
    ....
    Application.Current.Dispatcher.Invoke(
                    DispatcherPriority.Send,
                    (DispatcherOperationCallback)(arg =>
                    { 
             //do UI updating here ...
            }), null);
    

    I'm not recommending this code, since I'm still trying to understand the WPF thread model, how Dispatcher works, and why in my case the UI wouldn't update until the process was complete even with event handler getting called as expected (by design?). But this has worked for me so far.

    I found these two links helpful:

    http://www.nbdtech.com/blog/archive/2007/08/01/Passing-Wpf-Objects-Between-Threads-With-Source-Code.aspx

    http://srtsolutions.com/blogs/mikewoelmer/archive/2009/04/17/dealing-with-unhandled-exceptions-in-wpf.aspx

    0 讨论(0)
  • 2020-11-28 07:50

    WPF automatically marshals property changes to the UI thread. However, it does not marshal collection changes, so I suspect your adding a message is causing the failure.

    You can marshal the add manually yourself (see example below), or use something like this technique I blogged about a while back.

    Manually marshalling:

    public void backgroundWorker_ReportProgress(object sender, ReportProgressArgs e)
    {
        Dispatcher.Invoke(new Action<string>(AddMessage), e.Message);
        OnPropertyChanged("Messages");
    }
    
    private void AddMessage(string message)
    {
        Dispatcher.VerifyAccess();
        Messages.Add(message);
    }
    
    0 讨论(0)
  • 2020-11-28 07:53

    This is more of an extension to the accepted answer but I did this with my event hander ...

    using System.Threading;
    
    private void Handler(object sender, RoutedEventArgs e)
    {
        if (Thread.CurrentThread == this.Dispatcher.Thread)
        {
            //do stuff to this
        }
        else
        {
            this.Dispatcher.Invoke(
                new Action<object, RoutedEventArgs>(Handler),
                sender,
                e);
        }
    }
    
    0 讨论(0)
  • 2020-11-28 07:53

    I handle the BackgroundWorker.ReportProgress event outside of my ViewModel and pass the actual BackgroundWorker instance and the ViewModel into my class which defines the async method(s).

    The async method then calls bgWorker.ReportProgress and passes a class which wraps a delegate as the UserState (as object). The delegate I write as an anonymous method.

    In the event handler, I cast it from object back to the wrapper type and then invoke the delegate inside.

    All this means that I can code UI changes directly from the code that's running asynchronously, but it just has this wrapping around it.

    This explains it in more detail:

    http://lukepuplett.blogspot.com/2009/05/updating-ui-from-asynchronous-ops.html

    0 讨论(0)
提交回复
热议问题