0

所以我遇到了一个小小的绊脚石，希望有人能帮助我。简而言之，我需要访问一串网页（每个页面上的名称列表已经输入，该代码可以正常工作）。当我的代码访问每个页面时，我需要拉回信息。不幸的是，有一个问题——在我得到“自动化错误未指定错误”之前，它甚至无法通过“A”列表，而且它永远不会出现在同一个位置。

我已经尝试了“正常”步骤来解决这个问题。我已经安装了 VB 6 控件，并且我已经取消注册并重新注册了 mscomctl.ocx，并且包括 On Error Resume Next（它不执行任何操作）。

它通常会在死亡之前达到 100 多个案例（正如我之前所说的那样随机）。并且在弹出错误之后，当我尝试重新运行它（有或没有更改）并且它在第一个错误时出错。如果我重新启动计算机，它会让我再试一次（无论出于何种原因），但它仍然没有完成。

代码是否太复杂，我需要减少它吗？我可能可以找到一种方法让它一次只运行每个字母（运行所有 A，然后运行 ​​B，等等），但我什至无法让它完成字母 A。

我注意到在另一个线程中有人建议不要使用 IE 来交换到 xmlhttp - 这是解决这个问题的方法吗？是这个脚本太长的问题吗？我在这里到底做错了什么？

Sub Lookup()
Range("AI1").Value = "Unique ID"
Range("AJ1").Value = "Name"
Range("AK1").Value = "Birth Year"
Range("AL1").Value = "Title"
Range("AM1").Value = "State"
Range("AN1").Value = "Position"
Range("AO1").Value = "Country"
Range("AP1").Value = "Appointed"
Range("AQ1").Value = "Credentials"
Range("AR1").Value = "Terminations"
Dim i As Integer
For i = 1 To 26
    If i = 24 Then
        Range("X:X").End(xlUp).Select
        ActiveCell.Value = ""
    Else
    Dim ic As String
    ic = LCase(ConvertToLetter(i))
    Range(ic & "5000").End(xlUp).Select
    Dim J As Integer
    J = ActiveCell.Row
    Dim k As Integer
    For k = 2 To J
        Range(ic & k).Select
        Dim Lookup As String
        Lookup = ActiveCell.Value
        Dim IE As Variant
        Set IE = CreateObject("InternetExplorer.Application")
        IE.Visible = False
        IE.navigate "http://history.state.gov/departmenthistory/people/" & Lookup
        Do
            DoEvents
        Loop Until IE.readyState = READYSTATE_COMPLETE
        Dim Doc As HTMLDocument
        Set Doc = IE.document
        Dim Italics As Integer
        Italics = 0
        Dim EachA As Integer
        For EachA = 64 To 100
            Dim Position As String
            Position = Doc.getElementsByTagName("a")(EachA).innerText
            If Position = "Home" Then
                Exit For
            Else
                Dim NameBY As String
                NameBY = Doc.getElementsByTagName("h2")(1).innerText
                Dim TitleST As String
                TitleST = Doc.getElementsByTagName("p")(1).innerText
                Range("AJ" & "90000").End(xlUp).Offset(1, 0).Select
                ActiveCell.Value = NameBY
                TitleState = Split(TitleST, vbLf)
                ActiveCell.Offset(0, 2).Value = TitleState(0)
                On Error GoTo 1037
                ActiveCell.Offset(0, 3).Value = TitleState(1)
                On Error GoTo 1037
1037
                ActiveCell.Offset(0, 4).Select
                ActiveCell.Value = Position
                Dim EachLi As Integer
                EachLi = EachA - 1
                If Doc.getElementsByTagName("li").Item(EachLi + Italics).innerHTML Like "<em>*" Then
                    Italics = Italics + 1
                Else
                End If
                Dim JobList As String
                JobList = Doc.getElementsByTagName("li")(EachLi + Italics).innerText
                Dim Job() As String
                Job() = Split(JobList, vbLf)
                Dim JCount As Integer
                For JCount = LBound(Job) To UBound(Job)
                    ActiveCell.Offset(0, 1).Select
                    ActiveCell.Value = Job(JCount)
                Next JCount
            End If
        Next EachA
    Next k
End If
Next i
End Sub
4

1 回答 1

1

我注意到的一件事是，您在循环中不断创建新的 IE 对象，并且您永远不会破坏它们或设置为Nothing. 创建 100 多个 IE 实例是没有意义的、昂贵的，并且可能是错误的来源。

我认为最初创建一个 IE 实例可能会有所帮助，然后在循环中使用相同的对象来导航所需的 URL。

所以代替这个：

Dim IE As Variant
Set IE = CreateObject("InternetExplorer.Application")

做这个：

Dim IE as Object
If IE Is Nothing Then Set IE = CreateObject("InternetExplorer.Application")
于 2014-07-11T15:16:05.087 回答