Delete Entire Row Skipping Rows

quick_question

New Member
Joined
May 31, 2011
Messages
32
Hello All,

I am trying to delete all rows with a value of "1" in column "Q" (or 17).

The issue that I am having is that the code works, kind of, as it deletes the majority of the rows, but skips some rows.

My observation is if there are say 5 rows with the following values in column Q (by row); 1, 0, 1, 1, 1.

It looks as though it's skipping rows, b/c as it deletes a row, the code is recognizing the next row as the row that was just deleted b/c it shifted up.

ie - deletes row 1, row 2 then shifts up to become row 1. If row 2 was supposed to be deleted as well (b/c it has a value of 1) the code skips this row b/c it recognizes it as row 1, which it has already addressed.

PLEASE HELP!!!!

--

Sub DeleteRows_Nexans()


Sheets("Sheet2").Activate

Dim i As Integer
Dim x As Integer

x = Sheets("Sheet2").Range("A1").Value 'Identifies # of rows of data


For i = 1 To x
If Cells(i, 17).Value = 1 Then
Rows(i).EntireRow.Delete
End If
Next i
End Sub
 

Excel Facts

Excel motto
Not everything I do at work revolves around Excel. Only the fun parts.
If you do things like this you need to step up rather than down. So x To 1 Step -1 rather than 1 To x
 
Upvote 0
When deleting rows, you need to work backwards...

Code:
Sub DeleteRows_Nexans()
Application.ScreenUpdating = False
 Sheets("Sheet2").Activate
 Dim i As Integer
 Dim x As Integer
 x = Sheets("Sheet2").Range("A1").Value 'Identifies # of rows of data

 For i =[COLOR=#FF0000] [B]x To 1 Step -1[/B][/COLOR]
    If Cells(i, 17).Value = 1 Then Rows(i).EntireRow.Delete
 Next i
 
 Application.ScreenUpdating = True
 
 End Sub
 
Last edited:
Upvote 0
you need to go backwards

Code:
For i = 1 To x

needs to change for the below

Code:
For i = x To 1 Step -1
 
Upvote 0

Forum statistics

Threads
1,213,544
Messages
6,114,249
Members
448,556
Latest member
peterhess2002

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