C# windows forms handle click spam

Question

У меня есть приложение со многими пользовательскими элементами управления и несколькими кнопками на каждом, каждая кнопка имеет событие OnClick, которое делает некоторые важные вещи, а затем отправляет новый пользовательский элемент управления.

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

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

EDIT: Будет ли это хорошим решением для вывода из Button и переопределить событие OnClick, поэтому он всегда проверяет «рабочую» переменную, и если это правда, она не запускает событие? Что-то вроде:

public class MyButton : Button 
{ 
private static bool isWorking = false; 

protected override void OnClick(EventArgs e) 
{ 
    if (!isWorking) 
    { 
     isWorking = true; 
     base.OnClick(e); 
     isWorking = false; 
    } 
    //Else do nothing 
} 

}

Answer 1

Если вы хотите, чтобы все кнопки ждать, пока работа одной кнопки не будет сделана, добавьте bool isProcessing переменные в форму. Заверните работу каждой кнопки внутри if (!isProcessing) и установите для этого флага значение true в первой строке внутри оператора if. Затем не забудьте установить его обратно в false перед тем, как выйти из if.

Я предполагаю, что вы делаете все это асинхронно, поскольку, если все это в одном потоке, форма будет блокироваться, пока она обрабатывает работу. Тем не менее, это решит вашу проблему.

Answer 2

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

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

Что-то вроде этого:

void EnableControls(bool enable) 
{ 
    foreach (Control ctl in this.Controls) 
     ctl.Enabled = enable; 
} 

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

Существует альтернативное решение для использования таймера - отключите кнопку, но включите ее через 1 секунду. Это предотвращает нажатие нескольких пользователей, если это может привести к повреждению данных (т. Е. Каждый клик рассматривается как новая операция).

Answer 3

Я бы назвал ту же функцию от каждой кнопки, а затем выполнить определенную задачу:

private void Button_Click(object sender, EventAgrs e) 
{ 
    Button btn = sender; 
    btn.disable = true; 
    switch (btn.AccessibleName) 
    // call specific function for the particular button or do it all here 
} 

Answer 4

Вы можете использовать некоторые timeStamp для задержки между 2 клика:

DateTime timeStamp; 
//this will handle the clicks with the allowed interval being 0.5 second 
//Note that a tick is equal to 1/10,000,000 of second. 
private void click_Handler(object sender, EventArgs e) { 
    if ((DateTime.Now - timeStamp).Ticks < 5000000) return; 
    timeStamp = DateTime.Now; 
    //your code goes here .... 
} 

Answer 5

Я не уверен, если это будет даже работать, а просто идея ...

Вы можете попробовать с аспектно-ориентированным подходом (например, с помощью Postsharp):

1. Создайте два аспекта: один для ввода метода и один для выхода метода. В методе ввода отметьте текущий метод как «обработка» (например, добавьте имя метода в хэш-набор).В методе exit отметьте метод как «не обрабатывать» (удалите его из набора хэшей). Затем в записи метода проверьте, обрабатывается ли метод и если он есть, затем отмените метод (например: https://stackoverflow.com/a/2437794/113858)
2. Отметьте все обработчики событий с этим аспектом.