Question

编写程序定义一个vector对象，其每个元素都指向string类型的指针，读取该vector 对象，输出每个string内容

和相应的长度，以下是源代码：
#include<iostream>
#include<vector>
//#include<cstring>
#include<string>
using namespace std;
int main()
{
vector<string*>vec;
string str;
string *p=new string;
while(cin>>str&&str!="0"){
*p=str;
vec.push_back(p);
}

for(vector<string*>::iterator iter=vec.begin();iter!=vec.end();++iter)
cout<<**iter<<" "<<(**iter).size()<<" ";
cout<<endl;
for(vector<string*>::iterator iter1=vec.begin();iter1!=vec.end();++iter1)
delete [] *iter1;
delete [] p;
return 0;
}
编译通过了，但是运行时出现错误，麻烦大虾帮忙修改以下，谢谢！

Answer 1

#include<iostream>
#include<vector>
//#include<cstring>
#include<string>
using namespace std;
int main()
{
vector<string*>vec;
string str;

while(cin>>str&&str!="0"){
string *p=new string(str);
vec.push_back(p);
}

for(vector<string*>::iterator iter=vec.begin();iter!=vec.end();++iter)
cout<<**iter<<" "<<(**iter).size()<<" ";
cout<<endl;
for(vector<string*>::iterator iter1=vec.begin();iter1!=vec.end();++iter1)
delete *iter1;
return 0;
}

追问

我的源代码是哪出问题了，还望指教！

追答

#include
#include
//#include
#include
using namespace std;
int main()
{
vectorvec;
string str;
string *p=new string;//要为每一个字符串分配一个对象，不能只分配一次，否则vector中所有元素都指向同一个对象
while(cin>>str&&str!="0"){
*p=str;
vec.push_back(p);
}

for(vector::iterator iter=vec.begin();iter!=vec.end();++iter)
cout::iterator iter1=vec.begin();iter1!=vec.end();++iter1)
delete [] *iter1;//因为分配时用的是new ，删除时要用delete而不是delete[]
delete [] p;//多余的，上面已经释放了，这里不用再释放
return 0;
}

追问

你在分配内存的时候没有*p=str 这段语句，但是执行的时候却是正确的，我想问下这为什么不用这条语句呢？还有用delete,和delete []之间具体的区别在哪？我一般都是用的delete [],请赐教...

追答

1.我没有使用*p=str是因为我使用了构造函数 string *p=new string(str);，这样对象在创建的时候就使用str来初始化了，其实也可以麻烦一点写成两句：string *p=new string; *p = str;结果是一样的，只是一个简洁，一个罗嗦。

2.delete和new对应，delete[]和new[]对应，乱用容易出问题。
使用示例：
int *ptr = new int[6]; delete[] ptr;
int *ptr2 = new int; delete *ptr2;