Fornire aggiornamenti di stato per la macro che non risponde allo stato fino al completamento
-
09-12-2019 - |
Domanda
Ho una macro VBA per cercare attraverso gli archivi di posta elettronica.
Durante la ricerca di decine di decine di migliaia di e-mail, (o anche solo un paio di centinaia della macchina di prova) visualizza lo stato per alcuni secondi, quindi entra in uno stato che non risponde mentre si passa attraverso il resto delle e-mail.
.Ciò ha portato gli utenti impazienti a chiudere il compito prematuramente, e vorrei rettificarlo fornendo aggiornamenti di stato.
Ho codificato la seguente soluzione e riteniamo che il problema sta nel modo in cui la GarbageCollector funziona in VBA durante il loop.
Public Sub searchAndMove()
UserForm1.Show
' Send a message to the user indicating
' the program has completed successfully,
' and displaying the number of messages sent during the run.
End Sub
Private Sub UserForm_Activate()
Me.Width = 240
Me.Height = 60
Me.Label1.Width = 230
Me.Label1.Height = 50
Dim oSelectTarget As Outlook.Folder
Dim oMoveTarget As Outlook.Folder
Dim oSearchCriteria As String
' Select the target folder to search and then the folder to
' which the files should be moved
Set oSelectTarget = Application.Session.PickFolder
Set oMoveTarget = Application.Session.PickFolder
oSearchCriteria = InputBox("Input search string: ")
Dim selectedItems As Outlook.Items
Set selectedItems = oSelectTarget.Items
Dim selectedEmail As Outlook.MailItem
Dim StatusBarMsg As String
StatusBarMsg = ""
Dim initialCount As Long
initialCount = selectedItems.count
Dim movedCounter As Long
movedCounter = 0
Dim x As Long
Dim exists As Long
' Function Loop, stepping backwards
' to prevent errors derived from modifying the collection
For x = selectedItems.count To 1 Step -1
Set selectedEmail = selectedItems.Item(x)
' Test to determine if the subject contains the search string
exists = InStr(selectedEmail.Subject, oSearchCriteria)
If Len(selectedEmail.Subject) > 999 Then
selectedEmail.Move oMoveTarget
Else:
If exists <> 0 Then
selectedEmail.Move oMoveTarget
movedCounter = (movedCounter + 1)
Else: End If
End If
Set selectedEmail = Nothing
StatusBarMsg = "Processing " & x & " out of " & initialCount & " messages."
UserForm1.Label1.Caption = StatusBarMsg
UserForm1.Repaint
Next x
Dim Msg As String
Dim Response
Msg = "SearchAndMove has detected and moved " & movedCounter & _
" messages since last run."
Response = MsgBox(Msg, vbOKOnly)
' Close the References to prevent a reference leak
Set oSelectTarget = Nothing
Set oMoveTarget = Nothing
Set selectedItems = Nothing
Set selectedEmail = Nothing
Unload Me
End Sub
. Soluzione
Cambia la riga
UserForm1.Repaint
a
DoEvents
Sì, aumenterà il tempo di esecuzione ma nel caso ci siano migliaia di e-mail, non hai molte opzioni.
Suggerimento: Inoltre potresti voler cambiare
StatusBarMsg = "Processing " & x & " out of " & initialCount & " messages."
a
StatusBarMsg = "Please do not interrupt. Processing " & x & " out of " & initialCount & " messages."
Inoltre è consigliabile informare il tuo utente all'inizio del processo che potrebbe richiedere del tempo e quindi possono eseguire il processo quando sono sicuri che non vogliono lavorare su quel PC?
Qualcosa come questo
.
Sub Sample()
Dim strWarning As String
Dim Ret
strWarning = "This process may take sometime. It is advisable to run this " & _
"when you don't intend to use the pc for sometime. Would you like to Continue?"
Ret = MsgBox(strWarning, vbYesNo, "Information")
If Ret <> vbYes Then Exit Sub
For x = SelectedItems.Count To 1 Step -1
'~~> Rest of the code
End Sub
HTH
Sid