2017-09-25 45 views
1

此函数是一项正在进行中的工作,用于从stdin读取简单整数计算器的数据。在输入一个操作符和两个操作数时,该函数按预期工作:+ 2 2.它也可以正确读取命令行中输入的具有相似格式的文件。尽管如此,如果用户在命令行上输入或达到文件末尾,它将会分段。我不确定我哪里出了问题?什么可能导致这种行为?只有当您按Enter键或到达文件末尾时,函数才会导致分段错误

void process_input() 
{ 

while(!std::cin.eof()) 
{ 
    std::string line; 
    std::string n; 


    getline(std::cin, line); 


    if (std::cin.fail()) 
     break; 
    else if (line == "quit") 
     return; 

    std::istringstream stream(line); 
    std::vector<std::string> input_cpy(0); 
    while (stream >> n) 
     input_cpy.push_back(n); 

    //Clear global_input_cpy on each iteration and load the new line into it 
    global_input_cpy.clear(); 
    for (int i = 0; i < input_cpy.size(); ++i) 
     global_input_cpy.push_back(input_cpy[i]); 



    if(input_cpy[0] == "+") 
     sum(std::stol(input_cpy[1]), std::stol(input_cpy[2])); 
    else if (input_cpy[0] == "*") 
     multiply(std::stol(input_cpy[1]), std::stol(input_cpy[2])); 
    else if (input_cpy[0] == "^") 
     exponent(std::stol(input_cpy[1]), std::stol(input_cpy[2])); 
    else 
     std::cout << "input '" << input_cpy[0] << "'" << " unrecognized. skipping." << "\n"; 

    input_cpy.clear(); 
} 
} 
+0

请提供一个完整的例子。未声明'global_input_cpy',未定义'sum','multiply','exponent'。 –

+0

'while(!std :: cin.eof())'可能不会在这里咬你,但知道它可能会导致有趣的错误和大量的头发拉动。更多在这里:[为什么iostream :: eof内部循环条件被认为是错误的?](https://stackoverflow.com/questions/5605125/why-is-iostreameof-inside-a-loop-condition-considered-wrong) – user4581301

回答

4

如果没有字符串被读取,因此input_cpy是空的,你还是尝试访问input_cpy[0],这将产生不确定的行为。

你需要一个说法的地方,input_cpy.size() >= 2为了使这些陈述已经定义的行为:

if(input_cpy[0] == "+") 
    sum(std::stol(input_cpy[1]), std::stol(input_cpy[2])); 
else if (input_cpy[0] == "*") 
    multiply(std::stol(input_cpy[1]), std::stol(input_cpy[2])); 
else if (input_cpy[0] == "^") 
    exponent(std::stol(input_cpy[1]), std::stol(input_cpy[2])); 
+0

像魅力一样工作!谢谢! –