Hello,

Here is my code:
 Dim valDOB As DateTime = CType(objRecruit.CheckDOB(txtDOB.Text), Date)

   objRecruit.DOB = valDOB.Date
   Response.Write(valDOB.Date)

Here's the Function Code:
Function CheckDOB(ByVal DOB As String) As DateTime

        Dim thedate As String
        thedate = Format(DOB, "mm/dd/yyyy")
        Try
            thedate = Format(Convert.ToDateTime(DOB), "mm/dd/yyyy")
            Return CType(thedate, Date)
        Catch ex As Exception
            Return CType(thedate, Date)
        End Try

    End Function

Please someone help...all I am trying to do is error handle, if a user types anything else than the proper date format which is mm/dd/yyyy.

Thanks in Adavnce... P.S. been at this still last Thursday!!

Mark

Hi there,

What error do you ge?

Also, I think your logic is a bit flawed here. A Try Catch is used to catch (un)expected errors, and handle them gracefully. In your situation, you try to format a date. If the format fails, you still try to convert the invalid data to a date in the Catch block.

So, suppose someone types "Bla" as the date in the text field txtDOB. The Try statement fails, and you end up in the Catch statement. The Catch statement then tries to convert Bla to a date, something that won't work.

What exactly are you trying to accomplish? What kind of input can you expect from txtDOB.Text? I think it'll make sense to return an error, or DateTime.MinValue for example when the string cannot be converted to a date, and explain to the user what went wrong.

Also, since CheckDOB returns a date, there is no need to cast (with CType) it to a date anymore in the first line of your code.

If you're worried about incorrect date formats, try three drop downs for the day, week and year element of a date, or look into the <asp:Calendar"> control.

Cheers,

Imar
---------------------------------------
Imar Spaanjaars
Everyone is unique, except for me.

Thanks for the reply...I have it figured out, using the IsDate.

Mark