C - 打印数组

Question

我正在尝试实时打印添加到数组中的元素 .

一切似乎都很好，但是

例如

我将数字 1 2 3 相加

但结果是： 9966656 2686588 1 2 3

我不知道为什么它也打印 9966656 2686588 而不仅仅是 1 2 3

    int numbers[100] , c , x;
    char answer;
    puts("Please insert a value");

GO:
    scanf("%d", &numbers[c]);
    getchar();
    puts("Do you want to add another value? y/n");
    scanf("%c",&answer);
    if (answer == 'y') {
        c = c + 1;
        puts("Please insert another value");
        goto GO;
    } else {
        x = c;

        for (c = 0; c < x + 1; c++) {
            printf("%d ",numbers[c]);
        }
    }

*======================

如果你有什么不明白的地方请告诉我*

Answer 1

您需要解决几个问题。如果您使用 Warnings Enabled 进行编译（例如，将 -Wall -Wextra 添加到您的编译字符串中），所有这些都会为您拼写出来。例如：

$ gcc -Wall -Wextra -o bin/go go.c

go.c: In function ‘main’:
go.c:17:5: warning: format ‘%c’ expects argument of type ‘char *’, but argument 2 has type ‘char (*)[30]’ [-Wformat=]
 scanf("%c",&answer);
 ^
go.c:18:15: warning: comparison between pointer and integer [enabled by default]
 if(answer == 'y')

如果您解决每个警告，直到您的代码在没有警告的情况下编译，您就会遇到主要问题：

for(c = 0; c < x + 1; c++)

x + 1 导致您的循环读取数据末尾之外的内容。由于您NOT 初始化了numbers，因此它正在读取默认情况下存在的垃圾值。这是另一个很好的教训：总是初始化你的变量。

您在管理x 和c 的方式上也会遇到问题。 c 应仅在用户成功转换小数时更新，如 scanf 所读取。 scanf 返回时提供成功转换的次数。您需要使用返回来检查用户输入的有效小数，然后仅在成功转换时更新c（而不是用户输入y/n 的结果。

稍微清理一下，将您的逻辑稍微重新安排为类似于以下内容会更好：

#include <stdio.h>

int main (void) {

    int numbers[100] = { 0 };
    int c = 0;
    int i = 0;
    char answer[30] = { 0 };

    printf (" Please insert a value: ");

GO:

    if (scanf ("%d", &numbers[c]) == 1)
        c++;
    getchar ();
    printf (" Do you want to add another value (y/n)? ");
    scanf ("%c", answer);
    if (*answer == 'y') {
        printf (" Please insert another value: ");
        goto GO;
    }

    for (i = 0; i < c; i++) {
        printf (" number[%2d] : %d\n", i, numbers[i]);
    }

    return 0;
}

输出

$ ./bin/go
 Please insert a value: 10
 Do you want to add another value (y/n)? y
 Please insert another value: 11
 Do you want to add another value (y/n)? y
 Please insert another value: 12
 Do you want to add another value (y/n)? n
 number[ 0] : 10
 number[ 1] : 11
 number[ 2] : 12

（注意：我将 x 更改为 i - 只是觉得迭代 i 更正常）

Answer 2

该行为未定义，因为c 未初始化，您将其用作“索引”(&numbers[c])。这也可能导致分段错误，甚至按您的预期行事。

你需要做的是一开始就将c设置为0。

另外请尽量避免使用goto，除非有真正合理的理由使用它。