c++测试一个文件是否比一组文件旧

Question

我正在为某些数据创建缓存，但当然，如果创建缓存的任何源文件被修改，我希望缓存变得无效。为此，我创建了这个函数：

bool CacheIsValid(
    const std::string& cache_shader_path,
    const std::vector<std::string>& shader_paths)
{
    // This is really messy because of the std stuff, the short of it is:
    // Find the youngest file in the shader paths, then check it against the
    // timestamp of the cached file.
    std::time_t youngest_file_ts = std::time(nullptr);
    for(auto file : shader_paths)
    {
        std::time_t current_timestamp =
            std::chrono::system_clock::to_time_t(
                std::chrono::file_clock::to_sys(fs::last_write_time(file)));
        double time_diff = difftime(youngest_file_ts, current_timestamp);
        if(time_diff > 0) youngest_file_ts = current_timestamp;
    }

    // All this is doing is comparing the youngest file time stamp with the cache.
    return fs::exists(cache_shader_path)
        && (difftime(youngest_file_ts, std::chrono::system_clock::to_time_t(
            std::chrono::file_clock::to_sys(fs::last_write_time(cache_shader_path)))) < 0);
}

我不知道我做错了什么，但即使输入文件被修改，它也总是返回 true。我正在使用 stat 检查时间戳，并且磁盘上的文件客观上比缓存文件更年轻，我还测试了该函数的输入是否正确，而且确实如此。

Answer 1

在我看来，你似乎经历了很多困难，这使得事情变得比实际需要的更加困难。 filesystem::last_write_time 返回一个 time_point¹ ，它支持比较。因此，至少据我所知，没有理由对 time_t 进行漫长的转换，然后使用 difftime 进行比较。

如果我理解您正在做的事情的想法，您想要确定该一个文件至少与矢量中指定的任何文件一样新。为了更直接地做到这一点，我会沿着这条总路线编写代码：

bool isNewer(std::string const &file, std::vector<std::string> const &otherFiles) {
    auto newest = std::max_element(otherFiles.begin(), otherFiles.end(), 
        [&](std::string const &a, std::string const &b) {           
            return fs::last_write_time(a) > fs::last_write_time(b);
        });

    return fs::last_write_time(file) >= fs::last_write_time(*newest);
}

我可能误解了您想要完成一个或两个比较的方向，在这种情况下，这可能不正确 - 但如果是这样，请更改比较(s) 以满足您的要求应该是微不足道的。

---

1. 其中，someclock 是满足 C++17 的一些指定要求的任意类型，或者是满足 C++20 的 std::chrono::file_clock 的任意类型。

Answer 2

因为你想找到 youngest_file_ts ->查找正在更改的文件的最近时间戳（更大的数字），但是

double time_diff = difftime(youngest_file_ts, current_timestamp);
if(time_diff > 0) youngest_file_ts = current_timestamp; // find greater number one

在 for 循环之后 youngest_file_ts 是正在更改的文件的最旧时间戳
因此

(difftime(youngest_file_ts, std::chrono::system_clock::to_time_t(
          std::chrono::file_clock::to_sys(fs::last_write_time(cache_shader_path)))) < 0) alway true.

它应该像这样改变

    if (shader_paths.empty()) {
        return false
    } else {
//initialize youngest_file_ts as last_write_time of first element in shader_paths
        std::time_t youngest_file_ts = std::chrono::system_clock::to_time_t(
                 std::chrono::file_clock::to_sys(fs::last_write_time(shader_paths.at(0))); 
        for (auto file : shader_paths)
        {
            std::time_t current_timestamp = std::chrono::system_clock::to_time_t(
                std::chrono::file_clock::to_sys(fs::last_write_time(file)));
            double time_diff = difftime(youngest_file_ts, current_timestamp);
            if (time_diff < 0) youngest_file_ts = current_timestamp;
        }
        // All this is doing is comparing the youngest file time stamp with the cache.
        return fs::exists(cache_shader_path)
            && (difftime(youngest_file_ts, std::chrono::system_clock::to_time_t(
                std::chrono::file_clock::to_sys(fs::last_write_time(cache_shader_path)))) < 0);
    }