文件与stdin读取的错误处理正确性及isatty用法疑问

我的程序想实现：提供文件名时从文件读取，否则从stdin读取，写了如下代码：

if (isatty(STDIN_FILENO)) {
    if ( !(f = fopen(argv[1], "r")) )
    {
        perror(argv[1]);
        return 1;
    }
}
else
    f = stdin;

while ((size = getline(&line, &len, f)) != -1) {
...

有几个疑问：

• 这样的错误处理是否正确？
• 是否需要对stdin做错误处理？有没有stdin打开失败的场景？
• 对isatty(STDIN_FILENO)的返回结果感到困惑：当它实际上“不是TTY”时，返回值为true？我原本以为这个判断逻辑应该是相反的。

1. 当前代码的逻辑与错误处理问题 #

你的代码逻辑和需求不匹配，错误处理也存在漏洞：

• 核心问题是isatty的用法错误，导致触发文件读取的条件完全不符合需求。你的目标是传了文件名就读文件，否则读stdin，但当前代码是当stdin是终端（TTY）时，才尝试打开argv[1]。如果用户直接运行程序（stdin是TTY）但没传参数，argv[1]是NULL，fopen(NULL, "r")会直接报错退出，这完全违背了“没传参数就读stdin”的设计。
• 正确的逻辑应该是先检查命令行参数数量，而非依赖isatty判断，示例代码如下：

FILE *f = stdin;
if (argc > 1) {
    if (!(f = fopen(argv[1], "r"))) {
        perror(argv[1]);
        return 1;
    }
}

2. stdin的错误处理与异常场景 #

• stdin是进程启动时默认打开的标准输入流，正常场景下不会出现打开失败的情况。但存在极端异常场景：比如启动程序时手动关闭stdin（例如执行./your_program <&-），此时stdin的文件描述符无效，后续操作会触发错误。
• 不需要单独对stdin做“打开”操作的错误处理，因为你不需要调用fopen去打开它。但在读取阶段（比如getline调用），必须检查返回值：当getline返回-1时，要区分是到达文件末尾（feof(f)为真）还是发生错误（ferror(f)为真），如果是错误可以用perror("stdin")输出具体错误信息。

3. isatty的返回值规则 #

isatty(STDIN_FILENO)的返回逻辑很明确：

• 如果传入的文件描述符指向终端设备（比如用户直接运行程序时的stdin），返回非0值（在C的布尔判断中视为true）
• 如果不是终端设备（比如stdin被重定向到文件、管道，或者已被关闭），返回0（视为false）
  你的困惑源于对返回值的误解，当前代码里的判断逻辑刚好和需求相反，这也是程序行为不符合预期的核心原因。

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