简单函数的麻烦

Trouble with simple functions

本文关键字：麻烦 函数 简单      更新时间：2023-10-16

我正在编写代码来调用与力，质量和加速度方程有关的函数。函数被正确地调用，但是输入没有像它们应该的那样相乘。第一个函数的输出是一个非常小的数，而第一个函数的输出总是0。这是我的代码。任何反馈都会很有帮助。谢谢。

#include <iostream>
#include <cstdlib>
using namespace std;
void displayMenu();
double force(double);
double secondForce(double,double);
int main(int argc, char** argv) 
{
    int menuOption;
    displayMenu();
    system("PAUSE");
    return 0;
}
void displayMenu(void)
{
    int menuOption;
    double weight, accel;
    cout << "           Main Menu" << endl;
    cout << "Enter 1 for Force calculation with acceleration = 9.8m/s^2.n";
    cout << "Enter 2 for Force calculation with user defined acceleration.n";
    cout << "Enter 3 to quit the program.n";
    cin >> menuOption;
        if(menuOption==1)
        {
            cout << "Enter a mass.n";
            cin >> weight;
            cout << "The force is ";
            cout << force(weight);  
            cout << "N.";
            }
        else if(menuOption==2){
            cout << "Enter a mass.n";
            cin >> weight;
            cout << "Enter an acceleration.n";
            cin >> accel;
            cout << "The force is ";
            cout << secondForce(weight, accel);
            cout << "N.";
        }
}
double force(double weight)
{
    double force, mass;
    force=(mass*(9.8));
    return force;
}
double secondForce(double secondMass, double secondWeight)
{
    double secondForce, mass, acceleration;
    secondForce=(mass*acceleration);
    return secondForce;
}

你在函数中使用了未初始化的变量。在程序中使用这些未初始化的变量是未定义的行为。

force=(mass*(9.8)); << mass has a garbage value
secondForce=(mass*acceleration); << mass and acceleration have a garbage value

我想你的意思是

double force(double weight)
{
    return weight * 9.8;
}
double secondForce(double secondMass, double secondAccel)
{
   return secondMAss * secondAccel;
}

问题是您在计算中使用未定义的变量。请参阅下面嵌入的注释:

double secondForce(double secondMass, double secondWeight)
{
    double secondForce, mass, acceleration;
    secondForce=(mass*acceleration); //what is the value of mass and acceleration here??
    return secondForce;
}

作为旁注:考虑在调试器(如gdb)中逐步执行代码，以便查看程序的执行情况。对于大型程序来说，试图通过它进行推理是非常困难的。在您的示例中，对于经验丰富的程序员来说很容易发现。

您正在将数据乘以垃圾值

double force(double weight)
{
double force, mass;
force=(mass*(9.8));
return force;
}
double secondForce(double secondMass, double secondWeight)
{
double secondForce, mass, acceleration;
secondForce=(mass*acceleration);
return secondForce;
}

这里mass和acceleration的值是多少?这些变量未初始化。你必须先给它们赋值。

您应该启用编译器警告并注意它们。例如，以下是MSVC在最低警告级别上的说明:

warning C4700: uninitialized local variable 'mass' used
warning C4700: uninitialized local variable 'mass' used
warning C4700: uninitialized local variable 'acceleration' used

在c++中，使用未初始化的变量从来不是个好主意。与其他语言不同，c++中的大多数变量都是而不是自动初始化为0值。当要访问变量中包含的值时，这会导致程序禁止未指定或未定义的行为。

安全正确。初始化!例如:

double force = 0.0;
double mass = 0.0;

请注意，该语言还允许您先初始化，然后赋值，然后使用变量。例如:

double force;
force = 0.0;
cout << force;

但那是糟糕的风格。为什么要保留未初始化的变量，然后在可以一步完成所有操作的情况下再赋值呢?先不初始化变量还可以防止将其设置为const。

确保所有变量的正确初始化将使程序的行为定义和一致。但是，您还会遇到其他问题，因为您乘以0 mass，结果总是为0，这当然不是您想要的。然而，确定性行为是修复错误数学的一个很好的起点。

相关文章：