Want to protect your cyber security and still get fast solutions? Ask a secure question today.Go Premium

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 633
  • Last Modified:

vba autofill range selection and fill

All,

I keep getting errors when trying to execute the below code. Can anyone see my problem?

With shtTo

Dim from As Range
        Dim too As Range
        
        'specifying addresses
        copXrow = xRows - 1
        copxRows = shtTo.Cells(1).CurrentRegion.Columns(1).SpecialCells(xlCellTypeVisible).Count
        
        'setting from range
        Set from = Range("M" & copXrow & ":" & "EA" & copXrow)
        
        'setting to range
        Set too = Range("M" & lastrow & ":" & "EA" & lastrow)
        
        'autofill from too
        Range("from").AutoFill Destination:=shtTo.Range("too")

End With

Open in new window

0
discogs
Asked:
discogs
  • 5
  • 4
  • 2
1 Solution
 
Michael FowlerSolutions ConsultantCommented:
What error are you getting?

Michael
0
 
discogsAuthor Commented:
On line 17. The error "Method 'Range' of object'_Worksheet failed

Does that help?

It seems so stupid to me...
0
 
discogsAuthor Commented:
I even changed line 17 to read:

Range("from").AutoFill Destination:=Range("too")

Open in new window

An I got Method Range of Object Global failed error as well...
0
Concerto's Cloud Advisory Services

Want to avoid the missteps to gaining all the benefits of the cloud? Learn more about the different assessment options from our Cloud Advisory team.

 
Michael FowlerSolutions ConsultantCommented:
OK I see the error. when you declare "from" it is a range object not a named range


Change the line to

from.AutoFill Destination:=shtTo.too

Open in new window

0
 
Saqib Husain, SyedEngineerCommented:
Change line 17 to

        from.AutoFill Destination:=Range(from, too)

to get rid of the error but I do not think this is what you would want. Can you explain what you want to achieve?
0
 
discogsAuthor Commented:
Okay.

I have not changed anything because you might be able to tell me how to do it better.

Essentially, I copy a large dynamic dataset into a spreadsheet within one workbook.

In doing so, I know the row number to paste to, I also know the number of records I am pasting. For example, rngTo = A150 and record count = 78.

After pasting the records, I then select all records including and out to the right of column(M). After this, I autofill the row I selected (in the above example it would be M149 to EA149) and copy the whole lot down to the bottom of my records as displayed in columns(A through L).

Is there a better way?
0
 
Saqib Husain, SyedEngineerCommented:
Can you make my life easier by uploading a few-row fake sample?...unless Michael74 or anyone else has understood it well and can set up his own example.
0
 
discogsAuthor Commented:
Done.

Please note that I have highlighted and inserted a formula in row 150 as an example.

All formulas from column(M) through to column(EA) require copying all the way to the bottom of my data located in columns(A ~ L) respectively.

Although column M is a static starting point, the end column (EA) is not. So it would be really beneficial if the solution could be dynamic enough to compensate for growth.

TA
Book1.xlsx
0
 
Saqib Husain, SyedEngineerCommented:
This would find the row 150 using the formula
Set strtSrc = Range("M1").End(xlDown)

Open in new window

and the last column using the formula
Set endSrc = Range("M1").End(xlDown).End(xlToRight)

Open in new window

If this is different from what you want then write back.

Sub copym2()
Dim strtSrc As Range
Dim endSrc As Range
Dim strtTgt As Range
Dim endTgt As Range
Set strtSrc = Range("M1").End(xlDown)
Set endSrc = Range("M1").End(xlDown).End(xlToRight)
Set strtTgt = Range("M1").End(xlDown).Offset(1)
Set endTgt = Range("A1").End(xlDown).Offset(, 12)
Range(strtSrc, endSrc).Copy Range(strtTgt, endTgt)
End Sub
0
 
Saqib Husain, SyedEngineerCommented:
Taking advantage of the used variables this can be reduced to

Sub a()
Dim strtSrc As Range
Dim endSrc As Range
Dim strtTgt As Range
Dim endTgt As Range
Set strtSrc = Range("M1").End(xlDown)
Set endSrc = strtSrc.End(xlToRight)
Set strtTgt = strtSrc.Offset(1)
Set endTgt = Range("A1").End(xlDown).Offset(, 12)
Range(strtSrc, endSrc).Copy Range(strtTgt, endTgt)
End Sub
0
 
discogsAuthor Commented:
Outstanding assistance. Thanks heaps it does exactly what I expected. TA
0

Featured Post

Free Tool: Site Down Detector

Helpful to verify reports of your own downtime, or to double check a downed website you are trying to access.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

  • 5
  • 4
  • 2
Tackle projects and never again get stuck behind a technical roadblock.
Join Now