Object-Oriented Programming
-
code:
#include
#include
#include
#include
class employee
{
private:
int nSalary;
char name[80];
int nDept;
public:
void SetSalary(int);
void PrintSalary();
void SetName(char *aString);
protected:
};//end class employee
void employee::SetSalary(int s)
{
nSalary=s;
}//end void employee::SetSalary
void employee:: PrintSalary()
{
cout<<"nSalary of "<}//end void employee:: PrintSalary()
void employee::SetName(char *aString)
{
if(strlen(aString)<=79)
{
strcpy(name,aString);
}
else
{
strcpy(name, "Default" );
}
}//end void employee::SetName(char *aString)
class manager:public employee //Derived class
{
private:
int nOffice;
public:
void SetOffice(int);
void PrintOffice();
protected:
};//end class manager:public employee
void manager::SetOffice(int o)
{
nOffice=o;
}//end void manager::SetOffice(int o)
void manager:: PrintOffice()
{
cout<<"nOffice of "<}//end void manager:: PrintOffice()
anyone knows how can I print out the name of the manager in the last line.
Employee is the base class,manager is the derived class.
manager.name is obviously not correct.
-
what language is this?
-
C++
as seen from cout
long time since I have touched it, so I cannot help. Sry -
Wah~ me like 10 yrs never touch C liao... no wonder can't recognise...
-
Not trained in this language... I think Fatum may know.
-
have u tried
cout<<"nOffice of "<
because I think derived class means it is still something within the manager class -
I guess you can try declaring the employee.name as a public variable instead, so that it can be accessed by its derived classes. But anyway, I'm not that good in OOP, so please correct me if I'm wrong.
Hence, the code should be:code:
class employee
{
private:
int nSalary;
int nDept;
public:
void SetSalary(int);
void PrintSalary();
void SetName(char *aString);
char name[80];
protected:
};//end class employee -
lol. i got A for that module. but then i forget everything liao lol.
-
havent read thru the code...
but it's bad to declare name as public
the standard way is to make a public assessor to access the name instead -
u can do thisOriginally posted by thoreldan:havent read thru the code...
but it's bad to declare name as public
the standard way is to make a public assessor to access the name instead
public:
char* PrintName();
-------------
inside just return the name lor -
Hmm let me learn something here.Originally posted by thoreldan:havent read thru the code...
but it's bad to declare name as public
the standard way is to make a public assessor to access the name instead
So you mean something like, under "public:" of the employee class, you have a method named GetName() which basically returns the private "name" property?
PS:
Argh you answered my question before I posted it. lol.
So what's so bad about making a property/var public? -
try it...see if it worksOriginally posted by LatecomerX:Hmm let me learn something here.
So you mean something like, under "public:" of the employee class, you have a method named GetName() which basically returns the private "name" property?
PS:
Argh you answered my question before I posted it. lol. -
error C2248: 'name' : cannot access private member declared in class 'employee'Originally posted by eagle:have u tried
cout<<"nOffice of "<
because I think derived class means it is still something within the manager class -
yup it works.thanks.Originally posted by thoreldan:try it...see if it works
care to explain more on ur method of accessing a private member variable of a private class? -
Your variable "name" in the employee class is private; it's protected. You can't access it this way. Will need to declare it as public if you want to access it.Originally posted by Xcert:error C2248: 'name' : cannot access private member declared in class 'employee' -
mi o level nia
-
yup.LatecomerX's method also works...but my problem is that "name" must be declared as private(a requirement) so I cant suka suka change it to public.Originally posted by ndmmxiaomayi:Your variable "name" in the employee class is private; it's protected. You can't access it this way. Will need to declare it as public if you want to access it. -
nowadays primary sch learn how to program robots.Originally posted by croco2006:mi o level nia 
-
Private can only access with private... as far as I know. LateComerX's method will work as well, because it's within the same class. It's not like another class is trying to access the private class.Originally posted by Xcert:yup.LatecomerX's method also works...but my problem is that "name" must be declared as private(a requirement) so I cant suka suka change it to public.
eagle's method is actually utilizing another class to access the employee class, so it runs foul, producing an error. The error is quite easy to understand:'name' : cannot access private member declared in class 'employee'
-
Yep, use LateComerX's method.
-
always use a accessor instead...Originally posted by ndmmxiaomayi:Your variable "name" in the employee class is private; it's protected. You can't access it this way. Will need to declare it as public if you want to access it.
declaring instance variable as public is very bad programming.
anyway.. private and protected can mean different thing -
I stand corrected. I've always used public.Originally posted by thoreldan:always use a accessor instead...
declaring instance variable as public is very bad programming.
anyway.. private and protected can mean different thing
-
Originally posted by ndmmxiaomayi:I stand corrected. I've always used public.
-
Me curious here, why is it a bad programming practice to declare such vars as public?Originally posted by thoreldan:always use a accessor instead...
declaring instance variable as public is very bad programming.
anyway.. private and protected can mean different thing
And to mayi: in the future, please spell my username as "LatecomerX', not with the "C" capitalized. A big "C" there looks...kinda weird. -
See wrongly.Originally posted by LatecomerX:Me curious here, why is it a bad programming practice to declare such vars as public?
And to mayi: in the future, please spell my username as "LatecomerX', not with the "C" capitalized. A big "C" there looks...kinda weird.