C# windows窗体处理点击垃圾邮件[重复]

Question

我有一个包含许多用户控件和每个按钮的应用程序，每个按钮都有一个 OnClick 事件，该事件执行一些重要的操作，然后发送到新的用户控件。

当用户多次点击非常快时，问题就出现了，事件代码在退出到新的用户控件之前执行了多次，从而导致问题。

我目前的解决方案是禁用事件处理程序第一行的按钮，但是对每个窗口和处理程序都这样做会很麻烦，我该怎么办？

编辑：从 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

您可以使用一些timeStamp 来延迟两次点击：

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 2

如果您希望所有按钮都等到一个按钮的工作完成，请在表单中添加一个bool isProcessing 变量。将每个按钮的工作包装在 if (!isProcessing) 中，并在 if 语句的第一行中将该标志设置为 true。然后不要忘记在退出 if 之前将其设置回 false。

我假设您正在异步执行所有这些操作，因为如果它们都在同一个线程中，表单将在处理工作时锁定。不过，这将解决您的问题。

Answer 3

在进行敏感操作时禁用控件是我一直采用的典型解决方案。

但由于在一个屏幕上可能有相当多的控件会受到一些点击或 UI 更改的影响，所以我通常会设计一种专门的方法来遍历所有受影响的控件并相应地禁用/启用它们。

类似这样的：

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

同样，您可以将控件分组到相关的存储桶中，以便仅禁用/启用其中一个等。取决于您的确切需求。

还有一种使用计时器的替代解决方案 - 禁用按钮，但在 1 秒后启用它。这可以防止紧张的用户多次单击会导致数据损坏（即每次单击都被视为新操作）。

Answer 4

我会从每个按钮调用相同的函数，然后执行特定任务：

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 5

我不确定这是否可行，但只是一个想法......

您可以尝试使用面向方面的方法（例如借助 Postsharp）：

1. 创建两个方面，一个用于方法入口，一个用于方法出口。在方法条目中，将当前方法标记为“处理”（例如，将方法名称添加到哈希集中）。在方法退出中，将该方法标记为“未处理”（将其从哈希集中删除）。然后在方法入口中检查方法是否正在处理，如果是，则取消该方法（如：https://stackoverflow.com/a/2437794/113858）
2. 使用此方面标记所有事件处理程序。