Macro moving cells from sheet to sheet

mericleed

New Member
Joined
Jan 3, 2024
Messages
2
Office Version
  1. 2016
Platform
  1. Windows
Hi,

I'm new to using macros in excel, so please forgive me for my lack of knowledge or if I'm doing something completely wrong.

I am trying to move cells that auto-populate from specific areas on a source sheet (CIP), to corresponding specific areas on a master log target sheet (CIP 2024). It also needs to find the next empty space in correct area on the target sheet and paste the data there.

The code I have written sort of works...except instead of moving it to column AJ row 4, in the target sheet, it moves to column A and all the way down to rows in the 900s.

Any help is greatly appreciated.



Here is my code thus far:
VBA Code:
Sub CopyDataToAnotherSheet()

    Dim sourceSheet As Worksheet
    Dim targetSheet As Worksheet
    Dim BTR2 As Range
    Dim lastRow As Long
    
    Set sourceSheet = ThisWorkbook.Sheets("CIPs")
    Set targetSheet = ThisWorkbook.Sheets("CIP 2024")
    
    Set BTR2 = sourceSheet.Range("U23:Y38")
    
    lastRow = targetSheet.Cells(targetSheet.Rows.Count, "AJ").End(xlUp).Row
    
    BTR2.Copy
    targetSheet.Cells(lastRow, 1).PasteSpecial Paste:=xlPasteValues
    Application.CutCopyMode = False
    
End Sub
 

Excel Facts

Excel Can Read to You
Customize Quick Access Toolbar. From All Commands, add Speak Cells or Speak Cells on Enter to QAT. Select cells. Press Speak Cells.
Couple of things to try. Firstly after this line:
Rich (BB code):
lastRow = targetSheet.Cells(targetSheet.Rows.Count, "AJ").End(xlUp).Row
put this:
Rich (BB code):
MsgBox lastRow
and see what the last row is in column AJ

Also, with this line:
Rich (BB code):
targetSheet.Cells(lastRow, 1).PasteSpecial
you're telling the code to copy the values into column 1 (= column A). If you want it to copy to column AJ, it should be this:
Rich (BB code):
targetSheet.Cells(lastRow, 36).PasteSpecial
 
Upvote 0
Thanks Rich! This helped me get to where I could figure it out!
If anyone wants to know, this ended up being the solution.
VBA Code:
Sub CopyDataToAnotherSheet()

    Dim sourceSheet As Worksheet
    Dim destSheet As Worksheet
      
    Set sourceSheet = ThisWorkbook.Sheets("CIPs")
    Set targetSheet = ThisWorkbook.Sheets("CIP 2024")
    
    Dim sourceBTR2 As Range, targetBTR2 As Range
    Dim BTR2i As Integer
    
    For BTR2i = 107 To 122
        
        If sourceSheet.Cells(BTR2i, 32).Value > 0 And sourceSheet.Cells(BTR2i, 42).Value > 0 Then
           Set sourceBTR2 = sourceSheet.Range(sourceSheet.Cells(23 + BTR2i - 107, 21), sourceSheet.Cells(23 + BTR2i - 107, 25))
            
           Set targetBTR2 = targetSheet.Cells(targetSheet.Rows.Count, 36).End(xlUp).Offset(1, 0)
            
            If IsEmpty(targetBTR2.Value) Then
                sourceBTR2.Copy
                targetBTR2.PasteSpecial Paste:=xlPasteValues
                Application.CutCopyMode = False
            Else
                Set targetBTR2 = targetSheet.Cells(targetSheet.Rows.Count, 36).End(xlUp).Offset(1, 0)
                sourceBTR2.Copy
                targetBTR2.PasteSpecial Paste:=xlPasteValues
                Application.CutCopyMode = False
            End If
        Else
            Exit For
        End If
    Next BTR2i
End Sub
 
Upvote 0
Solution
Hi, would someone be able to break down for me how this Macro works? (in above solution)

Trying to implement something similar for my own sheet: Need some help with a Macro but I don't know enough about Macros to try and solo this one....
 
Upvote 0

Forum statistics

Threads
1,215,076
Messages
6,122,983
Members
449,092
Latest member
Mr Hughes

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