Разработка триггера

@skew · Регистрация: 15.05.2012

Доброго вечера, форумчане.
Разрабатываю свою маленькую софтинку и столкнулся с небольшим казусом, а точнее хочу идеала, но чувствую, что не могу к нему придти.
Суть такова, моя программа является промежуточным звеном между сервером и клиентом и как прокси-сервер пропускает через себя пакеты. Я захотел создать некую возможность создавать триггеры - то есть отвечать на сервер/клиент если данные из пакета будут соответствовать заданным условиям.
Все сделал, все работает, но чувствую, что сделал не по всем законам ООП. Так же основная задача пополнять вручную эти триггеры и ответы на них. То есть чтобы через визуальный конструктор можно было создать триггер, в него добавить условие срабатывания и действия, которые будут выполняться.
Чтобы внести чуть наглядности - немного кода:

Базовый класс триггеров
Содержит условие и список действий. Первый минус, что условие одно, а не несколько.
В коде понятно, что Execute вызывает лист задач, а Match проверяет условие из наследуемых классов.

C#

  public abstract class BaseTriggerModel<T> where T : Packet
    {
        public ConditionTriggerModel<T> ConditionTrigger { get; set; }
        public ObservableCollection<IActionTriggerModel> Actions { get; set; }
        protected virtual void Initialize(T conditionValue)
        {
            Actions = new ObservableCollection<IActionTriggerModel>();
            ConditionTrigger = new ConditionTriggerModel<T>(conditionValue);
        }
 
        public async void Execute()
        {
            foreach (var action in Actions)
            {
               await action.Execute();
            }
        }
        public abstract bool Match(T conditionPacket);  
    }

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

C#

 public class ConditionTriggerModel<T> where T: Packet
    {
        public T Value { get; set; }
        public ConditionTriggerModel(T value)
        {
            Value = value;
        }         
    }

Интерфейс действия
Банальный интерфейс, содержащий Action (действие), которое будет вызываться, саму процедуру вызова и динамично изменяющиеся данные, которые возможно нужно поправить в Action пере вызовом (динамичность - это тоже плохо)

C#

public interface IActionTriggerModel
    {
        Action TriggerAction { get; set; }
        void HandleData(dynamic data);
        Task Execute();
    }

Класс ответа триггера
Во первых проблема в обобщении, прямое обобщение типа BaseTriggerModel<ObjectMove> не позволит добавлять этот триггер в список List<BaseTriggerModel<Packet>>, хоть тип Packet и является базовым от ObjectMove. (Кто знаком с обобщениями поймут). Поэтому приходится приводить к типу напрямую, что как по мне тоже моветон.

C#

 public class CounterSkillTriggerModel<T>: BaseTriggerModel<Packet> where T: ObjectMove
    {     
       public CounterSkillTriggerModel(Point coords)
       {
           base.Initialize(new ObjectMove{Coords = coords});   
       }  
 
       public override bool Match(Packet gPacket)
       {
           if (!(gPacket is T packet) || !(ConditionTrigger.Value is T condition))
               return false;
           
           return condition.Coords.Equals(gPacket.Coords);
        }
   }

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

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

@Evgen1337 · 19.06.2018, 14:30

Привет.

GetType пробовал? - https://stackoverflow.com/ques... th-gettype
Или dynamic...

@insite2012 · 19.06.2018, 16:12

skew, посмотрите в сторону этого шаблона. Исходя из вашего описания, он может помочь в вашей задаче.

@Evgen1337 248 / 161 / 68 Регистрация: 10.12.2017 Сообщений: 558
	19.06.2018, 14:30	2
	Привет. GetType пробовал? - https://stackoverflow.com/ques... th-gettype Или dynamic... 0

@insite2012 Модератор 5300 / 4095 / 1182 Регистрация: 12.10.2013 Сообщений: 11,909 Записей в блоге: 2
	19.06.2018, 16:12	3
	skew, посмотрите в сторону этого шаблона. Исходя из вашего описания, он может помочь в вашей задаче. 0

Опции темы