I need help with one function in my code....I think

Ok so I've been working on this code for a couple days, and I just have one little problem with computing the pay. Whenever the DisplayPay function outputs the pay for the employee I get all zeros. I was hoping somebody could help me with my problem. I'll attach my code. Thanks for the help.

You need to transfer the result of the calculation from your ComputePay to DisplayPay. As it stands right now, you have some local variables in the functions ProcessManufacturingWorker() and ProcessOfficeWorker(), that are not the same variable as in the functions that calculate pay. So you print the values that haven't been calculated.

--
Mats

What do I need to do to fix that. I've tried to change the variables around so many times. This program is sucking the life out of me.

Since it's written in C++, I would pass in the relevant result variables as references to the calculation routine (and remove the local ones in the calculation routine, of course).

If this makes no sense to you, check the Tutorial here on "references".

--
Mats

Is what I did with my code correct. That tutorial was pretty brief.

Does it work? I don't think it's quite right, becaus you still have local variables for

Code:

---

        double overtimePay = 0;
        double bonusPay = 0;

---

inside CalculateHourlyPay() - and you need those to be available in the printing function, don't you?

--
Mats

When I execute the program and I enter the hours worked and hourly rate when I get the output the numbers are like -92360000000000000000000000 for some reason.

I'll attach my code as it is now.

You put references in the wrong place.

References mean that you are "transferring an alias rather than a copy" of a variable.

This in turn means that whatever you pass in can be changed, and the resulting change appears in the calling code.

If you just pass in a regular variable (without a reference) you get a copy in the called function (aka callee), and changes are only made to the callee's copy, not the "original" variable.

So in this case, you need to pass in the values (as references) you want calculated by the calculation function, so that you can pass the calculated result to the display function. Currently, you calculate a local value, and then throw it away again.

--
Mats

So do I need to put the & operator after my variables in the computehourlypay function?

Quote:

---

Originally Posted by NicAuf

So do I need to put the & operator after my variables in the computehourlypay function?

---

No, you need to REMOVE the local variables, there and pass them (with the & sign, but it's a storage modifier at this point, not really an operator).

--
Mats

Like this.....

Not really - you still don't pass the all the components to compute the pay to the function, do you?

Does the code you just posted compile? [My "brain-compile" says that you should have about four errors for "undefined variabe" in the CalculateHourlyPay function].

Try to make your code compile before you post it - it sorts out some of the more basic bits missing. Also try to re-read my previous post.

--
Mats

Ok well I've been working on that for an hour and I still can't get it to work. Help anybody!