c++ - 堆栈声明段错误

Question

只是我的练习。//读一个包含“()”的句子，并将()中的所有内容替换为“(deleted)”。

1) 将句子读入 vector 。

2) 把vector压入栈中>.
3)找到两个“(”的位置 和“)”
4)删除元素并替换为“(删除)” vector
5) 输出 vector 。

即。输入“this is (a) duck”，输出应该是“this is (deleted) duck”。提前致谢。

#include <iostream>
#include <stack>
#include <string>
#include <vector>
using namespace std;
int main ()
{
    cout<<"please enter a series of words with parenthes"<<endl;
    vector<string> svec;
    string word;
    while (cin>>word)
        svec.push_back(word);
    stack<string, vector<string> > strStack(svec);
    vector<string>::iterator pos1 ,pos2;
    for (vector<string>::iterator counter=svec.end(); strStack.empty()==false; --counter)
    {
        if(strStack.top()==")") pos1=counter-1;
        if(strStack.top()=="(") { pos2=counter-1; break; }
        strStack.pop();     
    }
    svec.insert(svec.erase(pos2,pos1),"(deleted)");
    for(vector<string>::iterator iter=svec.begin(); iter!=svec.end(); )
        cout<<*iter++<<" "<<flush;
return 0;
}

对不起。输出是:我认为问题是 stack > strStack(svec);因为我无法计算堆栈；上面的代码应该没问题(我算出来了)；

please enter a series of words with parenthes
this is (a) duck
Segmentation fault

Answer 1

您的代码完全有问题:它只处理一些非常具体的情况，即只有一个括号组成的单个单词。如果做不到这一点，您永远不会分配 pos1 和 pos2，从而导致我们心爱的“未定义行为”。

至少，确保您永远不会使用未初始化的变量 (#1)。作为奖励，我们检查是否有匹配项(#2):

vector<string>::iterator pos1 = svec.end(), pos2 = pos1; // #1: never uninitialized

// ...

if (pos1 != pos2)   // #2: only if we found something
    svec.insert(svec.erase(pos2,pos1),"(deleted)");