Declaration terminated incorrectly

I am trying to finish up a project on file I/O, and when compiling, I get the error "Declaration Terminated Incorrectly", but it marks the opening braces for one of the functions. What am I doing wrong?

Code follows:


/*
Read input from a file, calculate and print an inventory report,
including items that need to be reordered.
*/

#include <stdio.h>

int GetInvData();

int PrintReport (int *PartNo, float*Price, int *QtyOnHand,int *Reorder,int *MinOrder);

void ReportHeaders();
void PrintHeading();

int main (void)
{
PrintHeading();
ReportHeaders();

GetInvData();

return 0;
}

int GetInvData();
{
FILE *fpInvData;

int PartNo;
float Price;
int QtyOnHand;
int Reorder;
int MinOrder;

fpInvData = fopen("inv.dat","r");

fscanf(fpInvData, "%04d", &PartNo);
fscanf(fpInvData, "%2.2f", &Price);
fscanf(fpInvData, "%4d",&QtyOnHand);
fscanf(fpInvData, "%4d", &Reorder);
fscanf(fpInvData, "%4d", &MinOrder);

while (! feof(fpInvData))
{
PrintReport(int *PartNo, float *Price, int *QtyOnHand, int *Reorder, int *MinOrder);
}

printf("\n\n***End of Report***\n\n");
fclose(fpInvData);

}

Quote:

---

Originally posted by Griffin2020
I am trying to finish up a project on file I/O, and when compiling, I get the error "Declaration Terminated Incorrectly", but it marks the opening braces for one of the functions. What am I doing wrong?

Code follows:


/*
Read input from a file, calculate and print an inventory report,
including items that need to be reordered.
*/

#include <stdio.h>

int GetInvData();

int PrintReport (int *PartNo, float*Price, int *QtyOnHand,int *Reorder,int *MinOrder);

void ReportHeaders();
void PrintHeading();

int main (void)
{
PrintHeading();
ReportHeaders();

GetInvData();

return 0;
}

---

Do you see anything odd about your function call to 'PrintHeading' ... notice anything that isn't there that should be? If you're trying to just prototype, you're doing it wrong there.


Oh, and it could be this line:

Quote:

---

int GetInvData();
{
FILE *fpInvData;

---

See that nifty little semicolon? :D :D :D

Quzah.

oops, I knew it was something silly.

Now I get thris error, "Expression syntax in function GetInvData()"

It tags the line:
PrintReport(int *PartNo, float *Price, int *QtyOnHand, int *Reorder, int *MinOrder);

Do you define PrintReport somewhere?

PrintReport() should return an int, but I don't think that's the problem...

> PrintReport(int *PartNo, float *Price, int *QtyOnHand, int *Reorder, int *MinOrder);


Duh. Of freeking course it's going to give you an error! You're not CALLING this function there, you're freeking prototyping it again! You don't use "int *" and all that crap when you're actually making a function call.

Quzah.

Yes, I define it earlier (the initial post has the rest of the code), and it does return an integer.

Thanks in advance.

Okay..but if I do not specify the parameters when calling the function, it tells me not enough parameters....

Quote:

---

Originally posted by Griffin2020
Yes, I define it earlier (the initial post has the rest of the code), and it does return an integer.

Thanks in advance.

---

No, you only made the protype like this:

Code:

---

int PrintReport (int *PartNo, float*Price, int *QtyOnHand,int *Reorder,int *MinOrder);

---

You define the function like this:

Code:

---

int PrintReport (int *PartNo, float*Price, int *QtyOnHand,int *Reorder,int *MinOrder)
{
  ... //Whatever it do

  return ReturnVariable;
}

---

And as Quzah says, you are prototyping it again. Leave out the ints and only use whatever variables you want to pass to it.

Code:

---

ReturnVar = PrintReport (&PartNo, &Price, &QtyOnHand, &Reorder, &MinOrder);

---

Sorry, I yes, I actually defined the function, but was not having problems with the function itself, just the call.

I got it...thanks for the help, people...