使用 std::remove_if() 时没有可行的重载 '='

Question

我正在尝试编写一个随时间滑动的滑动窗口，并将最旧的数据作为模板类删除。我使用 std::map 作为容器和 std::chrono 来操纵时间。我在负责删除早于 X 毫秒的数据的方法中有一个编译时错误：

include/c++/5.4.0/bits/stl_pair.h:170:8: error: no viable overloaded '='
    first = std::forward<first_type>(__p.first);

我不明白为什么std::chrono::steady_clock::time_point没有可行的“运算符 =”。我使用 clang++ 编译器。

这是模板代码：

#include <algorithm>
#include <chrono>
#include <map>
#include <utility>

template <class Data, class Clock, class Time = typename Clock::time_point>

class TimeSlideWindow {
private:
    std::map<Time, Data> mData;

public:    
    void insert(Data value)
    {
        mData.insert(std::make_pair(Clock::now(), value));
    }

    void clearOlderThan(std::chrono::milliseconds ms)
    {
        Time now = Clock::now();
        remove_if(mData.begin(),
                  mData.end(),
                  [ms, now](const std::pair<Time, Data> &elem) {
                      return elem.first < (now - ms);
                  });
    }
};

这里是模板实例化：

TimeSlideWindow<unsigned long, std::chrono::steady_clock> window;
window.clearOlderThan(std::chrono::milliseconds(3));

你能解释一下问题出在哪里吗？谢谢你。

Answer 1

remove_if通过分配工作，它不适用于地图（或集合），因为它们的键是const¹。（它实际上也没有从容器中删除元素，因此是擦除删除成语。）

LFTS v2 就是erase_if为了这个目的。链接页面还显示了您可以使用的实现。

此外，由于map保持其元素按排序顺序，您最好先进行搜索（使用lower_bound），然后进行范围擦除。假设Time比更精确milliseconds，这样now - ms可以转换为Time：

auto x = mData.lower_bound(now - ms);
mData.erase(mData.begin(), x);

_{¹从技术上讲，它不适const用于集合，而集合仅提供const对其元素的访问。}