Loop for Deleting Rows

Hello Experts

Whats wrong with my code ?

Please Login or Register  to view this content.

Thanks
Sharky

When you delete rows you should work in reverse - bottom up...

If you consider you have rows 2:10 and you loop from 2 to 10, when you delete row 3 then what was row 4 is now 3 yet you miss it as you go to 4 (which was 5), make sense ?

Better then to adopt something along the lines of:

Please Login or Register  to view this content.

Exactly right...and if you're deleting a LOT of rows, then the code will execute much faster if you turn off the screenupdating:

Please Login or Register  to view this content.

If speed were a concern then I would make the point that as much as ScreenUpdating will slow code, given Deletion of Rows is a Volatile action if you have Volatile functions you're best served forcing Calculation to Manual prior to implementation of Deletion Loop and restoring Calc to it's former setting on completion ... and if you do that you may as well do all three, ie: screenupdate, calculation & events.

An example below:

Please Login or Register  to view this content.