Thread: Enum w/in Struct Help!

It's best if you just copy-paste your code and name the errors you get. It's not a long piece of code and some of us may not want to download, open and compile your code just to see what is wrong. Please read the announcement at the top of this forum.

Anyway, you need to overload the input operator for each of your enumerations.

Code:

istream& operator>>(istream &is, SexType &obj) {
    int input;

    if (is >> input) {
        switch (input) {
            case 0: case 1:
                obj = SexType(input);
                break;
      }
   }

  return is;
}

And you need to do the same to the other enum.

In another approach, you would have to read in their class and gender in a temporary string. I would think then a simple if/else can then determine what you should assign to those fields. For example, with gender:

Code:

string temp;
cout<<"Enter the gender: ";
cin>>temp;

if (temp == "male")
    student.sex = Male;
else if(temp == "female")
    student.sex = Female;
else 
    student.sex = Undisclosed;

Something like that. In non-trivial programs, you'd want to make sure that the string was in lowercase so that the comparisons work like the example. Mario's approach doesn't work the best though, as this is particularly error-prone.

Originally Posted by citizen

Mario's approach doesn't work the best though

Actually, there's a fundamental flaw on my code.

Code:

/* ... */
switch (input) {
    case 0: case 1:
        obj = Sex(input);
        break;
    default:
        is.setstate(istream::badbit);
/* ... */

Now, at least it has some form of error-checking.

Oh. I was aware of that. Was just mentioning that adding to a poor solution I was also not doing a proper error checking on the stream