使用std :: get_time将time-string转换为std :: time_t:结果错误

时间:2017-09-04 06:56:33

标签: c++ clang++

我想按照时间顺序对照片进行排序。因此,我从EXIF数据中提取时间作为字符串,然后我将其转换为std::time_t。但是我有时会得到错误的结果。我把问题简化为这个最小的例子。它有三个时间串,相隔一秒:

#include <vector>
#include <string>
#include <iostream>
#include <ctime>
#include <iomanip>
#include <sstream>

int main()
{
  std::vector<std::string> vec;

  vec.push_back("2016:07:30 09:27:06");
  vec.push_back("2016:07:30 09:27:07");
  vec.push_back("2016:07:30 09:27:08");

  for ( auto & i : vec )
  {
    struct std::tm tm;
    std::istringstream iss;
    iss.str(i);
    iss >> std::get_time(&tm,"%Y:%m:%d %H:%M:%S");

    std::time_t time = mktime(&tm);

    std::cout << i << " time = " << time << std::endl;
  }
}

使用clang++ -std=c++14 test.cpp编译。

输出:

2016:07:30 09:27:06 time = 1469867226
2016:07:30 09:27:07 time = 1469863627
2016:07:30 09:27:08 time = 1469863628

这显然是错误的,它们应该1分开,这只适用于最后两个?

  

修改

     

由于clang std::get_time(和std::put_time)中似乎存在错误,因此这是我的版本信息:

$ clang --version
Apple LLVM version 8.1.0 (clang-802.0.42)
Target: x86_64-apple-darwin16.7.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

2 个答案:

答案 0 :(得分:3)

Here is another library you could use解决这个问题。这个库是可移植的。如果您有任何问题,我会在几小时内解决。它只是一个标题,没有来源,所以非常容易“安装”。只需将标题放在包含路径的某处:

#include "date.h"
#include <vector>
#include <string>
#include <iostream>
#include <sstream>

int main()
{
  std::vector<std::string> vec;

  vec.push_back("2016:07:30 09:27:06");
  vec.push_back("2016:07:30 09:27:07");
  vec.push_back("2016:07:30 09:27:08");

  for ( auto & i : vec )
  {
    date::sys_seconds tm;
    std::istringstream iss{i};
    iss >> date::parse("%Y:%m:%d %H:%M:%S", tm);

    std::cout << i << " time = " << tm.time_since_epoch().count() << std::endl;
    std::cout << date::format("%Y:%m:%d %H:%M:%S\n", tm);
  }
}

输出:

2016:07:30 09:27:06 time = 1469870826
2016:07:30 09:27:06
2016:07:30 09:27:07 time = 1469870827
2016:07:30 09:27:07
2016:07:30 09:27:08 time = 1469870828
2016:07:30 09:27:08

答案 1 :(得分:2)

正如我在评论中所说,你的所有输出都是错误的。这似乎是clang中的一个bug。我在the Bugzilla page of the LLVM project上搜索但没有找到任何结果。也许你想用你的示例代码提交错误报告。

解决方法是使用std::sscanf()手动解析字符串并填充std::tm结构。以下代码为您的输入字符串i执行此操作。查看整个example with the output of std::get_time() and the std::sscanf() method at ideone。如果字符串不同,那么根据您的需要调整该解决方案。

此解决方案使用%d:%d:%d %d:%d:%d作为格式字符串。您也可以使用%4d:%2d:%2d %2d:%2d:%2d,它只接受长度小于或等于%d之间数字的数字。

<强>输出:

2016:07:30 09:27:06 | sscanf() time =        1469870826
2016:07:30 09:27:06 | std::get_time() time = 1469870826
2016:07:30 09:27:07 | sscanf() time =        1469870827
2016:07:30 09:27:07 | std::get_time() time = 1469870827
2016:07:30 09:27:08 | sscanf() time =        1469870828
2016:07:30 09:27:08 | std::get_time() time = 1469870828

<强>代码:

#include <vector>
#include <string>
#include <iostream>
#include <ctime>
#include <iomanip>
#include <sstream>
#include <cstring>

int main()
{
   std::vector<std::string> vec;

   vec.push_back("2016:07:30 09:27:06");
   vec.push_back("2016:07:30 09:27:07");
   vec.push_back("2016:07:30 09:27:08");

   for (auto & i : vec)
   {
      struct std::tm tm;

      /* std::sscanf() method: */
      std::memset(&tm, 0, sizeof(tm));
      if (6 != std::sscanf(i.c_str(), "%d:%d:%d %d:%d:%d",
                           &tm.tm_year, &tm.tm_mon, &tm.tm_mday,
                           &tm.tm_hour, &tm.tm_min, &tm.tm_sec))
      {
         return -1;
      }

      /* correct the numbers according to:
       * see: http://en.cppreference.com/w/cpp/chrono/c/tm */
      --tm.tm_mon;
      tm.tm_year -= 1900;
      /* mktime determines if Daylight Saving Time was in effect
       * see: http://en.cppreference.com/w/cpp/chrono/c/mktime */
      tm.tm_isdst = -1;

      std::time_t time = std::mktime(&tm);

      std::cout << i << " | sscanf() time =        " << time << std::endl;

      /************************************************************/

      /* std::get_time() method: */
      std::istringstream iss;
      iss.str(i);
      iss >> std::get_time(&tm, "%Y:%m:%d %H:%M:%S");

      time = std::mktime(&tm);

      std::cout << i << " | std::get_time() time = " << time << std::endl;
   }
}

有必要减去其中一个tm_mon,因为它从0开始。tm_year成员自1900年以来就是年。请参阅description of std::tm here

此外,需要将tm_isdst设置为-1,让std::mktime()确定夏令时是否生效。

要将std::time_t Linux timestamp转换回您指定格式的std::string%Y:%m:%d %H:%M:%S,您可以将std::strftime()std::localtime()一起使用。 See live example on ideone

<强>输出:

time = 1469870826 | 2016:07:30 09:27:06

<强>代码:

#include <vector>
#include <string>
#include <iostream>
#include <ctime>

int main()
{
   char buff[20];
   time_t timestamp = 1469870826;
   std::strftime(buff, sizeof(buff), "%Y:%m:%d %H:%M:%S", std::localtime(&timestamp));
   std::string timeStr(buff);

   std::cout << "time = " << timestamp << " | " << timeStr;
}