Попытка добавить элементы в многомерный массив.Исключение выдается при попытке запустить приложение - PullRequest
0 голосов
/ 22 октября 2018
int main()
{
    int r;
    scanf("%d", &r);
    int **arr = (int *)malloc(r * r * sizeof(int));
    *(*(arr + r) + r);

    for (int i = 1; i <= r; i++)
    {
        for (int j = 1; j <= r; j++)
        {
            printf("Enter element %d.%d: \n", i,j);
            scanf("%d", &arr[i-1][j-1]);
        }

    }
    getch();
}

, так что это недавно произошло, в основном я хочу добавить матричные элементы в 2d массив, но там написано

'Исключение, выданное в 0x0F1B97AE (ucrtbased.dll) в Matrix.exe: 0xC0000005:Место записи о нарушении прав доступа 0xCDCDCDCD '

Не знаю, что делать :( помогите, пожалуйста

Ответы [ 4 ]

0 голосов
/ 22 октября 2018

Просто забудьте все об этой int** ерунде и выделите вместо нее двумерный массив:

#include <stdio.h>
#include <stdlib.h>

int main (void)
{
    int r;
    scanf("%d", &r);
    int (*arr)[r] = malloc( sizeof(int[r][r]) );

    for (int i = 0; i<r; i++)
    {
        for (int j = 0; j<r; j++)
        {
            arr[i][j] = 1; // some sort of input here
            printf("%d ", arr[i][j]);
        }
        printf("\n");
    }
    free(arr);
}

Пример вывода со входом 5:

5
1 1 1 1 1
1 1 1 1 1
1 1 1 1 1
1 1 1 1 1
1 1 1 1 1

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

Дополнительная информация: Правильное размещение многомерных массивов .

0 голосов
/ 22 октября 2018

Проблема в том, что вы не выделяете и не создаете «двумерный» массив.Ваше распределение памяти выделяет один массив r * r элементов.

Этот единственный массив не может использоваться в качестве массива массивов.

Обычное решение для созданиядинамический массив массивов предназначен для создания массива зубчатый :

int **arr = malloc(r * sizeof(int *));
for (unsigned i = 0; i < r; ++i)
{
    arr[i] = malloc(r * sizeof(int));
}

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

arr[i * r + j]

[Обратите внимание, что для вышеприведенного требуются нулевые индексы i и j]

0 голосов
/ 22 октября 2018
int **arr = (int *)malloc(r * r * sizeof(int));

Вы не можете создать двумерный массив с одним выделением, подобным этому, по крайней мере, ни один, к которому вы можете получить доступ с помощью int **.Вместо этого у вас есть пространство для r * r, объекты типа int, к которым можно получить доступ через int * и некоторую дополнительную арифметику.

Когда вы позже сделаете это:

*(*(arr + r) + r);

С первой разыменовкой все в порядке, поскольку arr указывает на выделенный буфер.Второе, однако, нет, потому что вы читаете неинициализированное значение из этого выделенного буфера и пытаетесь использовать его в качестве действительного адреса.Это вызывает неопределенное поведение , которое в этом случае приводит к сбою.

Вам необходимо выделить место для массива int *, они для каждого из них выделяют массив int:

int **arr = malloc(r * sizeof(int *));
for (int i=0; i<r; i++) {
    arr[i] = malloc(r * sizeof(int));
}
0 голосов
/ 22 октября 2018

Вы используете malloc'ed блок памяти как многомерный массив C.Вместо этого вам нужно использовать один набор квадратных скобок.

Вместо arr[i-1][j-1] вам нужно что-то вроде arr[i * r + j].

Я удивлен, что большинство компиляторов примут этопо умолчанию, потому что вы используете int * для инициализации и int **.

...