在 C 中用指针打印数组
Print array with pointer in 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);
。请注意,要打印的长度已指定,因此您无需添加终止符 null-character。
您的代码中存在多个问题:
- 你不包括
<stdio.h>
也不包括 <stdlib.h>
- 你应该释放
char_array
并将 big_array
存储到 char_array
所以 char_array
指向更大的分配数组。
printf(*char_array)
由于多种原因具有未定义的行为。您可以使用 fwrite(char_array, 1, index, 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", (int)index, char_array);
}
return 0;
}
将此视为评论^^
我建议您对编译器使用 警告 标志。例如 gcc
标志 -W -Wextra -Wshadow
很好用并且可以防止浪费时间。
我有一个指向 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);
。请注意,要打印的长度已指定,因此您无需添加终止符 null-character。
您的代码中存在多个问题:
- 你不包括
<stdio.h>
也不包括<stdlib.h>
- 你应该释放
char_array
并将big_array
存储到char_array
所以char_array
指向更大的分配数组。 printf(*char_array)
由于多种原因具有未定义的行为。您可以使用fwrite(char_array, 1, index, 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", (int)index, char_array);
}
return 0;
}
将此视为评论^^
我建议您对编译器使用 警告 标志。例如 gcc
标志 -W -Wextra -Wshadow
很好用并且可以防止浪费时间。