2D массив C++ (ввод и печать)

Question

//Variable Description 
int H,W; 
int map[H][W]; 


//Input 
cin>>W>>H; 

for(int i=0; i<H; i++) 
{ 
    for (int j=0; j<W; j++) 
    { 
     cin>>map[i][j]; 
    } 

} 

cout<<endl; 
//Print 
for(int i=0; i<H; i++) 
{ 
    for (int j=0; j<W; j++) 
    { 
     cout<<map[i][j]; 
    } 

} 
cout<<endl; 
return 0; 

`

Мой вывод:

3 3 
1 2 3 
4 5 6 
7 8 9 

789789789 

Почему я не получаю все номера в моем выходе? Почему только последняя строка?

Answer 1

Вы инициализируете размеры массива после объявления его !!!

Вы не можете использовать массивы переменной длины в C++. Прочитайте this ответ для получения дополнительной информации.

так, вы должны использовать std::vector, как это:

#include <iostream> 
#include <vector> 

using namespace std; 

int main() { 
    int H, W; 
    cin >> W >> H; 
    std::vector< std::vector<int> > map; 
    map.resize(H); // H rows 
    for(int i = 0; i < H; ++i) 
    map[i].resize(W); // in every row, create W columns 
    // thus this is equivalent to a HxD array 

    for (int i = 0; i < H; i++) { 
    for (int j = 0; j < W; j++) { 
     cin >> map[i][j]; 
    } 

    } 

    cout << endl; 
    //Print 
    for (int i = 0; i < H; i++) { 
    for (int j = 0; j < W; j++) { 
     cout << map[i][j] << " "; 
    } 

    } 
    cout << endl; 
    return 0; 
} 

Вы можете сделать это с массивом, но вы должны использовать динамическое выделение памяти, которое требует, чтобы отменить выделение памяти, когда вы сделано. Вот как это должно пойти:

#include <iostream> 
#include <vector> 

using namespace std; 

int main() { 
    int H, W; 
    cin >> H >> W; 

    int** map = new int*[H]; 
    for(int i = 0; i < H; ++i) 
    map[i] = new int[W]; 

    for (int i = 0; i < H; i++) { 
    for (int j = 0; j < W; j++) { 
     cin >> map[i][j]; 
    } 

    } 

    cout << endl; 
    //Print 
    for (int i = 0; i < H; i++) { 
    for (int j = 0; j < W; j++) { 
     cout << map[i][j] << " "; 
    } 

    } 
    cout << endl; 


    // DON'T FORGET TO FREE 
    for(int i = 0; i < H; ++i) { 
    delete [] map[i]; 
    } 
    delete [] map; 
    return 0; 
} 

Answer 2

Ваше заявление int map[H][W]; намерен очистить достаточно места для хранения всех цифр. Но он не знает, сколько места еще (W и H не определены в точке, где вы объявляете map). Таким образом, он очистит некоторое случайное (возможно, нулевое, возможно, смехотворное) количество пространства, в зависимости от того, что содержат слоты памяти, занятые H и W. Чтобы динамически распределять память, вам нужно будет это сделать (a) после H и W известны и (b) с использованием ключевого слова new (посмотрите его с помощью поискового термина, такого как «распределение динамической памяти в C++», и пока вы там, узнайте, почему это важно и для delete [] map, когда вы закончили с ним).