Как правильно использовать MVC паттерн в сетевой игре?

Question

[Jeen99]

Создаю сетевую игру на C# для конкурса. Нужно использовать MVC подход. Интересующий процесс игры: подключаемся, получаем данные от сервера для создание объектов, отрисовываем объекты, обновляем данные, отрисовываем изменения.

Сейчас при получении данных, объекты создаются в контроллере и помещаются в модель(игровые объекты храню в словаре). В контроллере также вызываю событие, что изменили модель.

Появились сомнения на счет правильности реализации MVC. Может стоит добавить метод для модели(сейчас идет прямое обращение к словарю) на добавление объектов, в этом методе также вызывать событие на обновление модели. И где тогда хранить логику с определением типа объекта при получении данных от сервера? Или вообще не стоит заморачиваться над данной темой и оставить все как есть?

Псевдокод

Модель
{
    Словарь<ИгровойОбъект> объекты;
    Событие изменение;
}
Контроллер
{
    ОбработчикСообщенийотСервера(Сообщение сообщение)
    {
         Если(сообщение.тип== Объект1)
         {
             модель.объекты[сообщение.ID] = создатьОбъект1(сообщение как СостояниеОбъект1);
             модель.Изменение();
          }
    }

    Объект1 создатьОбъект1(СостояниеОбъект1 состояние)
    {
          Объект1  объект = new Объект1();
          объект.УстановитьСостояние(состояние);
          вернуть объект;
     }
}

Answer 1

[Антон Шаманов]

И где тогда хранить логику с определением типа объекта

В классе реализующем Factory/Builder шаблон

Объект1 создатьОбъект1(СостояниеОбъект1 состояние)

вот таких методов в контроллере точно быть не должно

Comments

[Jeen99]

Должно получится что-то на подобии этого? Просто не совсем понимаю, что поменялось. Тут еще нюанс в том, что для некоторых объектов приходит сразу несколько состояний. Могли бы вы подробнее рассказать, как оно должно быть организованно.

Director director = new Director();
Builder builder;

If(msg.type == Object1)
       builder= new ConcreteObject1(msg as  StateObject1);
If(msg.type == Object2)
       builder= new ConcreteObject2(msg as StateObject2);

director.Construct(builder);
Object object= builder.GetResult();
model.objects[msg.ID] = object;

[Jeen99]

Этот код ведь в контроллере будет?

[Антон Шаманов]

Jeen99, нет, в билдер передаем данные объекта который нужно получить и на выходе получаем требуемый объект. в итоге не придется размазывать логику по контроллерам

[Антон Шаманов]

Jeen99, я с++ не особо помню т.ч. будет псевдокод:

class GameObjectFactory
{
    function create($data)
    {
        if ($data['...'] == '...') {
            $obj = new GameClass($data['config']);
        } else {
            $obj = new GameClass2($data['config']);
        }
        return $obj;
    }
}

[Jeen99]

Антон Шаманов, Кажется понял, попробую реализовать. Не против, если еще возникнут вопросы - помочь?

[Jeen99]

Антон Шаманов, Еще такой вопрос. Представление у нас должно иметь интерфейс или не обязательно. Сейчас контроллер на прямую общается к Виду. Хотел переделать на интерфейс, но не получилось, т.к. при вызове события на смену объекта, приходится вызывать через основной поток Вида view.Dispather.Invoke(() => {model.OnChange() }). В противном случае ругается на то что, изменения происходят не в основном потоке.

[Антон Шаманов]

Jeen99, я же говорю - с++ изучал давно и поверхностно, пишу в основном на php т.ч. с конретной реализацией я помочь вряд ли смогу. пиши если будут вопросы по шаблонам проектирования