Какой правильный узор? Наблюдатель между моделью представления и сервисом

Что ж, один из вариантов — возложить на ViewModel ответственность за явный вызов модели, а не за использование мессенджера. Таким образом ViewModel будет легче понять, что ей не нужно запускать запрос на это изменение.

Альтернативой модели является проверка вновь установленного значения, чтобы увидеть, соответствует ли оно ее собственному представлению о «текущем» значении. Вы на самом деле не сказали нам, что здесь происходит с точки зрения того, что такое ответ или что ищет сервер, но обычно я ожидаю, что это будет случай проверки того, является ли старое значение равно новому значению и игнорируя «изменение», если это так.

Если бы вы могли показать короткий, но полный пример всего этого, было бы легче говорить об этом.

Я повторно занялся этим проектом за последние несколько недель и, наконец, нашел решение. Учитывая комментарии и мысли людей, опубликованные выше, я не уверен, что кто-то, кроме меня, понимает, что я пытаюсь сделать, но я подумал, что, возможно, стоит опубликовать, как я решил эту проблему. Как минимум, если я напишу это, я пойму это я :-).

Резюмируя вопрос еще раз:

У меня есть сервер управления домом, который предоставляет объекты в моем доме через интерфейс SOAP. Home.LivingRoom.Fireplace, например, отображается как:

http://server/Home/LivingRoom/Fireplace?property=DisplayName http://server/Home/LivingRoom/Fireplace?property=PowerState

Выполнение HTTP GET для них приведет к ответу HTTP, содержащему значение свойства (например, «Камин в гостиной» и «Выкл» соответственно).

Ворота гаража (например, Home.Garage.EastGarageDoor) выставлены как:

http://server/Home/Upstairs/EastGarageDoor?property=DisplayName http://server/Home/Upstairs/EastGarageDoor?property=GarageDoorOpened http://server/Home/Upstairs/EastGarageDoor?property=Trigger

Здесь у нас есть свойство, которое, если установлено, вызывает действие (Trigger). Выполнение POST против этого с телом HTTP «True» приведет к открытию/закрытию двери.

Я создаю приложение WP7 в качестве интерфейса для этого. Я решил следовать модели Mvvm и использую Mvvm-Light.

В WP7 нет встроенного способа поддержки уведомлений от интерфейсов REST, и я еще не готов создавать свой собственный (хотя он у меня на примете). Поэтому, чтобы пользовательский интерфейс отображал актуальное состояние, мне нужно провести опрос. Количество сущностей и объем данных относительно невелики, и теперь я доказал, что могу заставить его хорошо работать с опросом, но есть некоторые оптимизации, которые я могу сделать, чтобы улучшить его (включая добавление интеллектуальных функций на сервер, чтобы включить уведомление, подобное системе). ).

В моем решении я размыл границы между моей моделью и моей моделью представления. Если вы действительно хотите быть в курсе этого, моя «Модель» - это просто классы низкого уровня, которые у меня есть для упаковки моих запросов Http (например, GetPropertyAsync(objectLocation, propertyName, completionMethod)).

В итоге я определил общий класс для свойств. Это выглядит так:

namespace Premise.Model
{
    //where T : string, bool, int, float 
    public class PremiseProperty<T>  
    {
        private T _Value;
        public PremiseProperty(String propertyName)
        {
            PropertyName = propertyName;
            UpdatedFromServer = false;
        }

        public T Value
        {
            get { return _Value; }

            set { _Value = value; }
        }
        public String PropertyName { get; set; }
        public bool UpdatedFromServer { get; set; }
    }
}

Затем я создал производный от ViewModelBase (из Mvvm-Light) базовый класс PremiseObject, который представляет базовый класс, на котором основан каждый объект в системе управления (например, который буквально называется «Объект»).

Наиболее важным методом для PremiseObject является переопределение RaisePropertyChanged:

    /// </summary>
    protected override void RaisePropertyChanged<T>(string propertyName, T oldValue, T newValue, bool sendToServer)
    {
        if (sendToServer)
            SendPropertyChangeToServer(propertyName, newValue);

        // Check if we are on the UI thread or not
        if (App.Current.RootVisual == null || App.Current.RootVisual.CheckAccess())
        {
            // broadcast == false because I don't know why it would ever be true
            base.RaisePropertyChanged(propertyName, oldValue, newValue, false);
        }
        else
        {
            // Invoke on the UI thread
            // Update bindings 
            // broadcast == false because I don't know why it would ever be true
            GalaSoft.MvvmLight.Threading.DispatcherHelper.CheckBeginInvokeOnUI(() =>
                base.RaisePropertyChanged(propertyName, oldValue, newValue, false));
        }

    }

Обратите внимание на несколько вещей: 1) Я переопределяю/переназначаю параметр broadcast. Если это правда, то изменение свойства «отправляется на сервер» (я делаю HTTP POST). Я больше нигде не использую широковещательные изменения свойств (и на самом деле я даже не уверен, для чего бы я его использовал). 2) Я всегда передаю трансляцию в False при вызове base..

PremiseObject имеет набор стандартных PremiseProperty свойств: Location (URL-адрес объекта), Name, DisplayName, Value (свойство value). Отображаемое имя выглядит так:

    protected PremiseProperty<String> _DisplayName = new PremiseProperty<String>("DisplayName");

    public string DisplayName
    {
        get
        {
            return _DisplayName.Value;
        }

        set
        {
            if (_DisplayName.Value == value)
            {
                return;
            }

            var oldValue = _DisplayName;
            _DisplayName.Value = value;

            // Update bindings and sendToServer change using GalaSoft.MvvmLight.Messenging
            RaisePropertyChanged(_DisplayName.PropertyName, 
                   oldValue, _DisplayName, _DisplayName.UpdatedFromServer);
        }
    }

Таким образом, это означает, что каждый раз, когда .DisplayName изменяется в моей программе, он передается на весь пользовательский интерфейс и ЕСЛИ И ТОЛЬКО ЕСЛИ _DisplayName.UpdatedFromServer имеет значение True, он также отправляется обратно на сервер.

Так как же устанавливается .UpdatedFromServer? Когда мы получаем наш обратный вызов из асинхронного запроса Http:

    protected void DisplayName_Get(PremiseServer server)
    {
        String propertyName = _DisplayName.PropertyName;

        _DisplayName.UpdatedFromServer = false;
        server.GetPropertyAsync(Location, propertyName, (HttpResponseArgs) =>
        {
            if (HttpResponseArgs.Succeeded)
            {
                //Debug.WriteLine("Received {0}: {1} = {2}", DisplayName, propertyName, HttpResponseArgs.Response);
                DispatcherHelper.CheckBeginInvokeOnUI(() =>
                {
                    DisplayName = (String)HttpResponseArgs.Response; // <-- this is the whole cause of this confusing architecture
                    _DisplayName.UpdatedFromServer = true;
                    HasRealData = true;
                });
            }
        });
    }

Всякий раз, когда пользовательскому интерфейсу требуются свежие данные, эти функции XXX_Get вызываются (например, по таймеру опроса, при изменении представления, запуске приложения и т. д.).

Мне приходится дублировать приведенный выше код для каждого свойства, которое я определяю, что довольно болезненно, но я еще не нашел способа обобщить его (поверьте мне, я пытался, но мои знания C# недостаточно сильны, и я просто продолжайте решать проблему). Но это работает, и работает хорошо.

Чтобы охватить все основы, вот пример свойства Trigger в классе GarageDoor:

    protected PremiseProperty<bool> _Trigger = new PremiseProperty<bool>("Trigger");
    public bool Trigger
    {
        set
        {
            if (value == true)
                RaisePropertyChanged(_Trigger.PropertyName, false, value, true);
        }
    }

Обратите внимание, как я задаю параметру broadcast значение RaisePropertyChanged равным true, и как это свойство «только для записи»? Это генерирует HTTP POST для URL-адреса GarageDoor.Location + ?propertyName= + value.ToString().

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

Мысли, комментарии, предложения?