Thread: Payroll program using Arrays and functions

I am having trouble with a payroll program working with arrays and functions. What do you see in the program that would have to be changed to get it to work? File is attached.

What do you see in the program that would have to be changed to get it to work?

For one, there's no function ot() defined, but you're passing its return value as an argument to printf. I'd try to compile and see what other not so obvious warnings/errors the compiler throws.

I declaired ot a functions and reduced the errors to nine. I have a attached image to show what the errors are. Other than that for now I am trying to figure out a way to make this program work. Any other pointers, things that stick out or ideas that would make this program work?

Well a majority of the errors you are getting should be self-explanatory.

If we start with the first:

>> 88 error: too few arguments to function 'ClockHours'

The number corresponds to the line number on the left margin, where the compiler thinks the error is. The text of the error, "too few arguments" is saying that you are calling ClockHours without supplying values for its parameters. A number of syntactical errors could cause this as well, like typing a period for a comma. Just scan over your file, specifically where you call ClockHours, and try to fix the call statements.

It may turn out that that error is caused by this error:

>> 89 error: subscripted value is neither array nor pointer

This means that you are giving a normal data variable a subscript (as in [index]) where it is not appropriate. You should be using the variable as it can be used. If it needs to be used like an array, you need to make appropriate changes for that to be right.

Just as this will generate the same error:

Code:

double payroll;
int i = 0;
double whatever = /* .. */
payroll[i] = whatever;

payroll ought to be declared as an array if that is correct usage of 'payroll'.

At that point, after fixing those two things, I'd try to compile again. The process repeats until the build log is empty.

This helpful programming tutorial brought to you by Apply Your Brain, Inc.

I would need some more example code to guide me in the right direction, the description is good but I would need some more example code to help me better to figure this out. I have been struggling with understanding functions.

Code:

int Add( int op1, int op2 )
{
        return op1 + op2;
}

This is an example of a simple function named "Add". It has two parameters: op1 and op2. When I call this function (say, from main() or another function) I must supply it two arguments. These arguments correspond to the parameters used in the function declaration/definition.

For example:
I can call the function from main as

Code:

int main( void )
{
        int sum;

        sum = Add( 12, 19 ); /*Or any other combinations of two numbers, for that matter*/

        return 0;
}

If I do not supply the function with enough arguments (in the above example, the arguments were 12 and 19) then the compiler will give me an error.

Code:

int main( void )
{
        int sum;

        sum = Add( 12 );

        return 0;
}

This code will give me an error as I supplied only one argument to the function "Add" but it's defined as taking two.

I have cleaned some of the errors, now I get this error C2120 'void' illegal with all types. Which I don't know how to fix it. When you think you know you have figured out the problem and think the program is going to work you get 12 errors when you recompile. Can anyone fix this program from here to get this program to work? The files are attached.

Look at your function:

Code:

void CalcGrossPay (float hours, float ot, float Gross_Pay, float Wage)
{
    int Hours;

    /* check for overtime and calculate gross*/
    if (ot > 0) {
	Gross_Pay = (STD_HOURS * Wage) /* regular pay */
                             + (ot * Wage);     /* ot pay */
    } /* if */
    else  /* not ot */
    {
       Gross_Pay = Wage * Hours;
    } /* else */
    return(Gross_Pay);
} /* CalcGrossPay */

You are trying to return a value from a void function. You should be defining this function to return a float.

Again, it's just about trying to understand your errors. Just from eyeballing the code you can see that some return values are mixed up. With minimal changes I got a clean compile:

Code:

float ClockHours (float Hours )
{
    /*int ClockNumber; unused variable?*/

    printf ( "Enter hours worked for clock %6f: ", Hours );
    scanf ( "%f", &Hours );
    return ( Hours );
}

float CalcGrossPay (float hours, float ot, float Gross_Pay, float Wage)
...

int main()
{
    ...
    return 0;
}

Another problem is comment styles. In strict C90 and earlier, only /* comments */ are allowed. The C++ style
// line comments
were added in C99. It's mostly a pet peeve of mine, but people should just stop turning to C99 just for comments.

Try to understand your errors.

Good point jimblumberg and whiteflags, Thank you. I'll continue to modify it and reply when I have another struggle.

I took the help from previous post and restructured the payroll program with reorganized functions now it works. Thank you everyone.