Thread: Skipping if statement

My code seems to skip an if statement. The function code is here:

Code:

while(flag == 0)
    {
        if(strcmp(current_plane_fuel->name, temp_fuel->name)==0)
        {
            flag == 1;
        }
        temp_fuel = temp_fuel->next_fuel;
        
        if(temp_fuel == NULL && flag == 0)
        {
            flag = 1;
            check_result = 4;
        }                    
    }

It basically checks 2 char arrays and if they are the same breaks the while loop.

temp_fuel being a linked list of structs that hold a name.

It's just checking if the fuel exists.

If they are different it breaks the loop and sets the int "check_result" to 4.

Even when I give it words that are the same it gives me the result of 4. And when I debug it, I check both values and they are the same. As I step through I see it skip over the whole if statement.

It doesn't even place the debuggers current line on the "if(strcmp...etc" line.

I can post the function in more detail if it helps, but I can't see why it won't even check the first if statement.

so temp_fuel may != NULL then.

Sorry I should have worded that better.
temp_fuel holds a name and a link to the next fuel

In my loop I set temp_fuel to the next fuel. If there is no next fuel it will equal NULL.
This works. And by stepping through it, I can see it does indeed go through all of my fuels until it hits NULL, where it sets check_result to 4 and then ends.

The problem is the correct fuel is in the list. And when stepping through it, I can see that the 2 char arrays held in name are in fact equal. Yet it still skips the first if statement.

My problem is the first if statement. The second one works fine.

Originally Posted by Subsonics

Shouldn't the first codeblock be

Code:

flag = 1;

Right now it's

Code:

flag == 1;

As far as I can tell it does nothing if the condition in the if statement is true.

Yes! Sorry. It has fixed it!

I can't believe I've been staring at this code for around half an hour now and had friends look at it, and no one's noticed. I think I've gone code blind.

It's often the little things.