VBA .cells распознает значение в некоторых ячейках, но не в других

Я пытаюсь перебрать список значений в Excel, чтобы заполнить определенные части документа Word. Скрипт VBA находит строки, перечисленные в массивах, объявленных вверху («revArrayY1» и т. д.), и заменяет этот текст соответствующими значениями в электронной таблице, используя функцию .Cells для получения значения. Сначала это работало, но при добавлении еще одной строки в список значений в Excel для заполнения функция .Cells не распознает значение в новой ячейке. Все, что я сделал, это сдвинул ячейки в Excel вверх, чтобы освободить место для новой строки, и добавил новую строку в массив.

Option Explicit
Sub ReplaceText()
    Dim wApp As Object ' Change to "Object" for late binding
    Dim wdoc As Object ' Change to "Object" for late binding
    Dim custN As String, path As String
    Dim revArrayY1 As Variant ' Declare as Variant for now
    Dim revArrayY2 As Variant ' Declare as Variant for now
    Dim revArrayY3 As Variant ' Declare as Variant for now
    Dim revArrayY4 As Variant ' Declare as Variant for now
    Dim revArrayY5 As Variant ' Declare as Variant for now
    
    Dim COGArrayY1 As Variant ' Declare as Variant for now
    Dim COGArrayY2 As Variant ' Declare as Variant for now
    Dim COGArrayY3 As Variant ' Declare as Variant for now
    Dim COGArrayY4 As Variant ' Declare as Variant for now
    Dim COGArrayY5 As Variant ' Declare as Variant for now
    
    Dim xlApp As Object
    Dim xlWB As Object
    Dim xlWS As Object
    
    ' Initialize Excel objects
    Set xlApp = CreateObject("Excel.Application")
    Set xlWB = xlApp.Workbooks.Open("C:[file path]/ Financials.xlsx")
    Set xlWS = xlWB.Sheets("Cashflow") ' Change "Cashflow" to your sheet name
    
    ' Sales Arrays
    revArrayY1 = Array("<BeerSalesY1>", "<WineSalesY1>", "<SpiritSalesY1>", "<KDSalesY1>", "<BoochSalesY1>", "<VenueSalesY1>", "<KCSalesY1>", "<MealSalesY1>", "<RevFcstY1>")
    revArrayY2 = Array("<BeerSalesY2>", "<WineSalesY2>", "<SpiritSalesY2>", "<KDSalesY2>", "<BoochSalesY2>", "<VenueSalesY2>", "<KCSalesY2>", "<MealSalesY2>", "<RevFcstY2>")
    revArrayY3 = Array("<BeerSalesY3>", "<WineSalesY3>", "<SpiritSalesY3>", "<KDSalesY3>", "<BoochSalesY3>", "<VenueSalesY3>", "<KCSalesY3>", "<MealSalesY3>")
    revArrayY4 = Array("<BeerSalesY4>", "<WineSalesY4>", "<SpiritSalesY4>", "<KDSalesY4>", "<BoochSalesY4>", "<VenueSalesY4>", "<KCSalesY4>", "<MealSalesY4>")
    revArrayY5 = Array("<BeerSalesY5>", "<WineSalesY5>", "<SpiritSalesY5>", "<KDSalesY5>", "<BoochSalesY5>", "<VenueSalesY5>", "<KCSalesY5>", "<MealSalesY5>")
    
    'COGS Arrays
    COGArrayY1 = Array("<COGBeerY1>", "<COGWineY1>", "<COGSpiritY1>", "<COGKDY1>", "<COGBoochY1>", "<COGVenueY1>", "<COGKCY1>", "<COGMealY1>")
    COGArrayY2 = Array("<COGBeerY2>", "<COGWineY2>", "<COGSpiritY2>", "<COGKDY2>", "<COGBoochY2>", "<COGVenueY2>", "<COGKCY2>", "<COGMealY2>")
    COGArrayY3 = Array("<COGBeerY3>", "<COGWineY3>", "<COGSpiritY3>", "<COGKDY3>", "<COGBoochY3>", "<COGVenueY3>", "<COGKCY3>", "<COGMealY3>")
    COGArrayY4 = Array("<COGBeerY4>", "<COGWineY4>", "<COGSpiritY4>", "<COGKDY4>", "<COGBoochY4>", "<COGVenueY4>", "<COGKCY4>", "<COGMealY4>")
    COGArrayY5 = Array("<COGBeerY5>", "<COGWineY5>", "<COGSpiritY5>", "<COGKDY5>", "<COGBoochY5>", "<COGVenueY5>", "<COGKCY5>", "<COGMealY5>")
    
    Dim i As Integer
    Dim r As Long
    r = 1
    
    ' Loop through rows of Excel worksheet
    Do While r < 2
        ' Create new instance of Word application
        Set wApp = CreateObject("Word.Application")
        wApp.Visible = True
        
        ' Open the Word document
        Set wdoc = wApp.Documents.Open(Filename: = "C:[file path]/Business Plan.dotx")
        
    ' Loop through Revenue Projections
        ' Loop through the array of strings
        i = 0
        For i = LBound(revArrayY1) To UBound(revArrayY1)
            ' Find and replace text in Word document
            With wdoc.Content.Find
                .Text = revArrayY1(i)
                 Debug.Print xlWS.Cells(7 + i, 4).Value
                .Replacement.Text = CStr(Round(xlWS.Cells(7 + i, 10).Value / 1000, 0))
                .Execute Replace:=2 ' wdReplaceAll constant value is 2
            End With
        Next i
        
        i = 0
        For i = LBound(revArrayY2) To UBound(revArrayY2)
            ' Find and replace text in Word document
            With wdoc.Content.Find
                .Text = revArrayY2(i)
                .Replacement.Text = CStr(Round(xlWS.Cells(22 + i, 10).Value / 1000, 0))
                .Execute Replace:=2 ' wdReplaceAll constant value is 2
            End With
        Next i
        
       i = 0
        For i = LBound(revArrayY3) To UBound(revArrayY3)
            ' Find and replace text in Word document
            With wdoc.Content.Find
                .Text = revArrayY3(i)
                .Replacement.Text = CStr(Round(xlWS.Cells(37 + i, 10).Value / 1000, 0))
                .Execute Replace:=2 ' wdReplaceAll constant value is 2
            End With
        Next i
        
        i = 0
        For i = LBound(revArrayY4) To UBound(revArrayY4)
            ' Find and replace text in Word document
            With wdoc.Content.Find
                .Text = revArrayY4(i)
                .Replacement.Text = CStr(Round(xlWS.Cells(53 + i, 10).Value / 1000, 0))
                .Execute Replace:=2 ' wdReplaceAll constant value is 2
            End With
        Next i
        
        i = 0
        For i = LBound(revArrayY5) To UBound(revArrayY5)
            ' Find and replace text in Word document
            With wdoc.Content.Find
                .Text = revArrayY5(i)
                .Replacement.Text = CStr(Round(xlWS.Cells(69 + i, 10).Value / 1000, 0))
                .Execute Replace:=2 ' wdReplaceAll constant value is 2
            End With
        Next i
        
' Loop through COGS projections
        i = 0
        For i = LBound(COGArrayY1) To UBound(COGArrayY1)
            ' Find and replace text in Word document
            With wdoc.Content.Find
                .Text = COGArrayY1(i)
                .Replacement.Text = CStr(Round(xlWS.Cells(7 + i, 13).Value / 1000, 0))
                .Execute Replace:=2 ' wdReplaceAll constant value is 2
            End With
        Next i
        
        i = 0
        For i = LBound(COGArrayY2) To UBound(COGArrayY2)
            ' Find and replace text in Word document
            With wdoc.Content.Find
                .Text = COGArrayY2(i)
                .Replacement.Text = CStr(Round(xlWS.Cells(22 + i, 13).Value / 1000, 0))
                .Execute Replace:=2 ' wdReplaceAll constant value is 2
            End With
        Next i
        
        i = 0
        For i = LBound(COGArrayY3) To UBound(COGArrayY3)
            ' Find and replace text in Word document
            With wdoc.Content.Find
                .Text = COGArrayY3(i)
                .Replacement.Text = CStr(Round(xlWS.Cells(38 + i, 13).Value / 1000, 0))
                .Execute Replace:=2 ' wdReplaceAll constant value is 2
            End With
        Next i
        
        i = 0
        For i = LBound(COGArrayY4) To UBound(COGArrayY4)
            ' Find and replace text in Word document
            With wdoc.Content.Find
                .Text = COGArrayY4(i)
                .Replacement.Text = CStr(Round(xlWS.Cells(54 + i, 13).Value / 1000, 0))
                .Execute Replace:=2 ' wdReplaceAll constant value is 2
            End With
        Next i

        i = 0
        For i = LBound(COGArrayY5) To UBound(COGArrayY5)
            ' Find and replace text in Word document
            With wdoc.Content.Find
                .Text = COGArrayY5(i)
                .Replacement.Text = CStr(Round(xlWS.Cells(70 + i, 13).Value / 1000, 0))
                .Execute Replace:=2 ' wdReplaceAll constant value is 2
            End With
        Next i
        
        ' Save the Word document
        custN = xlWS.Cells(r, 1).Value
        path = "C:[file path]\"
        wdoc.SaveAs2 Filename:=path, FileFormat:=16, AddToRecentFiles:=False ' FileFormat 16 is wdFormatXMLDocument
        
        ' Close Word document
        wdoc.Close
        r = r + 1
    Loop
    
    ' Close Excel workbook and application
    xlWB.Close SaveChanges:=False
    xlApp.Quit
    
    ' Release objects from memory
    Set xlWS = Nothing
    Set xlWB = Nothing
    Set xlApp = Nothing
    Set wdoc = Nothing
    Set wApp = Nothing
End Sub



Я могу напечатать индекс «i» и увидеть, что цикл FOR выполняет итерацию по всему массиву, и я трижды проверил, чтобы убедиться, что я использую правильные номера строк и столбцов. Я также поместил произвольное число в произвольную ячейку и просто попытался распечатать эту ячейку, но .Cells также не распознает это значение и не выполнит отладку. Распечатайте это значение. Кажется, наступил момент, когда он распознавал только те значения в ячейках, которые у меня были раньше. Я подумал, что, возможно, что-то не так с форматированием ячейки, поэтому я скопировал и вставил значение из рабочей ячейки, но она все еще не распознает это значение. Более того, даже несмотря на то, что это не вызывает ошибки в первых двух циклах for (это просто не работает для последних значений пары), когда он доходит до цикла FOR, перебирая «revArrayY3», он выдает «Ошибку несоответствия типов». даже несмотря на то, что все значения имеют одинаковый формат и циклы FOR идентичны.

Do While r < 2 — ваш цикл выполняется только один раз?
Tim Williams 03.06.2024 22:28

Да, это был всего лишь остаток предыдущей версии, которую я сохранил на всякий случай и пока ничего не делает.

Shepherd darquea 03.06.2024 22:39

Вы запускаете это в Excel?

Tim Williams 03.06.2024 22:46

Да, на листе с поддержкой макросов.

Shepherd darquea 03.06.2024 22:51
Преобразование HTML-таблицы в профессиональный документ Excel
Преобразование HTML-таблицы в профессиональный документ Excel
Это самый простой способ создания Excel из HTML-таблицы.
Импорт excel в laravel в базу данных
Импорт excel в laravel в базу данных
Здравствуйте, дорогой читатель, в этой статье я расскажу практическим и быстрым способом, как импортировать файл Excel в вашу базу данных с помощью...
0
4
66
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

Ответ принят как подходящий

Окончательный код после получения копии файла Excel. Части, связанные со Word, закомментированы, поскольку у меня нет документа-шаблона.

Sub ReplaceText()
    Dim wApp As Object, wdoc As Object
    Dim custN As String, path As String
    Dim revArray As Variant, revArrayRows  As Variant
    Dim COGArray As Variant, COGArrayRows
    Dim xlWB As Object, xlWS As Object
    Dim i As Long, yr As Long, rw As Long, lbl As String, c As Range, v As String
    
    'label Arrays
    revArray = Array("BeerSales", "WineSales", "SpiritSales", "KDSales", _
                     "BoochSales", "VenueSales", "KCSales", "MealSales", "RevFcst")
    
    COGArray = Array("COGBeer", "COGWine", "COGSpirit", "COGKD", "COGBooch", _
                    "COGVenue", "COGKC", "COGMeal", "COGFcst")
    
    'Set xlWB = Workbooks.Open("C:\Users\sheph\Documents\Project Files\Fats Industries\Botanical Brews\Business Plan and Financials/Botanical Brews Financials.xlsx")
    Set xlWS = ThisWorkbook.Worksheets("Cashflow") ' Change "Cashflow" to your sheet name
    
    'Set wApp = CreateObject("Word.Application") ' Create new instance of Word application
    'wApp.Visible = True
    'Set wdoc = wApp.Documents.Open(Filename: = "C:\Users\sheph\Documents\Project Files\Fats Industries\Botanical Brews\Business Plan and Financials/Botanical Brews Business Plan.dotx") ' Open the Word document
    
    For yr = 1 To 5  'loop years 1-5
        
        m = Application.Match("Year " & yr, xlWS.Columns("A"), 0) 'find the year on the data sheet
        If Not IsError(m) Then  'got a match?
            rw = m + 3 'data starts 3 rows down from year
            Debug.Print "-------Year " & yr & ": Row# " & rw & " ------------"
        Else
            MsgBox "No 'Year " & yr & "' in ColA on data sheet", vbExclamation
            Exit Sub
        End If
        
        For i = LBound(revArray) To UBound(revArray)
            lbl = "<" & revArray(i) & "Y" & yr & ">"
            Set c = xlWS.Cells(rw + i, "J")
            v = CStr(Round(c.Value / 1000, 0))
            'if you get a "type mismatch" error on the next line it's likely because `c` value is not numeric
            Debug.Print "Got value '" & v & "' from " & c.Address(False, False) & " for " & lbl
            'ReplaceAll wdoc, lbl, v
        Next i
        
        For i = LBound(COGArray) To UBound(COGArray)
            lbl = "<" & COGArray(i) & "Y" & yr & ">"
            Set c = xlWS.Cells(rw + i, "M")
            v = CStr(Round(c.Value / 1000, 0))
            Debug.Print "Got value '" & v & "' from " & c.Address(False, False) & " for " & lbl
            'ReplaceAll wdoc, lbl, v
        Next i
    Next yr
    'xlWB.Close SaveChanges:=False
        
'    custN = xlWS.Cells(1, 1).Value ' Save the Word document
'    path = "C:C:\Users\sheph\Documents\Project Files\Fats Industries\Botanical Brews\Business Plan and Financials\"
'    wdoc.SaveAs2 Filename:=path, FileFormat:=16, AddToRecentFiles:=False ' 16 is wdFormatXMLDocument
'    wdoc.Close
'    wApp.Quit
End Sub

'In Word document `doc`, replace all instances of `findWhat` with `replaceWith`
Sub ReplaceAll(doc As Object, findWhat As String, replaceWith As String)
    'Debug.Print "Replacing '" & findWhat & "' with '" & replaceWith & "'"
    With doc.Content.Find
        .Text = findWhat
        .Replacement.Text = replaceWith
        .Execute Replace:=2 ' wdReplaceAll constant value is 2
    End With
End Sub

Спасибо Тим Уильямс! Ты быстро это подстроил. Будучи новичком в VBA, это займет некоторое время, чтобы переварить. Однако я запустил его, и у меня те же проблемы, поэтому я предполагаю, что дело не в коде, а в чем-то в электронной таблице.

Shepherd darquea 03.06.2024 23:48

Отредактировано, чтобы добавить некоторые выходные данные отладки, которые могут быть полезны при выяснении проблемы... Если вы получаете сообщение об ошибке несоответствия типов, возможно, некоторые значения ячеек Excel не являются числовыми.

Tim Williams 04.06.2024 00:01

Он будет перебирать и заменять значения из J7-J13, но начиная с J14 он просто не распознает значение. Значения определенно числовые, а ячейки имеют денежный формат, как и те, которые работают. J14 и J15 — это «0» в слове «документ», и с этого момента они вообще не заменяют.

Shepherd darquea 04.06.2024 00:20

Для (например) «rev» у вас есть блоки по 9, то есть годы 1-5 это J7:J15, J22:J30, J37:J45, J53:J61 и J69:J77. Если это не то, чего вы хотите/ожидаете, вам нужно объяснить, чем все должно быть по-другому. Я могу посмотреть, если вы готовы поделиться файлом XL по электронной почте.

Tim Williams 04.06.2024 01:30

Я скопировал адрес. Перешлю на мгновение

Shepherd darquea 04.06.2024 01:47

Исправлена ​​одна строка выше (см. ##FIX). В противном случае отправленная вами копия читается нормально. Вы проверили вывод на панели «Немедленно»?

Tim Williams 04.06.2024 02:06

Макрос находится в отдельном файле или содержится в файле Excel, где находятся данные? Если он находится в том же файле, вы можете пропустить Set xlWB = Workbooks.Open(..) и вместо этого обратиться к ThisWorkbook.

Tim Williams 04.06.2024 02:07

Итак Set xlWS = ThisWorkbook.WorkSheets("Cashflow")

Tim Williams 04.06.2024 06:54

Другие вопросы по теме