enum in class

[toomanystars]

Hi guys,

Looking for a little help with enum. Here's the code:

No matter where I put the decleratiron "compensation comp;", in the constructor for example, or the input function, it's not recognized across both input and the print function.

class Employee{

public:
	Employee(){}
	void input(){
		char choice;
		compensation comp;//issue here in decleration
		cout<<"Enter the name: ";
		cin>>name;
		cout<<"'d' for Daily\n'w' for Weekly\n'm' for Monthly\n'y' for yearly\n";
		cin>>choice;
		switch(choice){
		case 'd': comp=Daily;break;
		case 'w': comp=Weekly;break;
		case 'm': comp=Montly;break;
		case 'y': comp=Yearly;break;
		default: break;
		}
		
	}
	void print(){
		switch(comp){//get error here
		case Daily:cout<<"Daily\n";break;
		case Weekly:cout<<"Weekly\n";break;
		case Montly:cout<<"Montly\n";break;
		case Yearly:cout<<"Yearly\n";break;
		default: break;
		}
	}
private:
	char name[60];
	enum compensation{Daily, Weekly, Montly, Yearly};
};

int main()
{
	Employee e1;
	e1.input();
	e1.print();
	_getch();	
    return 0;
}

However this works! But it's all buried in one functions which is not what I want.

class Employee{

public:
	Employee(){}
	void input(){
		char choice;
		compensation comp;
		cout<<"Enter the name: ";
		cin>>name;
		cout<<"'d' for Daily\n'w' for Weekly\n'm' for Monthly\n'y' for yearly\n";
		cin>>choice;
		switch(choice){
		case 'd': comp=Daily;break;
		case 'w': comp=Weekly;break;
		case 'm': comp=Montly;break;
		case 'y': comp=Yearly;break;
		default: break;
		}
		switch(comp){
		case Daily:cout<<"Daily\n";break;
		case Weekly:cout<<"Weekly\n";break;
		case Montly:cout<<"Montly\n";break;
		case Yearly:cout<<"Yearly\n";break;
		default: break;
		}
	}
	void print(){
	
	}
private:
	char name[60];
	enum compensation{Daily, Weekly, Montly, Yearly};
};

int main()
{
	Employee e1;
	e1.input();
	e1.print();
	_getch();	
    return 0;
}

Can anyone tell me where to declare the compensation, so it's recognized in all functions?

THanks,

Mike

[Zhuge]

Sounds like you want it as a member of the class. So put compensation comp; between lines 31 and 32.

[vlad from moscow]

Define

enum compensation{Daily, Weekly, Montly, Yearly} comp;

and remove

compensation comp;//issue here in decleration

from the input.

[toomanystars]

Well that did the trick. But I did not think it was possible to initialize inside the private member area of the class. Is "compensation comp" not a sort of initialization?

[zb0S216C]

It looks like the compiler cannot find the compensation enumeration you're trying to base comp on. For this to work, the enumeration, compenstion, must be declared before declaring comp. For instance:

struct Employee
{
    enum Compensation
    {
        // ...
    } comp_;
};

_Wazzak

[Zhuge]

Well that did the trick. But I did not think it was possible to initialize inside the private member area of the class. Is "compensation comp" not a sort of initialization?

It's not any different than what you're doing on line 31.