2012-05-02 10 views
-1

私は構造体のリストを作成しています。リストのメンバーの1つを新しい値で更新しようとしていて、リストに戻しています。intをインクリメントするとき、ベクトルの添え字が範囲外ですか?

私は構造体の値をイテレータでコピーできているようですが、構造体のメンバ(int ++を使用)の値を更新しようとすると、すべてのもののベクトルクラスに例外がスローされます。

ここで何が起こっているかについての説明は役に立ちます。

struct Blob 
{ 
    int x; 
    int y; 
}; 

list<Blob> blob; 
// Add a Blob to blob using .push_back(); here 
for(list<Blob>::iterator iterator=blob.begin(); iterator!=blob.end(); ++iterator) 
{ 
    Blob temp; 
    temp.x = ((Blob)*iterator).x; 
    temp.y = ((Blob)*iterator).y; 
    if (temp.x < 10 - 1) 
     temp.x++; /* Exception: vector: line 932 - "Vector subscript out of range" */ 
    ((Rain)*iterator) = temp; 
} 
+6

をこれは自分自身であり、これについてのベクトルを含んでいません。 – PlasmaHH

+2

'((Rain)* iterator)= temp;行とは何ですか? 「雨」とは何ですか、なぜあなたはキャストしていますか? – interjay

+3

「temp.x = iterator-> x'」と言ってみませんか? (キャストは必要ありませんが、それはあなたの問題ではありません) – Benj

答えて

0

オブジェクトの既存の値を更新する場合は、その値を参照してください。私は同じことを説明するためのサンプルコードを書いています

#include<list> 
#include<iostream> 
using namespace std; 
struct Test 
{ 
    int x; 
    int y; 
}; 

int main() 
{ 
    list<Test> lTest; 
    int i = 0; 
    for(i=0;i<5;i++) 
    { 
     Test t1; 
     t1.x = i; 
     t1.y = i*i; 
     lTest.push_back(t1); 
    } 

    list<Test>::iterator lIter = lTest.begin(); 
    for(;lIter != lTest.end();++lIter) 
    { 
     Test &t1 = *lIter; 
     cout<<"1 Val is:"<<t1.x<<"|"<<t1.y<<endl; 
     t1.x += 2; 
     t1.y += 2; 
     cout<<"2 Val is:"<<t1.x<<"|"<<t1.y<<endl; 
    } 

    lIter = lTest.begin(); 
    for(;lIter != lTest.end();++lIter) 
    { 
     Test t1 = *lIter; 
     cout<<"3 Val is:"<<t1.x<<"|"<<t1.y<<endl; 
    } 

    return 0; 
} 
+3

参照を取るのはどうしてですか? 'lter-> x + = 2'の何が間違っていますか? – Benj

+0

それは良いアイデアです..... iterで作業する方がはるかに優れています。 – Raghuram

0

もしあなたがループを書いているのであれば、それを行う可能性があります。あなたは使用することができstd::for_each

#include <list> 
#include <algorithm> 

struct Blob 
{ 
    int x; 
    int y; 
}; 

void incrementXIfLessThanNine(Blob& blob) 
{ 
    if(blob.x < 9) 
    { 
     blob.x++; 
    } 
} 

int main() 
{ 
    std::list<Blob> blobs; 
    std::for_each(blob.begin(), blob.end(), incrementXIfLessThanNine); 
    return 0; 
} 

あなたがC++ 11使用している場合:コンパイルし、同じ動作が無効であるコードスニペットよりもはるかに有用であろう発揮する完全なテストケースを表示

#include <list> 

struct Blob 
{ 
    int x; 
    int y; 
}; 

int main() 
{ 
    std::list<Blob> blobs; 
    for(Blob& blob: blobs) 
    { 
     if(blob.x < 9) 
     { 
      blob.x++; 
     } 
    } 
    return 0; 
} 
関連する問題