[HeAdHuNtErKg]

#include <stdio.h>

#define SIZE 20 
#define MAX_LEN 30

struct employee {
  char first[MAX_LEN];
  char last[MAX_LEN];
  double payperhr;
  double gross;
  double taxes;
  double hours_in_week;
};

  

int main() {

int i, j, n, k, m, hrin, minin, hrout, minout, temphours;
char first, last, L;

FILE* ifp;
ifp = fopen("clock2.txt","r");

struct employee workers[SIZE];

for (i=0; i<n; i++) {
fscanf (ifp,"%s%s%lf", workers [i].first, workers [i].last, workers [i].payperhr);
}

fscanf(ifp,"%d", &k);

for(i=0; i<k; i++) {
    for(j=0; j<n; j++) {
        workers [j].hours_in_week = 0;
        }
        
        fscanf(ifp, "%d", &m);
    
    for(i=0; i<m; i++) {
       fscanf(ifp, "%s%s%d%d%d%d", last, first, &hrin, &minin, &hrout, &minout);
       temphours = calc_hours(hrin, minin, hrout, minout);
       
       for(L=0; L<n; L++) {
          if(strtcmp(first, workers[L].first) == 0 && (last, workers[L].last) == 0) {
             workers[L] += temphours; //line where I receive the error
             }
             }
}             
                              
     for(j=0; j<n; j++) {
         if(workers[j].hours_in_week > 40) {
            workers [j].gross += (40 * workers[j].payperhr) + (workers[j].hours_in_week        
            -40) * 1.5 * workers[j].payperhr;
            
            workers[j].taxes += 0.1 * (40 * workers[j].payperhr) + 0.2 * 
            (workers[j].hours_in_week - 40) * 1.5 * workers[j].payperhr;
            
            }
            }
}            
fclose(ifp);

FILE *ofp;
ofp = fopen("w2.txt", "w");

fprintf(ofp, "w2 Form\n");
fprintf(ofp, "-------\n");

for(j=0; j<n; j++)
    fprintf(ofp, "Name %s  %s\n", workers[j].first, workers[j].last);
    fprintf(ofp, "Gross Pay %lf\n", workers[j].gross);
    fprintf(ofp, "Taxes Withheld %lf\n", workers[j].taxes);
    fprintf(ofp, "Net Pay %lf", workers[j].gross - workers[j].taxes);
    

system("PAUSE");
return 0;
}

double calchours(int hrin, int minin, int hrout, int minout) {
       int hours = hrout - hrin;
       int min = minout - minin;
       if(min < 0) {
          min += 60;
          hours--;

}
double tothours = hours + 1.0 * min/60;
return tothours; 
}          

I apologize for my stying if it's difficult to read. Line 46 is where I receive the error. Thanks in advance for any help

[ButchDean]

The compiler doesn't know what to invoke in order to add to a 'type' of 'employee'. You need to add code to do this.

[HeAdHuNtErKg]

Thanks for the response. Would you mind explaining this? I'm relatively new to programming and not really sure what code to add.

[r.stiltskin]

workers[L] is a worker, not a number.
temphours is an integer, not a worker.

Now can you see that you can't simply "add" these two things together?

Presumably you wanted to add the number of temphours to the hours_in_week field that belongs to that employee, but you have to tell that to the compiler explicitly.

[HeAdHuNtErKg]

Ahhh I see. Wow that was so obvious. I really appreciate the help

[HeAdHuNtErKg]

I have one more question: When I try to compile I get linker errors. These include undefined reference to 'calchours' and undefined reference to 'strtcmp'. I'm not sure what this means.
Thanks again!

[ButchDean]

Can you copy any paste the exact messages, please?

[JackOfAllTrades]

It's not strtcmp, it's strcmp. And you should #include <string.h> if you're using strcmp.

[HeAdHuNtErKg]

Of course:

[Linker error] undefined reference to `calc_hours'
[Linker error] undefined reference to `strtcmp'
ld returned 1 exit status

[ButchDean]

JackOfAllTrades, on 03 December 2010 - 06:15 PM, said:

It's not strtcmp, it's strcmp. And you should #include <string.h> if you're using strcmp.

Oh yeah!

[HeAdHuNtErKg]

Thanks JackOfAllTrades. That got rid of the linker error but now I get this error message:

45 [Warning] passing arg 1 of `strcmp' makes pointer from integer without a cast.

[ButchDean]

You've called

double calchours(int hrin, int minin, int hrout, int minout)

Before declaring it.

Put this at the top of your code after your struct definition.

double calchours(int hrin, int minin, int hrout, int minout);

[HeAdHuNtErKg]

I'm not sure if this thread is still active but I'm still getting one error in my code:

"[Warning] passing arg 1 of `strcmp' makes pointer from integer without a cast"

This shows up for line 45 of my code. I just can't seem to fix it

This post has been edited by HeAdHuNtErKg: 03 December 2010 - 05:11 PM

[r.stiltskin]

You're supposed to be comparing two strings. On line 45, your arg 1 (the first argument you are passing to strcmp) is first, which is a char, not a char array.

Also, I just noticed that you also declared last to be a char, rather than an array. You need to change both of those to arrays big enough to accomodate the strings that you're storing in them.

[HeAdHuNtErKg]

Do I declare these arrays in the struct employee definition before main or in the 'strcmp' itself? Sorry I'm just trying to grasp this stuff.