不从分派器计时器更新背景颜色绑定

时间:2019-03-31 01:51:51

标签: c# wpf mvvm-light

我在VS2017中创建了一个新的WPF项目,还通过NuGet导入了MVVM Light。

然后,我添加了一些代码,该代码应每25毫秒更改一次MainWindows网格的背景色。可悲的是,这种变化不会传播,我也不知道为什么它不会更新。也许有人可以帮助我。

代码如下:

MainViewModel.cs

using GalaSoft.MvvmLight;
using System;
using System.ComponentModel;
using System.Runtime.CompilerServices;
using System.Windows.Input;
using System.Windows.Media;
using System.Windows.Threading;

namespace Strober.ViewModel
{
    /// <summary>
    /// This class contains properties that the main View can data bind to.
    /// <para>
    /// Use the <strong>mvvminpc</strong> snippet to add bindable properties to this ViewModel.
    /// </para>
    /// <para>
    /// You can also use Blend to data bind with the tool's support.
    /// </para>
    /// <para>
    /// See http://www.galasoft.ch/mvvm
    /// </para>
    /// </summary>
    public class MainViewModel : ObservableObject
    {
        private DispatcherTimer timer; 
        public string Title { get; set; }
        private Brush _background;
        public Brush Background
        {
            get
            {
                return _background;
            }

            set
            {
                _background = value;
                OnPropertyChanged("Background");
            }
        }
        /// <summary>
                 /// Initializes a new instance of the MainViewModel class.
                 /// </summary>
        public MainViewModel()
        {
            Background = new SolidColorBrush(Colors.Black);
            timer = new DispatcherTimer();
            timer.Tick += Timer_Tick;
            timer.Interval = new TimeSpan(0, 0, 0,0,100);

            timer.Start();
        }

        private void Timer_Tick(object sender, System.EventArgs e)
        {
            if (Background == Brushes.Black)
            {
                Background = new SolidColorBrush(Colors.White);
                Title = "White";
            }
            else
            {
                Background = new SolidColorBrush(Colors.Black);
                Title = "Black";
            }
        }


        #region INotifiedProperty Block
        public event PropertyChangedEventHandler PropertyChanged;

        protected void OnPropertyChanged([CallerMemberName] string PropertyName = null)
        {
            if (PropertyChanged != null)
            {
                PropertyChanged(this, new PropertyChangedEventArgs(PropertyName));
            }
        }
        #endregion
    }
}

ViewModelLocator.cs

    /*
  In App.xaml:
  <Application.Resources>
      <vm:ViewModelLocator xmlns:vm="clr-namespace:Strober"
                           x:Key="Locator" />
  </Application.Resources>

  In the View:
  DataContext="{Binding Source={StaticResource Locator}, Path=ViewModelName}"

  You can also use Blend to do all this with the tool's support.
  See http://www.galasoft.ch/mvvm
*/

using GalaSoft.MvvmLight;
using GalaSoft.MvvmLight.Ioc;
using CommonServiceLocator;

namespace Strober.ViewModel
{
    /// <summary>
    /// This class contains static references to all the view models in the
    /// application and provides an entry point for the bindings.
    /// </summary>
    public class ViewModelLocator
    {
        /// <summary>
        /// Initializes a new instance of the ViewModelLocator class.
        /// </summary>
        public ViewModelLocator()
        {
            ServiceLocator.SetLocatorProvider(() => SimpleIoc.Default);

            ////if (ViewModelBase.IsInDesignModeStatic)
            ////{
            ////    // Create design time view services and models
            ////    SimpleIoc.Default.Register<IDataService, DesignDataService>();
            ////}
            ////else
            ////{
            ////    // Create run time view services and models
            ////    SimpleIoc.Default.Register<IDataService, DataService>();
            ////}

            SimpleIoc.Default.Register<MainViewModel>();
        }

        public MainViewModel Main
        {
            get
            {
                return ServiceLocator.Current.GetInstance<MainViewModel>();
            }
        }

        public static void Cleanup()
        {
            // TODO Clear the ViewModels
        }
    }
}

MainWindow.xaml(MainWindow.xaml.cs只是定期生成的文件)

<Window x:Class="Strober.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:local="clr-namespace:Strober"
        mc:Ignorable="d"
        DataContext="{Binding Main, Source={StaticResource Locator}}"
        Title="{Binding Title}" Height="450" Width="800">
    <Grid Background="{Binding Background}">        
    </Grid>
</Window>

App.xaml

<Application x:Class="Strober.App" xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:local="clr-namespace:Strober" StartupUri="MainWindow.xaml" xmlns:d="http://schemas.microsoft.com/expression/blend/2008" d1p1:Ignorable="d" xmlns:d1p1="http://schemas.openxmlformats.org/markup-compatibility/2006">
  <Application.Resources>
        <ResourceDictionary>
      <vm:ViewModelLocator x:Key="Locator" d:IsDataSource="True" xmlns:vm="clr-namespace:Strober.ViewModel" />
    </ResourceDictionary>
  </Application.Resources>
</Application>

greg

1 个答案:

答案 0 :(得分:3)

代码中的主要问题是System.Windows.Media.SolidColorBrush不会覆盖Equals()方法,因此您的表达式Background == Brushes.Black永远不会true。由于您正在创建SolidColorBrush对象的显式新实例,并且由于==运算符只是在比较实例引用,因此始终将笔刷值与内置Brushes.Black实例之间的比较失败。

修复代码的最简单方法是仅使用实际的Brushes实例:

private void Timer_Tick(object sender, System.EventArgs e)
{
    if (Background == Brushes.Black)
    {
        Background = Brushes.White;
        Title = "White";
    }
    else
    {
        Background = Brushes.Black;
        Title = "Black";
    }
}

然后,当您比较实例引用时,它们实际上是可比的,并且您将根据需要检测“黑色”条件。

我会注意到,由于您也没有针对PropertyChanged属性的更改而提高Title,因此绑定也无法按预期进行。

对于它的价值,我会完全避免您的设计。首先,视图模型对象应避免使用特定于UI的类型。可以肯定的是,这将包括Brush类型。可以说,它还包含DispatcherTimer,因为它存在于UI服务中。除此之外,DispatcherTimer是一个相对不精确的计时器,并且它主要存在于拥有计时器的调度程序线程上引发其Tick事件的计时器,因为WPF会自动封送来自UI线程的任何其他线程,在此示例中它几乎没有用。

这是您的程序的一个版本,恕我直言更符合典型的WPF编程实践:

class MainViewModel : NotifyPropertyChangedBase
{
    private string _title;
    public string Title
    {
        get { return _title; }
        set { _UpdateField(ref _title, value); }
    }

    private bool _isBlack;
    public bool IsBlack
    {
        get { return _isBlack; }
        set { _UpdateField(ref _isBlack, value, _OnIsBlackChanged); }
    }

    private void _OnIsBlackChanged(bool obj)
    {
        Title = IsBlack ? "Black" : "White";
    }

    public MainViewModel()
    {
        IsBlack = true;
        _ToggleIsBlack(); // fire and forget
    }

    private async void _ToggleIsBlack()
    {
        while (true)
        {
            await Task.Delay(TimeSpan.FromMilliseconds(100));
            IsBlack = !IsBlack;
        }
    }
}

该视图模型类使用了我用于所有视图模型的基类,因此我不必一直重新实现INotifyPropertyChanged

class NotifyPropertyChangedBase : INotifyPropertyChanged
{
    public event PropertyChangedEventHandler PropertyChanged;

    protected void _UpdateField<T>(ref T field, T newValue,
        Action<T> onChangedCallback = null,
        [CallerMemberName] string propertyName = null)
    {
        if (EqualityComparer<T>.Default.Equals(field, newValue))
        {
            return;
        }

        T oldValue = field;

        field = newValue;
        onChangedCallback?.Invoke(oldValue);
        PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }
}

您会注意到,视图模型类没有任何特定于UI的行为。只要该程序具有对PropertyChanged事件做出反应并利用视图模型中的值的方式,它就可以与WPF或其他任何程序一起使用。

为使此工作有效,XAML变得更加冗长:

<Window x:Class="TestSO55437213TimerBackgroundColor.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:p="http://schemas.microsoft.com/netfx/2007/xaml/presentation"
        xmlns:l="clr-namespace:TestSO55437213TimerBackgroundColor"
        mc:Ignorable="d"
        Title="{Binding Title}" Height="450" Width="800">
  <Window.DataContext>
    <l:MainViewModel/>
  </Window.DataContext>
  <Grid>
    <Grid.Style>
      <p:Style TargetType="Grid">
        <Setter Property="Background" Value="White"/>
        <p:Style.Triggers>
          <DataTrigger Binding="{Binding IsBlack}" Value="True">
            <Setter Property="Background" Value="Black"/>
          </DataTrigger>
        </p:Style.Triggers>
      </p:Style>
    </Grid.Style>
  </Grid>
</Window>

(注意:我已将http://schemas.microsoft.com/netfx/2007/xaml/presentation XML名称空间明确命名为与<Style/>元素一起使用,仅作为Stack Overflow的XML标记处理不足的解决方法,否则无法识别<Style/>元素作为实际的XML元素。在您自己的程序中,您可以随意忽略它。)

此处的关键是,UI关注事项的整个处理都在UI声明本身中。视图模型不需要知道UI如何表示黑色或白色。它只是切换一个标志。然后,UI监视该标志,并将属性设置器适当地应用于其当前值。

最后,我要指出的是,要像这样在UI中反复更改状态,另一种方法是使用WPF的动画功能。这超出了此答案的范围,但是我鼓励您阅读它。这样做的一个优点是,动画使用的分辨率比我上面使用的基于线程池的Task.Delay()方法还要高分辨率,因此通常可以提供更平滑的动画(尽管随着间隔的增加越来越小(例如您的帖子指示您打算使用25ms的时间),无论如何,WPF都无法顺利保持;有时,您会发现WinForms,WPF,Xamarin,等等。只是无法在如此精细的计时器级别上运行)。