Thread: Structures, passing array of structures to function

Originally Posted by Dave_Sinkula

Code:

if(major!='e'||'h'||'m'||'c'||'b'||'u'){
			printf("ERROR! Invalid Error Code Entered\n");
			count--;
		}

You are comparing major to 1.

Code:

if(major!='e'||major!='h'||major!='m'||major!='c'||major!='b'||major!='u'){
			printf("ERROR! Invalid Error Code Entered\n");
			count--;
		}

Keep the switch and make this the default.

Thats not working either for some reason.

Originally Posted by Dave_Sinkula

D'oh! I knew I should have spent more time before posting. The || need to be &&.

Or again, go with the switch.

Code:

   switch ( major )
   {
   case 'e': case 'h': case 'm': case 'c': case 'b': case 'u':
      /* do stuff */
      break;

   default:
      printf("ERROR! Invalid Error Code Entered %c\n", major);
      return 0;
   }

Oh, I should have figured that out. Thanks.

Code:

scanf("%s",&person);

incorrect i belive it should be

Code:

scanf("%s",person);

Its an array. so u dont need the &

anyways its probably not the major concern

Originally Posted by tama00

Code:

scanf("%s",&person);

incorrect i belive it should be

Code:

scanf("%s",person);

Its an array. so u dont need the &

anyways its probably not the major concern

Hmm.. It seems to work either way.