私はベクトル<int *>に要素を追加するとき、私はここにいくつかの問題

Question

を満たすためには、私のコーディングです：

std::vector<int *>a; 
for(i=0;i<3;i++) 
{ 
    int j=i+1; 
    int *p=&j; 
    a.push_back(p); 
} 
for(auto &c:a) 
{ 
    std::cout<<*c<<std::endl; 
} 

私は出力が1,2,3ことにしたいが、出力が3,3,3であります私は同じポインターをaに追加したようですが、循環に別のポインターを追加したいのですが、私は何をすればいいのですか？

Answer 1

jは自動ストレージ期間を有し、そしてそれへのポインタは、forループの特定反復においてのみ有効です。

つまり、というダングリングポインタのベクトルを作成しました。このようなポインタを逆参照する動作は、未定義です。

代わりにstd::vector<int>とします。

（あなたは出力で繰り返し同じ値を参照するという事実は、おそらくにより同じメモリ位置にjを再作成するC++ランタイムにである。しかし、ははそれに依存しないでください。C++標準が主張していません

Answer 2

std::vector<int *>a; 
for(i=0;i<3;i++) 
{ 
    int j=i+1; 
    int *p=&j; 
    a.push_back(p); 
} 

pはスタック内のアドレスです。さらに、jとpは、ループの各反復でのみ有効です。ループが次の反復に進むか、終了すると、もはや有意ではありません（jは破棄されます）。

実際にポインタのベクトルが必要な場合は、ポインタを有効にする必要があります。 1つのオプションは、ポインタがヒープ内のオブジェクトを指し示すことです。もう1つの選択肢は、ポインタが、スコープから外れることのない、すなわち決して破壊されないスタック内のオブジェクトを指すことである。

もう1つ、おそらくより良いオプションは、オブジェクト自体のベクトルを持つことです。 vector<int>。