Post the newly revised code.
Post the newly revised code.
Originally Posted by Adak
Code:#include <iostream>#include <fstream> #include <string> #include <iomanip> #include <cmath> int main() { using namespace std; string emp_name; double hours_worked; int hourly; double total_pay; double regular_pay = 40; double overtime; int id; double regular; cout << "Please enter your first and last name:" << endl; getline(cin, emp_name); cout << "Please enter employee ID number: " << endl; cin >> id; cout << "Please enter your hourly classifcation number 1-5:" << endl; // enter for the switch statement to clairify which one to pick cin >> hourly; cout << "Please enter the amount of hours:" << endl; // enter for the if else statement to pick cin >> hours_worked; switch (hourly) // At this point the person will choose 1 of the following employee hourly pay wages { case 1: hourly = 5.50; cout << "Hourly Wage: $" << hourly << endl; break; case 2: hourly = 6.00; cout << "Hourly Wage: $" << hourly << endl; break; case 3: hourly = 7.00; cout << "Hourly Wage: $" << hourly << endl; break; case 4: hourly = 9.00; cout << "Hourly Wage: $" << hourly << endl; break; case 5: hourly = 12.00; cout << "Hourly Wage: $" << hourly << endl; break; default: hourly = 5.50; break; } if (hours_worked < 40) // if else if statments to decide how much needs to be calculated. { total_pay = hours_worked * hourly; cout << "Name: " << emp_name << endl; cout << "Employee Id: " << id << endl; if (hourly == 1 || hourly == 2 || hourly == 3 || hourly == 4 || hourly == 5) { cout << "Job Classification: " << hourly << endl; } else { cout << "Job Classification not found please try again" << endl; } cout << "Regular Pay: $" << total_pay << endl; cout << "Hours Worked: " << hours_worked << endl; cout << "You need to work more hours! Your total pay is: $" << total_pay << endl; } else if (hours_worked == 40) { total_pay = regular_pay * hourly; cout << "Name: " << emp_name << endl; cout << "Employee Id: " << id << endl; if (hourly == 1 || hourly == 2 || hourly == 3 || hourly == 4 || hourly == 5) { cout << "Job Classification: " << hourly << endl; } else { cout << "Job Classification not found please try again" << endl; } cout << "Regular Pay:" << total_pay; cout << "Hours Worked: " << hours_worked << endl; cout << "Good job you worked 40 hours! Your pay is: $" << total_pay << endl; } else if (hours_worked > 40) { regular = regular_pay * hourly; overtime = (hours_worked - 40) * 1.5 * hourly; total_pay = overtime + regular_pay * hourly; cout << "Name: " << emp_name << endl; cout << "Employee Id: " << id << endl; if (hourly == 1 || hourly == 2 || hourly == 3 || hourly == 4 || hourly == 5) { cout << "Job Classification: " << hourly << endl; } else { cout << "Job Classification not found please try again" << ". Default Pay of 5.50 being calculated." << endl; } cout << "Hours Worked: " << hours_worked << endl; cout << "Regular Pay: $" << regular << endl; cout << "Overtime: $" << overtime << endl; cout << "You worked an excessive amount of hours! Your total pay is: $" << total_pay << endl; } return 0; }
You are using the same variable name, hourly, to refer to 2 things - (a) the hourly classification number (lines 26-27) and (b) the hourly pay inside the switch statements (lines 35, 38, etc). Since the latter are not scoped the value of the variable hourly, post switch, is now one of the hourly wages that is != any of the hourly classification numbers and so one of the messages "Job Classification not found ... " is printed later on.
So in order to fix this the code I would need to input would be:
Than change:Code:cout << "Please enter your hourly classifcation number 1-5:" << endl; // enter for the switch statement to clairify which one to pick cin >> hourly && x;
Because that as well gives me an error or did I mess that up as well?Code:if(hourly == 1 || hourly == 2 || hourly == 3 || hourly == 4 || hourly == 5) { cout << "Job Classification: " << x << endl; } else { cout << "Job Classification not found please try again" << ". Default Pay of 5.50 being calculated." << endl;
Just scope the switch statments:
Sample OutputCode:switch (hourly) // At this point the person will choose 1 of the following employee hourly pay wages { case 1: { double hourly = 5.50; cout << "Hourly Wage: $" << hourly << endl; } break; case 2: { double hourly = 6.00; cout << "Hourly Wage: $" << hourly << endl; } break; case 3: { double hourly = 7.00; cout << "Hourly Wage: $" << hourly << endl; } break; case 4: { double hourly = 7.00; cout << "Hourly Wage: $" << hourly << endl; } break; case 5: { double hourly = 9.00; cout << "Hourly Wage: $" << hourly << endl; } break; default: { double hourly = 9.00; cout << "Hourly Wage: $" << hourly << endl; } break; }
PS: please double check your payment calculations as well, @ an hourly wage of $7 I'd not be happy to receive $190 after working 45 hours!Code:Please enter your first and last name: John Smith Please enter employee ID number: 42345 Please enter your hourly classifcation number 1-5: 4 Please enter the amount of hours: 45 Hourly Wage: $7 Name: John Smith Employee Id: 42345 Job Classification: 4 Hours Worked: 45 Regular Pay: $160 Overtime: $30 You worked an excessive amount of hours! Your total pay is: $190
Btw my calculations were working before you fixed my switch statements funny enough lol.
Not sure where I went wrong the math on paper in the code seems right.
Yes, that's because hourly is now back to being the classification system and not the wage and so multiplying hours worked with hourly to get total pay will give you a lower number.
You declare the hourly pay variables inside the switch statements and want to use the values of these variable outside of them to get total pay and therein lies our problem. Try and declare the hourly pay variables at a scope level where they can be used both by the switch statements and by the total pay calculations
