Объявление и инициализация глобальной 2D массив

Question

//gives the error expecting a declaration 

int** my_array; 
for (int i=0; i<200; i++) 
{ 
    my_array[i]= new int [200]; 
} 

//initialize array with zeroes 
for (int i = 0; i<200; i++) 
{ 
    for (int j = 0; j<200; j++) 
    { 
     my_array[i][j] =0; 
    } 
} 

Вкратце:

2D-массив объявлены глобально

Он инициализируется в главном. Основной вызывает функцию, которая сохраняет 0's или 1's в специфических клетках массива

массив затем печатается основной

Также совет правильно ли поток?

Answer 1

Если вам нужен массив из массива из 200 bools, вы можете использовать std :: array и std :: bitset.

#include <bitset> 
#include <array> 
//... 
std::array<200, std::bitset<200>> my_array; 

Во-вторых, ваш код, даже если вы исправите ошибку синтаксиса, неверен.Вы должны были бы создать my_array таким образом:

int** my_array; 
//... 
my_array = new int * [200]; 
for (int i = 0; i < 200; ++i) 
    my_array[i] = new int [200]; 

Тогда где вы должны удалить эту память:

for (int i = 0; i < 200; ++i) 
    delete [] my_array[i]; 
delete [] my_array; 

---

Альтернативным, который создает непрерывный блок и использует только два вызова new[]:

my_array = new int * [200]; 
int *pool = new int [200*200]; 
for (int i = 0; i < 200; ++i, pool += 200) 
    my_array[i] = pool; 

Тогда где-то вы должны удалить эту память:

delete [] my_array[0]; 
delete [] my_array; 

Однако я советую вам использовать контейнер, такие как станд :: массив, станд :: BitSet, станд :: вектор и т.д. вместо int** подхода.

Answer 2

Что касается синтаксической ошибки, возможно, это связано с чем-то перед кодом, который вы указали.

Показанный код,

int** my_array; 
for (int i=0; i<200; i++) 
{ 
    my_array[i]= new int [200]; 
} 

//initialize array with zeroes 
for (int i = 0; i<200; i++) 
{ 
    for (int j = 0; j<200; j++) 
    { 
     my_array[i][j] =0; 
    } 
} 

компилируется нормально, как тело функции.

---

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

auto my_array = vector<vector<bool>>(200, vector<bool>(200)); 

vector где находится std::vector из заголовка <vector>.

Это позволяет избежать грубых указателей, new -ing`, петель и т. Д., Все, что вам кажется проблематичным.

---

В качестве альтернативы вы можете объявить его как это:

vector<vector<bool>> my_array(200, vector<bool>(200)); 

---

Обратите внимание, что глобальная переменная, как правило, хлопотно, лучше избегать.