C语言不使用scanf且不读取换行符的输入读取方法及优化建议

你的实现核心思路是对的：逐字符读取、截断超长输入、清空残留缓冲区，比直接使用无防护的scanf、不处理换行符的fgets可靠性高很多，完成课程作业完全够用，但代码里存在几个隐藏bug，也有更简洁稳妥的实现方式。

现有代码的问题 #

1. 数值读取逻辑存在严重缺陷
   readint和readfloat用strpbrk跳转到第一个数字位置再转换的写法有两个问题：一是会丢失负号，比如输入abc-123会被转换为123；二是如果用户输入全是非数字内容，strpbrk会返回NULL，传给atoi/atof会直接触发段错误，对银行系统这类对输入稳定性有要求的场景来说是致命问题。另外atoi/atof本身没有错误检测能力，无法区分正常输入的0和非法输入。

2. readstr存在越界写风险
   for循环的判断顺序有问题：逻辑与是短路求值，你会先执行*ptr = getchar()把字符写入缓冲区，再判断ptr - buff < bytes -1的边界条件，当缓冲区已经写满时，已经多写入了1字节的越界内容，存在缓冲区溢出隐患。

3. 没有处理EOF异常
   如果用户输入Ctrl+D（类Unix系统）/Ctrl+Z（Windows）触发输入结束，现有函数没有传递错误状态，上层逻辑无法区分是正常读到0还是输入异常终止。

优化后的实现 #

1. 修正readstr的边界问题

调整判断顺序，先检查缓冲区剩余空间再读入字符，增加入参校验，对齐fgets的错误返回语义：

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

char *readstr(char *buff, size_t bytes) {
    if (bytes == 0 || buff == NULL) return NULL;
    int c;
    size_t idx = 0;
    // 先判断空间，再读字符，从根源避免越界写
    while (idx < bytes - 1 && (c = getchar()) != '\n' && c != EOF) {
        buff[idx++] = (char)c;
    }
    buff[idx] = '\0';
    // 读满缓冲区时清空剩余的输入残留，避免影响后续读取
    if (c != '\n' && c != EOF) {
        while ((c = getchar()) != '\n' && c != EOF);
    }
    // 遇到EOF且未读到任何内容时返回NULL，标记输入异常
    return (c == EOF && idx == 0) ? NULL : buff;
}

2. 用strtol/strtod替代atoi/atof做数值转换

这两个标准库函数支持错误检测，能正确识别正负号、小数点，比atoi/atof安全得多：

int readint(void) {
    char buf[256];
    if (readstr(buf, sizeof(buf)) == NULL) return 0; // EOF场景可根据需求调整处理逻辑
    char *endptr;
    errno = 0;
    long res = strtol(buf, &endptr, 10);
    // 简单校验：如果转换结束指针还在缓冲区开头，说明输入全是无效字符
    if (endptr == buf) return 0; // 作业场景可以在这里加输入无效的提示，让用户重输
    return (int)res;
}

double readfloat(void) {
    char buf[256];
    if (readstr(buf, sizeof(buf)) == NULL) return 0.0;
    char *endptr;
    errno = 0;
    double res = strtod(buf, &endptr);
    if (endptr == buf) return 0.0;
    return res;
}

char readchar(void) {
    char buf[2];
    if (readstr(buf, sizeof(buf)) == NULL) return '\0';
    return buf[0];
}

更简洁的实现思路 #

如果你不想自己写逐字符读取的逻辑，完全可以基于fgets做封装，代码更短，也不容易写错边界：

char *readstr(char *buff, size_t bytes) {
    if (bytes == 0 || buff == NULL) return NULL;
    if (fgets(buff, bytes, stdin) == NULL) return NULL;
    // 手动去掉fgets保留的换行符
    buff[strcspn(buff, "\n")] = '\0';
    // 如果没读到换行，说明输入超长，清空缓冲区残留
    if (!strchr(buff, '\n')) {
        int c;
        while ((c = getchar()) != '\n' && c != EOF);
    }
    return buff;
}

这个版本和你自己写的逐字符读取版本效果完全一致，代码量小很多，也更不容易出边界错误。

作业场景额外建议 #

• 银行系统对金额、账号这类输入的准确性要求高，可以在数值读取函数里加循环校验，遇到非法输入时提示用户重新输入，不要直接返回默认值。
• 读取姓名、备注这类可能包含空格的字符串时，不要用scanf("%s")，它遇到空格就会截断，你现在封装的readstr完全可以满足需求。
• 所有字符串读取都要明确传入缓冲区长度，不要硬编码长度值，用sizeof(缓冲区)传参更稳妥。

内容的提问来源于stack exchange，提问作者Gabriel Vasconcelos Fruet