C ++ range-v3：尝试将转换链接在一起

Question

我对范围库完全陌生，所以我不应该对这段代码没有编译并且我不知道为什么感到惊讶：

#include <iostream>
#include <algorithm>
#include <fstream>
#include <iterator>
#include <vector>

#include <range/v3/all.hpp>
#include <range/v3/view/all.hpp>
using namespace ranges::v3;


std::ifstream open_file(const std::string &filename) {
    return std::ifstream{filename};
}

int count_lines(std::ifstream &in) {
    return std::count(std::istreambuf_iterator<char>{in},
                      std::istreambuf_iterator<char>{}, '\n');
}

std::vector<int>
count_lines_in_files(const std::vector<std::string> &filenames) {
    auto a1 = filenames | view::transform(open_file) | view::transform(count_lines);
    return a1;
}

int main() {
    const std::vector<std::string> files{"listing1_1.cpp",
                                         "listing1_2.cpp",
                                         "listing1_4.cpp",
                                         "listing1_5.cpp"};
    const auto result = count_lines_in_files(files);
    std::cout << ranges::view::all(result) << '\n';
}

看来投诉是关于a1，编译器告诉我“错误：变量的类型不完整'void'。”

谁能看到我做错了什么，或者告诉我如果可能的话如何正确地将它们链接在一起？

提前致谢！

Answer 1

正如Porsche9II 所述，“std::ifstream 没有复制构造函数”。您可以在此处找到有关此主题的更多信息：

Why are iostreams not copyable?

C++11 为std::basic_ifstream 引入了一个移动构造函数（6），所以你可以写

auto open_file(const std::string &filename) {
    return std::ifstream{filename};
}

auto count_lines(std::ifstream &&in) {
    return std::count(std::istreambuf_iterator<char>{in},
                      std::istreambuf_iterator<char>{}, '\n');
}

可测试HERE。

Answer 2

std::ifstream 没有复制构造函数 - 通过函数返回 std::ifstream 不是一个好主意。一种可能的解决方案：打开和计数应该在一个函数中进行。