使用std :: unique_ptr和std :: istream吗？

Question

我正在用c ++编写程序，可以从文件（作为命令行arg传递给它），std :: cin或std :: istringstream接收输入。 它工作正常，并使用std :: istream *管理。 有人告诉我，这很糟糕，处理原始指针，所以我决定将其包装在std :: unique_ptr（即std :: unique_ptr）中。 问题是它不会编译。 据我可以从错误中看出来，std :: istream保护自己免受用来分配内存。 我已经尝试使用Google搜索，但是我不认为之前有人发布过这样的问题（因为我只看到了有关std :: unique_ptr的问题）。 有谁知道如何实现这一目标？

edit: errors: In file included from /usr/include/c++/4.8/iostream:40:0,
                 from /home/dtscode/Desktop/SLang/src/main.cpp:1:
/usr/include/c++/4.8/istream: In function ‘int main(int, char**)’:
/usr/include/c++/4.8/istream:606:7: error: ‘std::basic_istream<_CharT, _Traits>::basic_istream() [with _CharT = char; _Traits = std::char_traits<char>]’ is protected
       basic_istream()
       ^
compilation terminated due to -Wfatal-errors.
make[2]: *** [CMakeFiles/slang.dir/src/main.cpp.o] Error 1
make[1]: *** [CMakeFiles/slang.dir/all] Error 2
make: *** [all] Error 2

响应此行：std :: unique_ptr Stream（new std :: istream（））;

我还尝试了它，而没有调用istreams构造函数，并且在unique_ptrs构造函数中没有任何内容

编辑2：

#include <iostream>
#include  <fstream>
#include  <sstream>
#include   <vector>
#include   <string>
#include   <memory>

#include <lexer.hpp>

int main(int argc, char *argv[]) {
    std::unique_ptr<std::istream> Stream(new std::istream());
    std::vector<std::string> Arguments(argv, argv + argc);

    switch(argc) {
        case 1:
            Stream = &std::cin;
            break;

        case 2:
            Stream = new std::ifstream(Arguments[1]);
            break;

        case 3:
            if(Arguments[1] == "-c") {
                Stream = new std::istringstream(Arguments[2]);
                break;
            }

        default:
            std::cerr<<"error: invalid arguments."<< std::endl;
            return 0;
    }

    Token::Lexeme CurrentToken = Token::Start;

    while(*Stream) {
        CurrentToken = getToken(Stream);
        lex_debug();
    }

    if(Stream != &std::cin) {
        delete Stream;
    }

    return 0;
}

Answer 1

我会说这只是分解系数很差的代码。 为什么不这样：

void process_input(std::istream & is);

int main(int argc, char * argv[]) {
    if (argc == 1) {
        process_input(std::cin);
    } else if (argc == 2) {
        std::ifstream is(argv[1], "rb");
        process_input(is);
    } else if (argc == 3 && strcmp(argv[1], "-c") == 0) {
        std::istringstream is(argv[2]);
        process_input(is);
    } else {
        std::cerr << "Unrecognized invocation.\n";
        return EXIT_FAILURE;
    }
}

Answer 2

您显示的错误是由于std :: istream构造函数调用所致，它是基类，无法实例化。 即使代码可以编译，std :: unique_ptr还是一个拥有的指针，因此尝试删除std :: cin会遇到问题。 您需要一个无所有权的指针，因此原始指针或std :: weak_ptr都是正确的。

编辑：如果可能的话，我建议改用引用。

Answer 3

我经常去选择像这样的流：

#include <iostream>
#include <fstream>
#include <istream>
#include <sstream>
#include <vector>

int main(int argc, char *argv[])
{
    std::vector<std::string> args(argv, argv + argc);

    std::ifstream ifs(argc == 2 ? args[1]:"");
    std::istringstream iss(argc == 3 ? args[2]:"");

    std::istream& is = argc == 2 ? ifs : argc == 3 && args[1] == "-c" ? iss : std::cin;

    while(is)
    {
        // process stream
    }
}

这就是我避免使用指针的方式。