Enum issue

I read a few tutorials on enum types, thought it'd be easy to use them until i had this idea of practicing it with classes, here's what i came up with:

person.h

Code:

using namespace std;

class Person
{
    public:
        enum Sex {FEMALE, MALE};
        // constructors
        Person();
        Person(string, string, enum Sex, int);

        // getters
        string GetFullName() const { return string(FirstName+" "+LastName); }
        string GetFirstName() const { return string(FirstName); }
        int GetAge() const { return (2010 - YearBorn); }

        string GetSex(string, string) const;

        // setters
        void SetFirstName(string str) {this->FirstName = str;}
        void SetLastName(string str) {this->LastName = str;}
        //void SetSex  stuck

    private:

        string FirstName;
        string LastName;
        Sex Sex;
        int YearBorn;

};

Person::Person()
{
    cout<<"PARAMETERLESS CONSTRUCTOR SHOULD NOT BE CALLED."<<endl;
}

Person::Person(string fName, string lName, enum Sex sex, int yBorn)
    : FirstName(fName), LastName(lName), Sex(sex), YearBorn(yBorn)
    {

    }

string Person::GetSex(string strMale="man", string strFemale="woman") const
{
    string returnStr;
    returnStr = (this->Sex) ? strMale : strFemale;
    return returnStr;
}

main.cpp

Code:

#include <iostream>
#include <string>
#include "person.h";

using namespace std;

int main()
{
    Person p1("John","Willians",MALE,1989);
    cout<<p1.GetFullName()<<" is a "<<p1.GetSex("guy","chick");

    return 0;
}

i just cant figure out how make enums work with classes.

btw, any help non regarding the enum issue is also WELCOME. thanks

Code:

Person p1("John","Willians",Person::MALE,1989);

Since your enum is inside the Person class then it's under the scope of that class.

SourceForge.net: Do not remove parameter names - cpwiki
Aside from that, some other heads up:

>> string GetFullName() const { return string(FirstName+" "+LastName); }
>> string GetFirstName() const { return string(FirstName); }
Casting to a string is not necessary here.

>> void SetFirstName(string str) {this->FirstName = str;}
>> void SetLastName(string str) {this->LastName = str;}
The "this->" is not necessary here.

Furthermore, it is bad practice to use using namespace in headers, because that would force the using namespace on anyone using the header. Suggest you avoid that and use explicit qualification, eg std::string instead. Using namespace is fine in source files.
Also, why do you have a parameterless constructor if you don't want anyone to call it?

Code:

Person::Person()
{
    cout<<"PARAMETERLESS CONSTRUCTOR SHOULD NOT BE CALLED."<<endl;
}

As to parameterless constructor, just remove it from the class, and the compiler will make sure that it won't be called.

You can declare the enum outside the class if you don't want to have to write Person::FEMALE or Person::MALE

Code:

enum Sex {FEMALE, MALE};

Your enumeration is wrong. Gender is a spectrum. Use float instead.

I can't tell if that's a joke or not, because that makes no sense...

Originally Posted by Elysia

I can't tell if that's a joke or not, because that makes no sense...

I have a strange sense of humor.