ループ コードがフォルダー内の同じ Excel スプレッドシートからコピーされ続ける

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

  •  02-01-2020
  •  | 
  •  

質問

そこで、フォルダー内に Excel ファイルのリスト (ファイル名とパス) を作成し、For ループを使用して、リストされているすべてのファイルの指定されたワークシートをコピーして、大きい。これまでのところ、すべてのファイルではなく同じファイルがコピー&ペーストされ続けるという事実を除いて、すべてが機能します。マクロは正しい回数だけループしますが、すべての Excel ファイルが使用されるわけではありません。

コードは次のとおりです。

フォルダー内のファイルを一覧表示する最初の部分

Private Sub btn_LeaveReport()
Dim objFSO As Object
Dim objFolder As Object
Dim objFile As Object
Dim i As Integer

'Create an instance of the FileSystemObject
Set objFSO = CreateObject("Scripting.FileSystemObject")
'Get the folder object
Set objFolder = objFSO.GetFolder("D:\Administration\Time Sheets")
i = 2
'loops through each file in the directory and prints their names and path
For Each objFile In objFolder.Files
    'print file name
    Cells(i + 1, 2) = objFile.Name
    'print file path
    Cells(i + 1, 3) = objFile.Path
    i = i + 1
Next objFile
End Sub

これがループの部分です

Private Sub btn_PullData()
'Declared Variables
Dim wbk As Workbook
Dim i As Integer
Dim StartAt As Integer
Dim EndAt As Integer
Dim CopyPath As String
Dim CopyPathRow As Integer
Dim iRow As Integer

'Ranges
StartAt = 1
EndAt = Val(ThisWorkbook.Worksheets("LeaveReport").Range("A1"))
CopyPathRow = 3
CopyPath = ThisWorkbook.Worksheets("LeaveReport").Range("C" & CopyPathRow)
iRow = 3

'Loop de loop
For i = StartAt To EndAt
    Application.ScreenUpdating = False
    Set wbk = Workbooks.Open(CopyPath)
    Sheets("TIMESHEET").Select
    Range("C12:S34").Select
    Selection.Copy

    ThisWorkbook.Activate
    Sheets("Pastebin").Select
    Range("a" & iRow).PasteSpecial Paste:=xlPasteValues
    Application.CutCopyMode = False

    iRow = iRow + 39
    CopyPathRow = CopyPathRow + 1
    wbk.Close True
Next i

Sheets("Pastebin").Select
Range("A:A").SpecialCells(xlCellTypeBlanks).EntireRow.Delete
Application.ScreenUpdating = True
MsgBox "Timesheet Data Imported"
End Sub

エラーの原因に基づいて、つまり同じファイルが使用されているため、問題は次の部分にあると思います。

CopyPath = ThisWorkbook.Worksheets("LeaveReport").Range("C" & CopyPathRow)

これにより、For ループ内で更新されることが「想定されています」:

コピーパス行 = コピーパス行 + 1

役に立ちましたか?

解決

ラインを移動する

CopyPath = ThisWorkbook.Worksheets("LeaveReport").Range("C" & CopyPathRow)

ループ内では、その値は CopyPath は決して変更されませんが、の値は CopyPathRow は。

編集:これも再帰とは呼びません。

ライセンス: CC-BY-SA帰属
所属していません StackOverflow
scroll top