Thread: Help checking a character in a structure

I am currently writing a function that checks whether a character string contains an "F" or "M". I have tried strcmp() and a if statement. Here is the code I have tried, any help would be much appreciated.

Code:

void printFemale(Employee s[], int num) {
    int i;
    char *F;
    char female[1];
    female[0] = 'F';
    *F = female[0];
     printf( "The Female Employees Are\n");
     for( i = 0; i < num; ++i )
    {
    Trying this statement --->   if (s[i].sex[i] == 'F')

    Also have tried ----->    if (strcmp(s[i].sex, *F) == 0){
            printf("");
            printf("%.2f\n", s[i].pay);
        }
                    
    }
}

The code for the struct is as follows:

Code:

typedef struct Employee
{
    char first[8];
    char initial[2];
    char last[10];
    Address ad;
    int age;
    char sex[2];
    int years;
    double pay;
}Employee;

I have a number of other structures that work, but when checking the character in sex[2] I just seem to be missing something.

Thank you again.

I commented your code to show the different problems with what you tried.

Code:

void printFemale(Employee s[], int num) {
    int i;
    char *F;  // this is a char pointer, but it doesn't point anywhere meaningful, it points to garbage
    char female[1];  // this doesn't need to be an array, a single char would work
    female[0] = 'F';  // you set the one char in the array to 'F' -- this is okay
    *F = female[0];  // you assign the char pointed to by F (in garbage memory) to 'F', from the array -- this will likely seg fault or cause other nasty problems
     printf( "The Female Employees Are\n");
     for( i = 0; i < num; ++i )
    {
    Trying this statement --->   if (s[i].sex[i] == 'F')  // if i is not 0 or 1, you go out of bounds of the sex array.  perhaps s[i].sex[0] would work better
 
    Also have tried ----->    if (strcmp(s[i].sex, *F) == 0){  // F is a char pointer.  if F actually pointed somewhere useful, *F would be the single char it pointed to, which is not what strcmp wants, it wants a char pointer

            printf("");
            printf("%.2f\n", s[i].pay);
        }
                     
    }
}

In your case, I would do

Code:

if (s[i].sex[0] == 'F')
// or
if (strcmp(s[i].sex, "F") == 0)

Actually, if you really only intend to store a single character for the sex field in your struct, don't use an array/string, it just confuses you. Just make it char sex;

I tried that before, the if statement which is

Code:

void printFemale(Employee s[], int num) {
    int i;
    
    
     printf( "The Female Employees Are\n");
     for( i = 0; i < num; ++i )
    {
        if (s[i].sex[0] == 'F'){
            printf("HELLO\n");
        }

    }
}

I think that there might be something wrong with the way the structure is set up. I have been playing around with this for hours.

Any ideas?

Something wrong with the parameters of the call? I thought it was supposed to be the structure and the then the integer in the call, but when I try Employee[] it says type not allowed and I have played with other ways of using something other than employ. Am i on the right track, or is there something else to consider?

Thank you

So you created an array that contains an "F", and then I can use that to check it against s[i].sex[0]? would i use strcmp. Brand new to programming so I'm sorry if I don't make myself clear. Really trying to understand this.

Thanks again.

So inside of the function i created (well you did)

Code:

void printFemale(Employee s[], int num) {
    int i;
    Employee empList[1] = {{"F"}};
 
   
    
     printf( "The Female Employees Are\n");
     printf("%c", empList[0]);
     for( i = 0; i < num; ++i )
    {
        if (s[i].sex[0] == 'F'){
            printf("HELLO\n");
        }

    }
}

I read through the tutorial, and from your NOTE: in the struct declaration you two are saying to change char sex[2] to just char sex;. If I do that a lot of other things wont work. Is there a was I can :

Code:

if (strcmp(s[i].sex[0], emplist[0]) == 0)
     print();

Thanks again for your help, I really appreciate it

Help Please

Don't bump your thread (read this: forum guidelines, rule #3). We'll get to it when we can -- we volunteer in our spare time after all, and we do have lives of our own .

I gave you the strcmp solution in my original post. Also, look at your initialization of empList:

Code:

Employee empList[1] = {{"F"}};

That is incorrect, and not what Tim posted.