missing ')' before constant

When I debug the definitions for all my functions give me missing ')' before constant, and missing ';' before constant. It all looks good to me, just wondering what would make it give me that error and how I might be able to fix it?

Code:

---

#include <iostream>
#include <fstream>
#include <iomanip>
#include <cmath>
#define size 50

using namespace std;

void headings(ofstream &outFile);
float readData(ifstream &inFile, float customer[], float balance[], float);
void sort(ifstream &inFile, float balance[], float);
void printResults(ofstream &outFile, ifstream &inFile, float customer[], float balance[], float);

int main()
{
        //declare variables
        float customer[size];
        float balance[size];
       
        //open files
        ifstream inFile;
        ofstream outFile;
        inFile.open("prog7.txt");
        outFile.open("prog7.out");

        // funtion to print headings
        headings(outFile);
       
        // read from file
        readData(inFile, customer, balance, size);
       
        // function to sort in descending order
        sort(inFile, balance, size);

        // function to print results
        printResults(outFile, inFile, customer, balance, size);

        // close files
        outFile.close();
        inFile.close();
        return 0;
}
        void headings(ofstream &outFile)
        {
                outFile <<setw(10)<< "Customer ID"
                                <<setw(10)<< "Balance"
                                <<endl;
        }
        float readData(ifstream &inFile, float customer[], float balance[], float size)
        {
                for (int i = 0; i <size; i++)
                {
                        if (!(inFile>> customer[i] >>balance[i]))
                }
                        break;
                                return i;
        }
        void sort(ifstream &inFile, float balance[], float size)
        {
                float temp = 0;                       
        for(int j=1; j< size; j++)
                {
                        for(int i=0; i< size-1; i++)
                        {
                                if(balance[i] > balance[i+1])
                                {
                                        temp=balance[i];
                                        balance[i]=balance[i+1];
                                        balance[i+1]=temp;
                                }
                        }
                }
        }
        void printResults(ofstream &outFile, ifstream &inFile, float customer[], float balance[], float size)
        {
                outFile << setprecision(2) << fixed << showpoint;
                for (int i=0; i < size; i++)
                outFile <<setw(10)<< customer[i]
                                <<setw(10)<< balance[i]
                                <<endl << endl;
        }

---

I got some errors that look very different from what you are describing. The ones you describe simply mean that you are missing one of those characters when there should be one, and that it may cause some ambiguity in your code. However, I would recommend that you post the specific error messages you are getting, the line numbers, etc... and all that other good stuff.

Before doing that though, I recommend you proofread your code. You have a break statement in there for no apparent reason - wrong location, perhaps?

you have a conflict due to a macro:

Code:

---

#define size 50
void sort(ifstream &inFile, float balance[], float size)

---

becomes

Code:

---

void sort(ifstream &inFile, float balance[], float 50)

---

The convention is for all macros to be all capital letters ie: SIZE

Also, you might want to simply make it a constant instead of a macro (it helps prevent errors such as these). Convention is also all upper-case for constants as well.

Hi Barfchunk. I see that you have an error here:

Code:

---

        float readData(ifstream &inFile, float customer[], float balance[], float size)
        {
                for (int i = 0; i <size; i++)
                {
                        if (!(inFile>> customer[i] >>balance[i]))
                }
                        break;
                                return i;
        }

---

...after fixing this:

Code:

---

using namespace std;

const int SIZE = 50; // meets the standard!

void headings(ofstream &outFile);
/*...*/

---

I have fixed it for you:

Code:

---

float readData(ifstream &inFile, float customer[], float balance[], float size)
        {
                for (int i = 0; i <size; i++)
                {
                        if (!(inFile>> customer[i] >>balance[i]))
                            break;
                        return i;
                }
        }

---

...unless this isn't the logic that you really want. Anyway, it now compiles error-free.

Like I said before, it's very important to be careful. Compilers will find such errors and alert you to them. Correcting them will put you closer to getting your mission accomplished.

Best,

Steve

Thanks for all the help guys, got everything to work perfect now... these boards really help with the learning process.