C++编写判断输入数值是否为迄今最大值的程序问题求助

看起来你的代码逻辑里缺少了比较当前输入值和历史最大值的关键环节，所以不管输入什么数值，程序都会直接判定它是当前最大值，这就是结果不符合预期的核心原因。我来帮你修正这个问题：

原代码的核心问题 #

• 没有维护一个专门记录「当前最大值」的变量，每次输入后直接默认它是最大值，完全没有和之前输入的数值做比较
• 循环内的输入逻辑重复调用cin，容易导致输入流处理混乱

修正后的完整代码 #

#include <iostream>
#include <cmath> // 用于使用-INFINITY，确保初始值足够小

int main() {
    double currentNum;
    double maxNum = -INFINITY; // 初始化最大值为负无穷，确保第一个输入的数一定能成为初始最大值

    std::cout << "Please enter a number: \n";
    // 循环读取输入，直到用户输入非数值内容（比如字母）结束
    while (std::cin >> currentNum) {
        // 比较当前输入和最大值
        if (currentNum > maxNum) {
            maxNum = currentNum; // 更新最大值
            std::cout << currentNum << " is the greatest number so far\n";
        } else {
            std::cout << currentNum << " is not the greatest number so far. Current maximum is " << maxNum << "\n";
        }
        std::cout << "Enter a new number: \n";
    }

    std::cout << "\nProgram ended. Final maximum was: " << maxNum << "\n";
    return 0;
}

代码逻辑解释 #

• 用maxNum变量持续跟踪当前输入的最大值，初始设为-INFINITY（比任何实数都小），这样第一个输入的数肯定会成为初始最大值
• 每次读取新的currentNum后，和maxNum做比较：
  • 如果更大，更新maxNum并告知用户这是当前最大值
  • 如果更小或相等，明确告诉用户当前的最大值是多少
• 循环会在用户输入非数值（比如输入字母q）时自动结束，最后还会输出最终的最大值

额外优化建议 #

如果不想引入<cmath>库，也可以用第一个输入的数来初始化maxNum，示例如下：

#include <iostream>

int main() {
    double currentNum;
    double maxNum;

    std::cout << "Please enter a number: \n";
    std::cin >> currentNum;
    maxNum = currentNum;
    std::cout << currentNum << " is the greatest number so far\n";

    // 循环读取后续数值
    std::cout << "Enter a new number: \n";
    while (std::cin >> currentNum) {
        if (currentNum > maxNum) {
            maxNum = currentNum;
            std::cout << currentNum << " is the greatest number so far\n";
        } else {
            std::cout << currentNum << " is not the greatest number so far. Current maximum is " << maxNum << "\n";
        }
        std::cout << "Enter a new number: \n";
    }

    std::cout << "\nProgram ended. Final maximum was: " << maxNum << "\n";
    return 0;
}

这种方式不需要依赖-INFINITY，适合对标准库依赖有要求的场景

内容的提问来源于stack exchange，提问作者håkon knudsen