c++简单函数返回大值-无论输入是什么，总是相同的值- 4309838

C ++ simple function returning huge value - always the same value no matter the inputs - 4309838

本文关键字：4309838 是什么 函数 简单 返回 c++ 输入      更新时间：2023-10-16

我正在学习编码，这是一个非常简单的函数，但它每次都返回相同的答案，4309838，我不知道为什么。它是用来计算薪水的，加班时加50。如有任何帮助，不胜感激。

float payCheck(int ratePar, float hoursPar)
{
    if (hoursPar>40)
        payCheck = ratePar*hoursPar + 50;
    else
        payCheck = ratePar*hoursPar;
        return payCheck;
}
int main()
{   
int rate;
    float hours, pay;
    cout<<"Enter hours worked and pay rate "<<endl;
    cin>>hours>>rate;
    pay = payCheck(rate, hours);
    cout<<"Your paycheck is "<<pay<<endl;
}

您没有在payCheck方法中声明payCheck变量。如此:

float payCheck(int ratePar, float hoursPar)
{
float payCheck;
if (hoursPar>40)
    payCheck = ratePar*hoursPar + 50;
else
    payCheck = ratePar*hoursPar;
return payCheck;
}

您需要声明在函数中返回的变量。

float payCheck(int ratePar, float hoursPar)
{
float payCheck; //Observe this
if (hoursPar>40)
    payCheck = ratePar*hoursPar + 50;
else
    payCheck = ratePar*hoursPar;
return payCheck;
}

虽然缺少payCheck局部变量的声明是问题中问题的主要原因，但我不同意只引入这样一个局部变量的答案(变量名称更改为与函数名称消除歧义):

float payCheckAmount;
// or alternatively:
// float payCheckAmount = 0.0f  // looks better at first, but is not
// ... do something in an if-else
return payCheckAmount;

问题在于payCheckAmount处于无意义状态(从函数的目的来看)，直到在if构造中的某个地方发生计算值的实际赋值。这是一个等待发生的错误。在某些时候，某人将更改if构造，使其不再覆盖所有可能的条件，并且将返回payCheckAmount值的无意义初始值，导致难以在程序的其他地方跟踪错误。

只要可能，用一种不可能以非法状态结束的方式编写程序。当你这样做的时候，试着预测未来可能对代码的修改。

对于payCheck()函数，可以完全去掉局部变量:

float payCheck(int ratePar, float hoursPar)
{
    if (hoursPar > 40) {
        return ratePar*hoursPar + 50;
    }
    return ratePar*hoursPar;
}
然而，这样做的缺点是在代码中重复了速率*小时的计算。如果它改变了，你必须记住改变两个位置。出于这个原因，我将使用以下实现: 
float payCheck(int ratePar, float hoursPar)
{
    float payCheckAmount = ratePar*hoursPar;
    if (hoursPar > 40) {
        payCheckAmount += 50;
    }
    return payCheckAmount;
}

只要你计算的基本逻辑仍然是"每小时基本工资加上加班奖金"，你就不可能最终返回一个无意义的值。

相关文章：