Why my code is throwing this error String was not recognized as a valid DateTime?

Asked May 22 '17 at 11:10

Active May 22 '17 at 11:51

Viewed 100 times

Getting this error:

String was not recognized as a valid DateTime.

DateTime dtOldOpeningDateTime = (lblOldOpeningDate.Text+" "+lblOldOpeningTime.Text).ToDate();

lblOpeningDate.Text contains 26-03-2017 and lblOldOpeningTime.Text contains 02:00:00.

Why this error ?

Update: toDate() function is like this:

public static DateTime ToDate(this object obj)
{
    return Convert.ToDateTime(obj);
}

edited May 22 '17 at 11:51

DigiFriend

1,164
5
10

asked May 22 '17 at 11:10

Covert

@AndyKorneyev - not really the same, as there is no explicit format string involved. – DigiFriend May 22 '17 at 11:13
What is the culture this code is running under? The runtime is trying to parse the string using that culture, so if there is a mismatch, you will get that error. – DigiFriend May 22 '17 at 11:14
1

Your code is working with that string http://rextester.com/SHBOAM56571 . Check textBox values – Samvel Petrosov May 22 '17 at 11:14
@S.Petrosov Sir, I checked but same error and the values i provided are the same – Covert May 22 '17 at 11:16
1

@S.Petrosov - that completely depends on the default culture when the code is running. – DigiFriend May 22 '17 at 11:16
@DigiFriend what's the solution then ? – Covert May 22 '17 at 11:17
There's an overload to `Convert.ToDateTime` that takes an `IFormatProvider`, this can be any culture, such as `CultureInfo.InvariantCulture`. – DigiFriend May 22 '17 at 11:19
@DigiFriend I tried overloading with this but same error – Covert May 22 '17 at 11:23
DateTime dtOldOpeningDateTime = Convert.ToDateTime((lblOldOpeningDate.Text + " " + lblOldOpeningTime.Text), CultureInfo.InvariantCulture); – Covert May 22 '17 at 11:23
I was just giving an example. You need to use a culture where the date separator is `-`. I don't know what culture that would be. – DigiFriend May 22 '17 at 11:28
I also used DateTime dtOldOpeningDateTime = DateTime.ParseExact((lblOldOpeningDate.Text + " " + lblOldOpeningTime.Text), "dd/MMM/yyy HH:mm", CultureInfo.InvariantCulture); but no luck – Covert May 22 '17 at 11:30
`MMM` is not the correct string. Check the docs. – Matt Johnson-Pint May 22 '17 at 17:47

Why my code is throwing this error String was not recognized as a valid DateTime?

0 Answers0