【发布时间】:2019-12-10 03:23:28
【问题描述】:
我正在尝试熟悉 ranges-v3 库,它将成为 C++20 标准的一部分。为此,我试图通过用新的可用构造替换(在合适的情况下)经典迭代器和算法来重构一些玩具代码。在这个特定的示例中,我无法弄清楚如何将返回的对 ranges::min_element 的调用(它取代了对 std::min_element 的调用)的调用传递给我的另一个函数,该函数接受经典的 iterator 作为参数。
我在 documentation of the library 中搜索了一些类似 smartIt2classicIt 的函数,但没有成功。
这是一个最小的例子
void f(std::vector<int>& v, std::vector<int>::iterator it); // old function that I want to reuse
auto predicate = [](int i){ return true; }; // check function
std::vector<int> v;
// auto min_el = std::min_element(...); // old code
auto filtered_range = v | ranges::view::filter(predicate); // to avoid a dangling iterator
auto min_el = ranges::min_element(filtered_range);
f(v, min_el); // pass min_el to f: doesn't compile with the new code
起初我以为ranges::min_element 的结果可以隐式转换为经典迭代器,但我错了:编译器返回一个长错误,说cannot convert a ranges::basic_iterator bla bla bla to a std::vector bla bla bla iterator。鉴于此错误,我推断ranges::min_element 确实返回了某种迭代器,但是如何以旧方式使用它?
我看到了三种可能的解决方案:
- 更改
min_el传递给f的方式 - 更改
f的第二个参数的类型(可能以向后兼容的方式) - 改变这两件事
但我不知道如何实现它们。也许还有另一种解决方案?我还看到了另一个可能的麻烦来源,因为返回的迭代器可能指的是filtered_range而不是v...欢迎任何帮助!
【问题讨论】:
-
"这将成为 C++20 标准的一部分" 它不会。不是全部。你应该注意what is actually in C++20。
-
@NicolBolas 你的措辞很不友好......
-
想象一下你做了类似
min_el = v | takeEverySecondElement()的事情。递增min_el当然应该在向量内推进两个元素,但f大概期望传递的迭代器只推进一个元素。也许您真正想要的是向量的索引?基本上,试着在心理上改变问题的各个部分,看看你会期望什么以及它暴露了哪些要求/冲突(例如,如果你有一个std::map而不是一个向量怎么办?) -
请注意,
f获取v的副本(您通过值传递它)。这是故意的吗?这意味着您传递给f的迭代器指向的向量与f内部的向量不同。 -
@sebrockm 这是一个错字,我正在编辑问题。谢谢你的来信!