在C中使用指针打印数组

问题描述

我有一个指向char数组的指针,我需要打印整个数组(不是一个一个,而是整个数组)。我遇到错误zsh: segmentation fault。我不明白为什么。 我的代码stdin获取文本,并将字符放入数组中。如果第一个数组char_array不够大,我将它们放在第二个数组bigger_array中。

int main() {
    int c;
    int index = 0;
    size_t size = 100;                        
    char *char_array = malloc(size * sizeof(char)); 
    if (char_array != 0) {
        while ((c = getchar()) != EOF && index < size) {
            char_array[index] = c;
            index++;
        }
        size_t newSize = size * 100;
        char *bigger_array = realloc(char_array,newSize * sizeof(char)); 
        if (bigger_array != 0) {
            bigger_array = char_array;
            while ((c = getchar()) != EOF && index < newSize) {
                bigger_array[index] = c;
                index++;
            }
            size = newSize;
        }
    }
    printf(*char_array);
    return 0;
}

解决方法

  • bigger_array = char_array;应该是char_array = bigger_array;。否则,指向新分配的缓冲区的指针将被覆盖为指向旧缓冲区的指针,并且会发生错误的事情。
  • printf(*char_array);不好,因为它传递了需要指针的整数(一个字符)。它应该是printf("%.*s",index,char_array);。请注意,已指定要打印的长度,因此您无需添加终止的空字符。
,

您的代码中存在多个问题:

  • 您不包括<stdio.h><stdlib.h>
  • 您应该释放char_array并将big_array存储到char_array,以便char_array指向更大的分配数组。
  • printf(*char_array)具有未定义的行为,原因有多种。您可以使用fwrite(char_array,1,stdout)printf("%.*s\n",(int)index,char_array)打印字符。
  • 不需要单独的循环,只需按需重新分配数组即可。

这是修改后的版本:

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

int main() {
    int c;
    size_t index = 0;
    size_t size = 100;
    char *char_array = malloc(size);
    if (char_array != NULL) {
        while ((c = getchar()) != EOF) {
            if (index == size) {
                size_t new_size = size + size / 2 + size / 8; /* increase by the golden ratio */
                char *new_array = realloc(char_array,new_size);
                if (new_array == NULL) {
                    fprintf(stderr,"out of memory\n");
                    free(char_array);
                    return 1;
                }
                size = new_size;
                char_array = new_array;
            }
            char_array[index++] = c;
        }
        printf("%.*s\n",char_array);
    }
    return 0;
}
,

将此作为评论^^

我建议您在编译器中使用标志进行警告。例如gcc标志-W -Wextra -Wshadow的示例很好使用,可以防止浪费时间。