C++構造体のベクタを初期化する正しい方法

Question

私はここでたくさん検索しましたが、C++の高度な初心者のための私の正しい説明はありません。以前は構造体のベクトルで作業していましたが、現在はセグメンテーションフォールトが発生しています...

このようなオブジェクトが実際にどのように機能するのか、それが正しい方法であるかを知りたいのですが！私はそれのサイズを変更し、値でそれを埋める前に

std::vector<numberOfSpecies> size; 

：

は私が

struct numberOfSpecies { 
    int predator; 
    int prey1; 
    int prey2; 
}; 

のような構造体と、それのベクトルを持っています。

size.resize(100); 

これは実際に何をしていますか？これは構造体のための権利ですか？ それがゼロで初期化されているように見えます...

は今、私はこのようなをしています：

size[t].predator=0; 
size[t].prey1=0; 
size[t].prey2=0; 

for(int k = 0; k < N; ++k){ 
    size[t].predator++; 
    size[t].prey1++; 
    size[t].prey2++; 
} 

は、この権利ですか？起こりうる問題はどこですか？それをより良くするには？

Answer 1

ここで、最も簡単で「正しい」解決策はただ、

size.resize(100,{0,0,0}); //aggregate initialization 

for(int k = 0; k < N; ++k) 
{ 
    size[t].predator++; 
    size[t].prey1++; 
    size[t].prey2++; 
} 

のようなものを集計初期化（あなたがC++ 11と上へのアクセス権を持っている場合）とベクトルオブジェクトに属し resize()機能を使用することが考えられます

各numberOfSpeciesオブジェクトのすべてのメンバーは0に初期化されます。

Answer 2

デフォルトコンストラクタを構造体に追加できます。新しいコードは次のようになります：

struct numberOfSpecies { 
    numberOfSpecies(): predator(0), prey1(0), prey2(0) { } // default constructor 
    int predator; 
    int prey1; 
    int prey2; 
}; 

このように、サイズ変更が適用されると、構造体はベクター内で適切に初期化されます。

Answer 3

これ：

size[t].predator=0; 
size[t].prey1=0; 
size[t].prey2=0; 

はsizeのt番目の要素にゼロ書き込む - または有用であってもなくてもよい。

この：

for(int k = 0; k < N; ++k){ 
    size[t].predator++; 
    size[t].prey1++; 
    size[t].prey2++; 
} 

はt thをインクリメントします要素はsize N回です。これは非常に有用であるとは思われません。

size[0].predator=0; // Technically not needed because .resize() 
size[0].prey1=0;  // will have initialized it to zero anyway 
size[0].prey2=0;  // *BUT* explicit is always better than implicit. 

// Initialize each element of size to be one greater than previous. 
for(int k = 1; k < N; ++k){ 
    size[k].predator = size[k-1].predator + 1; 
    size[k].prey1 = size[k-1].prey1 + 1; 
    size[k].prey2 = size[k-1].prey2 + 1;; 
}