Сбой программы 2d динамического массива

Question

Сбой программы 2d динамического массива

0

Я написал динамически выделенный 2D-массив, и по какой-то причине он не работает, как я его намерен. Я выделил для массива 14x5 для хранения значений в.

Вот код:

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

int main()
{
   int i, j, count, count2;
   int size, keysize, column;

   size = 7;  keysize = 10; column = 5;
   int row = (size * keysize) / column;  // 14

   char **arr = (char **)malloc(column * sizeof(char *));
   for(i=0; i<row; i++)
      arr[i] = (char *)malloc(row * sizeof(char *));

   count=0;
   count2=0;

   for(i=0; i<keysize; i++)
   {
      for(j=0; j<size; j++)
      {
         printf("arr[%d][%d]=", count2, count);
         arr[count2][count] = 'C';
         printf("%c\n", arr[count2][count]);
         count++;

         if(count == 5)
         { 
            count = 0;
            count2++;
         }
      }
   }
   return 0;
}

Кажется, что программа вылетает, когда значение должно быть вставлено в место arr [6] [0]. До этого он отлично работает. Я вставил некоторые заявления printf, чтобы зафиксировать это.

Я не уверен, почему это делается, распределение памяти выглядит хорошо со мной, но я не эксперт.

Спасибо за любую помощь.

SmalekTF 29 нояб. 2014, в 20:52

Источник

0

примечание: пожалуйста, избегайте приведения возврата malloc()
Sourav Ghosh 29 нояб. 2014, в 19:40
0

arr[i] = (char *)malloc(row * sizeof(char *)); , Вы должны выделить (rows * elementsize) но не сразу понятно, сколько строк или какого размера у каждого элемента.
Weather Vane 29 нояб. 2014, в 19:41
1

column = 5; malloc(column * sizeof(char *)); Это освобождает место для 5 указателей на строки, а затем вы вызываете arr[6][0] чтобы получить доступ через конец массива.
shuttle87 29 нояб. 2014, в 19:42
1

@SouravGhosh у него есть теги C и C ++, и если он хочет, чтобы он компилировался с помощью компилятора C ++, ему нужно привести его, C ++ не делает неявных преобразований произвольных указателей. (Я знаю, что это не рекомендуется, но некоторые люди просто пишут программы, которые компилируются на компиляторах C ++ и C)
PeterT 29 нояб. 2014, в 19:43
0

@PeterT #include <stdio.h> и malloc() не для C++ . Кроме того, тегирование C и c++ в основном рассматривается как спам тегов.
Sourav Ghosh 29 нояб. 2014, в 19:46

Показать ещё 3 комментария

Теги:

c++

arrays

c

malloc

2 ответа

Ещё вопросы

примечание: пожалуйста, избегайте приведения возврата malloc()
arr[i] = (char *)malloc(row * sizeof(char *)); , Вы должны выделить (rows * elementsize) но не сразу понятно, сколько строк или какого размера у каждого элемента.
column = 5; malloc(column * sizeof(char *)); Это освобождает место для 5 указателей на строки, а затем вы вызываете arr[6][0] чтобы получить доступ через конец массива.
@SouravGhosh у него есть теги C и C ++, и если он хочет, чтобы он компилировался с помощью компилятора C ++, ему нужно привести его, C ++ не делает неявных преобразований произвольных указателей. (Я знаю, что это не рекомендуется, но некоторые люди просто пишут программы, которые компилируются на компиляторах C ++ и C)
@PeterT #include <stdio.h> и malloc() не для C++ . Кроме того, тегирование C и c++ в основном рассматривается как спам тегов.

shuttle87 · Answer 1 · 2014-11-29T17-06-00.000Z

Я замечаю вопрос, отмеченный тегом c++, ясным способом сделать это в c++ является использование std::vector. Поэтому я отвечаю на это, как на вопрос C:

Похоже, вы не выделили достаточно места здесь.

column = 5;
char **arr = (char **)malloc(column * sizeof(char *));

Это делает пространство для 5 указателей для строк. Затем позже, когда вы зацикливаетесь:

for(i=0; i<keysize; i++)//note keysize is 10
{
    for(j=0; j<size; j++){ // note size is 7
        arr[count2][count];
        count++;
        if(count == 5){
            count = 0;
            count2++;//gets incremented every time the inner loop runs
        }
    }
}

Поскольку count2 получает приращение каждый раз, когда выполняется внутренний цикл, он в конечном итоге становится таким же большим, как и keysize. Это означает, что arr[keysize][0] должен быть действительным, а это не потому, что вы действительно сделали arr[5][14]. Первый сбой - это когда вы вызываете arr [6] [0], который обращается к концу массива и вызывает сбой вашей программы.

Если целью здесь является инициализация переменных, то для исправления вы, вероятно, захотите перебрать те же значения для измерений, которые вы использовали для выделения памяти.

BLUEPIXY · Answer 2 · 2014-11-29T16-52-00.000Z

изменить на

char **arr = (char **)malloc(row * sizeof(char *));
for(i=0; i<row; i++)
   arr[i] = (char *)malloc(column * sizeof(char));