自定义事件多次触发

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

自定义事件多次触发

Custom event firing multiple times

c#

asp.net

webforms