MVVM我实现这个正确吗?

本文关键字:实现 MVVM | 更新日期: 2023-09-27 17:49:33

我一直在试图把我的头围绕mvvm上周或更多,仍然挣扎一点。我看了杰森·多林杰的MVVM视频,也上了里德·科普西的课,但我还是想知道我做得对不对……我创建了一个非常简单的时钟应用程序,我将在下面发布。程序的输出与预期的一样,但是我更感兴趣的是我是否正确地使用了模式。如有任何意见、评论等,我将不胜感激。

我的模型
using System;
using System.Threading;
namespace Clock
{
    public class ClockModel
    {
        private const int TIMER_INTERVAL = 50;
        private DateTime _time;
        public event Action<DateTime> TimeArrived;
        public ClockModel()
        {
            Thread thread = new Thread(new ThreadStart(GenerateTimes));
            thread.IsBackground = true;
            thread.Priority = ThreadPriority.Normal;
            thread.Start();
        }
        public DateTime DateTime
        {
            get
            {
                return _time;
            }
            set
            {
                this._time = value;
                if (TimeArrived != null)
                {
                    TimeArrived(DateTime);
                }
            }
        }
        private void GenerateTimes()
        {
            while (true)
            {
                DateTime = DateTime.Now;
                Thread.Sleep(TIMER_INTERVAL);
            }
        }
    }
}

我的观点

<Window x:Class="Clock.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:ViewModels="clr-namespace:Clock"
        Title="MainWindow" Height="75" Width="375">
    <Window.DataContext>
        <ViewModels:ClockViewModel />
    </Window.DataContext>
    <StackPanel Background="Black">
            <TextBlock Text="{Binding Path=DateTime}" Foreground="White" Background="Black" FontSize="30" TextAlignment="Center" />
    </StackPanel>
</Window>

my view model

using System;
using System.ComponentModel;
namespace Clock
{
    public class ClockViewModel : INotifyPropertyChanged
    {
        private DateTime _time;
        private ClockModel clock;
        public ClockViewModel()
        {
            clock = new ClockModel();
            clock.TimeArrived += new Action<DateTime>(clock_TimeArrived);
        }
        private void clock_TimeArrived(DateTime time)
        {
            DateTime = time;
            this.RaisePropertyChanged("DateTime");
        }
        public DateTime DateTime 
        {
            get
            {
                return _time;
            }
            set
            {
                _time = value;
            }
        }

        /// <summary>
        /// Occurs when a property value changes.
        /// </summary>
        public event PropertyChangedEventHandler PropertyChanged;
        /// <summary>
        /// Raises the property changed event.
        /// </summary>
        /// <param name="propertyName">Name of the property.</param>
        private void RaisePropertyChanged(string property)
        {
            if (PropertyChanged != null)
            {
                PropertyChanged(this, new PropertyChangedEventArgs(property));
            }
        }
    }
}

MVVM我实现这个正确吗?

你这样做很好。我只需要改变一件事:将对RaisePropertyChange的调用移动到属性的setter。这通常是这样做的,它可以防止你在设置属性时忘记触发通知。

在我看来,你的实现在关注点分离方面看起来很好,尽管你可能有兴趣将你的Model方法委托给Command,然后你可以将它附加到让我们说你的主UI的Loaded事件。这绝对是个人偏好,但作为良好实践,我倾向于在View: ViewModelModel.Method: Command之间保持1:1的关系

对于一些正常的功能,使用MVVM是相当容易的,当你开始触摸显示消息框。显示单独的窗口,以及视图和视图模型之间的通信。然后你会发现一些棘手的事情…