Я пытаюсь перебрать список значений в 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 идентичны.
Да, это был всего лишь остаток предыдущей версии, которую я сохранил на всякий случай и пока ничего не делает.
Вы запускаете это в Excel?
Да, на листе с поддержкой макросов.
Окончательный код после получения копии файла 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, это займет некоторое время, чтобы переварить. Однако я запустил его, и у меня те же проблемы, поэтому я предполагаю, что дело не в коде, а в чем-то в электронной таблице.
Отредактировано, чтобы добавить некоторые выходные данные отладки, которые могут быть полезны при выяснении проблемы... Если вы получаете сообщение об ошибке несоответствия типов, возможно, некоторые значения ячеек Excel не являются числовыми.
Он будет перебирать и заменять значения из J7-J13, но начиная с J14 он просто не распознает значение. Значения определенно числовые, а ячейки имеют денежный формат, как и те, которые работают. J14 и J15 — это «0» в слове «документ», и с этого момента они вообще не заменяют.
Для (например) «rev» у вас есть блоки по 9, то есть годы 1-5 это J7:J15, J22:J30, J37:J45, J53:J61 и J69:J77. Если это не то, чего вы хотите/ожидаете, вам нужно объяснить, чем все должно быть по-другому. Я могу посмотреть, если вы готовы поделиться файлом XL по электронной почте.
Я скопировал адрес. Перешлю на мгновение
Исправлена одна строка выше (см. ##FIX). В противном случае отправленная вами копия читается нормально. Вы проверили вывод на панели «Немедленно»?
Макрос находится в отдельном файле или содержится в файле Excel, где находятся данные? Если он находится в том же файле, вы можете пропустить Set xlWB = Workbooks.Open(..)
и вместо этого обратиться к ThisWorkbook
.
Итак Set xlWS = ThisWorkbook.WorkSheets("Cashflow")
Do While r < 2
— ваш цикл выполняется только один раз?