C语言中使用fgets、sscanf读取输入时创建全局缓冲区是否为良好实践？

你提供的代码片段如下：

#include <stdio.h>

char buffer[2048];

int main()
{
    int a;

    do {
        printf("Enter an integer: ");
        fgets(buffer, sizeof(buffer), stdin);
        if (sscanf(buffer, "%d", &a) < 1)
            printf("Invalid input\n");
    } while (sscanf(buffer, "%d", &a) < 1);

    return 0;
}

你提问中提到的定义全局buffer缓冲区的写法不属于良好的C语言编程实践，核心原因如下：

• 无意义的内存占用：全局变量的内存会在程序完整运行周期内持续占用，而该缓冲区仅在main函数的输入逻辑中使用，没有必要长期保留内存空间。哪怕2KB的占比很小也属于冗余占用，这类写法在大型项目中累积后会带来明显的内存浪费。
• 存在全局污染风险：全局变量对所有函数可见，任意函数都可以直接修改该缓冲区的内容，后续新增功能如果误操作修改了这个buffer的值，会直接导致输入逻辑异常，且这类问题的排查难度远高于局部变量问题。
• 违反变量最小作用域原则：通用编程规范要求变量的作用域尽可能缩小到仅使用它的范围内，这个缓冲区只有main函数使用，完全可以定义为main函数内的局部变量，随main函数执行分配、结束自动回收，没有额外开销的同时也避免了全局污染。

另外你当前的代码还存在可优化点：循环中两次调用sscanf解析缓冲区属于重复操作，可以将第一次sscanf的返回值存储下来复用，优化后的参考写法如下：

#include <stdio.h>

int main()
{
    char buffer[2048];
    int a, parse_ret;

    do {
        printf("Enter an integer: ");
        fgets(buffer, sizeof(buffer), stdin);
        parse_ret = sscanf(buffer, "%d", &a);
        if (parse_ret < 1)
            printf("Invalid input\n");
    } while (parse_ret < 1);

    return 0;
}

内容的提问来源于stack exchange，提问作者Kain