Code to copy columns not working

Hi Excel Forum gurus,

I'm having a second go, from a different angle, at a problem I posted on here about at the end of last week - I got it to do what I wanted, but the macro was taking well over 5 minutes to run, a time for the task that I couldn't really handle.

Anyway, I've come up with a different, much more efficient method, but the code I've come up with doesn't seem to be working. If someone can spot the obvious error before I start uploading more workbooks, that would be fantastic!

Thanks again.

Please Login or Register  to view this content.

Hi,

Change this

Please Login or Register  to view this content.

to

Please Login or Register  to view this content.

What you where doing was passing the range an integer when you should pass it the value from the columns array

Ok, I managed to sort one element of the problem, but have run headlong into another. The uploader is saying my file is too large to upload I'm afraid.
The current code is

Please Login or Register  to view this content.

You might like to try telling people what is actually going wrong rather than leaving them to guess...

I was trying to upload an excel file to illustrate it better than I could describe, but the forum attachment uploader isn't letting me. :@ Basically, It just pastes row upon row of the headers of the column, right down to the row limit of the spreadsheet.