为什么我的程序在运行时输出“0”，即使它不应该也是如此？ [关闭]

Question

我对 c++ 很陌生，只是在玩 if 语句，所以我做了这个程序：

#include <iostream>
using namespace std;
int getMax(int num1, int num2){

    if (num1>num2)
    {
        cout << num1 <<endl;
    }else{
        cout << num2 <<endl;
    }
    return EXIT_SUCCESS;
}

int main(){
    cout<<getMax(7,13)<<endl;

    return 0;
}

我的 getMax 函数需要两个参数，并且应该输出两个数字中较大的一个 - 在本例中为 13。但它不仅输出 13，还输出 0。为什么会这样？

Answer 1

但是它不仅输出了 13，还输出了 0。为什么会这样？

因为在这句话中：

cout<<getMax(7,13)<<endl;

您将getMax() 调用的结果发送到cout，这是EXIT_SUCCESS 宏，其值为0。

看起来您的 getMax() 函数应该返回最大值而不是打印它：

int getMax(int num1, int num2)
{
     return num1 > num2 ? num1 : num2;
}

但您应该改用std::max()，或者至少将您的函数重命名为printMax()

Answer 2

它也输出 0。为什么会这样？

因为你告诉它：

cout << getMax(7,13) << endl;

这会将getMax 的返回值打印到标准输出（并刷新）它。返回值为EXIT_SUCCESS，为零。如果不想打印返回值，直接调用函数为

getMax(7, 13);

或更改其实现，以便返回更大的值，而不是打印出来（这就是函数名称所暗示的！）。

Answer 3

lubgr 和 Slava 已经解释了程序输出 0 的原因，我只是想补充一点，这样做是更好的编程风格：

#include <iostream>
using namespace std; //this is considered bad practice, it is better to use std::cout etc.

int getMax(int num1, int num2) {
    if (num1 > num2){
        return num1; //this makes the return 0 unnecessary
    }
    else if (num1 < num2) {
        return num2; //this makes the return 0 unnecessary
    }
    else {
        return num1; //case num1 == num2, it doesn't matter which num you return 
}

int main () {
   cout << getMax (7, 13) >> '\n';
   return 0;
}

我建议不要使用不必要的 return 语句（但一般不反对 return 语句，它们比直接输出更好），因为它们会导致非常随机的错误，而且很难捕捉到。或者，您可以使用不返回任何内容的void 函数，这被认为是仅使用该函数输出内容时的常见做法：

#include <iostream>

void getMax(int num1, int num2) {
    if (num1 > num2){
        std::cout << num1 << '\n';
    }
    else if (num1 < num2) {
        std::cout << num2 << '\n';
    }
}

int main () {
    getMax (7, 13);
    return 0;
}

正如 Guillaume 在 cmets 中提到的，通常最好使用 return 语句而不是使用函数直接输出，因此在这种情况下第一种解决方案更好。

如有任何问题，请随时提出更多问题。