Question

Why is my code not calculating the pay and not printing entire data at the end?

//

// main.cpp

// Project 14

//

// Created by Esmeralda Martinez on 5/13/19.

// Copyright © 2019 Esmeralda Martinez. All rights reserved.

//

#include<iostream>

#include<fstream>

#include<cstdlib>

#include<regex>

#include <iomanip>

using namespace std;

float grossPay(float hrsWorked, float payrate);

float grossPay(float hrsWorked, float payrate){

return hrsWorked*payrate;

  

  

}

int main(){

  

//opening an output file

ifstream outFile("Employee.txt", ios::in);

//Read variables

string depId,emp_num,firstName,lastName,email,hrs_worked,pay_rate,ch="y";

  

//Regex patterns

regex integerPattern("(\\+|-)?[[:digit:]]+");

regex stringPattern("^[a-zA-Z]{1,20}$");

regex emailPattern("(\\w+)(\\.|_)?(\\w*)@(\\w+)(\\.(\\w+))+");

regex floatPattern("[0-9]*\\.[0-9]+|[0-9]+");

  

cout << "Please select 'y' to begin, anything else to exit"<<endl;

cin >> ch;

  

while(ch=="y" || ch=="Y"){

  

cout << "Enter department ID number: ";

cin >> depId;

while (!regex_match(depId, integerPattern)) {

cout << "Department ID is incorrect!!Only numbers allowed" << endl;

cout << "Enter department number: ";

cin >> depId;

}

//Prompt for employee number and error check

cout << "Enter employee number: ";

cin >> emp_num;

while (!regex_match(emp_num,integerPattern)) {

cout << "Employee ID is incorrect!!Only numbers allowed" << endl;

cout << "Enter employee number: ";

cin >> emp_num;

}

//Prompt for first name and error check

cout << "Enter first name: ";

cin>>firstName;

if (!regex_match(firstName, stringPattern)) {

firstName = firstName.substr(0, 20);

}

//Prompt for first name and error check

cout << "Enter last name: ";

cin >> lastName;

if (!regex_match(lastName, stringPattern)) {

lastName = lastName.substr(0, 20);

}

//Prompt for email and error check

cout << "Enter email address: ";

cin >> email;

while (!regex_match(email, emailPattern) ){

cout << " Please enter valid email address" << endl;

cout << "Enter email address: ";

cin >> email;

}

//Promtp for hours worked and error check

cout << "Enter hours worked:";

cin >> hrs_worked;

while (!regex_match(hrs_worked, floatPattern)) {

cout << "Incorrect value, please re enter " << endl;

cout << "Enter hours worked: ";

cin >> hrs_worked;

  

}

  

//Promtp for pay rate and error check

cout << "Enter hourly pay rate:";

cin >>pay_rate;

while (!regex_match(pay_rate, floatPattern)) {

cout << "Incorrect value, please re enter" << endl;

cout << "Enter hours pay rate:";

cin >> pay_rate;

}

}

//convert into fload

float payrate = stof(pay_rate);

float hrsWorked = stof(hrs_worked);

  

//call to function

grossPay(hrsWorked, payrate);

if(hrsWorked>40){

return (payrate*40)+(hrsWorked-40)*payrate*1.5;

  

  

  

  

  

//display menu

cout<<"Employee ID: "<<emp_num<<" Employee's Name: "<<lastName<<" , "<<firstName<<" Email Address: "<<email<<" Regular hours worked: "<<hrs_worked<<" Overtime hours worked: "<<hrsWorked-40<<" Current Pay Rate "<<&rand<<" GrossPay: "<<&grossPay<<endl;

  

//Prompt to continue or end

cout<<endl<<endl<<" Would you like to review more files? (y/n) ";

cin>>ch;

  

}

return 0;

}

My program is supposed to calculate pay and then print out the calculation along with the user input. At this moment, however, my program ends after gathering the data.

How can I fix that?

Answer 1

if(hrsWorked>40){

return (payrate*40)+(hrsWorked-40)*payrate*1.5;

There is a problem with the above return statement. I also suppose there must be compilation errors in this program too as the brace of if condition isn't closed.

Anyways I see that you have already defined the function grossPay right in the beginning. Modify it in the following way..

float grossPay(float hrsWorked, float payrate){

if(hrsWorked>40){

return (payrate*40)+(hrsWorked-40)*payrate*1.5;

}

else {

return hrsWorked*payrate;

}

Remove the below piece of code.

if(hrsWorked>40){

return (payrate*40)+(hrsWorked-40)*payrate*1.5;

Because when you call the grossPay function, it checks for the hrsWorked>40. If it is true, then, it returns (payrate*40)+(hrsWorked-40)*payrate*1.5; otherwise it returns hrsWorked*payrate;

Do get in touch after making the above modifications. For some reasons I couldn't compile the code. Probably the above change should fix the issue.