Macro running painfully slow.

[Dspace]

Good afternoon-
I'm running what I consider to be a fairly simple code. It works but it takes roughly 30 seconds to run which is driving me a little bonkers. I have ala carte pricing from rows 18 to 463. What this codes is saying is if my sales rep enters a value in column H to not hide that row. If the cell is left blank then hide that row. Looking for insight on how to streamline this code so it speeds up the macro. Any insight is appreciated. Thanks

Sub HideAllRows()
'Hides everything without a quantity in column H. Used for all three sales teams as well as Bid Desk when hiding everything if we've added something custom
Unprotect Password:="PW"
Application.ScreenUpdating = True
For Each cell In Range("H18:H463")

cell.EntireRow.Hidden = cell.Value = 0
Next cell
ActiveSheet.Protect Password:="PW", AllowFormattingCells:=True
End Sub

 

[Joe4]

See if this helps any:

Sub HideAllRows()
'Hides everything without a quantity in column H. Used for all three sales teams as well as Bid Desk when hiding everything if we've added something custom
Unprotect Password:="PW"
Application.ScreenUpdating = False
For Each cell In Range("H18:H463")

cell.EntireRow.Hidden = cell.Value = 0
Next cell
Application.ScreenUpdating = True
ActiveSheet.Protect Password:="PW", AllowFormattingCells:=True
End Sub

 

[Dspace]

Joe4 for the win! Simple fix. Big Thank you!

 

[Joe4]

You are welcome.

Please note that when marking a post as the solution, please mark the actual thread that contains the solution (not your own post acknowledging that some other post is the solution).

You will notice that once you mark a post as the solution, it also appears at the very top under the original question, so anyone can easily see at quick glance the original question and the solution. So marking any post that does not actually contain the solution would kind of defeat the purpose of that.

I have gone ahead and change the posted solution for you.

 

[Dspace]

You are welcome.

Please note that when marking a post as the solution, please mark the actual thread that contains the solution (not your own post acknowledging that some other post is the solution).

You will notice that once you mark a post as the solution, it also appears at the very top under the original question, so anyone can easily see at quick glance the original question and the solution. So marking any post that does not actually contain the solution would kind of defeat the purpose of that.

I have gone ahead and change the posted solution for you.

Yea, that was dumb on my part. I went back to fix it but you beat me to the punch. Again, thanks for the fix.