问题

我是C的新手,我不明白为什么我收到此错误.在类似的3个标记错误的5个语句中,其他两个没问题.错误在主函数中.

     #include <iostream>
using namespace std;

// Function declaration
void getGallons(int wall);
void getHours(int gallons);
void getCostpaint(int gallons, int pricePaint);
void getLaborcharges(int hours);
void getTotalcost(int costPaint, int laborCharges);

// Function definition
void getGallons(int wall)
{
    int gallons;

    gallons = wall / 112;

    cout << "Number of gallons of paint required: " << gallons << endl;


}

// Function definition
void getHours(int gallons)
{
    int hours;

    hours = gallons * 8;

    cout << "Hours of labor required: " << hours << endl;


}

// Function definition
void getCostpaint(int gallons, int pricePaint)
{
    int costPaint;

    costPaint = gallons * pricePaint;

    cout << "The cost of paint: " << costPaint << endl;
}

// Function definition
void getLaborcharges(int hours)
{
    int laborCharges;

    laborCharges = hours * 35;

    cout << "The labor charge: " << laborCharges << endl;
}

// Funtion definition
void getTotalcost(int costPaint, int laborCharges)
{
    int totalCost;

    totalCost = costPaint + laborCharges;

    cout << "The total cost of the job: " << totalCost << endl;
}

// The main method
int main()
{
    int wall;
    int pricePaint;

    cout << "Enter square feet of wall: ";
    cin >> wall;

    cout << "Enter price of paint per gallon: ";
    cin >> pricePaint;

    getGallons(wall);

    getHours(gallons); // error here

    getCostpaint(gallons, pricePaint);

    getLaborcharges(hours); // error here

    getTotalcost(costPaint, laborCharges); //error here

    return 0;

}
 

这个教训侧重于在代码中使用函数和传递参数.我不应该使用全局变量.如果您有更好的方法来做到这一点,请分享.

  最佳答案

减少到三行(其他错误类似):

 int wall;    
getGallons(wall);
getHours(gallons); // error here
 

虽然wall被定义,但gallons不是.你想从哪里获得gallons?结果隐藏在另一个函数中.你想从那里获取它?

嗯,您需要一个返回值:

   int getGallons(int wall)
//^^^ !
{
     int gallons = wall / 112;
     // ...
     return gallons; // !
}
 

这样,您可以使用这样的函数:

 int gallons = getGallons(wall);
// now gallons is defined and you can use it:
getHours(gallons);
 

类似于其他函数和变量。

通常,在相同的fucntion中混合逻辑(计算)和输出是不好的想法.所以我宁愿将写入控制台转换为main函数:

 int getGallons(int wall) { return wall / 112; }
int getHours(int gallons) { return gallons * 8; }

int wall;
std::cin >> wall;
int gallons = getGallons(int wall);
std::cout << ...;
int hours = getHours(gallons);
std::cout << ...;
 

注意?所有输入/输出现在都是相同的级别...

附注:如果在定义之前没有使用它们,则在定义它们之前没有必要声明函数:

 //void f(); // CAN be ommitted
void f() { };
void g() { f(); }
 

反例:

 void f();
void g() { f(); } // now using f before it is defined, thus you NEED do declare it
void f() { };
 

如果您仍然希望保留声明是一个风格问题(但是在管理不同编译单元中的代码时会变得重要,因为您将声明放在头文件中 – 您将在接下来的课程中遇到).

  相同标签的其他问题

c++variablesparametersundefinedidentifier