problem...

i am just learning how to do classes, and i made this program, but when i try to compile, it gives me an "declaration terminated incorrectly". what am i doing wrong?

Code:

#include <iostream.h>
void public::sayhello()
{
    cout<<"hello there!";
}
void public::saygoodbye()
{
    cout<<"goodbye!";
}
int main()
{
int choice
cout<<"choose one:";
cout<<"[1] hello";
cout<<"[2] goodbye";
cin>>choice;
if(choice==1)
{
   sayhello();
}
if(choice==2)
{
   saygoodbye();
}
return 0;
}

int choice

should be

int choice;

drop the public:: in both of the following two lines:

void public::sayhello()

void public::saygoodbye()

When that is done you will have an example of a program that uses two simple functions and uses a simple menu to allow user to make a single choice each time the program is done. Unfortunately, this program has little to do with classes. To have the two functions belong to a class you would need to do something like this:

Code:

#include <iostream.h>

//declare the class 
class communicate
{
public:
  void sayhello();
  void saygoodbye();
};


//provide member function definitions
void  communicate::sayhello()
{
    cout<<"hello there!";
}
void communicate::saygoodbye()
{
    cout<<"goodbye!";
}


int main()
{
int choice;
communicate talk;
cout<<"choose one:";
cout<<"[1] hello";
cout<<"[2] goodbye";
cin>>choice;
if(choice==1)
{
   talk.sayhello();
}
if(choice==2)
{
   talk.saygoodbye();
}
char dummy;
cin >> dummy;
return 0;
}

At some point you will want to start putting class definitions in your own header files so you can reuse classes in more than one program, but based on the code as posted, I think there are a few more lessons to be learned between now and then.

uh, ok. i guess i didnt have that clear of an idea of what a class is. removing the public:: works fine, however, your second one produces a whole bunch of error messages in Borland C++ compiler 5.5.

thanks anyways!

I corrected the mistake in the original post for the class example. It runs fine now in BCB as a console program.

//declare class class communicate

should have been

//declare class
class communicate

Sorry

ohh, i see now, you had it on the same line so it just read as a comment! Now why didnt _i_ catch that?