Question

请指出具体在哪里以及需要进行哪些新的特定编辑。我总是遇到相同的错误，也不知道哪里出了问题。我已经折断了一百万人次了，很确定我做对了：

cpp：36：错误：“ {”标记之前不允许在此处进行功能定义
cpp：44：错误：在'{'标记之前不允许在此处进行功能定义
cpp：58：错误：“ double”之前需要使用初始化程序
cpp：63：错误：在'{'标记之前不允许在此处进行功能定义
cpp：69：错误：“ {”标记之前不允许在此处进行功能定义

代码：

preg_match()

Answer 1

您必须将函数放在main之前

void instructions() 
{
    cout << "This payroll program calculates an individual employee pay and";
    cout << "\ncompany totals using data from a data file payroll.txt.\n"; 
    cout << "\n\nA payroll report showing payroll information ";
    cout << " is displayed.\n\n";
}

顺便说一句，为了保持一致性并提高可读性，您应该将所有行更改都放在开头或结尾处。否则，很难看到，例如，payroll.txt和A payroll report之间有3行...

// Other functions here…
// If some functions are dependant on others, those need to be declared before they are used.

int main()
{
      // Some code here…

      // Call your function
      instructions();

      // More code afterwards…

      return 0; 
}

或者，您只能像这样在main之前声明函数：

void instructions();
void reportTitle();
double calculateGross(double hours, double hourly);
double calculateNet(double grosspay, double netpercent);

// For documentation purpose, you should name your arguments.
// Also the body of your function does not appears to do what its name suggest.
void displayEmployeeInfo(const string &, double, double, double, double, double);

// Show probably named displayTotalAmounts
void totalAmounts(double tgross, double tnet);

您需要知道，在调用函数时必须传递适当的参数。例如：

int main() // partial implementation
{
    double tgross = 1.0; // Whatever code you need to have desired value...
    double tnet = 0.90;

    totalAmounts(tgross, tnet);

    return 0;
}

如果使用该选项，则可以在此处（在main之后）定义其他功能。

这给出了如何构造程序的基本思想。

阅读所有其他注释以查找代码中的其他问题！

还有一些其他事情：

您定义了变量fileName，甚至对其进行了初始化，但随后您使用字符串打开了文件。
如果您需要修改传递给函数的参数的变量，以便调用者可以看到更改，则需要通过引用将其传递。例如：double &tnet。
通常，最好避免在生产代码中使用using namespace std。
最好在首次使用变量时声明它们。
某些变量行net似乎从未被初始化。
如所写，指令if(!fin.is_open())似乎可疑。假定没有错误，此时文件将打开，但在这种情况下，您可能要显示标题！
用string初始化= ""是没有用的，因为字符串有一个默认构造函数将其创建为空。
我建议您在if或while之类的关键字和左括号之间添加一个空格。
此外，您应该保持一致。 totalAmounts之后是否有空格，而其他功能却不是。
也为您命名变量。当fileName小写时，为什么要为taxrate使用驼峰大小写。如果使用小写字母，则应使用_分隔单词（例如tax_rate），因为这样更易于阅读。
命名变量时应避免缩写。 total_net（或totalNet）比tnet更容易理解。
通常，当您遇到编译器错误时，第一个问题是在编译器报告的位置附近。修复该错误，然后检查其他错误是真正的错误还是第一个错误的结果。在这种情况下，它有助于编译单个文件（对于具有数百个文件的大型生产项目）。

Answer 2

此代码给出的警告少了一些，但您实际上必须自己整理代码。我也不想说，全局声明所有变量是一个很好的解决方案。

#include <iostream>
#include <fstream>
#include <iomanip>
#include <string>
using namespace std;


string item = "";
ifstream fin;
double tgross = 0;
double tnet = 0;
double hourly;
double hours;
double taxrate;
double net;

void instructions() 
{
    cout << "This payroll program calculates an individual employee pay and";
    cout << "\ncompany totals using data from a data file payroll.txt.\n"; 
    cout << "\n\nA payroll report showing payroll information ";
    cout << " is displayed.\n\n";
}

void reportTitle() 
{
    cout << setprecision(2) << fixed << showpoint << left
        << setw(20) << "Employee" << setw(10) << "Hourly" << setw(10) << "Hours"
        << setw(10) << "Tax" << setw(10) << "Gross" << setw(10) << "Net" << endl;
    cout << setw(20) << "Name" << setw(10) << "Rate" << setw(10) << "Worked"
        << setw(10) << "Rate" << setw(10) << "Amount" << setw(10) << "Amount" << endl;
}

double calculateNet(double grosspay, double netpercent)
{
    return grosspay - grosspay*netpercent/100.0;
}

void displayEmployeeInfo(const string &, double, double, double, double, double)
{
    tgross += grosspay;
    tnet += net;
}

void totalAmounts (double tgross, double tnet)
{
    cout << "Totals" << setprecision(2) << fixed << showpoint << right
        << setw(50) << tgross << setw(10) << tnet << endl;
}

int main()
{   
    string fileName = "payroll.txt";    
    fin.open("payroll.txt");

    if(!fin.is_open())
    {   
        instructions();
        reportTitle();
    }

    while(!fin.eof())
    {
        getline(fin,item,'#');
        fin >> hourly >> hours >> taxrate;

        double calculateGross(double hours, double hourly);
    }

    fin.close();
}

不断收到相同的错误...怎么了？ C ++

2 个答案: