Обеспечение обновлений статуса для макроса, который входит в не отвечает государству до завершения

StackOverflow https://stackoverflow.com//questions/9622983

  •  09-12-2019
  •  | 
  •  

Вопрос

У меня есть макрос VBA для поиска в архивах электронной почты.

При поиске через десятки тысяч электронных писем, (или даже всего лишь пару сотен на моем тестовой машине) он отображает статус в течение нескольких секунд, затем входит в не отвечает состояние во время пробега через остальные электронные письма.

.

Это привело нетерпеливые пользователи, чтобы закрыть задачу преждевременно, и я хотел бы исправить это, предоставляя обновления статуса.

Я закодировал следующее решение, и полагаю, что проблема заключается в том, как Гарбагольтектор функционирует в VBA во время петли.

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
.

Это было полезно?

Решение

Изменить строку

UserForm1.Repaint

к

DoEvents

Да, это увеличит время выполнения, но в случае, если в том случае, если у вас нет много вариантов электронного письма.

<Сильный> Совет: Также вы можете изменить

StatusBarMsg = "Processing " & x & " out of " & initialCount & " messages."

к

StatusBarMsg = "Please do not interrupt. Processing " & x & " out of " & initialCount & " messages."

Также рекомендуется сообщить своему пользователю в начале процесса, который может потребоваться время и, следовательно, они могут запускать процесс, когда они уверены, что они не хотят работать на этом ПК?

что-то вроде этого

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

Лицензировано под: CC-BY-SA с атрибуция
Не связан с StackOverflow
scroll top