C++ if文の動作について

Question

C++のプログラムなのですが、for文の中のif文が想定していた動作と同じ動作をしていません。

この順番でのインプットを想定していて、2回目のturnOrMove, directionが, 't', 'R'となりif(turnOrMove[i] == 't' && direction[i] == 'R')の中の内容が実施されるはずなのですが、毎回else{
cout << "上記以外の動作" << endl;}の内容が実施されてしまいます。

想定しているインプットの一覧は下記の内容です。
2 2
2 1 1 1
8
m F
t R
m F
m F
t B
m L
m L
m B

#include <bits/stdc++.h> 
#include <string> 

using namespace std; 

// function to find  final position of 
// robot after the complete movement

string askTurnOrMove(string turnOrMove){
    cin >> turnOrMove;
    return turnOrMove;
}

string askDirection(string direction){
    cin >> direction;
    return direction;
}

void finalPosition(int initialX, int initialY, int front, int right, int back, int left, int numberOfCommands) { 
    //int numCommands = numberOfCommands.size(); 
    int countUp = 0, countDown = 0; 
    int countLeft = 0, countRight = 0; 
    string turnOrMove, direction;
    string AngleOfRobot;
    AngleOfRobot = 'N';
    cout << AngleOfRobot << endl;

// traverse the instruction string 'move' 
    for (int i = 0; i < numberOfCommands; i++) { 
        // for each movement increment its respective counter 
        cout << "------" << i << "s loop" << endl;
        
        turnOrMove = askTurnOrMove(turnOrMove);
        direction = askDirection(direction);
        
        cout << "AngleOfRobot in here: " << AngleOfRobot << endl;
        cout << "Turn or move: " << turnOrMove << endl;
        cout << "Direction: " << direction << endl;
        
        if(turnOrMove[i] == 't' && direction[i] == 'R'){
            cout << "REACH" << endl;
            if(AngleOfRobot[i] == 'N'){
                AngleOfRobot = 'E'; 
                cout << "Reach to Angle change to E" << endl;
            }
            else if(AngleOfRobot[i] == 'E'){
                AngleOfRobot = 'S';
            }
            else if(AngleOfRobot[i] == 'S'){
                AngleOfRobot = 'W';
            }
            else if(AngleOfRobot[i] == 'W'){
                AngleOfRobot = 'N';
            }
            else{
                cout << "REACH to else" << endl;
            }
        }
        else{
            cout << "上記以外の動作" << endl;           
        }

        // ロボットが北向きの時
        if (direction[i] == 'F' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'N'){
            countUp += front; 
            cout << "Reach to East" << endl;
        }
        else if (direction[i] == 'B' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'N'){ 
            countDown += back; 
        }
        else if (direction[i] == 'L' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'N'){
            countLeft += left; 
        }
        else if (direction[i] == 'R' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'N'){
            countRight += right; 
        }
            
        // ロボットが南向きの時
        else if (direction[i] == 'F' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'S'){ 
            countDown += front; 
        }
        else if (direction[i] == 'B' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'S'){
            countUp += back; 
        }
        else if (direction[i] == 'L' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'S'){
            countRight += left; 
        }
        else if (direction[i] == 'R' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'S'){
            countLeft += right; 
        }
            
        // ロボットが東向きの時
        else if (direction[i] == 'F' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'E'){
            countRight += front; 
            cout << "Reach when robot angle is E" << endl;
        }
        else if (direction[i] == 'B' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'E'){
            countLeft += back; 
        }
        else if (direction[i] == 'L' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'E'){
            countUp += left; 
        }
        else if (direction[i] == 'R' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'E'){ 
            countDown += right; 
        }

        // ロボットが西向きの時
        else if (direction[i] == 'F' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'W'){ 
            countLeft += front; 
        }
        else if (direction[i] == 'B' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'W'){ 
            countRight += back; 
        }
        else if (direction[i] == 'L' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'W'){ 
            countDown += left; 
        }
        else if (direction[i] == 'R' && turnOrMove[i] == 'm' && AngleOfRobot[i] == 'W'){ 
            countUp += right; 
        }
        else{
            cout << "上記以外の動作" << endl;
        }
    } 
    // required final position of robot 
    cout << "Final Position: ("
         << initialX + (countRight - countLeft)
         << ", " << initialY + (countUp - countDown)
         << ")" << endl; 
}

// Driver program to test above 
int main() { 
    int initialX, initialY;
    int front, right, back, left;
    int numberOfCommands;
    cin >> initialX;
    cin >> initialY;
    cin >> front;
    cin >> right;
    cin >> back;
    cin >> left;
    cin >> numberOfCommands;
    
    finalPosition(initialX, initialY, front, right, back, left, numberOfCommands); 
    return 0; 
}

Comments

[cateye]

インデントがちゃんと分かるように <code>を選んで出てきた```と```の間にソースを貼り付けましょう。
また、環境（OS,開発環境等）を記述しましょう。・・・人があなたと同じ環境を持っていると、想定しないように。

Answer 1

        cout << "Turn or move: " << turnOrMove << endl;
        cout << "Direction: " << direction << endl;

の部分をifの式のように

        cout << "Turn or move: " << turnOrMove[i] << endl;
        cout << "Direction: " << direction[i] << endl;

と書けば間違っていることに素早く気付けると思います。

あと、３行目の入力データが一つしかないため、turnOrMoveとdirectionが入れ替わってしまうような気がします。

Answer 2

turnOrMove[i]

これはturnOrMove文字列のi番目の文字を取り出す処理だと思いますが、期待した動作になっていますか？

Comments

[Naohito4869]

1回目のforループのみ期待した動作になっているのですが、2回目以降が毎回elseの方の条件に当てはまってしまいます。2回目のt Rの場合は、if(turnOrMove[i] == 't' && direction[i] == 'R')の内容が実行されると思ったのですが、elseの内容になってしまいます。

[takabosoft]

いや、そういう事を言っているわけではありません。

turnOrMove[i] は turnOrMove文字列のi番目の文字を取り出す処理
と言っています。

あたながしたいことはi番目の文字を取得することですか？違いますよね？