Mismatch Error?

[crburke92]

Having a mismatch error trying to compare a range of cells (Which is my date column) to todays date. I have Cell A1 =TODAY(), and range B2:B29 are expiry dates. I initially had the message box within the "For each cell" statement, which did work and format the cells properly, but then gave the message box for every expired date it found. Tried bringing the message box outside of the For Statement so it only came up once and BAM! Mismatch error.

Private Sub Workbook_Open()
Dim TDate As Date
Dim ExDate As Date
TDate = Range("A1")
ExDate = Range("B2:B29")
If TDate < ExDate - 10 Then
    MsgBox "Something is about to expire!"
End If
For Each Cell In Range("B2:B20")
    If Cell.Value < TDate - 10 And Cell.Value <> "" Then
            Cell.Interior.ColorIndex = 19
            Cell.Font.Bold = True
                If Cell.Value > TDate - 10 And Cell.Value <> "" Then
                    Cell.Interior.ColorIndex = 2
                    Cell.Font.Bold = False
                End If
    End If
Next
End Sub

Thanks in advance!

 

[steve the fish]

Try like this:

Dim TDate As Long, ExDate As Long

With Sheets("Sheet1")
    TDate = .Range("A1")
    ExDate = .Application.Min(Range("B2:B29"))
    If TDate > ExDate - 10 Then MsgBox "Something is about to expire!"
End With

You need to use the sheet name really.

 

[steve the fish]

Range("B2:B29") should be .Range("B2:B29"). See the dot/ period in red.

 

[crburke92]

That did the trick for the message box, but now I'm having an issue with my last If statememnt to revert the formatting back? Any ideas why?

Private Sub Workbook_Open()
Dim TDate As Long
Dim ExDate As Long
With Sheets("Sheet1")
    TDate = .Range("A1")
    ExDate = Application.Min(Range("I3:I29"))
    If ExDate < TDate - 10 Then MsgBox "Something is about to expire!"
End With
For Each Cell In Range("I3:I29")
    If Cell.Value < TDate - 10 And Cell.Value <> "" Then
            Cell.Interior.ColorIndex = 19
            Cell.Font.Bold = True
                If Cell.Value > TDate - 10 And Cell.Value <> "" Then
                    Cell.Interior.ColorIndex = 2
                    Cell.Font.Bold = False
                End If
    End If
Next
End Sub

 

[crburke92]

Nevermind....I nested the second If within the First so it was jumping over it. I'm dumb