C++での2Dベクトルの初期化

Question

2次元ベクトルで2D行列を入力したいと思っています。しかし、私が下のようにしている間、プログラムはクラッシュした!!!しかし、なぜ？

int m, n; 
cin>>m>>n; 

vector<vector<int> > v; 

for(int i=0; i<m; i++) 
{ 
    for(int j=0; j<n; j++) 
    { 
     int a; 
     cin>>a; 
     v[i].push_back(a); 
    } 
} 

私のコードに従った解決策は何でしょうか。

v[i].push_back(a);

Answer 1

がforループ内で呼び出される時点で、ベクターvは空であり、v[i]を使用して、未定義の動作につながります。

あなたのコードは、最初べきpush_backvector<int>：あなたは行列の大きさを知っているので

int m, n; 
cin>>m>>n; 
vector<vector<int> > v(m); 

for(int i=0; i<m; i++) 
{ 
    for(int j=0; j<n; j++) 
    { 
     int a; 
     cin>>a; 
     v[i].push_back(a); 
    } 
} 

Answer 2

だけサイズを変更V：

for (int i=0; i<m; i++) 
{ 
    v.push_back(vector<int>()); 
    for (int j=0; j<n; j++) 
    { 
     int a; 
     cin >> a; 
     v[i].push_back(a); 
    } 
} 

代替は、明示的にサイズmにベクトルvを初期化するだろうあなたがそれを記入する前に：

vector<vector<int> > v(m); 
for(int i=0; i<m; i++) 
{ 
    v[i].resize(n); 
    for(int j=0; j<n; j++) 
    { 
     cin >> v[i][j]; 
    } 
} 

Answer 3

あなたはVを呼び出すときに[i]が構築されていない

V [i]を.push_back（）、あなただけのV first.itのシンプルを初期化する必要があります。

vector<vector<int>> v; 
v.reserve(m); 
for(...) 
...