Как конвертировать unix time в std::string?

Question

[Дмитрий]

Основная цель: сконвертировать UNIX time в std::string


Когда я попытался заюзать подобный код:

std::string unixtimeToString( time_t ts )
{
    // _CRT_SECURE_NO_WARNINGS
    enum  { numCStringSize = 0x80 };
    const char strTimestampFmt[] = "%a, %Y-%b-%d, %H:%M:%S";

    char timeCStr[ numCStringSize ];

    std::tm * tmInfo = gmtime( &ts );

    std::strftime( &timeCStr[0], numCStringSize, strTimestampFmt, tmInfo );

    return std::string( timeCStr );
}

То получил справедливый варнинг от MS Visual Studio 2008 компиллера с кодом 4996.


Поэтому переписал на более безопасный:

std::string  unixtimeToString( uint32_t timeStamp )
{
    using namespace boost::posix_time;
    using namespace boost::gregorian;

    ptime unixEpoche( date( 1970, Jan, 1 ) );
    ptime ts(  unixEpoche + time_duration( 0, 0, timeStamp ) );

    return to_simple_string( ts );
}

Но при таком коде моя тулза работающая на большом кол-ве файлах(папка c:\windows\ на Win 7 x64 sp1) начала притормаживать.


Сейчас передо мною три варианта:

1) Подавить сообщение о варнинге 4996 #pragma warning(push) -> #pragma warning(disable:4996) -> мой код -> #pragma warning(pop)

2) Подавить на уровне проекта, заюзав в опциях проекта _CRT_SECURE_NO_WARNINGS

3) Тоже самое что и п.1, но вынести отключение\включение во внешние инклуды вида disable_warning\restore_warning и это даст возможность еще и портабельно варнинги отключать.


Мне оба первых варианта не нравятся. В случае п.2 приходится согласиться на то что компиллер совсем меня не будет извещать о возможных багах. В случае п.1 падает читабельность кода, пусть даже и не намного, но все равно как-то не хочется.


В виду того что в голове каша хочется спросить у других кто как поступал?


ЗЫ:

Сильно не бейте ;)

Comments

[skor]

А почему вы не хотите воспользоваться советом компилятора и воспользоваться gmtime_s?

std::string unixtimeToString( time_t ts )
{
    const size_t numCStringSize = 0x80;
    const char strTimestampFmt[] = "%a, %Y-%b-%d, %H:%M:%S";

    char timeCStr[ numCStringSize ];

    std::tm tmInfo;
    if (gmtime_s( &tmInfo, &ts ) != 0)
        return "invalid time";

    std::strftime( timeCStr, numCStringSize, strTimestampFmt, &tmInfo );

    return timeCStr;
}

P.S.
Объявлять константы через enum — это какой-то C-style.

[Дмитрий]

Потому что это привязка к MS

[Дмитрий]

Хотя Ваш вопрос привел к найденному в гугле gist.github.com/2701145

Answer 1

[bogolt]

По-моему самое разумное вынести код функции в отдельный файл, и обернуть его pragma push/pop с отключением надоедливого ворнинга.

Насчет медленной работы вашей программы используя преобразование через буст — это немного странно ведь если вы обращаетесь к каждому файлу а затем преобразовываете время, то очевидно что именно обращение к файловой системе должно быть узким местом.

Comments

[Дмитрий]

Да, именно это меня и смутило. Добавления фичи тулза на виндовой папке работала где-то 50 мин., после добавления стала 2.5 часа, начал смотреть, из всех сложных операций в фиче это конвертация времени.

[bogolt]

А вы как делаете — открываете первый файл, проводите с ним все операции, потом следующий… или вначале открываете все файлы, сохраняете данные а потом в уже с данными проводите все нужные операции?

[Дмитрий]

Тулза работает с одним файлом, а в целях тестирования она гоняется в питон скрипте на заданной папке переданной в аргументе ком. строки скрипта.

[bogolt]

Хм, думаю правильно было бы пройтись профайлером, либо добавить код который будет измерять скорость выполнения разных функций.

[Дмитрий]

Измерено, перепроверено. Пожирание найдено. Было устроено так:
struct Item
{
//other type
std::string timestampUTC0;
}

В цикле после операции TimeUtils::unixtimeToString( tm ); присваивалось в этот Item.timestampUTC0, затем в itemlist.push_back( item )

Сухой остаток: строка std::string достаточно часто создавалась

Чуть переписал алгоритм, стало значительно быстрее.

Спасибо за совет глянуть в результат профайлера