Copy and paste columns

[Nelson78]

Good morning everybody.

I've my workbook with a sheet named "One" with four columns, "Alpha", "Beta", "Gamma" and "Delta".
Another sheet is called "Two" where you can find a lot of columns among which "Alpha", "Beta", "Gamma" and "Delta" that could change their position every day.

Now, I need to copy the datas of the four columns from "Two to "One" with no considering the header (they are already set).


My attempt (at the moment I'm not able to exclude the headers)

Sub Copycolumns()

Dim header As Variant
header = Array("Alpha", "Beta", "Gamma", "Delta")
Dim i As Long 

For i = 0 To UBound(header)
Sheets("Two").Select    

On Error GoTo ErrHandler

Cells.Find(What:=Titles(i), After:=Range("A1"), _
        LookIn:=xlFormulas, LookAt:=xlPart, SearchOrder:=xlByColumns).EntireColumn.Copy

On Error GoTo 0

    Sheets("One").Select
    Range("A1").Offset(0, i).Select
    ActiveSheet.Paste
    Application.CutCopyMode = False

NextOne:
Next i
Exit Sub

ErrHandler:
Resume NextOne

End Sub

Thank you.

 

[Nelson78]

Re: Copy and past columns

Could Excel 2007 be a problem?

 

[JackDanIce]

Re: Copy and past columns

I wouldn't expect so with that code, I'm using 2010 to test on.

 

[Nelson78]

Re: Copy and past columns

With the pc of my workmate, Excel 2016, same error.

A mystery.

 

[JackDanIce]

Re: Copy and past columns

Then, there's some set up I haven't been able to replicate here. You could upload a dummy file and post the link to this thread?

 

[Fluff]

Re: Copy and past columns

@Nelson78
Do all the columns you are copying have data in them?

 

[Nelson78]

Re: Copy and past columns

With the help of JackDanIce, I've found the error.
Was really a banal one... simply the names of the sheets were inverted...

Sub CopyCols()

    Dim header()    As String
    Dim arr()       As Variant
    Dim x           As Long
    Dim y           As Long
    Dim LR          As Long
    Dim LC          As Long
    
    header = Split("Alpha,Beta,Gamma,Delta", ",")

    With Sheets("[COLOR=#ff0000][B]One[/B][/COLOR]")
        LC = .Cells(1, .Columns.count).End(xlToLeft).column
        For x = LBound(header) To UBound(header)
            y = .Cells(1, 1).Resize(, LC).find(header(x), LookIn:=xlValues).column
            LR = .Cells(.Rows.count, y).End(xlUp).row
            arr = .Cells(1, y).Resize(LR).Value
            Sheets("[B][COLOR=#ff0000]Two[/COLOR][/B]").Cells(1, x + 1).Resize(UBound(arr, 1), UBound(arr, 2)).Value = arr
            Erase arr
        Next x
    End With
        
    Erase header
    
End Sub

 

[JackDanIce]

Re: Copy and past columns

Glad it's resolved!