自定义事件多次触发

本文关键字：事件 自定义 | 更新日期: 2023-09-27 17:59:10

我有一个类似于这里发布的问题：事件触发的次数越来越多

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

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

用户控制

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;
}

自定义事件多次触发

您的代码看起来不错。只有在再次检查后，我才发现出了什么问题。

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

因此，您需要做的是使事件处理程序非静态：

public event QuickViewClickEventHandler QuickViewClicked;

您的页面您的page_Load应该是您使用用户控件实例的位置：

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

编辑：我没有注意到控制不在页面上，而是在中继器上。因此，为了使用中继器实现同样的效果（但同样可以在页面中完成，而不需要在page_Load中完成），需要设置OnQuickViewClicked（On+EventHandler名称），这相当于代码背后的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;
}
相关文章：