vectorのerase
解決string型配列のvectorと宣言し
std::vector<std::string> rdata[30];
で、このvectorのｎ番目の要素を、削除したいのですが、以下のコードではどこが
まずいでしょうか。
void Foo::remove( int n )
{
std::vector<std::string>::iterator itr = rdata[30].begin();
itr += n;
for(int i=0; i<30; i++)
{
m_rdata[i].erase( itr );
}
}
突っ込みどころだらけなんだが・・・
vector 自体が３０個あるの？それとも
要素数３０個の vector が１個あるの？まずはそこから。
要素数３０個のvectorを宣言し
これにpush_backしています。
> std::vector<std::string> rdata[30];
これだと vector が 30個ありますよ。
ますます突っ込みどころが増えている気のせいがするが・・・
vector<string> v; と書いたら、どう解釈されるかはわかる？
・ v という vector が１個
・デフォルト初期化される（要素数０個）
int i; と書いたのと同じ。
vector<string> a[30]; であれば
・ a という「 vector の配列」が作られ、その配列の要素数は３０
つまり３０個の vector が作られる
・それぞれの vector はいずれもデフォルト初期化される (要素数０)
int x[30]; と同じ。
３０個の要素を持つ vector １つを作りたいのであれば記述方式が違う。
答え書いちゃうのは簡単だけどまずは調べて味噌。
次に vector は要素数可変な配列ということを忘れちゃいけない。
push_back は要素を１個追加する、ということだ。
既に３０個の要素がある vector に push_back すると要素数は３１になる。
初期化時点で要素数０個の vector を作ったとしても、その必要があれば
push_back を繰り返して好きな要素数にできる。
逆に初期化時点で３０個の要素数を確保しておき、それを増減させたくないのであれば
push_back を使うのでなく v[0]=string("abc"); のほうが適切。
# ないしは個数可変機能が無い boost::array を使う
んで本題のｎ個目の要素を削除したい、って案件に対しては erase １回でいい。
どこを(=n番目)削除するか、その iterator を決めるのには std::advance を使う。
これもまた今ここで答え書いちゃうのは簡単だけど、もうちょっと考えて味噌。
要素数３０個の vector から１個要素を削除すると、要素数２９個になるぞ。
それが望みの動作なのかどうか、要検討
すいません。
vectorが３０個でした。
rdata[i].pushu_back(xxx);
とやってます。
rdata.push_backが出来る？なんて、頭になかったもんで適当に答えてしまいました。
rdata[i]のiは状況によって増減します。
で、rdata[i][j]のもj変化します。
この配列のjがn番目のものをすべて削除したいんですが。
と言う事で、最初のコードを試してみたんですが、
この場合だと、std::advanceを使うんでしょうか？
勉強してみます。
> vectorが３０個でした。
ぎゃっふん。
テキトーに問うとテキトーな答えしか返らない/返しようがないのはおｋ？
プログラムなんて１文字違っても動かないんだしさぁ・・・もっと慎重に返事してよ。
rdata[*][n] を削除したい、ということなら
iterator は各 vector に対して１つづつ別に計算する必要があるわけで
for (i=0; i<30; ++i) {
std::vector<std::string>::iterator p(rdata[i].begin());
std::advance(p, n); // p+=n; と実質同じだがこれなら list に対しても通用する
rdata[i].remove(p);
}
// エラーチェック類は省略 たとえば n>=size だったらとか・・・
ご参考:
> for (i=0; i<30; ++i) {
> std::vector<std::string>::iterator p(rdata[i].begin());
> std::advance(p, n); // p+=n; と実質同じだがこれなら list に対しても通用する
> rdata[i].erase(p);
> }
vector以外に通用しなくていいならコレ↓で十分。
for (i=0; i<30; ++i) {
rdata[i].erase(rdata[i].begin()+n);
}
すいません、Ｘ適当に ○単純に
そうか、個別のvectorなので、iteratorも別なんですね。
わかりました。
rdata[i].erase(rdata[i].begin()+n);
このような、書き方もできるんですね。
ありがとう、ございまいした。
| ツイート |  |