[VBA] [Data Consolidation][Speed up]

[zeromax1]

Hi Professional, I would like to ask the help for the VBA code speed up & improvement.

I have to consolidate the sub-workbooks to my master workbook. Each workbooks have the same tab name to contain the data.

Eg: I have to copy the data in column F from "Monday" Tab in sub workbooks to my master workbook in the column B of "Monday" Tab and subtract the number.
I have to copy the data in column C from "Monday" Tab in sub workbooks to my master workbook in the column F of "Monday" Tab.


I found a VBA code to consolidate the data, but the speed seems too slow and the code is not perfect.

For the convenient, I skip to paste the similar code of "Tuesday","Wednesday", etc tab.

Sub Consolidatfiles()

On Error Resume Next

    Application.ScreenUpdating = False

    Dim FileName As Variant, wkbSource As Workbook, wkbDest As Workbook
    Set wkbDest = ThisWorkbook
    
    FileName = Application.GetOpenFilename("Excel Files (*.xlsx*),*.xlsx*", , "Select Excel Files", , True)
    
    If Not IsArray(FileName) Then
        MsgBox "No File Selected"
        Exit Sub
    End If
    
    For i = LBound(FileName) To UBound(FileName)
        Set wkbSource = Workbooks.Open(FileName(i))
        With wkbSource
            .Worksheets("Monday").Range("C5:C" & Range("C" & Rows.Count).End(xlUp).Row).Copy _
            Destination:=wkbDest.Worksheets("Monday").range("F7").End(xlUp)

            .Worksheets("Monday").Range("F5:F" & Range("F" & Rows.Count).End(xlUp).Row-1).Copy _
            Destination:=wkbDest.Worksheets("Monday").range("F7").End(xlUp).PasteSpecial Paste:=xlPasteAll, Operation:=xlSubtract 

            wkbSource.Close savechanges = False
        End With
    Next i
    
    Application.CutCopyMode = False
    Application.ScreenUpdating = True
  
    
End Sub

Thank you very much.

 

[zeromax1]

I'm glad to help you. Thanks for the feedback.

Hi DanteAmor, I would like to modify the code to copy a range data from A2 to Z & lastrow of a another workbook to the existing workbook. Would you give a help for this question?

Just copy the data to the existing workbook with no change. The data include number, date, text etc.

Sub Consolidatfiles()
  Dim wb1 As Workbook, sh1 As Worksheet
  Dim FileName As Variant, sheetName As Variant, arr As Variant
  Dim i As Long, j As Long, lr1 As Long, lr2 As Long
  Dim a() As Variant, b() As Variant
  
  Application.ScreenUpdating = False
  
  FileName = Application.GetOpenFilename("Excel Files (*.xlsx*),*.xlsx*", , "Select Excel Files", , True)
  
  If Not IsArray(FileName) Then
    MsgBox "No File Selected"
    Exit Sub
  End If
  
  arr = Array("Matched")
  
  For i = LBound(FileName) To UBound(FileName)
    Set wb1 = Workbooks.Open(FileName(i))
    
    For Each sheetName In arr
      Set sh1 = wb1.Sheets(sheetName)
      Erase a
      lr1 = sh1.Range("A2:Z" & Rows.Count).End(3).Row

      
      With ThisWorkbook.Sheets(sheetName)
        lr2 = .Range("A" & Rows.Count).End(3).Row + 1
        .Range("A" & lr2).Resize(UBound(a)).Value = sh1.Range("A2").Resize(UBound(a)).Value
      End With
    Next sheetName
    wb1.Close False
  Next i
  
  Application.ScreenUpdating = True
End Sub

 

[zeromax1]

Hi DanteAmor, I would like to modify the code to copy a range data from A2 to Z & lastrow of a another workbook to the existing workbook. Would you give a help for this question?

Just copy the data to the existing workbook with no change. The data include number, date, text etc.

I updated the code by below:

If Not IsArray(FileName) Then
        MsgBox "No File Selected"
        Exit Sub
    End If
     
    For i = LBound(FileName) To UBound(FileName)
        Set wkbSource = Workbooks.Open(FileName(i))

        With ThisWorkbook

            .Worksheets(1).Range("A4:Z" & Range("C" & Rows.Count).End(xlUp).Row).Value = _
            wkbSource.Worksheets(1).Range("A2:Z" & Range("C" & Rows.Count).End(xlUp).Row).Value
            
            wkbSource.Close savechanges = False
            
        End With
    Next i