Почему не обновляется View, когда обновляется property во viewmodel?

Question

[Антон Иванов]

Внутри ViewModel выполняется асинхронный блок.
Перед этим блоком значение свойства ControlsEnabled меняется на false, внутри этого блока меняется назад на True.

Примерно так:

private void DoTheThing(string clientType)
{
    var applicationWorker = new ApplicationWorker();
    applicationWorker.StatusUpdated += (sender, s) =>
    {
            _syncContext.Post(o => StatusText = s, null); // обновление статуса в статусбаре
    };
    ControlsEnabled = false; // отключаем контролсы

    Task.Run(() =>
    {
             var result = applicationWorker.DoTheThing();
             
            _syncContext.Post(o => ControlsEnabled = true, null); // включаем контролсы (выполнение в потоке UI)
            ResetDataToDefaults();
    }).ContinueWith((t) =>
    {
        if (t.IsFaulted) SendFaultLog(t.Exception);
    }, TaskScheduler.FromCurrentSynchronizationContext()); ;
}

Как можно понять, ControlsEnabled блокирует контролсы на форме, чтобы пользователь ничего не натыкал, пока идет процесс. Проблема в том, что у клиента очень часто, хоть и не всегда, ControlsEnabled = true не срабатывает. При этом видно, что ResetDataToDefaults срабатывает (поля на форме очищаются, хоть и остаются "отключенными").

_syncContext я получаю в конструкторе вот так: _syncContext = SynchronizationContext.Current
До того, как я начал использовать _syncContext, я использовал

Application.Current.Dispatcher.Invoke(delegate { ... })

В этом случае у меня иногда не работала смена статусов (их штук 10 меняется в процессе ожидания окончания работы applicationWorker.DoTheThing). Показывался первый статус (и изредка один из середины) и все. Когда поменял на _syncContext, все стало работать как часы.

Сейчас у меня локально все работает верно в 100% случаев, а у заказчика 4 раза из пяти проблема.
Я даже не знаю, в какую сторону копать.
Если это важно, класс ApplicationWorker находится в другой сборке

P.S. Только сейчас заметил, что ResetDataToDefaults выполняется не в потоке UI. И, при этом, он срабатывает. Значит проблема в _syncContext.Post?

Comments

[Петр]

Попробуйте

dispatcher = Dispatcher.CurrentDispatcher; // Убедитесь, что эта часть выполняется в UI потоке

и уже в тасках
dispatcher.Invoke(() => { ... })

[Борис Животное]

И ещё не мешало бы использовать async await, а не то, что сейчас.

Того, что ты выложил мало. Скинь проект с имитацией проблемы. У тебя проблема там явно где-то в другом месте. С чего ты взял, что ты ищешь там где нужно и привёл именно нужный код?

[Борис Животное]

В твоём коде, если посмотреть, выключать контроЛЫ на форме нужно в первую очередь. Это так, мелкое замечание.

Зачем создавать снаружи Task.Run
var applicationWorker = new ApplicationWorker();
непонятно.

[Борис Животное]

_syncContext.Post(o => ControlsEnabled = true, null);

Может не сработать, если произойдёт исключение

var result = applicationWorker.DoTheThing();
в части кода, которую я выложил выше.

[Борис Животное]

P.S. Только сейчас заметил, что ResetDataToDefaults выполняется не в потоке UI. И, при этом, он срабатывает. Значит проблема в _syncContext.Post?

Нет, проблема в целом в реализации всего кода.

Проблема ещё в том, что ты выложил какие-то обрывки, из которых непонятно, что как реализовано.

[Антон Иванов]

Борис Животное, насчет контролсов - верное замечание.
Создание воркера было вынесено за пределы Task.Run когда разбирался с необновляющимися статусами.
Исключение в потоке точно не происходит, так как тогда выполнение попало бы в блок ContinueWith. Да и код, который за

syncContext.Post(o => ControlsEnabled = true, null);

выполняется.
Вряд ли проблема в другом месте, так как когда я вывел ControlsEnabled на форму (true/false), у клиента все работает в 100% случаев...

[Борис Животное]

Антон Иванов, вот так это делается примерно (обновил код). И никаких проблем с потоками. Кстати, контекст синхронизации в моём примере даже не нужен, так как объект создаётся в UI потоке. Но для примера сойдёт и объект можно создать и в не UI потоке, будет работать.

using System;
using System.Threading;
using System.Threading.Tasks;
using System.Windows.Input;
using GalaSoft.MvvmLight;
using GalaSoft.MvvmLight.Command;

namespace WpfApp1.ViewModels
{
    public class MainViewModel : ViewModelBase
    {
        private bool _controlsEnabled = true;
        private string _status;

        public bool ControlsEnabled
        {
            get { return _controlsEnabled; }
            private set { Set(ref _controlsEnabled, value); }
        }

        public string Status
        {
            get { return _status; }
            private set { Set(ref _status, value); }
        }

        public ICommand DoTheThingCommand { get; }


        public MainViewModel()
        {
            DoTheThingCommand = new RelayCommand(DoTheThing);
        }

        private void ResetDataToDefaults()
        {
        }

        private async void DoTheThing()
        {
            ControlsEnabled = false;
            ApplicationWorker worker = null;
            try
            {
                worker = new ApplicationWorker();
                worker.StatusUpdated += StatusUpdated;
                await worker.DoTheThing(CancellationToken.None);
            }
            catch (OperationCanceledException ex)
            {
                // Ignored
            }
            catch (Exception ex)
            {
                SendFaultLog(ex);
            }
            finally
            {
                if (worker != null)
                {
                    worker.StatusUpdated -= StatusUpdated;
                }
                ResetDataToDefaults();
                ControlsEnabled = true;
            }
        }

        private void StatusUpdated(object sender, WorkerStatusEventArgs args)
        {
            Status = args.Status.ToString();
        }

        private void SendFaultLog(Exception ex)
        {
            // Асинхронно отправляем
        }
    }

    public class ApplicationWorker
    {
        private readonly SynchronizationContext _syncContext;

        public event EventHandler<WorkerStatusEventArgs> StatusUpdated;

        public ApplicationWorker()
        {
            _syncContext = SynchronizationContext.Current ?? new SynchronizationContext();
        }

        public Task DoTheThing(CancellationToken cancellationToken)
        {
            return Task.Run(async () =>
            {
                OnStatusUpdated(new WorkerStatusEventArgs(WorkerStatus.InProgress));
                await Task.Delay(3000, cancellationToken);
                OnStatusUpdated(new WorkerStatusEventArgs(WorkerStatus.Completed));
            }, cancellationToken);
        }

        private void OnStatusUpdated(WorkerStatusEventArgs args)
        {
            _syncContext.Post(localArgs =>
            {
                StatusUpdated?.Invoke(this, (WorkerStatusEventArgs)localArgs);
            }, args);
        }
    }

    public class WorkerStatusEventArgs
    {
        public WorkerStatus Status { get; }

        public WorkerStatusEventArgs(WorkerStatus status)
        {
            Status = status;
        }
    }

    public enum WorkerStatus
    {
        InProgress,
        Completed
    }
}

Answer 1

[soloveid]

Возможно у Вас не воспроизводится потому, что у себя тестируете в Debug режиме,
а у пользователя наверное уже Release.