C++中如何从istream读取字符？验证函数始终返回false问题排查

核心问题 #

你的代码存在两个关键问题导致始终返回false：

1. 类型不匹配：你要求输入格式为(a)，但代码中用int i读取括号内的内容——a是字符，无法被int类型的输入操作符正确读取，这会导致输入流进入错误状态，后续的input >> b2操作直接失败，b2的值永远不会是')'，自然无法触发返回true的条件。
2. 未检查流状态：代码没有判断每次读取操作是否成功，哪怕输入格式完全正确，只要流出现错误（比如读取失败、到达文件尾），后续逻辑都会失效。

修复代码 #

情况1：括号内是字符（如(a)） #

把int i改为char i，同时添加流状态检查：

bool foo(std::istream& input) {
    char b1;
    char i;
    char b2;

    // 读取左括号并检查是否成功
    if (!(input >> b1) || b1 != '(') {
        return false;
    }

    // 读取括号内的字符并检查
    if (!(input >> i)) {
        return false;
    }

    // 读取右括号并检查
    if (!(input >> b2) || b2 != ')') {
        return false;
    }

    return true;
}

情况2：括号内是整数（如(123)） #

如果实际需求是验证整数格式，保留int i，但必须确保输入为数字，同时检查流状态：

bool foo(std::istream& input) {
    char b1;
    int i;
    char b2;

    if (!(input >> b1) || b1 != '(') {
        return false;
    }

    if (!(input >> i)) {
        return false;
    }

    if (!(input >> b2) || b2 != ')') {
        return false;
    }

    return true;
}

额外说明 #

• 输入流的>>操作符会自动跳过空白字符，如果你的输入格式允许括号和内容之间有空格（如( a )或( 123 )），上述代码依然有效；如果要求严格无空格的紧凑格式，可以用input.get()代替>>来读取字符，避免跳过空白。
• 一旦流进入错误状态，后续操作都会失败，所以每次读取后必须检查流的状态（input在布尔上下文中会自动转换为!input.fail()）。

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