VBA code to insert rows based on value. Current code breaking on zero

D

davie_aus

New Member
Joined
Jul 5, 2018
Messages
6
Hey all,
Ive found some code on a forum elsewhere and tried to retrofit it solve a problem im facing with formatting.

I need to insert a certain amount of blank rows based on a value. I.E: If the value is 3, i need to insert 3 blank rows below this cell. The code ive found works nicely, but when it gets to 0 it doesnt work. Can anyone cast your eye over it and let me know if you find anything that may help.

Current Code:
Sub InsertRowsIf()
Dim lr As Long, R As Range, i As Long
lr = Range("C" & Rows.Count).End(xlUp).Row
Set R = Range("C1", "C" & lr)

Application.ScreenUpdating = False
For i = R.Rows.Count To 1 Step -1

If IsNumeric(R.Cells(i, 1).Value) And Not IsEmpty(R.Cells(i, 1)) Then
R.Cells(i, 1).Offset(1, 0).Resize(R.Cells(i, 1).Value).EntireRow.Insert

End If
Next
End Sub
Click to expand...

Input:

[TABLE="class: grid, width: 500"]
<tbody>[TR]
[TD]Day/Night Shift[/TD]
[TD]Date Code[/TD]
[TD]Value[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/1/2018[/TD]
[TD]1[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/2/2018[/TD]
[TD]2[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/2/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/2/2018[/TD]
[TD]1[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/3/2018[/TD]
[TD]3[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/3/2018[/TD]
[TD]3[/TD]
[/TR]
</tbody>[/TABLE]


Desired output:

[TABLE="class: cms_table_grid"]
<tbody>[TR]
[TD]Day/Night Shift[/TD]
[TD]Date Code[/TD]
[TD]Value[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/1/2018[/TD]
[TD]1[/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/2/2018[/TD]
[TD]2[/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/2/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/2/2018[/TD]
[TD]1[/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/3/2018[/TD]
[TD]2[/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/3/2018[/TD]
[TD]2[/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD="bgcolor: #FAFAFA"][/TD]
[TD="bgcolor: #FAFAFA"][/TD]
[/TR]
</tbody>[/TABLE]
 

Excel Facts

Select all contiguous cells
Pressing Ctrl+* (asterisk) will select the "current region" - all contiguous cells in all directions.
Sort by date Sort by votes
Hi. Try after replacing
And Not IsEmpty(R.Cells(i, 1)) Then

by this
And R.Cells(i, 1).Value > 0 Then

Don't forget to reset ScreenUpdating to True at the code end:
Application.ScreenUpdating = True
End Sub



 
Upvote 0
davie_aus said:
Hey all,
Ive found some code on a forum elsewhere and tried to retrofit it solve a problem im facing with formatting.

I need to insert a certain amount of blank rows based on a value. I.E: If the value is 3, i need to insert 3 blank rows below this cell. The code ive found works nicely, but when it gets to 0 it doesnt work. Can anyone cast your eye over it and let me know if you find anything that may help.

Current Code:


Input:

[TABLE="class: grid, width: 500"]
<tbody>[TR]
[TD]Day/Night Shift[/TD]
[TD]Date Code[/TD]
[TD]Value[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/1/2018[/TD]
[TD]1[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/2/2018[/TD]
[TD]2[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/2/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/2/2018[/TD]
[TD]1[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/3/2018[/TD]
[TD]3[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/3/2018[/TD]
[TD]3[/TD]
[/TR]
</tbody>[/TABLE]


Desired output:

[TABLE="class: cms_table_grid"]
<tbody>[TR]
[TD]Day/Night Shift[/TD]
[TD]Date Code[/TD]
[TD]Value[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/1/2018[/TD]
[TD]1[/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/1/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/2/2018[/TD]
[TD]2[/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/2/2018[/TD]
[TD]0[/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/2/2018[/TD]
[TD]1[/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD]Day[/TD]
[TD]1/3/2018[/TD]
[TD]2[/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD]Night[/TD]
[TD]1/3/2018[/TD]
[TD]2[/TD]
[/TR]
[TR]
[TD][/TD]
[TD][/TD]
[TD][/TD]
[/TR]
[TR]
[TD="bgcolor: #FAFAFA"][/TD]
[TD="bgcolor: #FAFAFA"][/TD]
[/TR]
</tbody>[/TABLE]
Click to expand...
just replace your
Code: 
IsNumeric(R.Cells(i, 1).Value)

to

Code: 
R.Cells(i, 1).Value <> 0

and if you have non numeric values on the 3rd column just add
Code: 
R.Cells(i, 1).Value <> 0

it should look like this:
Code: 
R.Cells(i, 1).Value <> 0 And IsNumeric(R.Cells(i, 1).Value) And Not IsEmpty(R.Cells(i, 1))
 
Last edited:
Upvote 0
You must log in or register to reply here.

Similar threads

D
  • Solved
Need help creating a formula based on VBA function!
Replies
5
Views
319
Trixterz
T
T
  • Question Question
VBA help to count Daytime,Nighttime and Summary
2 3
Replies
20
Views
797
Alex Blakenburg
A
B
  • Solved
Need code that converts colors from the RGB code to the Natural Color System (NCS)
Replies
6
Views
174
PeteWright
PeteWright
P
  • Question Question
putting a snippet of code from one to another somehow
Replies
5
Views
286
Patriot2879
P
J
  • Solved
Selenium VBA - loop to align new data with old data in worksheet
Replies
8
Views
323
Anthony47
A

Forum statistics

Threads
1,223,098
Messages
6,170,100
Members
452,301
Latest member
QualityAssurance

Share this page

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