Copying rows from multiple worksheets onto one worksheet based on the contents in one cell

kinnoyu

New Member
Joined
Apr 5, 2021
Messages
2
Office Version
  1. 365
Platform
  1. Windows
I've been having a bit of trouble trying to get some code to work. I have a workbook with multiple sheets. Each sheets has a bunch of rows of data, and I want to organize the data onto specific sheets based on the contents in one cell. So any of the rows that have the word "BASE" in the A column from all worksheets I want copied onto a sheet named BASE.

Here's what I have so far

VBA Code:
Sub NewSheetData()

With Application
    .ScreenUpdating = False
    .EnableEvents = False
End With

Dim ws As Worksheet
Dim Rng As Range

For Each ws In ThisWorkbook.Worksheets

Set Rng = Range([A1], Range("A" & Rows.Count).End(xlUp))

On Error Resume Next
    With Rng
        .AutoFilter , field:=1, Criteria1:="BASE"
        .SpecialCells(xlCellTypeVisible).EntireRow.Copy Destination:=Sheets("BASE").Range("A1").Offset(n, 0)
        n = n + 1
        
        .AutoFilter
    End With
On Error GoTo 0

Next ws

Application.EnableEvents = True


End Sub


It runs fine when I remove the "For Each, Next" code which I added to try and make it run through all the worksheets. Then it won't do anything at all; it won't give me an error, it just won't do anything.
 

Excel Facts

Excel Wisdom
Using a mouse in Excel is the work equivalent of wearing a lanyard when you first get to college

jolivanes

Well-known Member
Joined
Sep 5, 2004
Messages
1,824
Office Version
  1. 2013
  2. 2007
Platform
  1. Windows
Try
Code:
Sub Maybe()
Dim i As Long
Application.ScreenUpdating = False
    For i = 1 To ThisWorkbook.Worksheets.Count
        If Sheets(i).Name <> "BASE" Then
            If WorksheetFunction.CountIf(Sheets(i).Columns(1), "BASE") <> 0 Then
                With Sheets(i).UsedRange
                    .AutoFilter 1, "BASE"
                        .Offset(1).Copy Sheets("BASE").Cells(Rows.Count, 1).End(xlUp).Offset(1)
                    .AutoFilter
                End With
            End If
        End If
    Next i
Application.ScreenUpdating = True
End Sub
 

Fluff

MrExcel MVP, Moderator
Joined
Jun 12, 2014
Messages
62,606
Office Version
  1. 365
Platform
  1. Windows
Another option
VBA Code:
Sub NewSheetData()

With Application
    .ScreenUpdating = False
    .EnableEvents = False
End With

Dim ws As Worksheet
Dim Rng As Range

For Each ws In ThisWorkbook.Worksheets
   If UCase(ws.Name) <> "BASE" Then
      Set Rng = ws.Range([A1], ws.Range("A" & Rows.Count).End(xlUp))
      
      On Error Resume Next
          With Rng
              .AutoFilter , field:=1, Criteria1:="BASE"
              .Parent.AutoFilter.Range.Offset(1).EntireRow.Copy Sheets("BASE").Range("A" & Rows.Count).End(xlUp).Offset(1)
              .AutoFilter
          End With
      On Error GoTo 0
   End If
Next ws

Application.EnableEvents = True


End Sub
 

Forum statistics

Threads
1,141,777
Messages
5,708,465
Members
421,571
Latest member
ChaosPup

We've detected that you are using an adblocker.

We have a great community of people providing Excel help here, but the hosting costs are enormous. You can help keep this site running by allowing ads on MrExcel.com.
Allow Ads at MrExcel

Which adblocker are you using?

Disable AdBlock

Follow these easy steps to disable AdBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the icon in the browser’s toolbar.
2)Click on the "Pause on this site" option.
Go back

Disable AdBlock Plus

Follow these easy steps to disable AdBlock Plus

1)Click on the icon in the browser’s toolbar.
2)Click on the toggle to disable it for "mrexcel.com".
Go back

Disable uBlock Origin

Follow these easy steps to disable uBlock Origin

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back

Disable uBlock

Follow these easy steps to disable uBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back
Top