Ошибка с указателем на массив (нарушение прав доступа при чтении по адресу)

[Anton911]

Приветствую уважаемые форумчане, изучаю с++ и написал код который хорошо работает с двумерным статическим массивом (без указателей). Но всё было в одной большой функции и хотелось массив динамический, решил в учебных целях по работать с указателем и переделал код, я так понял с динамическими массивами в с++ работаем через указатели? Процедура massInitElements работает хорошо, но в функции homeWork6 возникает ошибка в строчке которая выделена красным (нарушение прав доступа при чтении по адресу).

void homeWork6() {
   const int massSize = 8; //Эта константа реликт от статических массивов, допустим я ввел с клавиатуры произвольное значение
   int** mass1 = new int* [massSize];  //Мой основной массив указателей на массив указателей на целочисленные значения (у меня двумерный массив) 
   massInitElements(mass1[0], massSize); 
   cout <<mass1[0][0];       //Почему же я не могу прочитать этот элемент? 
}


void massInitElements(int* mass, int massSize) {
   srand(time(0));  //Запускаем генератор значений 
   int** massNew = new int*[massSize]; //инициализация указателя на массив указателей
   massNew[0] = mass; //теперь указатель на указатель имеет ссылку на мой массив из аргумента процедуры
   for (int k = 0; k < massSize; k++) {
      massNew[k] = new int[massSize]; //инициализация подмассива указателей
      for (int n = 0; n < massSize; n++) {
         massNew[k][n] = rand() % 10; //забиваем рандомными значениями от 0 до 9
         printf("%2d", massNew[k][n]);
      }
      printf("\n");
   }
}

P.S. Я сознательно убрал из кода много всего, что не относится к ошибке (delete [] mass1 и все остальное), чтобы уменьшить количество кода .

[Anton911]

В общем я вижу что я затупил и 2 раза выделил память под массив указателей на указатели.
Первый раз в процедуре homeWork6 в строчке

int** mass1 = new int* [massSize];

А второй раз в функции massInitElements в строчке

int** massNew = new int*[massSize];

А значит ошибка возникала потому-что я инициализировал и забивал значениями один участок в памяти, а в функции homeWork6 пытался читать другой, не инициализированный участок памяти.
Но если я уберу из функции massInitElements

int** massNew = new int*[massSize];

, то вообще не компилится.

void massInitElements(int* mass, int massSize) {
   int** massNew;
   massNew[0] = mass; //В этой строчке пишет - использована неинициализированная локальная переменная "massNew"	

   for (int k = 0; k < massSize; k++) {
      massNew[k] = new int[massSize];
      for (int n = 0; n < massSize; n++) {
         massNew[k][n] = rand() % 10;
         printf("%2d", massNew[k][n]);
      }
      printf("\n");
   }
}

Как правильно сделать?
Или может быть я вообще зря полез в указатели? Грубо говоря я хочу в отдельной функции (от основной процедуры) забить значениями динамический массив. В функцию пытаюсь передать ссылку на мой массив.


Вот полный код, объединил все это в одну функцию, так работает всё.

int  homeWork6() {
   const int massSize = 8;
   int** mass1 = new int* [massSize];
   int summDiagonalElements = 0;
   srand(time(0));
   printf("Матрица:\n");
   for (int k = 0; k < massSize; k++) {
      mass1[k] = new int[massSize];
      for (int n = 0; n < massSize; n++) {
         mass1[k][n] = rand() % 10;
         printf("%2d", mass1[k][n]);
      }
      printf("\n");
   }
   cout <<mass1[0][0];
   printf("Суммирующиеся значения: ");
   for (int k = 0; k < massSize; k++) {
      for (int n = 0; n < massSize; n++) {
         if (k == n) { 
            summDiagonalElements += mass1[k][n]; 
            cout << mass1[k][n] << " ";
         }
         if ((k == massSize - 1 - n) && (n == massSize - 1 - k)) {
            summDiagonalElements += mass1[k][n];
            cout << mass1[k][n] << " ";
         }
      }
   }
   for (int k = 0; k < massSize; k++) delete[] mass1[k];
   delete [] mass1;
   
   printf("\nСумма диагональных элементов матрицы: ");
   if (massSize % 2 != 0) {
      summDiagonalElements -= mass1[massSize / 2 + 1][massSize / 2 + 1];
   }
      
   return summDiagonalElements;
}

Но как же разделить? как вынести этот код в отдельную процедуру?

 for (int k = 0; k < massSize; k++) {
      mass1[k] = new int[massSize];
      for (int n = 0; n < massSize; n++) {
         mass1[k][n] = rand() % 10;
         printf("%2d", mass1[k][n]);
      }
      printf("\n");
   }

[BDA]

Выделяйте память в одной из функций, чтобы не путаться.

#include <iostream>
#include <cstdlib>
#include <cstdio>
#include <ctime>

using namespace std;

int** massInitElements(int massSize)
{
   int** massNew = new int*[massSize];
   for (int k = 0; k < massSize; k++) {
      massNew[k] = new int[massSize];
      for (int n = 0; n < massSize; n++) {
         massNew[k][n] = rand() % 10;
         printf("%2d", massNew[k][n]);
      }
      printf("\n");
   }
   return massNew;
}

void homeWork6()
{
   const int massSize = 8;
   int** mass1 = massInitElements(massSize);
   cout << mass1[0][0] << endl;
   for (int k = 0; k < massSize; k++)
        delete[] mass1[k];
    delete[] mass1;
}

void massInitElements2(int** massNew, int massSize)
{
   for (int k = 0; k < massSize; k++) {
      for (int n = 0; n < massSize; n++) {
         massNew[k][n] = rand() % 10;
         printf("%2d", massNew[k][n]);
      }
      printf("\n");
   }
}

void homeWork62()
{
   const int massSize = 8;
   int** mass1 = new int*[massSize];
   for (int k = 0; k < massSize; k++)
      mass1[k] = new int[massSize];
   massInitElements2(mass1, massSize);
   cout << mass1[0][0] << endl;
   for (int k = 0; k < massSize; k++)
        delete[] mass1[k];
    delete[] mass1;
}


int main()
{
    srand(time(0));
    homeWork6();
    homeWork62();
    return 0;
}

[Anton911]

BDA, спасибо большое, оба варианта работают. Не знаю как вам спасибку поставить, раньше вроде были кнопки.