使用fgets结合strtol替代scanf是否为优质方案？是否有更好选择？

先看你提供的代码，首先存在一个致命问题：

void fill_boxes(box *boxes, int length)
{
    char *n; // 此处n是未初始化的野指针，直接用fgets写入会触发未定义行为（崩溃/内存破坏）
    
    for (int i = 0; i < length; i++) {
        printf("Enter box %d id: ", i+1);
        //scanf("%d",&boxes[i].id);
        fgets(n,25,stdin); // 野指针写入，风险极高
        boxes[i].id = strtol(n,NULL,10);

        printf("Enter box %d name: ", i+1);
        //scanf(" %49[^\n]",boxes[i].name);
        fgets(boxes[i].name,50,stdin);

    }
}

核心结论：fgets+类型转换是scanf的优秀替代方案 #

scanf的最大痛点是对输入缓冲区的处理不直观——比如输入数字后按下的换行符会留在缓冲区，导致后续读取字符串时直接读入空行；而且它对非法输入的容错性极差，容易让程序陷入死循环。而用fgets读取整行输入，再转换为对应类型的方式，逻辑更接近Python的input()，能更精准地控制输入流程，也更容易处理错误，确实是更健壮的选择。

你的代码需要修正的问题 #

• 野指针问题：必须给n分配实际内存，比如声明为数组char n[25];，或者用malloc动态分配空间。
• 换行符残留：fgets会把输入末尾的\n一并读入字符串，虽然strtol转换整数时会自动忽略，但读取名称时如果不对换行符处理，最终存储的name会带换行符，影响后续使用。
• 转换错误判断：strtol转换失败时返回0，但0也可能是合法的id值，需要通过endptr参数判断转换是否真的成功。

优化后的代码示例 #

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

typedef struct {
    int id;
    char name[50];
} box;

void fill_boxes(box *boxes, int length)
{
    char id_buf[25]; // 分配固定大小的缓冲区，避免野指针
    char *endptr;
    
    for (int i = 0; i < length; i++) {
        // 带验证的id输入
        while (1) {
            printf("Enter box %d id: ", i+1);
            if (fgets(id_buf, sizeof(id_buf), stdin) == NULL) {
                fprintf(stderr, "输入错误或已到文件末尾\n");
                return;
            }
            // 去除换行符
            id_buf[strcspn(id_buf, "\n")] = '\0';
            // 转换并校验合法性
            boxes[i].id = strtol(id_buf, &endptr, 10);
            if (endptr == id_buf) {
                printf("无效的id，请输入数字。\n");
                continue;
            }
            break;
        }

        // 名称输入处理
        printf("Enter box %d name: ", i+1);
        if (fgets(boxes[i].name, sizeof(boxes[i].name), stdin) == NULL) {
            fprintf(stderr, "输入错误或已到文件末尾\n");
            return;
        }
        // 去除换行符（如果存在）
        boxes[i].name[strcspn(boxes[i].name, "\n")] = '\0';
    }
}

更优的扩展方向 #

• 封装输入函数：把整数输入、字符串输入封装成独立工具函数（比如int get_int(const char *prompt)、void get_string(const char *prompt, char *buf, size_t buf_size)），提升代码复用性。
• 处理缓冲区溢出：如果用户输入的内容超过fgets的缓冲区大小，剩余字符会留在输入缓冲区，需要额外读取并丢弃，避免影响后续输入。
• 业务级验证：根据需求添加更严格的校验，比如id是否为正整数、名称是否为空等。

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