Это правильный способ конвертировать между часовыми поясами в Nodatime?

Question

Целью является создание функции, которая преобразует время из одного часового пояса в другое, используя Nodatime. Я не просто ищу отзыв о том, корректен ли результат, но обратная связь, специфичная для «как» я делаю это (действительно ли это правильно или есть тонкие дыры).

Вот функция:

static ZonedDateTime ConvertDateTimeToDifferentTimeZone(DateTime p_from_datetime, string p_from_zone, string p_to_zone) 
{ 
    DateTimeZone from_zone = DateTimeZoneProviders.Tzdb[p_from_zone]; 
    LocalDateTime from_local = new LocalDateTime(p_from_datetime.Year, p_from_datetime.Month, p_from_datetime.Day, p_from_datetime.Hour, p_from_datetime.Minute); 
    ZonedDateTime from_datetime = from_zone.AtStrictly(from_local); 

    var from_instant = from_datetime.ToInstant(); 

    DateTimeZone to_zone = DateTimeZoneProviders.Tzdb[p_to_zone]; 
    ZonedDateTime to_datetime = to_zone.AtStrictly(from_local); 

    var to_offset_datetime = from_instant.WithOffset(to_datetime.Offset); 

    return to_zone.AtStrictly(to_offset_datetime.LocalDateTime); 
} 

Путь я бы назвал это выглядит следующим образом:

DateTime from_datetime = new DateTime(2016, 10, 15, 16, 30, 0); 
    string from_zone = "US/Central"; 
    string to_zone = "US/Eastern"; 
    var x = ConvertDateTimeToDifferentTimeZone(from_datetime, from_zone, to_zone); 

    Console.WriteLine(from_datetime.ToString() + " (" + from_zone + ") = " + " (" + x.ToString() + " (" + to_zone + ")"); 

Я пропускаю что-нибудь или делать что-нибудь неправильно?

Answer 1

Я бы придерживаться типов Нода времени, насколько это возможно (и конвенций .NET именования). Преобразование между зонами должно быть выполнено с помощью ZonedDateTime.WithZone, что означает, что вы действительно спрашиваете о конвертации в ZonedDateTime. Если вы действительно, действительно должны использовать DateTime вместо типов Нода времени, вы, вероятно, хотите что-то вроде:

public static DateTime ConvertDateTimeToDifferentTimeZone(
    DateTime fromDateTime, 
    string fromZoneId, 
    string tozoneId) 
{ 
    LocalDateTime fromLocal = LocalDateTime.FromDateTime(fromDateTime); 
    DateTimeZone fromZone = DateTimeZoneProviders.Tzdb[fromZoneId]; 
    ZonedDateTime fromZoned = fromLocal.InZoneLeniently(fromZone); 

    DateTimeZone toZone = DateTimeZoneProviders.Tzdb[toZoneId]; 
    ZonedDateTime toZoned = fromZoned.WithZone(toZone); 
    LocalDateTime toLocal = toZoned.LocalDateTime; 
    return toLocal.ToDateTimeUnspecified(); 
} 

Обратите внимание на использование InZoneLeniently здесь - это значит, что если местное время вы дали недействителен из-за скачок в смещении UTC (как правило, из-за летнего сбережения), он все равно вернет значение, а не исключает исключение. Подробнее см. в документах.

Трудно понять, как будет выглядеть метод, если бы вы использовали Noda Time повсюду, так как мы не знаем, начинаете ли вы с LocalDateTime или ZonedDateTime. Например, вы могли бы:

public static LocalDateTime ConvertDateTimeToDifferentTimeZone(
    LocalDateTime fromLocal, 
    string fromZoneId, 
    string tozoneId) 
{ 
    DateTimeZone fromZone = DateTimeZoneProviders.Tzdb[fromZoneId]; 
    ZonedDateTime fromZoned = fromLocal.InZoneLeniently(fromZone); 

    DateTimeZone toZone = DateTimeZoneProviders.Tzdb[toZoneId]; 
    ZonedDateTime toZoned = fromZoned.WithZone(fromZone); 
    return toZoned.LocalDateTime; 
} 

Answer 2

Я думаю, что метод написан красиво. Насколько я могу судить, вы ничего не замечаете и ничего не делаете. Но вот немного улучшенная версия методы:

static ZonedDateTime ConvertDateTimeToDifferentTimeZone(DateTime p_from_datetime, string p_from_zone, string p_to_zone) 
{ 
    DateTimeZone from_zone = DateTimeZoneProviders.Tzdb[p_from_zone]; 
    var from_local = LocalDateTime.FromDateTime(p_from_datetime); 
    ZonedDateTime from_datetime = from_zone.AtStrictly(from_local); 

    DateTimeZone to_zone = DateTimeZoneProviders.Tzdb[p_to_zone]; 
    return from_datetime.WithZone(to_zone); 
}