CODEWITHSUNDEEP
c#datetimestringformattingiformatprovider
Refracted Paladin
Refracted Paladin

Reputation: 12216

Format a string to display the Date correctly

I need help converting this string --> 20090727 10:16:36:643 to --> 07/27/2009 10:16:36

The original date and time are being returned by the SynchronizationAgent.LastUpdated() function, which returns a String in the above format.

Original question:preserved for reference

I have this -->

 HUD.LastSyncDate = mergeSubscription.SynchronizationAgent.LastUpdatedTime;

Which is setting a property that looks like this -->

public static string LastSyncDate
    {
        get { return _lastSyncDate; }
        set
        {
            _lastSyncDate = String.Format(CultureInfo.InvariantCulture,"{0:G}", value);
        }
    }

Unfortunately, with or without the String.Format the date that is displayed looks like this --> 20090727 10:16:36:643

I have tried multiple variations to Format it the way I want. What am I missing?

Based on the below suggestions(Mostly Joel's), I implemented the suggested changes but I am still getting a "String is not a valid DateTime error"

I also tried implementing this --> 

HUD.LastSyncDate = DateTime.ParseExact(mergeSubscription.SynchronizationAgent.LastUpdatedTime,"yyyyMMdd HH:mm:ss:fff",CultureInfo.InvariantCulture);

but still nothing.

Upvotes: 0

Views: 800

Answers (3)

Daniel Brückner
Daniel Brückner

Reputation: 59675

What do you want to do? You get a string, pass it to String.Format() and store it in a string field. Do you want to reformat the string? In this case you have to parse the string back to DateTime and format this value again.

DateTime dateTime;
if (DateTime.TryParse(value, out dateTime))
{
    lastSyncDate = String.Format(CultureInfo.InvariantCulture,"{0:G}", dateTime);
}
else
{
    HandleInvalidInput(value);
}

Upvotes: 0

Joel Coehoorn
Joel Coehoorn

Reputation: 416039

HUD.LastSyncDate = DateTime.Parse(mergeSubscription.SynchronizationAgent.LastUpdatedTime).ToString("MM/dd/yyyy")

You can put any format string you want there. But it sounds like what you really want is something more like this:

private static DateTime _lastSyncDate;
public static DateTime LastSyncDate
{
    get { return _lastSyncDate; }
    set { _lastSyncDate = value;}
}

public static string LastSyncDateString
{
    get { return LastSyncDate.ToString("MM/dd/yyyy"); }
}

Keep it as a datetime in the background and just use the string property for display.

Upvotes: 4

tvanfosson
tvanfosson

Reputation: 532555

It appears to me that LastUpdatedTime is actually a string (since you can do the assignment) not a DateTime. In that case, the format applied won't do anything. You'll want to parse the LastUpdatedTime into a DateTime then reformat into the format that you want before assigning it to your string.

DateTime lastUpdated = DateTime.Parse( mergeSubscription.SynchronizationAgent.LastUpdatedTime );
HUD.LastSyncDate = string.Format( "{0:G}", lastUpdated );

public static string LastSyncDate { get; set; }

Note that you may need to use ParseExact instead.

DateTime lastUpdated = DateTime.ParseExact( "yyyyMMdd HH:mm:ss:fff",
                                            ...,
                                            CultureInfo.InvariantCulture );

Upvotes: 1

Related Questions