动态创建字符串数组时出错

时间:2015-11-02 14:42:57

标签: c arrays malloc

我正在尝试从文件中读取字符串并将其插入矩阵中。每一行都是一个字。

FILE *fp = fopen("zadanie4.txt","r");

if( fp == NULL)
{
  perror("Error while opening the file.\n");
  exit(EXIT_FAILURE);
}


int symbol, num_of_lines = 0, len_of_string = 0, max_len = 0;
do {
    symbol = fgetc(fp);
    len_of_string++;
    if (symbol == '\n' || feof(fp)) {
        num_of_lines++;

        if(len_of_string > max_len){
            max_len = len_of_string;
        }
        len_of_string = 0;
    }
} while (symbol != EOF);
fclose(fp);

printf("Number of words: %d\n", num_of_lines);
printf("Longest word: %d\n", max_len);

fp = fopen("zadanie4.txt","r");
char (*arr)[num_of_lines] = calloc(num_of_lines, sizeof(char*) * max_len); 
int index = 0;

while(fscanf(fp, "%s", arr[index++]) == 1) {  
    printf("%s\n", arr[index - 1]); //first check to see what is written into array
}
close(fp);

printf("--------------------------\n");

int i;
for(i = 0; i < num_of_lines; i++){
    printf("%s\n", arr[i]); //second check
}

我找出最长字符串的大小,并为字符串数量*最长字符串分配内存。

如果最长的单词是5(+1为空'\ 0'),输出如何显示:

Number of words: 6
Longest word: 6
AAAAA
BBBBB
CCCCC
DDDDD
EEEEE
FFFFF
--------------------------
AAAAA
BBBBB
CCCCC
DDDDD
EEEEE
FFFFF

如果我在每一行添加另一个字符:

Number of words: 6
Longest word: 7
AAAAAa
BBBBBb
CCCCCc
DDDDDd
EEEEEe
FFFFFf
--------------------------
AAAAAaBBBBBbCCCCCcDDDDDdEEEEEeFFFFFf
BBBBBbCCCCCcDDDDDdEEEEEeFFFFFf
CCCCCcDDDDDdEEEEEeFFFFFf
DDDDDdEEEEEeFFFFFf
EEEEEeFFFFFf
FFFFFf

注意:在此示例中,每个字符串的大小都相同,但我希望它适用于各种大小。

有人可以帮我解释如何为这个数组正确分配内存吗?

3 个答案:

答案 0 :(得分:0)

arr的声明和分配是错误的,不匹配。

您将arr声明为指向num_of_lines个字符数组的指针。然后为角色分配max_len个指针。

您可以为外部和内部数组使用可变长度数组:

char arr[num_of_lines][max_len + 1];  // +1 for string terminator

根本不需要动态分配。

答案 1 :(得分:0)

我相信你并不真正理解calloc是如何运作的。您无法使用

将内存分配给数组成员
char (*arr)[num_of_lines] = calloc(num_of_lines, sizeof(char*) * max_len);

calloc的第一个参数是为数组分配的块数,第二个参数是每个块的大小(以字节为单位)。

相反,你想要像这样定义数组

char *arr[num_of_lines];

现在你需要遍历每个成员并为其分配内存

for(int i = 0; i < num_of_lines; i++) {
  arr[i] = malloc(sizeof(char) * max_len);
}

或者,您可以将数组定义为静态数组,因为您知道max_len的大小。

答案 2 :(得分:0)

您正在看到这种行为,因为当您添加额外字符时,您在每个字符串的末尾覆盖了字符串终结符(空字符== 0)。

不是使用一个连续的内存块,而是要保留一个字符串指针数组,并分别分配每个字符串:

char *strings[];
strings = calloc(sizeof(char*),NUMBER_OF_STRINGS);
for i = 0; i < NUMBER_OF_STRINGS; i++ {
   strings[i] = calloc(sizeof(char), MAX_STRING_LENGTH+1);
}

然后,当您想要为字符串添加字符时,如果新的总数小于最大字符串长度,请使用strcat:

 strcat(strings[i],"-suffix");

或者,如果它更长,则需要重新分配存储空间:

 strings[i] = realloc(strings[i], MAX_STRING_LEN+EXTRA_BYTES+1);
 strcat(strings[i],"-suffix");