Solved

VBA to delete rows in filtered table

Posted on 2012-03-13
7
1,530 Views
Last Modified: 2012-03-13
Hi Experts!

Attached is some code I'm using to Filter one column in a Table and then delete the visible rows.  I had difficulty with Selection.EntireRow.Delete but have replaced it with RngToDelete.Delete.

My problem is that the code stops and asks if I want to "Delete Entire Sheet Row?".  I click "Yes" and it's fine.  How can I have the code eliminate that step as well so it runs from start to finish without stopping?

This is very likely a no brainer for all of you. My attempt at code is below.  Thank you!

Sub Filter_Sales1()
'
    Dim RngToDelete As Range
    Range("TblFTE").Select
   
    ActiveSheet.ListObjects("TblFTE").Range.AutoFilter Field:=10, Criteria1:= _
        Array("Business Development", "G&A", "Marketing", "R&D", "Solutions", "Support"), _
        Operator:=xlFilterValues
       
    Set RngToDelete = Selection.SpecialCells(xlCellTypeVisible)
     
    RngToDelete.Delete
       
    Range("TblFTE[[#Headers],[EE ID]]").Select
    ActiveSheet.ShowAllData
End Sub
0
Comment
Question by:xllvr
  • 4
  • 3
7 Comments
 
LVL 41

Accepted Solution

by:
dlmille earned 500 total points
ID: 37717688
try this (see line 12 modification):

Sub Filter_Sales1()
'
    Dim RngToDelete As Range
    Range("TblFTE").Select
    
    ActiveSheet.ListObjects("TblFTE").Range.AutoFilter Field:=10, Criteria1:= _
        Array("Business Development", "G&A", "Marketing", "R&D", "Solutions", "Support"), _
        Operator:=xlFilterValues
        
    Set RngToDelete = Selection.SpecialCells(xlCellTypeVisible)
     
    RngToDelete.EntireRow.Delete
        
    Range("TblFTE[[#Headers],[EE ID]]").Select
    ActiveSheet.ShowAllData
End Sub

Open in new window


Also, when you want to avoid Excel prompting and you don't know exactly how to do the statement so you don't get the prompt (and some don't have - e.g., for deleting sheets, you would get prompted), then use:

Application.DisplayAlerts = False

before the command that would cause the prompt

Then,

Application.DisplayAlerts = True 'continue as normal
0
 
LVL 1

Author Comment

by:xllvr
ID: 37717714
Hey dlmille!  Nice to see you here.

Thanks for posting.  The code hung up on line 11 when I changed line 12.  Thoughts?
0
 
LVL 1

Author Comment

by:xllvr
ID: 37717733
Hang on a second.  Just tried your suggestion with Application.DisplayAlerts=False along with the code I already had and it seems to work.
0
Netscaler Common Configuration How To guides

If you use NetScaler you will want to see these guides. The NetScaler How To Guides show administrators how to get NetScaler up and configured by providing instructions for common scenarios and some not so common ones.

 
LVL 41

Expert Comment

by:dlmille
ID: 37717744
I'm not sure what you mean by "hung up" and I don't see a line 11.

Yes, displayalerts would also work, but it shouldn't be needed if you delete using the entirerow property.

Dave
0
 
LVL 1

Author Comment

by:xllvr
ID: 37717765
Apologies...Line 10.  The code halted at that point.  I originally used EntireRow and had problems with it.  I have read quite a few posts that showed the very same problem with using EntireRow in the code so I replaced it with what others used successfully.  Using RngToDelete.Delete worked fine except I didn't want to have to stop and click "Yes" to Delete Entire Sheet Row to continue the code.  

Using RngToDelete.Delete with the Display Alert message seems to be working.  Thanks for sending that along as a potential solution!
0
 
LVL 41

Expert Comment

by:dlmille
ID: 37717779
Perhaps it was choking on EntireRow because it was a table?  Yes, any time there is an Excel Prompt you want to circumvent (almost anytime, there are a few exceptions), you can override and take the default "YES" with Application.DisplayAlerts = False.

Dave
0
 
LVL 1

Author Comment

by:xllvr
ID: 37717791
That's the only conclusion I could come up with as well.  Grateful for the prompt tip, Dave!

-Nancy
0

Featured Post

Master Your Team's Linux and Cloud Stack

Come see why top tech companies like Mailchimp and Media Temple use Linux Academy to build their employee training programs.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

This code takes an Excel list of URL’s and adds a header titled “URL List”. It then searches through all URL’s in column “A”, looking for duplicates. When a duplicate is found, it is moved to the top of the list. The duplicate URL’s are then highlig…
Freeze panes is an option within all variants of Excel to enable parts of a sheet to remain stationary when the cursor is in another part of the sheet. This is a very useful feature which is overlooked or under used.
Graphs within dashboards are meant to be dynamic, representing data from a period of time that will change each time the dashboard is updated with new data. Rather than update each graph to point to a different set within a static set of data, t…
This Micro Tutorial demonstrates in Microsoft Excel how to consolidate your marketing data by creating an interactive charts using form controls. This creates cool drop-downs for viewers of your chart to choose from.

832 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question