help with one minor error

ok...this is really simple (even though i can't get it). My program only has one error which i can't seem to get out. Here is my code:

Code:

#include <iostream.h>
#include <fstream.h>
#include <stdio.h>

class person
	{
public:
	person()
	{
		fName = "\0";
		lName = "\0";
		number = 0;
	
	~person()
	{
	}

private:
	char fName[40];
	char lName[40];
	int number;
};

int main()
{
	
	for(;;)
	{
		int choice = 0;
		person p;
		cout << "Enter the person's first name: ";
		cin >> p.fName;
		cout "\nEnter the person's last name: ";
		cin >> p.lName;
		cout << "\nEnter the person's telephone number: ";
		cin >> p.number;

		file("data.txt", ios::ate);

		file << "\nID = ";
		file << i;
		file << "\nName: ";
		file << fName;
		file << " ";
		file << lName;
		file << "\nTelephone Number: ";
		file << number;
		file << "-----------------------------------\n\n";

		cout << "Data stored in data.txt...\n";
		cout << "Enter another?(y or n) ";
		cin >> choice;

		if(choice == 'n')
		{
			break;
		}
	}
	return 0;
}

and i keep getting this error:

Code:

fatal error C1004: unexpected end of file found

This error is pointing to just under the last close brace. I have tried everything like adding a brace/adding several braces and i have tried to take it away, but that doesn't work. BTW, i am using MSVC++ 6.0

Thanks
-Chris

Where is the ending '}' after the constructor in the person-class?

As I see it there must be more then one error, or ther will come more when you correct the missing }. Because you have made the class variables private, and trying to directly acces them with the object of the class. I might have missed something but as I see it, if you want to do that you must make the variables public, or make functions in the class that, from wich you can access them.

Yeah, I saw that too. I thought I maybe had misunderstood it so I didn't say anything.

you have a semicolon after the } on you private constructor

private:
char fName[40];
char lName[40];
int number;
};


^ the semi colon

My concerns include:

At the end of the declaration of a class/struct interface there should be a semicolon, so I think the semicolon issue brought up is in error, not the code as written, on that issue.

I agree with the missing } at the end of the default constructor.

I agree with need for public accessor/mutator functions to when using private member class variables within main().

You don't declare the variables i or file anywhere, yet you are using tbem.

the variable i is never incremented, I suspect it should be based on what I think your program is trying to do.

you declare the variable choice to be of type int but you assign it a char. This may cause a problem, may not. But unless you know what is happening I suggest you keep variable type matching as simple as you can.

You may need to use strcpy() in your constructor if the compiler feels the = sign means assignment rather than intialization in the function body. If it works as is, fine. If not, that's a change I would make.

So I agree. You are likely to uncover other errors once you fix the first one.

sorry about all of the dumb errors, guys but this code was written up pretty fast and wasn't given much thought so I decided to fix everything up except I can't fix this (crazy) error that I keep getting.

My new code:

Code:

#include <iostream.h>
#include <fstream.h>
#include <stdio.h>

class person
	{
public:
	person()
	{
		
		number = 0;
	}
	
	~person()
	{
	}


	char fName[40];
	char lName[40];
	int number;
};

person p;
int i = 1;
int main()
{
	
	for(;;)
	{
		int choice = 0;
		person p;
		cout << "Enter the person's first name: ";
		cin >> p.fName;
		cout "\nEnter the person's last name: ";
		cin >> p.lName;
		cout << "\nEnter the person's telephone number: ";
		cin >> p.number;

		ofstream file("data.txt", ios::ate);

		file << "\nID = ";
		file << i;
		file << "\nName: ";
		file << p.fName;
		file << " ";
		file << p.lName;
		file << "\nTelephone Number: ";
		file << p.number;
		file << "-----------------------------------\n\n";

		i++;
        cout << "Data stored in data.txt...\n";
		cout << "Enter another?(y or n) ";
		cin >> choice;

		if(choice == 'n')
		{
			break;
		}
	}
	return 0;
}

And here is the error:

Code:

  error C2143: syntax error : missing ';' before 'string'

My response to this error:
"...WHAT THE...??????"

The error happend on this line:

Code:

 		cout "\nEnter the person's last name: ";

I know, I haven't got the slightest clue either but has anybody go any ideas?

cout "\nEnter the person's last name: ";


don't you mean:

cout << "\nEnter the person's last name: ";

You forgot the <<.

The errors don't always tell you exactly where the error is. Sometimes you have to look for them a few lines up.

Anyway, fix the << and it should work. It worked for me

Thanks,
Marc

WHOA!!!
How the hell could I have missed THAT?? Thanks for pointing out that (SILLY) mistake....

no problem

this is dragging on a bit, isn't it? Well I have another problem but first, here is my new code...

Code:

#include <iostream.h>
#include <fstream.h>
#include <stdio.h>

class person
	{
public:
	person()
	{
		
		number = 0;
	}
	
	~person()
	{
	}


	char fName[40];
	char lName[40];
	int number;
};

int choice = 0;
person p;
int i = 1;
int main()
{
	
	for(;;)
	{
		
		cout << "Enter the person's first name: ";
		cin >> p.fName;
		cout << "\nEnter the person's last name: ";
		cin >> p.lName;
		cout << "\nEnter the person's telephone number: ";
		cin >> p.number;

		ofstream file("data.txt", ios::ate);

		file << "\nEntry # ";
		file << i;
		file << "\nName: ";
		file << p.fName;
		file << " ";
		file << p.lName;
		file << "\nTelephone Number: ";
		file << p.number;
		file << "\n-----------------------------------\n\n";

		i++;
        cout << "Data stored in data.txt...\n";
		cout << "Enter another?(y or n) ";
		cin >> choice;

		if (choice == 'n')
		{
			return 0;
		}
	
}
	return 0;
}

My new problem is that at the end of my prog (when it asks you to input a 'y' or an 'n') it goes CRAZY! Just compile it and take a look!

Any help will be apprieciated (is that spelt right...?)

Naughty!.. what's this?.. int choice = 0;
And you're trying to input a character right?

Well, you must replace int choice = 0; with char choice = 'x';

It should work now

Hope this helps,
Marc

oh another thing

Why don't you use a do...while loop?

do {
//code
}while (choice != 'n');

The code will be much cleaner that way

force of habit - I usualy say "...press a 1, 2 or 3 bla bla bla..." in a switch statment. That must be why I did that

it doesn't matter if you use char or int, you can do the same thing with both.