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

Remove leading & trailing spaces
Save as CSV to remove all leading and trailing spaces. It is faster than using TRIM().

Fluff

MrExcel MVP, Moderator
Joined
Jun 12, 2014
Messages
56,671
Office Version
  1. 365
Platform
  1. Windows
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
 

JonathanSass

New Member
Joined
Apr 15, 2021
Messages
14
Office Version
  1. 2019
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?
 

Fluff

MrExcel MVP, Moderator
Joined
Jun 12, 2014
Messages
56,671
Office Version
  1. 365
Platform
  1. Windows
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
 

JonathanSass

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

ADVERTISEMENT

You're a grand champion. Thank you very much
 

Fluff

MrExcel MVP, Moderator
Joined
Jun 12, 2014
Messages
56,671
Office Version
  1. 365
Platform
  1. Windows
You're welcome & thanks for the feedback.
 

JonathanSass

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

ADVERTISEMENT

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
 

Fluff

MrExcel MVP, Moderator
Joined
Jun 12, 2014
Messages
56,671
Office Version
  1. 365
Platform
  1. Windows
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
 

JonathanSass

New Member
Joined
Apr 15, 2021
Messages
14
Office Version
  1. 2019
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
 

Fluff

MrExcel MVP, Moderator
Joined
Jun 12, 2014
Messages
56,671
Office Version
  1. 365
Platform
  1. Windows
Can you post the codes that you are using.
 

Watch MrExcel Video

Forum statistics

Threads
1,129,795
Messages
5,638,382
Members
417,025
Latest member
MusterDuster

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
Top