VBA - Help!

[ChrisFoster]

Hi,

I have the below code which looks for any dates from current year in column B and deletes those rows. Every time I run this code it will be on data from a specific month but could be different years (so always November but could be November 2000 all the way to November 2019). It seems to work fine when using any month that isn't the current month.

Dim c As Range
    With Range("B1:B" & Cells(Rows.Count, "B").End(xlUp).Row)
    Set c = .Find(What:=Year(Date), LookIn:=xlFormulas, lookAt:=xlPart, SearchOrder:=xlByRows, SearchDirection:=xlNext, MatchCase:=False, SearchFormat:=False)
    If Not c Is Nothing Then
        .AutoFilter Field:=2, Criteria1:=xlFilterThisYear, Operator:=xlFilterDynamic
        .Offset(1).SpecialCells(xlCellTypeVisible).EntireRow.Delete
        .AutoFilter
    End If
    End With

I'm trying to use the code for data from October and it debugs at the below stage. Please can someone help?

.AutoFilter Field:=2, Criteria1:=xlFilterThisYear, Operator:=xlFilterDynamic

Regards,

Chris

 

[steve the fish]

You dont have a field 2 in a single column range.

 

[ChrisFoster]

You dont have a field 2 in a single column range.

My VBA skills are really quite limited, so apologies in advance, I got this code from searching the web and from help on here. What do you mean by your above statement?

The strange thing though is that it works perfectly fine if I manually delete any rows that contain '2019' before running the code. And works fine for any data that is for another month other than current month.

 

[steve the fish]

Really? Far as i can tell that can never work as supplied regardless of the data. You are using B1:B whatever. There is only a field 1 as there is only one column being filtered. Change the Field:=2 to a 1.

 

[ChrisFoster]

Really? Far as i can tell that can never work as supplied regardless of the data. You are using B1:B whatever. There is only a field 1 as there is only one column being filtered. Change the Field:=2 to a 1.

Brill! That's sorted it, thanks so much.
Also, thanks for the info, it makes more sense to me now.