Назначить указатель на указатель в цикле C++

Question

Я пытаюсь сохранить 2-мерный массив в указателе на указатель итеративно. Я заполняю его случайными значениями Гаусса, но это не важная часть. Я пытаюсь заполнить массив, на который указывает A, но получить ошибку «плохой код доступа». Как правильно заполнить массив/указатель? Благодарю.

INT основной() {

int N = 100; 
double **A[N][N]; 
for(int i = 0; i< N; i++){ 
    for(int j =0; j < N; j++){ 
     random_device rd; 
     default_random_engine generator; 
     generator.seed(rd()); 
     normal_distribution<double> distribution(0.0,1.0); 
     **A[i][j] = distribution(generator); // error here, bad access code 1 
    } 
} 

Answer 1

Во-первых, изменить double **A[N][N]; к double **A;. Вам нужен двойной указатель, а не матрица двойных указателей. Затем вы должны выделить память следующим образом: A=new double*[N];, а затем for(i=0;i<N;i++) A[i]=new double[n]; Тогда все будет работать нормально. Вы получите доступ к элементу матрицы обычным способом, A[i][j], а не **A[i][j]! Этот пример кода познакомит вас с концепцией:

#include <iostream> 

int main() { 

    int n=100; 

    double **a; 
    a=new double*[n]; 
    for(int i=0;i<n;i++) a[i]=new double[n]; 

    a[1][2]=3; 

    std::cout<<a[1][2]; 

    for(int i=0;i<n;i++) delete a[i]; 
    delete[] a; 

    return 0; 
} 

Answer 2

Ошибка здесь:

**A[i][j] = distribution(generator); 

A[i][j] хранит значение типа double**. Поэтому вам нужно сначала сохранить ваше случайное значение в действительном адресе, а затем сохранить его в A.

/*Assign a new double* pointer to A[i][j]*/ 
A[i][j] = new double*; 

/*Assign a new double pointer which is addressed by A[i][j]*/ 
*A[i][j] = new double; 

/*Assign value to **A[i][j] */ 
**A[i][j] = distribution(generator); 

Answer 3

Вы не можете объявить статический массив, используя не-const integer для своих измерений.

Вы объявляете массив указателей, но указатели не указывают на какую-либо действительную память, поэтому вы получаете ошибки памяти.

Попробуйте вместо этого:

const int N = 100; 

double A[N][N]; 

random_device rd; 
default_random_engine generator; 
generator.seed(rd()); 
normal_distribution<double> distribution(0.0,1.0); 

for(int i = 0; i < N; ++i) { 
    for(int j = 0; j < N; ++j) { 
     A[i][j] = distribution(generator); 
    } 
} 

С другой стороны, выделить массив в куче, вместо стека (то вы можете использовать неконстантную Int для измерений):

int N = 100; 

double **A = new double*[N]; 
for(int i = 0; i < N; ++i) { 
    A[i] = new double[N]; 
} 

random_device rd; 
default_random_engine generator; 
generator.seed(rd()); 
normal_distribution<double> distribution(0.0,1.0); 

for(int i = 0; i < N; ++i) { 
    for(int j = 0; j < N; ++j) { 
     A[i][j] = distribution(generator); 
    } 
} 

... 

for(int i = 0; i < N; ++i) { 
    delete[] A[i]; 
} 
delete[] A; 

Или это :

int N = 100; 

double *A = new double[N*N]; 

random_device rd; 
default_random_engine generator; 
generator.seed(rd()); 
normal_distribution<double> distribution(0.0,1.0); 

for(int i = 0; i < N; ++i) { 
    for(int j = 0; j < N; ++j) { 
     A[(i*N)+j] = distribution(generator); 
    } 
} 

... 

delete[] A; 

с учетом сказанного, то лучше использовать std::vector вместо new[] вручную:

int N = 100; 

std::vector<std::vector<double> > A(N); 
for(int i = 0; i< N; i++) { 
    A[i].resize(N); 
} 

random_device rd; 
default_random_engine generator; 
generator.seed(rd()); 
normal_distribution<double> distribution(0.0,1.0); 

for(int i = 0; i < N; ++i) { 
    for(int j = 0; j < N; ++j) { 
     A[i][j] = distribution(generator); 
    } 
} 

Или:

int N = 100; 

std::vector<double> A(N*N); 

random_device rd; 
default_random_engine generator; 
generator.seed(rd()); 
normal_distribution<double> distribution(0.0,1.0); 

for(int i = 0; i < N; ++i) { 
    for(int j = 0; j < N; ++j) { 
     A[(i*N)+j] = distribution(generator); 
    } 
}