Run time error 450: Copying non zero values from non adjacent columns to another sheet (non adjacent columns)

JonathanSass

New Member
Joined
Apr 15, 2021
Messages
14
Office Version
  1. 2019
Hi all,

I have tried to use the following code for my issue described in the header:

VBA Code:
Sub CopyDataToJournal()

Dim erow As Long, lastrow As Long, i As Long

lastrow = Sheets("Sheet1").Cells(Rows.Count, 12).End(xlUp).Row

For i = 4 To lastrow
    If Sheet1.Cells(i, 12) <> "0" Then
    Sheets("Sheet1").Range(Cells(i, 12), Cells(i, 13), Cells(i, 17), Cells(i, 18), Cells(i, 19), Cells(i, 20)).Copy
    Sheets("JOURNAL").Activate
    erow = JOURNAL.Cells(Row.Count, 1).End(xlUp).Offset(1, 0).Row
    ActiveSheet.Paste Destination:=Sheets("JOURNAL").Range(Cells(erow, 1), Cells(erow, 2), Cells(erow, 6), Cells(erow, 7), Cells(erow, 8), Cells(erow, 9))
    Sheets("Sheet1").Activate
    End If
Next i
Application.CutCopyMode = False
   
End Sub

I am new to Excel VBA.

Thank you very much in advance!!
/Jonathan
 
Last edited by a moderator:

Excel Facts

Which came first: VisiCalc or Lotus 1-2-3?
Dan Bricklin and Bob Frankston debuted VisiCalc in 1979 as a Visible Calculator. Lotus 1-2-3 debuted in the early 1980's, from Mitch Kapor.
Ok, how about
VBA Code:
Sub CopyDataToJournal()

Dim erow As Long, lastrow As Long, i As Long

erow = Sheets("JOURNAL").Cells(Rows.Count, 12).End(xlUp).Offset(1, 0).Row

With Sheets("Sheet1")
   lastrow = .Cells(Rows.Count, 12).End(xlUp).Row
   
   For i = 4 To lastrow
      If .Cells(i, 12) <> "0" Then
         .Cells(i, 12).Resize(, 2).Copy Sheets("Journal").Range("A" & erow)
         .Cells(i, 17).Resize(, 4).Copy Sheets("Journal").Range("F" & erow)
         erow = erow + 1
      End If
   Next i
End With
Application.CutCopyMode = False
   
End Sub
 
Upvote 0
Thanks a whole lot it copied all the data and correctly!!

However, as I an copying formulas, is there a way to make it paste the data as values?
 
Upvote 0
How about
VBA Code:
      If .Cells(i, 12) <> "0" Then
         Sheets("Journal").Range("A" & erow).Resize(, 2).Value = .Cells(i, 12).Resize(, 2).Value
         Sheets("Journal").Range("F" & erow).Resize(, 4).Value = .Cells(i, 17).Resize(, 4).Value
         erow = erow + 1
      End If
 
Upvote 0
You're welcome & thanks for the feedback.
 
Upvote 0
Hi again,

I'm doing this exercise for several sheets into 1 journal tab. With the code now it deletes the previous data in the journal tab. It simply needs to be entered after the data already there. Can this be done?

Thanks a lot
 
Upvote 0
That code should not be deleting anything. It should be adding data to the journal sheet after the last row with a value in col L
 
Upvote 0
Unfortuantely this is what happens. It is like the three modules I have created right now (for three tabs that each should be copied in to the journal sheet) delete each other when I run the respective module
 
Upvote 0
Can you post the codes that you are using.
 
Upvote 0

Forum statistics

Threads
1,214,864
Messages
6,121,981
Members
449,058
Latest member
oculus

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