Macro to Hide Rows

billythedj66

Board Regular
Joined
Jan 6, 2003
Messages
126
Hello,

I have a worksheet named "IGE". This worksheet is used as a printout sheet that gets it's data from various other sheets within the workbook. The range of cells are A1:H343. What I am trying to do is have a macro that if the cell in column "A" of a particular row is blank, then the whole row is hidden, if not then the row is visible and has autofit applied to it. I have the following macro, but it takes about two minutes to complete:

Sub IGE()

' IGE Macro
' Edited 1/22/2003 by Bill

Sheets("IGE").Visible = True
Sheets("IGE").Select
Application.EnableEvents = False
Application.ScreenUpdating = False
Dim c As Range
For Each c In Range("A1:A" & Range("A344").End(xlUp).Row)
If c.Value = "" Then
c.EntireRow.Hidden = True
Else: c.EntireRow.AutoFit
End If
Next c
Sheets("IGE").PrintOut
ActiveSheet.UsedRange.EntireRow.Hidden = False
Application.ScreenUpdating = True
Application.EnableEvents = True
Sheets("IGE").Visible = False
Sheets("Opening").Select
Range("B17").Select


End Sub


Any ideas on why it is so slow, or is there a better way to do the same function?

Thanks in Advance!!
 

Excel Facts

How to create a cell-sized chart?
Tiny charts, called Sparklines, were added to Excel 2010. Look for Sparklines on the Insert tab.

Von Pookie

MrExcel MVP
Joined
Feb 17, 2002
Messages
13,686
Untested, but does this work any better for you?

Code:
Sub IGE()
Dim i As Range, lRow As Long

With Application
    .EnableEvents = False
    .ScreenUpdating = False
End With

With Sheets("IGE")
    .Visible = True
    lRow = .Cells(Rows.Count, 1).End(xlUp).Row
    For i = 1 To lRow
        If .Cells(i, 1) = "" Then
            .Rows(i).Hidden = True
        Else:
            .Rows(i).AutoFit
        End If
    Next i
    .PrintOut
    .UsedRange.EntireRow.Hidden = False
    .Visible = False
End With

With Application
    .Goto Sheets("Opening").Range("B17")
    .ScreenUpdating = True
    .EnableEvents = True
End With

End Sub
 
Upvote 0

billythedj66

Board Regular
Joined
Jan 6, 2003
Messages
126
I tried that macro, but when I ran it I received the following error:

Compile Error:

Type mismatch

Then the debugger highlights the lower case "i" in the following:

For i = 1 To lRow

Bill
 
Upvote 0

Forum statistics

Threads
1,191,274
Messages
5,985,702
Members
439,974
Latest member
sjoerdbosch

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