switch statement converting struct char to struct int

oceaneyes2 · 12-10-2003, 03:40 PM

is there any way to make this work? If not how should i convert char to int when defined in a struct?

switch (char Employee[i].payCode)
{
case 'A': case 'a':
Employee[i].pay = 20;
break;

case 'B': case 'b':
Employee[i].pay = 15;
break;

case 'C': case 'c':
Employee[i].pay = 10;
break;

default
cout << "Invalid";
}

deiussum · 12-10-2003, 04:21 PM

Based on your previous post, I'm guessing payCode is defined in the struct as:

char payCode[2];

This is a char array, which cannot be used in a switch statement.

However, it appears from your case statements that you ONLY care about the first character of that. So...

Why not define it as char payCode instead? Either that or you could do the switch on Employee[i].payCode[0] instead...

Also, you have a char in that switch, which you probably don't want. You are trying to declare a new char variable named Employee[i].payCode[0], which is not only not what you want, it is also probably giving you compiler errors.

mr_segfault · 12-10-2003, 04:30 PM

Hi oceaneyes2,

First off there are a couple of small errors in your code as highlighted here in red:

Code:

switch(char Employee[i].payCode) // Point 1
{
  case 'A': case 'a':
    Employee[i].pay = 20;
    break;

  case 'B': case 'b':
    Employee[i].pay = 15;
    break;

  case 'C': case 'c':
    Employee[i].pay = 10;
    break;

    default  // Point 2
      cout << "Invalid";
}

1) The typename char is not needed here.
2) There is a colon (':') missing from the default keyword.

Here is some correct code.

Code:

struct employeeT
{
  char payCode;
  int pay;
};

int main()
{
  employeeT Employee;  

  doStuffWithEmployee(Employee);
  
  switch (Employee[i].payCode)
  {
    case 'A': 
    case 'a':
      Employee[i].pay = 20;
      break;

    case 'B':
    case 'b':
      Employee[i].pay = 15;
      break;

    case 'C': 
    case 'c':
        Employee[i].pay = 10;
        break;

    default:
       cout << "Invalid";
  }
}

Or even more simple (yet possibly less efficient)

Code:

#include <ctype.h> // for toupper

struct employeeT
{
  char payCode;
  int pay;
};

int main()
{
  employeeT Employee;  

  doStuffWithEmployee(Employee);
  
  switch (toupper(Employee[i].payCode))
  {
    case 'A': 
      Employee[i].pay = 20;
      break;

    case 'B':
      Employee[i].pay = 15;
      break;

    case 'C': 
        Employee[i].pay = 10;
        break;

    default:
       cout << "Invalid";
  }
}

Hope this helps..

Cheers..