本地时间替代方案不会覆盖提供的结构

Question

本质上，我想做的是检查文件的上次访问时间并将其与字符串进行比较。这是相关的块：

struct stat file;
char timeStr[ 100 ];

stat(nodes.at(0), &file);
strftime(timeStr, 100, "%H:%M:%S-%m/%d/%y", localtime(&file.st_atime)); /* problem */

nodes 是文件路径的向量；我不确定它是否相关，但我将包含用于设置节点的代码：

vector<char*> nodes;
DIR *dir;
struct dirent *cur

if((dir = opendir(searchPath.c_str())) == NULL) {
    cout << "Error opening search path. Are you sure '" 
        << searchPath.c_str() << "' is a valid search path?" << endl;
    return 0;
}
while((cur = readdir(dir)) != NULL) {
    if(string(cur->d_name) == "." || string(cur->d_name) == "..") continue;
    nodes.push_back(cur->d_name);
}
closedir(dir);

其中 searchPath 是用户输入的字符串。

问题：当“问题”行运行时，节点上的节点是垃圾向量。我想知道是否可以完成这项任务而不将节点变成垃圾。

由于这是家庭作业，并且正如您可能会告诉我的那样，我不习惯 C++，因此如果朝着正确的方向坚定地推动，将会得到“接受”。

谢谢。

Answer 1

它与您的 strftime 调用无关，但与以下事实有关：（来自 此处）：

readdir() 返回的指针指向的数据可能会被同一目录流上的另一个 readdir() 调用覆盖。

由于您只是推送一个指向可能被后续调用 readdir 覆盖的数据的字符指针，因此您很可能会得到垃圾。

您可以通过使用 C 字符串的副本来修复它，例如：

nodes.push_back (strdup (cur->d_name)); // plus error handling if need be.

并且，如果您的实现没有 strdup （它不是标准的一部分） ，你可以使用我的（发现此处）。

Answer 2

nodes.push_back(cur->d_name);

您在向量中存储的指针立即变得无效（cur 在下一次 readdir 或 closedir 调用之前一直有效）。最好的解决方法是编写您想要的代码 - 将 nodes 设为 string 的向量。最简单的修复：

nodes.push_back(strdup(cur->d_name));