Copying selective range of column data from mutiple sheets to one sheet

vignesh_thegame

New Member
Joined
Sep 30, 2013
Messages
48
Hi All,

I have a workbook with 4 sheets. Each sheet has data in 10 columns. all sheets column headers are same.

I want to consolidate selective column (A: C, F:H) data from all four sheets into a new sheet under the same column header.

I have a code, but it is copying the entire columns from different sheets and consolidating in new sheet with out header.

Can any one help me to get this done?

Sub CopyDataWithoutHeaders()
Dim sh As Worksheet
Dim DestSh As Worksheet
Dim Last As Long
Dim shLast As Long
Dim CopyRng As Range
Dim StartRow As Long

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

'Delete the sheet "RDBMergeSheet" if it exist
Application.DisplayAlerts = False
On Error Resume Next
ActiveWorkbook.Worksheets("RDBMergeSheet").Delete
On Error GoTo 0
Application.DisplayAlerts = True

'Add a worksheet with the name "RDBMergeSheet"
Set DestSh = ActiveWorkbook.Worksheets.Add
DestSh.Name = "RDBMergeSheet"

'Fill in the start row
StartRow = 2

'loop through all worksheets and copy the data to the DestSh
For Each sh In ActiveWorkbook.Worksheets

'Loop through all worksheets except the RDBMerge worksheet and the
'Information worksheet, you can ad more sheets to the array if you want.
If IsError(Application.Match(sh.Name, _
Array(DestSh.Name, "Information"), 0)) Then

'Find the last row with data on the DestSh and sh
Last = LastRow(DestSh)
shLast = LastRow(sh)

'If sh is not empty and if the last row >= StartRow copy the CopyRng
If shLast > 0 And shLast >= StartRow Then

'Set the range that you want to copy
'Need to check here-- Vignesh
'Set CopyRng = sh.Range("A3:G3, AH3:AJ3, AL3")
Set CopyRng = sh.Range(sh.Rows(StartRow), sh.Rows(shLast))

'Test if there enough rows in the DestSh to copy all the data
If Last + CopyRng.Rows.Count > DestSh.Rows.Count Then
MsgBox "There are not enough rows in the Destsh"
GoTo ExitTheSub
End If

'This example copies values/formats, if you only want to copy the
'values or want to copy everything look below example 1 on this page
CopyRng.Copy
With DestSh.Cells(1 + Last, "A")
.PasteSpecial xlPasteValues
.PasteSpecial xlPasteFormats
Application.CutCopyMode = False
End With

End If

End If
Next

ExitTheSub:

Application.GoTo DestSh.Cells(1)

'AutoFit the column width in the DestSh sheet
DestSh.Columns.AutoFit

With Application
.ScreenUpdating = True
.EnableEvents = True
End With
End Sub
 

Excel Facts

Workdays for a market open Mon, Wed, Friday?
Yes! Use "0101011" for the weekend argument in NETWORKDAYS.INTL or WORKDAY.INTL. The 7 digits start on Monday. 1 means it is a weekend.
If I have understood correctly, here is one way.

Code:
Sub CopyData()
  Dim sh As Worksheet, DestSh As Worksheet
  Dim Last As Long
  
  Application.ScreenUpdating = False
  Application.DisplayAlerts = False
  On Error Resume Next
  ActiveWorkbook.Worksheets("RDBMergeSheet").Delete
  On Error GoTo 0
  Application.DisplayAlerts = True
  Sheets.Add(After:=Sheets(Sheets.Count)).Name = "RDBMergeSheet"
  Set DestSh = Sheets(Sheets.Count)
  For Each sh In Worksheets
    Select Case sh.Name
      Case "Information", "RDBMergeSheet"
      Case Else
        If IsEmpty(DestSh.Range("A1").Value) Then sh.Range("A1:H1").Copy Destination:=DestSh.Range("A1")
        Last = DestSh.UsedRange.Rows.Count
        With sh.UsedRange
          .Offset(1).Resize(.Rows.Count - 1, 8).Copy Destination:=DestSh.Range("A" & Last + 1)
        End With
    End Select
  Next sh
  DestSh.Columns("D:E").Delete
  DestSh.UsedRange.Columns.AutoFit
  Application.ScreenUpdating = True
End Sub
 
Upvote 0
Hi Peter,

Your code works perfectly till consolidating upto three sheets, after 4th sheet we could see a blank blank rows inserted before pasting the sheet 4 data...

[TABLE="width: 561"]
<colgroup><col><col span="2"><col><col></colgroup><tbody>[TR]
[TD]Header 1[/TD]
[TD]Header 2[/TD]
[TD]Header 3[/TD]
[TD]Header 6[/TD]
[TD]Header 7[/TD]
[/TR]
[TR]
[TD]Cell A2 of Sheet1[/TD]
[TD]Cell B2 of Sheet1[/TD]
[TD]Cell C2 of Sheet1[/TD]
[TD]Cell F2 of Sheet1[/TD]
[TD]Cell G2 of Sheet1[/TD]
[/TR]
[TR]
[TD]Cell A3 of Sheet1[/TD]
[TD]Cell B3 of Sheet1[/TD]
[TD]Cell C3 of Sheet1[/TD]
[TD]Cell F3 of Sheet1[/TD]
[TD]Cell G3 of Sheet1[/TD]
[/TR]
[TR]
[TD]Cell A4 of Sheet1[/TD]
[TD]Cell B4 of Sheet1[/TD]
[TD]Cell C4 of Sheet1[/TD]
[TD]Cell F4 of Sheet1[/TD]
[TD]Cell G4 of Sheet1[/TD]
[/TR]
[TR]
[TD]Cell A2 of Sheet2[/TD]
[TD]Cell B2 of Sheet2[/TD]
[TD]Cell C2 of Sheet2[/TD]
[TD]Cell F2 of Sheet2[/TD]
[TD]Cell G2 of Sheet2[/TD]
[/TR]
[TR]
[TD]Cell A3 of Sheet2[/TD]
[TD]Cell B3 of Sheet2[/TD]
[TD]Cell C3 of Sheet2[/TD]
[TD]Cell F3 of Sheet2[/TD]
[TD]Cell G3 of Sheet2[/TD]
[/TR]
[TR]
[TD]Cell A4 of Sheet2[/TD]
[TD]Cell B4 of Sheet2[/TD]
[TD]Cell C4 of Sheet2[/TD]
[TD]Cell F4 of Sheet2[/TD]
[TD]Cell G4 of Sheet2[/TD]
[/TR]
[TR]
[TD]Cell A2 of Sheet3[/TD]
[TD]Cell B2 of Sheet3[/TD]
[TD]Cell C2 of Sheet3[/TD]
[TD]Cell F2 of Sheet3[/TD]
[TD]Cell G2 of Sheet3[/TD]
[/TR]
[TR]
[TD]Cell A3 of Sheet3[/TD]
[TD]Cell B3 of Sheet3[/TD]
[TD]Cell C3 of Sheet3[/TD]
[TD]Cell F3 of Sheet3[/TD]
[TD]Cell G3 of Sheet3[/TD]
[/TR]
[TR]
[TD]Cell A4 of Sheet3[/TD]
[TD]Cell B4 of Sheet3[/TD]
[TD]Cell C4 of Sheet3[/TD]
[TD]Cell F4 of Sheet3[/TD]
[TD]Cell G4 of Sheet3[/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD]Cell A2 of Sheet4[/TD]
[TD]Cell B2 of Sheet4[/TD]
[TD]Cell C2 of Sheet4[/TD]
[TD]Cell F2 of Sheet4[/TD]
[TD]Cell G2 of Sheet4[/TD]
[/TR]
[TR]
[TD]Cell A3 of Sheet4[/TD]
[TD]Cell B3 of Sheet4[/TD]
[TD]Cell C3 of Sheet4[/TD]
[TD]Cell F3 of Sheet4[/TD]
[TD]Cell G3 of Sheet4[/TD]
[/TR]
[TR]
[TD]Cell A4 of Sheet4[/TD]
[TD]Cell B4 of Sheet4[/TD]
[TD]Cell C4 of Sheet4[/TD]
[TD]Cell F4 of Sheet4[/TD]
[TD]Cell G4 of Sheet4[/TD]
[/TR]
</tbody>[/TABLE]
 
Upvote 0
I would say that Sheet3 probably has some cells at the bottom that appear empty but contain formulas returning a null string ("") or perhaps formatting that causes them to be included in the sheet's UsedRange.
Try this vrersion to see if it fixes the problem.
Code:
Sub CopyData_v2()
  Dim sh As Worksheet, DestSh As Worksheet
  Dim Last As Long, shLast As Long
  
  Application.ScreenUpdating = False
  Application.DisplayAlerts = False
  On Error Resume Next
  ActiveWorkbook.Worksheets("RDBMergeSheet").Delete
  On Error GoTo 0
  Application.DisplayAlerts = True
  Sheets.Add(After:=Sheets(Sheets.Count)).Name = "RDBMergeSheet"
  Set DestSh = Sheets(Sheets.Count)
  For Each sh In Worksheets
    Select Case sh.Name
      Case "Information", "RDBMergeSheet"
      Case Else
        If IsEmpty(DestSh.Range("A1").Value) Then sh.Range("A1:H1").Copy Destination:=DestSh.Range("A1")
        Last = DestSh.UsedRange.Rows.Count
        With sh
          shLast = .Cells.Find(What:="*", LookIn:=xlValues, SearchOrder:=xlByRows, SearchDirection:=xlPrevious, SearchFormat:=False).Row
          .UsedRange.Offset(1).Resize(shLast - 1, 8).Copy Destination:=DestSh.Range("A" & Last + 1)
        End With
    End Select
  Next sh
  DestSh.Columns("D:E").Delete
  DestSh.UsedRange.Columns.AutoFit
  Application.ScreenUpdating = True
End Sub
 
Upvote 0
Great!!!!

Thanks and its works perfectly.. i am impressed by your work:)

Just for my learning purpose, Please clarify the below query,
In case, if the header is occupied on 2 columns (i.e column A1 and A2 is header) and i want to copy the the data from A3 range with A1 and A2 as header, what needs to be change in my coding...
[TABLE="width: 225"]
<colgroup><col><col></colgroup><tbody>[TR]
[TD]A[/TD]
[TD]B[/TD]
[/TR]
[TR]
[TD]Header[/TD]
[TD]Header[/TD]
[/TR]
[TR]
[TD]Sub Header[/TD]
[TD]Sub Header[/TD]
[/TR]
[TR]
[TD]Data 1[/TD]
[TD]Data 1[/TD]
[/TR]
[TR]
[TD]Data 2[/TD]
[TD]Data 2[/TD]
[/TR]
</tbody>[/TABLE]
 
Upvote 0
In case, if the header is occupied on 2 columns ..
I think you mean rows?

Here is one way that allows any number of header rows, so long as each sheet is the same.

1. Add this just below the 'Dim' statements to record how many header rows you have in each sheet.
Rich (BB code):
Const HdrRws As Long = 2
Then change these two lines
Rich (BB code):
If IsEmpty(DestSh.Range("A1").Value) Then sh.Range("A1:H" & HdrRws).Copy Destination:=DestSh.Range("A1")

.UsedRange.Offset(HdrRws).Resize(shLast - HdrRws, 8).Copy Destination:=DestSh.Range("A" & Last + 1)
 
Upvote 0
Thank you so much. its working fine....

i would like to learn vba, is there any best website or book to understand the vba structure?
 
Upvote 0

Forum statistics

Threads
1,223,894
Messages
6,175,250
Members
452,623
Latest member
Techenthusiast

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
Back
Top