Want to win a PS4? Go Premium and enter to win our High-Tech Treats giveaway. Enter to Win

x
?
Solved

Filter criteria for Excel using VBA in Access

Posted on 2009-04-01
12
Medium Priority
?
520 Views
Last Modified: 2013-11-27
Hi Experts,

I'm having a application defined error when I run the following VBA code in Access that tried to do a filter in an Excel file:


Dim myDate as Date
Dim getRow As Integer
myDate = Date     
 
With wb.Worksheets("Sheet1")
        .AutoFilterMode = False 'To turn off the filter in case there is any
        With .Rows(8)
         .AutoFilterMode = True 'To turn on the filter
         .AutoFilter Field:=4, Criteria1:="New"
         .AutoFilter Field:=9, Criterial:=myDate 'This is the line that error happens
         .AutoFilter Field:=29, Criteria1:="=s-*"
        End With
        getRow = .Range("B2").End(xlDown).Row
        .Rows("2:" & getRow).Delete shift:=xlUp
End With

Open in new window

0
Comment
Question by:LizzJ
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 6
  • 6
12 Comments
 
LVL 85

Expert Comment

by:Rory Archibald
ID: 24038059
You need to pass myDate as a String, formatted the same way the cells are.
HTH
Rory
0
 

Author Comment

by:LizzJ
ID: 24038116
the cells contain dates.

can you provide the right way of doing such conversion? tried a few, but doens't seem to work
0
 

Author Comment

by:LizzJ
ID: 24038119
btw, the dates are in format: dd/mm/yyyy
0
Visualize your virtual and backup environments

Create well-organized and polished visualizations of your virtual and backup environments when planning VMware vSphere, Microsoft Hyper-V or Veeam deployments. It helps you to gain better visibility and valuable business insights.

 
LVL 85

Expert Comment

by:Rory Archibald
ID: 24038246
I would recommend not using just Rows(8) to set the filter, but use the actual range you want filtered - it's much more reliable. Then your original syntax should work if the dates are real dates.
0
 

Author Comment

by:LizzJ
ID: 24046972
Can you provide the codes for that in such a case? Not much idea on how to specify the range...
0
 
LVL 85

Expert Comment

by:Rory Archibald
ID: 24047120
Sure - is there a column that should always have data for each row?
0
 

Author Comment

by:LizzJ
ID: 24047161
the column will be D
0
 
LVL 85

Expert Comment

by:Rory Archibald
ID: 24047211
Ok - I'll put something together when I'm back in the office, hopefully later this morning?
0
 

Author Comment

by:LizzJ
ID: 24047360
okie. Thanks. Please also let me know how to fix the date error
0
 
LVL 85

Expert Comment

by:Rory Archibald
ID: 24047762
Quick question - your code starts the filter from row 8, which I assume is where the headers are, but then deletes from row 2 downwards - is that correct?
0
 
LVL 85

Accepted Solution

by:
Rory Archibald earned 2000 total points
ID: 24047854
I've just looked at your code more closely - on the date line that errors, you have written:
 CRITERIAL:=
 rather than:
 CRITERIA1:=
 
if you change the L to a 1 that should fix that error. I just ran a quick test using this code, and it worked fine:
 
 

Sub FilterXL()
   Dim myDate As Date
   Dim getRow As Long
   Dim wb As Excel.Workbook, appXL As Excel.Application
   myDate = Date
   Set appXL = CreateObject("Excel.Application")
   appXL.Visible = True
   Set wb = appXL.workbooks.Open("C:\test\filter_test.xls")
   With wb.Worksheets("Sheet1")
      ' get last used row in column D
      getRow = .cells(.Rows.Count, "D").End(xlUp).Row
      .AutoFilterMode = False 'To turn off the filter in case there is any
      With .Range("A8", .cells(getRow, 29))
         ' filter col D
         .AutoFilter Field:=4, Criteria1:="New"
         ' filter col I
         .AutoFilter Field:=9, Criteria1:=myDate 'This is the line that error happens
         ' filter col AC
         .AutoFilter Field:=29, Criteria1:="=s-*"
      End With
      .Range("B2", .cells(getRow, "B")).SpecialCells(xlCellTypeVisible).EntireRow.Delete shift:=xlUp
   End With
 
End Sub

Open in new window

0
 

Author Comment

by:LizzJ
ID: 24047933
Thanks a lot! it works! Couldn't imagine the silly mistake bothered me whole day...

Anyway, I've got another related question regarding looping through the filtered records. But I'll post another question. Maybe you can also help me look through that?
0

Featured Post

Technology Partners: We Want Your Opinion!

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

Question has a verified solution.

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

This article describes how to use a set of graphical playing cards to create a Draw Poker game in Excel or VB6.
Instead of error trapping or hard-coding for non-updateable fields when using QODBC, let VBA automatically disable them when forms open. This way, users can view but not change the data. Part 1 explained how to use schema tables to do this. Part 2 h…
This Micro Tutorial will demonstrate in Microsoft Excel how to add style and sexy appeal to horizontal bar charts.
Have you created a query with information for a calendar? ... and then, abra-cadabra, the calendar is done?! I am going to show you how to make that happen. Visualize your data!  ... really see it To use the code to create a calendar from a q…

618 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