Новый DispatcherTimer создается вместе со старым, должен запускаться только новый

Question

Я столкнулся с проблемой на днях. Я выяснил, почему это происходит, но я никогда не сталкивался с такой проблемой, поэтому не знаю, как ее решить.

У меня есть приложение, где в DashboardView (основной вид) DispatcherTimer запускается в DashboardViewModel. Когда отметится таймер, мы получим данные из базы данных, этот список будет привязан к данным между View и ViewModel. Когда появятся новые данные, которые привели к изменению базы данных, будет воспроизводиться звук.

Пользователь может перейти к другим представлениям. Когда пользователь возвращается к DashboardView, DashboardViewModel снова создается, как и DispatcherTimer.
Теперь есть 2 таймера, и они оба запускают событие Tick, создавая запутанный сценарий для пользователя.

Это мое наблюдение того, что происходит в приложении прямо сейчас:
Мой таймер тикает каждую минуту. Когда я запускаю приложение, открывается DashboardView #1. Запускается DashboardViewModel #1 и DispatcherTimer #1.
Я переключаюсь в другое представление и обновляю данные (новое электронное письмо), чтобы при включении таймера список в DashboardView изменялся и воспроизводился звук.
Когда Таймер # 1 составляет 30 секунд, я переключаюсь на DashboardView, который создается заново, создавая таким образом View&ViewModel&Timer #2. Через 1 минуту, таймер №1 тикает, появляются новые данные, поэтому он обновляет базу данных и воспроизводит звук, но список в представлении не обновляется. Я думаю, что это потому, что View #2 показывает более # 1. Я знаю, потому что иначе я бы увидел наложение, говорящее, что это освежает.
View #2 привязан к ViewModel #2. Таймер # 1 обновил ViewModel #1, поэтому изменения не будут отображаться, так как мы не видим View # 1, так как он заменен / перекрыт View #2. Через 1 мин 30 с тикает таймер № 2, получает данные из БД, не воспроизводит звук, так как БД уже был обновлен Таймером № 1, и показывает данные в новом состоянии.
(Надеюсь, это имело смысл)

Итак, TLDR: есть 2 таймера, в то время как только 1 должен быть активным (я думаю, самый новый). Как мне этого добиться?

Вот (часть) DashboardViewModel, как у меня сейчас:

namespace QRM.ViewModel
{
    class DashboardListViewModel : INotifyPropertyChanged
    {
        private static readonly log4net.ILog log = log4net.LogManager.GetLogger(System.Reflection.MethodBase.GetCurrentMethod().DeclaringType);
        DBServer dbServer = new DBServer();

        #region Constructor
        public DashboardListViewModel()
        {
            log.Info("Dashboard Initializing - Starting...");
            MyObservableCollection<View_server> listDashboard = new MyObservableCollection<View_server>();
            ListDashboard = dbServer.ReadDashboard();
            listBoxCommand = new RelayCommand(() => SelectionHasChanged());

            // Refresh to get all new emails, errors, etc.
            GetListDashboard();

            IsRefreshing = Visibility.Collapsed;

            // Make a timer to renew the data in the Dashboard automatically. 
            DispatcherTimer timer = new DispatcherTimer();
            timer.Tick += new EventHandler(timer_Tick);
            timer.Interval = Properties.Settings.Default.Timer_interval; // hours, minutes, seconds.
            timer.Start();

            //Receive the Notification sent after DashboardDetailsViewModel has handled the button commands, and call a respond method for the List.
            App.Messenger.Register("RefreshServers", (Action)(() => GetListDashboard()));
            App.Messenger.Register("ClearSelection", (Action)(() => SelectedServer = null));
            App.Messenger.Register("ErrorSolved", (Action)(() => KeepSelection(selectedServer)));
            App.Messenger.Register("WarningSound", (Action)(() => HasNewError = true));
            log.Info("Dashboard Initializing - Done.");
        }
        #endregion

        #region Get list dashboard
        private void GetListDashboard()
        {
            HasNewError = false;
            log.Info("Dashboard - Checking for Email...");

            // The old Outlook class and methods
            //EmailManager checkMail = new EmailManager();
            //checkMail.GetEmail();

            // First, check for mail.
            IMAPManager checkMail = new IMAPManager();
            checkMail.GetEmail();

            log.Info("Dashboard - Checking for linked Errors...");
            // Check if the emails have Errors linked to them. If not, add the Error from the Email to the DB
            ErrorManager checkError = new ErrorManager();
            checkError.GetNewErrors();

            log.Info("Dashboard List - Starting...");
            // Load the dashboard.
            ListDashboard = dbServer.ReadDashboard();
            System.Diagnostics.Debug.WriteLine("REFRESHED THE DASHBOARD");
            log.Info("Dashboard List - Done.");
        }

        private void KeepSelection(View_server keepSelection)
        {
            GetListDashboard();
            SelectedServer = keepSelection;
            SelectionHasChanged();
        }
        #endregion

        #region Timer
        //This method runs every time the timer ticks.
        private async void timer_Tick(object sender, EventArgs e)
        {
            log.Info("Dashboard - Refreshing...");
            System.Diagnostics.Debug.WriteLine(">>Timer tick");
            IsRefreshing = Visibility.Visible;

            // To make sure the overlay is visible to the user, let it be on screen for at least a second (2x half a second)
            await Task.Delay(500);

            if (selectedServer != null)
            {
                KeepSelection(selectedServer);
            }
            else
            {
                GetListDashboard();
            }

            // 2nd half second.
            await Task.Delay(500);
            IsRefreshing = Visibility.Collapsed;

            if (hasNewError == true)
            {
                System.Diagnostics.Debug.WriteLine("List has new error");
                PlayWarningSound();
                HasNewError = false;
            }
            else
            {
                System.Diagnostics.Debug.WriteLine("List has no new error");
                HasNewError = false;
            }
            System.Diagnostics.Debug.WriteLine(">>End timer");

            log.Info("Dashboard - Refreshed.");
        }        
        #endregion
    }
}

Answer 1

Здесь происходит несколько вопросов. Давайте начнем с самого основного:

уборка

Когда DashboardListViewModel утилизируется или закрывается, вам нужно DispatcherTimer.Tick обработчик события invoke .Stop() а затем позвоните .Finalize(), MSDN. Это гарантирует, что ваш System.Windows.Threading.DispatcherTimer правильно очищен.

Async / Await & Event Handlers

Так же DispatcherTimer.Tick Обработчик события определяется как async void, Это неправильное использование async ключевое слово. Вместо этого используйте это:

private void timer_Tick(object sender, EventArgs e)
{
    log.Info("Dashboard - Refreshing...");
    System.Diagnostics.Debug.WriteLine(">>Timer tick");
    IsRefreshing = Visibility.Visible;

    // To make sure the overlay is visible to the user, let it be on screen for at least a second (2x half a second)
    Thread.Sleep(500);

    if (selectedServer != null)
    {
        KeepSelection(selectedServer);
    }
    else
    {
        GetListDashboard();
    }

    // 2nd half second.
    Thread.Sleep(500);
    IsRefreshing = Visibility.Collapsed;

    if (hasNewError == true)
    {
        System.Diagnostics.Debug.WriteLine("List has new error");
        PlayWarningSound();
        HasNewError = false;
    }
    else
    {
        System.Diagnostics.Debug.WriteLine("List has no new error");
        HasNewError = false;
    }
    System.Diagnostics.Debug.WriteLine(">>End timer");

    log.Info("Dashboard - Refreshed.");
}

Я обычно никогда не советую использовать Thread.Sleep но поскольку вы уже находитесь в контексте таймера потоков, это имеет смысл.

Последнее беспокойство

Вы уверены, что App.Messenger.Register может быть вызван несколько раз, как это происходит каждый раз, когда создается ваша модель представления? Я бы предположил, что это будет то, что вы бы хотели сделать только один раз, в static контекст.