cannor convert parameter...

[SpEkTrE]

I'm having this error occur and I'm not sure how to resolve it...

error C2664: 'readfile' : cannot convert parameter 1 from 'employee [14]' to 'int *'

here is where it is pointing to...

readfile(records, buf);

and here is the readfile portion...

    do{
        fgets(buf,200,fPtr);
        sscanf(buf, "%d %s %f %d", &records[i].id, &records[i].name, &records[i].pr, &records[i].csalary);

tks in advance

[itsme86]

I'm guessing the records array is of type struct employee. What does the prototype for readfile() look like? It looks like it should be void readfile(struct employee *records, char *buf);

[Salem]

Check your prototype / definition of the function

> 'employee [14]'
This is what you tried to call it with

> 'int *'
This is what it expected.

Solution: make them the same type

[SpEkTrE]

I'm guessing the records array is of type struct employee. What does the prototype for readfile() look like? It looks like it should be void readfile(struct employee *records, char *buf);

that seem to have worked, now I have another error..

 error C2228: left of '.csalary' must have class/struct/union type

from here ..

sscanf(buf, "%d %s %f %d", &records[i].id, &records[i].name, &records[i].pr, &records[i].csalary);

is this an error in my declaration?

[itsme86]

I think it would help if you showed us your struct employee definition.

[Salem]

Re-reading your book on what a struct is would help as well.

[SpEkTrE]

I think it would help if you showed us your struct employee definition.

struct employee 
{
    int id;
    char name[10];
    float pr;
    int csalary;
    int raise;
    double rate;
    int nsalary;
};

[quzah]

You sure do like to provide the absolute minimum in information, don't you? You never show what exactly 'employee' is. Is it an array? Is it a single instance? Is it even anything to do with your structure? We wouldn't know, because you never provide that information.

Do you show us the function prototype for 'readfile' so we can see how it's set up? No. Do you show us the variable declarations of the variables you're passing to it? No again.

If you want some real help, try actually helping us help you, instead of sitting around with your head up your ass wondering why you can't see clearly.

Quzah.

[SpEkTrE]

You sure do like to provide the absolute minimum in information, don't you? You never show what exactly 'employee' is. Is it an array? Is it a single instance? Is it even anything to do with your structure? We wouldn't know, because you never provide that information.

Do you show us the function prototype for 'readfile' so we can see how it's set up? No. Do you show us the variable declarations of the variables you're passing to it? No again.

If you want some real help, try actually helping us help you, instead of sitting around with your head up your ass wondering why you can't see clearly.

Quzah.

calm down there killer..i just didnt want to post the whole code...but here it is..

FYI: trail.txt is a file that consist of employee's

...

[quzah]

Much better, now all you need to do is learn to indent. Oh, and you may also want to actually pay attention to your compiler:

void readfile(struct employee *records, char *buf);

...

void readfile(int *records, char *buf)
{
    FILE *fPtr; 
    int i;
    do{
        fgets(buf,200,fPtr);
        sscanf(buf, "%d %s %f %d", &records[i].id, &records[i].name, &records[i].pr, &records[i].csalary);
    }
    while (!(feof(fPtr)));
}

While you're at it, read the FAQ on why using feof to control loops is a bad idea.

Quzah.

[itsme86]

Anoter thing is, you'll need to either pass fPtr from main to readfile() or open the file in readfile(). You declare an fPtr in readfile() but it isn't initialized. The one in main() is the one that's initialized.

[SpEkTrE]

I'm almost there...

But I believe it is not reading the data from the first file. Mainly because I have output of zero's. I did try to place a print statement just after the fgets and the sscanf, but I come up with the same outcome...

void readfile(struct employee *records, char *buf)
{
	FILE *fPtr; 
	int i;
	fPtr = fopen("lab2.txt", "r");
	do{
        fgets(buf,200,fPtr);
		sscanf(buf, "%d %s %f %d", &records[i].name, &records[i].name, &records[i].pr, &records[i].csalary);
	}
	while (!(feof(fPtr)));
}

looks okay to me, but of course, its not

[Stack Overflow]

Hello,

Small glitch:

sscanf(buf, "%d %s %f %d", &records[i].name, &records[i].name, ...

Looks like name is in there twice, with a %s and %d flag.


- Stack Overflow

[itsme86]

You might want to check the return value of fopen() to make sure it didn't fail. If it did, fPtr would be NULL after the call to fopen().

Also, why did you delete your code from your previous post?

[Stack Overflow]

Hello,

C++ library reference: fopen()


- Stack Overflow