Question

我遇到类似于此处发布的问题：Event fires more and more times

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

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

用户控制

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 + =： / p>

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

自定义事件多次触发

1 个答案: