1 回答

TA貢獻1773條經驗 獲得超3個贊
對于vector一般不要用erase(),因為很多情況下他要和<algorithm>中的remove()一塊用!erase()的使用會使迭代器失效如果刪除的不是最后面的元素的話。你的程序中if(*iter%2==0)
ivec.erase(iter);
可以換成:(記著加頭文件<algorithm>)
if
(*iter%2
==
0)
ivec.erase(remove(ivec.begin(),
ivec.end(),
*iter),
ivec.end())
但如果你用if
(*iter%2
!=
0)
ivec.erase(remove(ivec.begin(),
ivec.end(),
*iter),
ivec.end())
輸出結果則包含11,結果錯誤!
不知道你為什么數組,向量,鏈表都用,為了和你一致,我也只能改成這樣:
#include
<iostream>
#include
<vector>
#include
<list>
#include
<algorithm>
#include
<functional>
#include
<iterator>
using
namespace
std;
int
main
()
{
vector<int>
ivec;
list<int>
ilis;
int
vec[]
=
{2,
3,
4,
5,
6,
7,
8,
9,
10,
11};
remove_copy_if
(vec,
vec+10,
back_inserter(ivec),
bind2nd(modulus<int>(),
2));
remove_copy_if
(vec,
vec+10,
back_inserter(ilis),
not1(bind2nd(modulus<int>(),
2)));
for
(vector<int>::const_iterator
iter
=
ivec.begin();
iter
!=
ivec.end();
++iter)
cout
<<
*iter
<<
'\t';
cout
<<
endl;
for
(list<int>::const_iterator
liter
=
ilis.begin();
liter
!=
ilis.end();
++liter)
cout
<<
*liter
<<
'\t';
cout
<<
endl;
}
最好的做法是:
#include
<iostream>
#include
<vector>
#include
<list>
#include
<algorithm>
#include
<functional>
#include
<iterator>
using
namespace
std;
int
main
()
{
vector<int>
ivec,
odd_vec,
even_vec;
for
(int
i
=
2;
i
!=
12;
++i)
ivec.push_back(i);
remove_copy_if
(ivec.begin(),
ivec.end(),
back_inserter(even_vec),
bind2nd(modulus<int>(),
2));
remove_copy_if
(ivec.begin(),
ivec.end(),
back_inserter(odd_vec),
not1(bind2nd(modulus<int>(),
2)));
copy
(even_vec.begin(),
even_vec.end(),
ostream_iterator<int>(cout,
"\t"));
cout
<<
endl;
copy
(odd_vec.begin(),
odd_vec.end(),
ostream_iterator<int>(cout,
"\t"));
cout
<<
endl;
}
- 1 回答
- 0 關注
- 151 瀏覽
添加回答
舉報