c# - 自定义事件多次触发

Question

我有一个类似于此处发布的问题:Event fires more and more times

但是该解决方案对我不起作用。我有一个子控件，它在单击按钮时触发事件，在父页面上有一个监听器。当点击事件发生并调用该事件时，它会在父页面上多次触发。每次递增一个。

页面加载(在父级上)和按钮单击(在子级上)事件只触发一次，只是事件方法会运行多次。

用户控制

public delegate void QuickViewClickEventHandler(int jobId, int bayId);

public static event QuickViewClickEventHandler QuickViewClicked;

protected void QuickViewLinkButton_OnClick(object sender, EventArgs e)
{
    // code removed for clarity
    OnQuickViewClicked(jobId, bayId);
}

protected void OnQuickViewClicked(int jobId, int bayId)
    {
        var handler = QuickViewClicked;
        if (handler != null)
        {
            handler(jobId, bayId);
        }
    }

父页面

<asp:Repeater runat="server" ID="BayRepeater" OnItemDataBound="BayRepeaterStuff_ItemDataBound">
    <ItemTemplate>
        <uc:BayViewItem ID="BayViewItemControl" runat="server" />
    </ItemTemplate>
</asp:Repeater>

protected void Page_Load(object sender, EventArgs e)
{
    BayViewItem.QuickViewClicked += BayViewItem_QuickViewClicked;
}

private void BayViewItem_QuickViewClicked(int jobId, int bayId)
{
    // code removed for clarity

    // unregistering the event seems to work but only after the first time
    // initial page load will still cause it to fire multiple times
    BayViewItem.QuickViewClicked -= BayViewItem_QuickViewClicked;
}

Answer 1

您的代码看起来不错。只有在另一次检查之后，我才知道出了什么问题。

在您的页面中，您有一个用户控件的实例。您应该订阅该用户控件的事件处理程序，因此它只会在您的页面范围内。如果你这样做，你就不会因为有人同时请求这个页面而多次触发同一个事件。没有理由为什么这里的事件应该是静态的，基本上使这个静态会导致这些问题。

所以您需要做的是让您的事件处理程序成为非静态的:

public event QuickViewClickEventHandler QuickViewClicked;

你的页面你的 Page_Load 应该是你使用用户控件实例的地方:

protected void Page_Load(object sender, EventArgs e)
{
    BayViewItemInstance.QuickViewClicked += BayViewItem_QuickViewClicked;
}

编辑: 我错过了控件不在页面中而是在转发器中。因此，要使用中继器实现相同的效果(但可以在页面中完成相同的操作，而无需在 Page_Load 中进行)设置 OnQuickViewClicked (On + EventHandler-name) 相当于代码隐藏中的 .QuickViewClicked += :

<asp:Repeater runat="server" ID="BayRepeater" OnItemDataBound="BayRepeaterStuff_ItemDataBound"> 
    <ItemTemplate> 
        <uc:BayViewItem ID="BayViewItemControl" runat="server" OnQuickViewClicked="BayViewItem_QuickViewClicked" />
    </ItemTemplate> 
 </asp:Repeater>

现在您不需要注销任何东西，因为事件处理程序不在静态范围内:

private void BayViewItem_QuickViewClicked(int jobId, int bayId)
{
    // code removed for clarity

    // unregistering the event seems to work but only after the first time
    // initial page load will still cause it to fire multiple times
    //BayViewItem.QuickViewClicked -= BayViewItem_QuickViewClicked;
}