Сбой сегментации 2D-массива

Question

Я пишу программу, и в своей программе мне нужно скопировать информацию из 1-го массива 1D в 2D-массив, но каждый раз, когда в массиве 1n находится массив \ n, он предполагает перейти в разные слоты в 2D-массиве. Например, если массив 1D равен Hello \ nWorld в массиве 2d он станет привет/n в первом слоте и мир во втором слоте.

Вот мой код Но я получаю ошибку сегментации. Массив, называемый символами, уже сделан в моей программе до этого шага.

words = (char**) malloc(numWords*sizeof(char)); 
    int copyCountForChars=0; 
    int copyCountForWords=0; 


    while(copyCountForWords <= numWords) 
    { 

     words[copyCountForWords][copyCountForChars] = chars[copyCountForChars]; 
     // printf("%c",chars[copyCountForChars]);              
     if(chars[copyCountForChars] == '\n') 
     { 
      // printf("%c",chars[copyCountForChars]);            

      copyCountForWords++; 

     } 

     copyCountForChars++; 
    } 

Answer 1

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

int main(){ 
    char str[100] = "hello\nworld\nstack\noverflow"; 

    int numWords = 4; // you can handle this part 
    char **words = malloc(numWords * sizeof(char *)); 

    char *tok = strtok(str, "\n"); 
    int counter = 0; 
    while(tok != NULL){ 
     words[counter] = tok; 
     counter++; 
     tok = strtok(NULL, "\n"); 
    } 

    printf("%s\n", words[0]); // hello 
    printf("%s\n", words[1]); // world 
    printf("%s\n", words[2]); // stack 
    printf("%s\n", words[3]); // overflow 

} 

Я представляю указатель отношения как этот

---

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

int main(){ 
    char *str = "stackoverflow"; 
    int size = strlen(str); 
    char str2[size+1]; 
    strcpy(str2, str); 
    strcpy(str2+size, "\n"); 

    printf("%s", str2); 
} 

Answer 2

Распределение памяти для 2D-массива должно выполняться следующим образом.

words = malloc(sizeof(char *) * size1); 
for(i = 0; i< size1; i++) 
{ 
    words[i] = malloc(sizeof(char) * size2); 
} 

Answer 3

WhozCraig является правильным. Ваш malloc для слов не выделяет достаточно памяти и, вероятно, вызывает сбои в доступе к памяти за пределами границ. Вот почему: Say numWords = 2, то следующая строка:

слова = (символ **) таНос (numWords * SizeOf (Char));

фактически делает malloc (2 * 1). sizeof (char) равен 1.

Вы выделили только 2 байта памяти.

Возможно, вы имели в виду размерof (chars) с точки зрения распределения? Это даст вам размер 1-мерного массива. Даже тогда вам все равно придется копировать каждый байт за один раз из одного массива в другой, что в настоящее время не выполняется.