Pergunta

Então, eu tenho esse menu de contexto funcionando, exceto que o evento por trás da seleção real do item de menu parece disparar várias vezes. Primeira vez que clico nele, ele dispara uma vez, depois duas vezes e depois três vezes. Então, no exemplo que acabei de dar, por 3 cliques, teria disparado um total de 6 vezes (1 + 2 + 3). Por que é que?

Abaixo está o meu código sobre como estou criando os itens do menu. Eu o tirei para as peças relevantes; Eu omiti coisas como .Tag,. Visíveis e. Estou construindo isso usando .NET 3.5 e vs 2008.

Desde já, obrigado!!

Private WithEvents ActiveExplorerCBars As Office.CommandBars
Private app As New Outlook.Application

Private Sub ThisAddIn_Startup(ByVal sender As Object, ByVal e As System.EventArgs) Handles Me.Startup
     ActiveExplorerCBars = app.ActiveExplorer.CommandBars
     AddHandler ActiveExplorerCBars.OnUpdate, AddressOf ActiveExplorerCBars_OnUpdate
End Sub

//This seems to get hit A LOT    
Private Sub ActiveExplorerCBars_OnUpdate()
    Dim bar As Office.CommandBar

    If IgnoreCommandbarsChanges Then Exit Sub

    bar = ActiveExplorerCBars.Item("Context Menu")

    If Not bar Is Nothing Then
        Dim addMenu As Boolean = False
        //this For loop just makes sure the context is only available when the user right-clicks over a mail item
        For Each mail As Outlook.MailItem In Application.ActiveExplorer().Selection
            addMenu = True
            Exit For
        Next
        If addMenu Then
            AddContextDropdown(bar)
        End If
    End If
End Sub

Private Sub AddContextDropdown(ByVal ContextMenu As Office.CommandBar)
    Dim RootPopup As Office.CommandBarPopup
    Dim popupTaskItem As Office.CommandBarPopup
    RootPopup = ContextMenu.FindControl(Type:=Office.MsoControlType.msoControlPopup, Tag:="Update task")

    If RootPopup Is Nothing Then
        ChangingBar(ContextMenu, Restore:=False)
        RootPopup = ContextMenu.Controls.Add(Type:=Office.MsoControlType.msoControlPopup)

        Dim thisTaskPopup As Office.CommandBarPopup
        popupTaskItem = ContextMenu.FindControl(Type:=Office.MsoControlType.msoControlPopup, Tag:=task.EntryID)
        If popupTaskItem Is Nothing Then
              popupTaskItem = RootPopup.Controls.Add(Type:=Office.MsoControlType.msoControlPopup)
              thisTaskPopup = popupTaskItem
              AddActionButtons(thisTaskPopup)
        End If
    End If
End Sub

Private Sub AddActionButtons(ByVal puItem As Office.CommandBarPopup)

    Dim puDeploy As Office.CommandBarPopup = puItem.Controls.Add(Type:=Office.MsoControlType.msoControlPopup)
    Dim btnActionItem As Office.CommandBarControl = puDeploy.Controls.Add(Type:=Office.MsoControlType.msoControlButton)
    Dim thisButton As Office.CommandBarButton = btnActionItem
    AddHandler thisButton.Click, AddressOf OnContextClick
End Sub

//Click event
Public Sub OnContextClick(ByVal ctrl As Microsoft.Office.Core.CommandBarButton, ByRef CancelDefault As Boolean)
   //This messagebox shows once the first time, twice the second, 3 times, etc
    MessageBox.Show("Clicked: " & ctrl.Caption)
End Sub
Foi útil?

Solução

Eu descobri. Tobrien, usei seu último comentário como veículo para chegar a essa conclusão, principalmente onde você disse:

Pode ser que você esteja realmente criando retornos de chamada adicionais (sinalizados idênticos)

O código que estou usando para adicionar o manipulador é:

AddHandler thisButton.Click, AddressOf OnContextClick

Como isso poderia ser idêntico? Bem, existe apenas um ONCONTEXTCLICK sub ... e então thisbutton?

For Each value As ActivityType.Request In [Enum].GetValues(GetType(ActivityType.Request))
        Dim btnActionItem As Office.CommandBarControl = puRequest.Controls.Add(Type:=Office.MsoControlType.msoControlButton)
        With btnActionItem
            .Tag = puRequest.Tag & "|" & value
            .Caption = [Enum].GetName(GetType(ActivityType.Request), value)
            .Visible = True
        End With
        Dim thisButton As Office.CommandBarButton = btnActionItem
        AddHandler thisButton.Click, AddressOf OnContextClick
    Next

Esse código é executado quando ocorre o OnUpdate, o que, como você sabe, acontece o tempo todo. Portanto, em essência, cada vez que o OnUpdate hits, estou adicionando um manipulador adicional para o mesmo botão exatamente, não considerando que o botão é basicamente criado recentemente cada vez que ocorre o OnUpdate e que seu manipulador é armazenado na memória.

Então, eu precisava tornar o controle de botão exclusivo:

.Tag = puRequest.Tag & "|" & value & "|" & Now.ToBinary().ToString()

Acabei de adicionar um agora.Tobinary (). ToString () no final da propriedade .TAG para garantir que cada vez que o botão seja criado para o usuário, ele possui uma tag exclusiva. Agora, os eventos são únicos e são apenas disparos uma vez por clique.

Tobrien, eu te soluto! Embora eu tenha respondido minha própria pergunta, não foi sem a sua orientação. Obrigado!

Licenciado em: CC-BY-SA com atribuição
Não afiliado a StackOverflow
scroll top