Я пытаюсь скопировать данные из целой кучи разных книг в один мастер-лист, вставив только значения в следующий пустой столбец. Все это кажется функциональным, но всегда терпит неудачу при попытке вставить в мастер-лист. Я пробовал искать подобные проблемы в других местах, но я не могу заставить их работать с тем, что я пытаюсь сделать.
Я взял большую часть этого кода где-то еще и изменил его, как вы можете понять из некоторых оставшихся комментариев.
Sub LoopAllExcelFilesInFolder()
'PURPOSE: To loop through all Excel files in a user specified folder and perform a set task on them
'SOURCE: www.TheSpreadsheetGuru.com
Dim wb As Workbook
Dim myPath As String
Dim myFile As String
Dim myExtension As String
Dim FldrPicker As FileDialog
Dim colDest As Long
Dim Dest As Worksheet
'Optimize Macro Speed
Application.ScreenUpdating = False
Application.EnableEvents = False
Application.Calculation = xlCalculationManual
'Retrieve Target Folder Path From User
Set FldrPicker = Application.FileDialog(msoFileDialogFolderPicker)
With FldrPicker
.Title = "Select A Target Folder"
.AllowMultiSelect = False
If .Show <> -1 Then GoTo NextCode
myPath = .SelectedItems(1) & "\"
End With
'In Case of Cancel
NextCode:
myPath = myPath
If myPath = "" Then GoTo ResetSettings
'Target File Extension (must include wildcard "*")
myExtension = "*.xls*"
'Target Path with Ending Extention
myFile = Dir(myPath & myExtension)
'Loop through each Excel file in folder
Do While myFile <> ""
'Set variable equal to opened workbook
Set wb = Workbooks.Open(fileName:=myPath & myFile)
'Ensure Workbook has opened before moving on to next line of code
DoEvents
'Change First Worksheet's Background Fill Blue this is where the work occurs
Set Dest = Workbooks("Master.xlsm").Worksheets(1)
colDest = Dest.Cells(1, Dest.Columns.Count).End(xlToRight).Column
wb.Worksheets(1).Range("b3:u83").Copy
Dest.Range(1, colDest).PasteSpecial Paste:=xlPasteValues
'Save and Close Workbook
wb.Close SaveChanges:=True
'Ensure Workbook has closed before moving on to next line of code
DoEvents
'Get next file name
myFile = Dir
Loop
'Message Box when tasks are completed
MsgBox "Task Complete!"
ResetSettings:
'Reset Macro Optimization Settings
Application.EnableEvents = True
Application.Calculation = xlCalculationAutomatic
Application.ScreenUpdating = True
End Sub
Обновлено: в этой строке возникает ошибка:
Dest.Range(1, colDest).PasteSpecial Paste:=xlPasteValues
Ошибка выполнения «1004»: Не удалось выполнить метод «Диапазон» объекта «_Worksheet».
EDIT2: изменение попытки вставки с попыткой записи значения в ячейку, т.е.:
Dest.Cells(1, colDest) = "Test"
Правильно вводит «Тест» в следующий доступный столбец на мастер-листе для каждой книги, которая была открыта из каталога. Очевидно, изменение «Диапазона» на «Ячейки» работает, что, как мне казалось, я пробовал вчера, но выдавал другую ошибку, жалуясь, что я не выбирал ячейку правильного размера.
я разместил это в спешке и не смог вытащить саму ошибку в то время, но она вылетает при попытке вставить, а не при попытке найти последний столбец. Я обновлю основной вопрос с ошибкой.
Попробуйте это, в основном вам нужно добавить 1 к colDest
, чтобы получить следующий пустой столбец.
Sub LoopAllExcelFilesInFolder()
'PURPOSE: To loop through all Excel files in a user specified folder and perform a set task on them
Dim wb As Workbook
Dim myPath As String
Dim myFile As String
Dim myExtension As String
Dim FldrPicker As FileDialog
Dim colDest As Long
Dim Dest As Worksheet
'Optimize Macro Speed
Application.ScreenUpdating = False
Application.EnableEvents = False
Application.Calculation = xlCalculationManual
'Retrieve Target Folder Path From User
Set FldrPicker = Application.FileDialog(msoFileDialogFolderPicker)
With FldrPicker
.Title = "Select A Target Folder"
.AllowMultiSelect = False
If .Show <> -1 Then GoTo NextCode
myPath = .SelectedItems(1) & "\"
End With
'In Case of Cancel
NextCode:
myPath = myPath
If myPath = "" Then GoTo ResetSettings
'Target File Extension (must include wildcard "*")
myExtension = "*.xls*"
'Target Path with Ending Extention
myFile = Dir(myPath & myExtension)
'Loop through each Excel file in folder
Do While myFile <> ""
'Set variable equal to opened workbook
Set wb = Workbooks.Open(fileName:=myPath & myFile)
'Ensure Workbook has opened before moving on to next line of code
DoEvents
'Change First Worksheet's Background Fill Blue this is where the work occurs
Set Dest = Workbooks("Master.xlsm").Worksheets(1)
colDest = Dest.Cells(1, Dest.Columns.Count).End(xlToLeft).Column + 1
wb.Worksheets(1).Range("b3:u83").Copy
Dest.Range(1, colDest).PasteSpecial Paste:=xlPasteValues
'Save and Close Workbook
wb.Close SaveChanges:=True
'Ensure Workbook has closed before moving on to next line of code
DoEvents
'Get next file name
myFile = Dir
Loop
'Message Box when tasks are completed
MsgBox "Task Complete!"
ResetSettings:
'Reset Macro Optimization Settings
Application.EnableEvents = True
Application.Calculation = xlCalculationAutomatic
Application.ScreenUpdating = True
End Sub
Эй, ты читал мой комментарий? Это вызовет ошибку. Вы не можете идти сразу после последнего столбца на листе. это xlToLeft
.
@Damian Damian Я изменил его на xlLeft, но, глядя на него сейчас, по какой-то причине этого не произошло.
Да, поскольку код OP был таким же, вы добавили правильный +1, но также нужно было идти влево, а не вправо.
@Damian Да, я знаю, что, должно быть, сделал что-то не так, а затем отменил изменения и забыл сделать xlToLeft.
Ниже приведены некоторые рекомендации о том, как найти последний столбец и значение импорта после последнего столбца.
Option Explicit
Sub Test()
Dim LastColumn As Long
With ThisWorkbook.Worksheets("Sheet1")
'Last Column using UsedRange (NOT A GOOD IDEA)
LastColumn = .UsedRange.Columns(.UsedRange.Columns.Count).Column
'Last Column using specific row 7
LastColumn = .Cells(7, .Columns.Count).End(xlToLeft).Column
'Add a value in row 5 & after last column
.Cells(5, LastColumn + 1).Value = ""
End With
End Sub
Set Dest = Workbooks("Master.xlsm").Worksheets(1)
colDest = Dest.Cells(1, Dest.Columns.Count).End(xlToLeft).Column + 1
wb.Worksheets(1).Range("b3:u83").Copy
Dest.Cells(1, colDest).PasteSpecial Paste:=xlPasteValues
Правильно вводит данные там, где они мне нужны, «Влево» имеет значение, но «Диапазон» не позволяет мне вставлять туда, где это делает «Ячейки».
Эта строка
colDest = Dest.Cells(1, Dest.Columns.Count).End(xlToRight).Column
должна быть:colDest = Dest.Cells(1, Dest.Columns.Count).End(xlToLeft).Column + 1
. Когда вы используетеDest.Columns.Count
, он возвращает вам максимальное количество столбцов для этого листа, поэтому переход вправо невозможен, переход влево даст вам последний записанный столбец для строки 1, а добавление +1 даст вам первый пустой столбец для строки 1.