Thread: If statement should be true, but isn't?

Hi all,

I seem to be having a silly issue with an IF statement that I can't figure out. I'm quite new to C++ but not to code in general. I understand how logical statements work but I might be missing something here.

Just so there's no confusion. I'm trying to create a basic draughts/checkers game and the function in question is being called to move the correct piece as referrence by a grid:

1 2 3 4 5 6 7 8
a
b
c
d
e
f
g
h

The code checks the first element of the "position_from" array and looks for a letter (A-H). If it finds any of these it will then continue in to a nested loop and check for 1-8 to complete the grid referrence.

I've only just begun so only have code which checks for 'A', then continues in to a loop and checks for 1-8. My problem is, the nested IF statement I am using within the loop, has no other option but to come true, but for some reason won't.

The following code will output:

Originally Posted by Code output

True - 'position_from[0] does equal 'A'
0 = A <- element zero contains 'A'.
1 = 1 <- element one contains '1'.

As you can see from the output, element[1] does indeed contain the number 1. With that in mind, why is the following code not working? The for loop checks 'position_from[1]' (which equals '1') against 'I', which equals 0 on the first loop then 1 on the second loop, then stops looping.

My code:

Code:

void move_piece(char *pieces, char *position_from, char *position_to)
{
     if(position_from[0] == 'A')
     {
           cout << "True - 'position_from[0]' does equal 'A'" << endl;
           for(int i = 0; i < 2; i++)
           {
                  if(position_from[1] == i) /* This statement must come true once */
                  {
                       cout << "True - 'position_from[1]' does equal 1";
                  }
                  else
                  {
                       cout << i << " = ";
                       cout << position_from[i] << endl;
                  }
           }
      }
      else
      {
            cout << "False - 'position_from[0] does not equal 'A'";
      }
}

Help is much appreciated. Thanks.

You are assuming, mistakenly, that the test if ('1' == 1) will be true. It will not. In the ASCII character set for example, the character '1' has a numeric value of 49.

Comparing characters with an integer compares the numeric value.

You are saying that position_from[i] contains the value of '0'. You are then comparing that against the value of i (which, in your code) can only have the value of 0 or 1. '0' will not compare equal with either 0 or 1. That is what the test "if (position_from[1] == i)" does.

Thanks for your replies. The problem has been solved. The above code needs to subtract '0' from the char array during the comparison, so it becomes:

if((position_from[1] - '0') == i)
instead of
if(position_from[1] == i)

Again, thanks. Would never have figured that out alone. I can't seem to change the title to [SOLVED], would a mod be able to? This thread may help someone in the future.

Code:

void move_piece(char *pieces, char *position_from, char *position_to)
{
     if(position_from[0] == 'A')
     {
           cout << "True - 'position_from[0]' does equal 'A'" << endl;
           for(int i = 0; i < 2; i++)
           {
                  if((position_from[1] - '0') == i) /* This statement must come true once */
                  {
                       cout << "True - 'position_from[1]' does equal 1";
                  }
                  else
                  {
                       cout << i << " = ";
                       cout << position_from[i] << endl;
                  }
           }
      }
      else
      {
            cout << "False - 'position_from[0] does not equal 'A'";
      }
}