Утилизировать таймер внутри анонимного метода

Одна вещь, которую вы можете сделать по-другому, — это переместить Dispose для таймера в начало анонимного метода. Таким образом, даже если вы вызовете исключение позже в методе, вы все равно удалите таймер. Я использовал этот шаблон раньше, и это достаточно чистый способ получить отложенный обратный вызов.

Если вы используете C#5, есть очень хороший Task.Delay вы можете ждать, чтобы получить обратный вызов таймера в асинхронном методе. Это часто используется для реализации тайм-аутов в сочетании с вызовом WaitAny следующим образом:

    public static async Task WithTimeout(this Task task, int timeout, string timeoutMessage = null, params object[] args)
    {
        var timeoutTask = Task.Delay(timeout);
        if (await Task.WhenAny(task, timeoutTask) == timeoutTask)
            throw new TimeoutException(timeoutMessage == null ? "Operation timed out" : string.Format(timeoutMessage, args));

        await task;
    }

Очень странная задача, но ладно.

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

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

Кэш нужен только в том случае, если вы создаете таймеры очень часто (сотни раз в секунду), потому что многократное создание и удаление такого тяжеловесного объекта, как Таймер (он использует некоторые системные ресурсы) может привести к проблемам с производительностью. Но из вашего подробного описания видно, что вы не собираетесь так часто создавать таймеры, так что можете оставить свое решение. Но вместо использования таймера вы можете использовать RX Interval наблюдаемая коллекция, и весь ваш код будет сокращен до 1 строки :

Observable.Interval(TimeSpan.FromSeconds(2)).Take(1).Subscribe(_ => MessageBox.Show("Do something after 2000ms"));

У меня не было серьезных проблем с производительностью при создании тысячи таймеров с вашим кодом.

Удаление Timer (или любого IDisposable) в анонимном методе вообще не проблема.

Однако ваш код написан не лучшим образом. Вместо этого попробуйте эту реализацию:

    var delayTimer = new Timer()
    {
        Interval = 2000,
        Enabled = true,
    };
    EventHandler tick = null;
    tick = (_s, _e) => 
    { 
        delayTimer.Tick -= tick;
        delayTimer.Stop();     
        delayTimer.Dispose(); 
        MessageBox.Show("Do something after 2000ms");
    };
    delayTimer.Tick += tick;

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

Тем не менее, мне нравится ответ Rx на этот вопрос как самый чистый путь. Хотя использование Rx не упорядочивает обратный вызов в потоке пользовательского интерфейса, если вы не сделаете это:

Observable
    .Timer(TimeSpan.FromSeconds(2.0))
    .ObserveOn(this) // assuming `this` is your form
    .Subscribe(_ => MessageBox.Show("Do something after 2000ms"));

Гораздо проще.