Нужны ли классы EventArg теперь, когда у нас есть дженерики

При использовании дженериков есть ли причина создавать специфические производные классы EventArg?

Кажется, что теперь вы можете просто использовать их на лету с общей реализацией.

Должен ли я выполнить все мои примеры и удалить мои классы eventArg (StringEventArgs, MyFooEventArgs и т. Д.)

public class EventArgs<T> : EventArgs
{
    public EventArgs(T value)
    {
        m_value = value;
    }

    private T m_value;

    public T Value
    {
        get { return m_value; }
    }
}

4 ответа

Решение

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

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

Допустим, у нас есть событие, указывающее, что какое-то деление было завершено, и оно содержит числитель, знаменатель и результат:

public event EventHandler<EventArgs<double, double, double>> Divided;

Обработчик событий имеет некоторую двусмысленность:

private void OnDivided(object sender, EventArgs<double, double, double> e)
{
    // I have to just "know" this - it is a convention

    var numerator = e.Value1;
    var denominator = e.Value2;
    var result = e.Value3;
}

Это было бы намного яснее с EventArgs представляющий событие:

private void OnDivided(object sender, DividedEventArgs e)
{
    var numerator = e.Numerator;
    var denominator = e.Denominator;
    var result = e.Result;
}

Универсальный многоразовый EventArgs занятия облегчают разработку механизма за счет выражения намерения.

Посмотрите на статью Custom Generic EventArgs, написанную Мэтью Кокраном, в этой статье он описывает, как еще больше расширить ее, добавив в нее два и три члена.

Использование универсальных EventArgs имеет свои применения и, конечно, их неправильное использование, поскольку информация о типах теряется в процессе.

public class City {...}

public delegate void FireNuclearMissile(object sender, EventArgs<City> args);
public event FireNuclearMissile FireNuclearMissileEvent;

public delegate void QueryPopulation(object sender, EventArgs<City> args);
public event QueryPopulation QueryPopulationEvent;

В следующем примере это типобезопасный, но немного более LOC:

class City {...}

public class FireNuclearMissileEventArgs : EventArgs
{
    public FireNuclearMissileEventArgs(City city)
    {
        this.city = city;
    }

    private City city;

    public City City
    {
        get { return this.city; }
    }
}

public delegate void FireNuclearMissile(object sender, FireNuclearMissileEventArgs args);
public event FireNuclearMissile FireNuclearMissileEvent;

public class QueryPopulationEventArgs : EventArgs
{
    public QueryPopulationEventArgs(City city)
    {
        this.city = city;
    }

    private City city;

    public City City
    {
        get { return this.city; }
    }
}

public delegate void QueryPopulation(object sender, QueryPopulationEventArgs args);
public event QueryPopulation QueryPopulationEvent;

Я думаю, что EventArgs в стиле Tuple полезны. Как и в случае с Tuple, ими можно злоупотреблять, но, похоже, моя лень сильнее моего чувства осторожности. Я реализовал следующее:

public static class TupleEventArgs
{
    static public TupleEventArgs<T1> Create<T1>(T1 item1)
    {
        return new TupleEventArgs<T1>(item1);
    }

    static public TupleEventArgs<T1, T2> Create<T1, T2>(T1 item1, T2 item2)
    {
        return new TupleEventArgs<T1, T2>(item1, item2);
    }

    static public TupleEventArgs<T1, T2, T3> Create<T1, T2, T3>(T1 item1, T2 item2, T3 item3)
    {
        return new TupleEventArgs<T1, T2, T3>(item1, item2, item3);
    }
}

public class TupleEventArgs<T1> : EventArgs
{
    public T1 Item1;

    public TupleEventArgs(T1 item1)
    {
        Item1 = item1;
    }
}

public class TupleEventArgs<T1, T2> : EventArgs
{
    public T1 Item1;
    public T2 Item2;

    public TupleEventArgs(T1 item1, T2 item2)
    {
        Item1 = item1;
        Item2 = item2;
    }
}

public class TupleEventArgs<T1, T2, T3> : EventArgs
{
    public T1 Item1;
    public T2 Item2;
    public T3 Item3;

    public TupleEventArgs(T1 item1, T2 item2, T3 item3)
    {
        Item1 = item1;
        Item2 = item2;
        Item3 = item3;
    }
}

Может использоваться следующим образом (при использовании с расширением события)

public event EventHandler<TupleEventArgs<string,string,string>> NewEvent;

NewEvent.Raise(this, TupleEventArgs.Create("1", "2", "3"));

Как уже сказали ТКК: используйте EventArgs<T> если вам нужно передать только одно значение, в противном случае производные от EventArgs (или же EventArgs<T>, что вы хотите).

Другие вопросы по тегам