Как удалить событие, добавленное в виде лямбда-выражения?

Question

[romaro]

При повторном вызове этого кода событие оказывается добавленным дважды:

_openMenuItem.Click -= (object? sender, EventArgs e) => Open_Click((int)id);
_openMenuItem.Click += (object? sender, EventArgs e) => Open_Click((int)id);

Answer 1

[joyrax-dev]

EventHabdler handler = (sender, e) => myfunction();

_openMenuItem.Click -= handler;
_openMenuItem.Click += handler;

В точности не помню синтаксис, но смысл понятен

Comments

[Роман]

В первой строке просто присваивание

[joyrax-dev]

Роман, и правда ;) щас исправим

Answer 2

[Сергей Соловьев]

В общем случае нельзя, так как объект лямбды создается каждый, поэтому нужно вынести функцию отдельно.
Но если регистрация и удаление обработчика происходят в одной функции/блоке, то ответ joyrax-dev будет правильным.
Но можно сделать такой костыль:

public class MyEvent
{
    public event EventHandler ActionHappened;

    public void FireEvent()
    {
        ActionHappened?.Invoke(this, EventArgs.Empty);
    }
}

public class SomeClass
{
    public void RegisterInner(MyEvent @event)
    {
        EventHandler handler = null!;
        handler = (sender, args) =>
        {
            Console.WriteLine("Hello, world");
            @event.ActionHappened -= handler;
        };

        @event.ActionHappened += handler;
    }
}
public class Program
{
    public static void Main()
    {
          var myEvent = new MyEvent();
          var someClass = new SomeClass();
          someClass.RegisterInner(myEvent);
          myEvent.FireEvent();
          myEvent.FireEvent();
          myEvent.FireEvent();
          myEvent.FireEvent();
          myEvent.FireEvent();
     }
}

Hello, world будет выведен только 1 раз

Comments

[romaro]

Спасибо! Я решил пойти немного другим путем. Задача, которую нужно было решить связана с отображением контекстного меню по клику на DataGridView (winforms).

Требуется, чтобы первый пункт меню становился бы видимым лишь в том случае, когда клик происходит по ячейке (событие CellMouseDown). Проблема заключалась в том, что обработчик этого события должен открывать форму объекта с тем идентификатором, к которому относится строка клика. Но обработчик клика ничего не знает об id, которое вычисляется при перехвате CellMouseDown. Я думал, что можно как-то изящно "замкнуть" состояние первого обработчика, а затем обнулить его, выполнив отписку.

Но пришлось-таки вынести id в глобальное состояние. Если все же есть более изящное решение моей задачи, буду рад помощи. Пока вот так:

// Потребителем этой переменной является только обработчик клика, поэтому не важно, что кеш подвисает в случае клика за пределами ячейки
        int? _idCache;
        void GridControl_CellMouseDown(object? sender, DataGridViewCellMouseEventArgs e)
        {
            _idCache = null;

            _gridControl.CurrentCell = _gridControl[e.ColumnIndex, e.RowIndex];
            var idCell = _gridControl.CurrentRow.Cells[AppSettings.IdColumnName];
            _idCache = idCell.Value == DBNull.Value ? null : (int)idCell.Value;

            if (_idCache.HasValue)
            {
                _openMenuItem.Visible = true;
                _openMenuItem.Click -= Open_Click;
                _openMenuItem.Click += Open_Click;
            }
        }

// Открываем форму
        void Open_Click(object? sender, EventArgs e)
        {
            App.ShowDataRecordForm(_dataDomainName, _idCache);
        }