VBA to copy number of rows to specific sheets based on a few cell values

stupurd

New Member
Joined
Jan 18, 2016
Messages
7
Hello.

I've search this site to try and find the answer but I what I require is a bit more specific than simply copying a list based on a quantity. I'll explain more:

I have a sheet and in this sheet lists tasks in column A, whether they are required or not in column B, The quantity in column C and finally a location (sheet) these tasks need to be copied to in column D. Hopefully, this is a bit clearer:

[TABLE="width: 500"]
<tbody>[TR]
[TD]Task[/TD]
[TD]Required[/TD]
[TD]Qty[/TD]
[TD]Location[/TD]
[/TR]
[TR]
[TD]Task 1[/TD]
[TD]Y[/TD]
[TD]4[/TD]
[TD]Project Lead[/TD]
[/TR]
[TR]
[TD]Task 2[/TD]
[TD]Y[/TD]
[TD]4
[/TD]
[TD]Project Lead[/TD]
[/TR]
[TR]
[TD]Task 3[/TD]
[TD]N[/TD]
[TD]3[/TD]
[TD]Mechanical[/TD]
[/TR]
[TR]
[TD]Task 4[/TD]
[TD]Y[/TD]
[TD]2[/TD]
[TD]Project Lead[/TD]
[/TR]
[TR]
[TD]Task 5[/TD]
[TD]Y[/TD]
[TD]2[/TD]
[TD]Project Lead[/TD]
[/TR]
[TR]
[TD]Task 6[/TD]
[TD]Y[/TD]
[TD]1[/TD]
[TD]Project Lead[/TD]
[/TR]
[TR]
[TD]Task 7[/TD]
[TD]Y[/TD]
[TD]1[/TD]
[TD]Project Lead[/TD]
[/TR]
</tbody>[/TABLE]

I'd like the task (Cell A2), if it is required (Cell B2), to be copied the numbers of times in the Qty column (B3) into the sheet shown in the Location column (Cell B4)

The cell within each sheet that this data is to be copied into will start at Cell D3.

I hope that makes sense.

Regards

Stuart
****** id="cke_pastebin" style="position: absolute; top: 166px; width: 1px; height: 1px; overflow: hidden; left: -1000px;">[TABLE="width: 500"]
<tbody>[TR]
[TD]Project Lead[/TD]
[/TR]
</tbody>[/TABLE]
</body>
 

Excel Facts

What do {} around a formula in the formula bar mean?
{Formula} means the formula was entered using Ctrl+Shift+Enter signifying an old-style array formula.
****CORRECTION: (I'm unable to edit posts yet)****

I'd like the task (Cell A2), if it is required (Cell B2), to be copied the numbers of times in the Qty column (C2) into the sheet shown in the Location column (Cell D2)

The cell within each sheet that this data is to be copied into will start at Cell D3.

Regards

Stuart

Edit
 
Upvote 0
So you only want the values in column A copied to other sheet. Not the entire row.
Is that correct?
 
Upvote 0
Try this:
Run script from Master sheet

Code:
Sub Test()
'Modified 5/1/2018 8:10 AM  EDT
Application.ScreenUpdating = False
Dim i As Long
Dim Lastrow As Long
Dim Lastrowa As Long
Lastrow = Cells(Rows.Count, "A").End(xlUp).Row
    For i = 2 To Lastrow
        If Cells(i, "B").Value = "Y" Then
            Lastrowa = Sheets(Cells(i, "D").Value).Cells(Rows.Count, "D").End(xlUp).Row + 1
            If Lastrowa < 5 Then Lastrowa = 5
            Cells(i, "A").Copy Sheets(Cells(i, "D").Value).Cells(Lastrowa, "D").Resize(Cells(i, 3).Value)
        End If
    Next
Application.ScreenUpdating = True
End Sub
 
Upvote 0
Alternate suggestion, try:
Code:
Sub MoveData()

    Dim arr()   As Variant
    Dim x       As Long
    Dim LR      As Long
    
    Application.ScreenUpdating = False
    
    With Sheets("Master")
         LR = .Cells(.Rows.Count, 1).End(xlUp).Row - 1
        arr = .Cells(2, 1).Resize(LR, 4).Value
    End With
    
    For x = LBound(arr, 1) To UBound(arr, 1)
        If UCase$(arr(x, 2)) = "Y" Then
            LR = Application.Max(5, Sheets(CStr(arr(x, 4))).Cells(Rows.Count, 4).End(xlUp).Row + 1)
            Sheets(CStr(arr(x, 4))).Cells(LR, 4).Resize(arr(x, 3)).Value = arr(x, 1)
        End If
    Next x
    Erase arr
    
    Application.ScreenUpdating = True
    
End Sub
 
Last edited:
Upvote 0
Try this:
Run script from Master sheet

Code:
Sub Test()
'Modified 5/1/2018 8:10 AM  EDT
Application.ScreenUpdating = False
Dim i As Long
Dim Lastrow As Long
Dim Lastrowa As Long
Lastrow = Cells(Rows.Count, "A").End(xlUp).Row
    For i = 2 To Lastrow
        If Cells(i, "B").Value = "Y" Then
            Lastrowa = Sheets(Cells(i, "D").Value).Cells(Rows.Count, "D").End(xlUp).Row + 1
            If Lastrowa < 5 Then Lastrowa = 5
            Cells(i, "A").Copy Sheets(Cells(i, "D").Value).Cells(Lastrowa, "D").Resize(Cells(i, 3).Value)
        End If
    Next
Application.ScreenUpdating = True
End Sub

Thank you very much. This worked perfectly on my test sheet. I had to change the column letters and it worked great on the real spreadsheet (The format of which is a little convoluted, a simple table was created to ask for help). I didn't know that .Resize could be used to paste the number of rows - you learn something new everyday.

Thanks again

Regards

Stuart
 
Upvote 0
Alternate suggestion, try:
Code:
Sub MoveData()

    Dim arr()   As Variant
    Dim x       As Long
    Dim LR      As Long
    
    Application.ScreenUpdating = False
    
    With Sheets("Master")
         LR = .Cells(.Rows.Count, 1).End(xlUp).Row - 1
        arr = .Cells(2, 1).Resize(LR, 4).Value
    End With
    
    For x = LBound(arr, 1) To UBound(arr, 1)
        If UCase$(arr(x, 2)) = "Y" Then
            LR = Application.Max(5, Sheets(CStr(arr(x, 4))).Cells(Rows.Count, 4).End(xlUp).Row + 1)
            Sheets(CStr(arr(x, 4))).Cells(LR, 4).Resize(arr(x, 3)).Value = arr(x, 1)
        End If
    Next x
    Erase arr
    
    Application.ScreenUpdating = True
    
End Sub

Thank you!
 
Upvote 0

Forum statistics

Threads
1,224,823
Messages
6,181,177
Members
453,021
Latest member
Justyna P

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