Compile error

[shawnt1993]

Write your question here.
I need help with an error that is preventing from compiling my code.
the error reads as:
error: 'Jon' was not declared in this scope.

Here is the code as written in vi

  
#include <iostream>
#include <fstream>
#include <string>
#include <iomanip>

using namespace std;

int main()
{
  ifstream infile;
  float hourly_rate;
  float regular_wage, overtime_wage, total_wage;
  int hours_worked;
  string name;

  infile.open("work_hours.txt");        //Open the file
  if (!infile) {                        //check if the file exists
    cout << "Cannot open the input file." << endl; // if not, exit the program
    return 1;
}

  if (hours_worked > 40) {
  regular_wage = 40 * hourly_rate;
  overtime_wage = (hours_worked - 40) * hourly_rate * 1.5;
  total_wage = regular_wage + overtime_wage;
}
  else
{
  total_wage = hours_worked * hourly_rate;
}

  getline(infile, Jon Doe);             //Read first employee name
  name = Jon Doe;

  infile >> hourly_rate;                //Read hourly pay rate

  infile >> hours_worked;               //Read hours worked

  cout << name << "'s weekly wage is $ "  << setprecision(2) << total_wage << endl;
}

[48T7M4Gy]

you have errors in lines 33 and 34

Jon Doe in line 33, the way you have written it, is a variable not the name of a perso on fil

You should have something like:

string name;
getline( infile, name );

( name is already declared at line 15 so it doesn't need to be repeated. )

[shawnt1993]

Thank you kemourt. I got the program to compile correctly now but it does not read the inserted file correctly. The program is supposed to read another file with that read as:

Jon Doe
20
30

it is supposed to take these numbers and use them to output Jon Doe's income of that week based off his hours. For whatever reason it comes out as $0 for his earnings. Do I need to declare more strings?

[48T7M4Gy]

It wouldn't be, total_wage = hourly_rate * hours_worked;, would it?

[shawnt1993]

You think that line isn't correct?

[48T7M4Gy]

I'm reasonably sure the line I wrote is correct. It's the same as your line 30 which I missed seeing. However line 30 is in the wrong place because the total_wage is being calculated before, not after which is the correct spot, you have read in the values.

Your calculation is 0 times 0 = $0.

[shawnt1993]

so if I switched lines 23-31 with lines 33-41 do you think the program would do the calculations after it reads the information from the other file?

[48T7M4Gy]

Shifting lines 23-31 should work. Leave the other lines where they are - you'll see they both don't have to be shifted. Try it. Don't forget to use 'undo/redo' if necessary.

But you are right, the details have to be read in before the calculation is done. You are dealing with a machine not a mind reader. :)

[shawnt1993]

That makes sense, I had gotten certain parts from my professor in a class and didn't even give thought to the order they should have been put in. Thanks for the help!