我的程序中有一个关于冲突的小错误

Question

我花了一个小时的时间试图解决我的问题
我要画出正在发生的事情。

char trap = 'Q'; 
char character = 'L';
.... 
.Q..
..L.
....

当L向上移动，Q向右移动时，它们发生碰撞，程序结束。但是：

....
.QL.
....
....

当L向左移动，Q向右移动时，它们不会像上面的例子那样碰撞，而是发生了什么：

....
..Q.
....
....

这是我的密码。抱歉我英语不好：（

测试移动是"w"、"a"、"s"还是"d"：

void cave::move(int& x, int& y, char m, char unit)
{
                if ( m == 'W' || m == 'w' ) // if moves up
                {
                    floor[x][y] = tile;
                    x -= 1;
                    for ( unsigned short int x = 0; x < 3; x++ )
                    {
                            if ( floor[x][y] == wall && floor[x][y] == trap[x] )
                            {
                                x += 1;
                                trapsMove();
                        }
                        }
                        floor[x][y] = unit;
                }
                else if ( m == 'A' || m == 'a' ) // if moves to left
                {
                            floor[x][y] = tile;
                            y -= 1;
                        for ( unsigned short int x = 0; x < 3; x++ )
                        {
                            if ( floor[x][y] == wall && floor[x][y] == trap [x] )        
                            {
                                   y += 1;
                                   trapsMove();
                                }
                        }
                            floor[x][y] = unit;
                }
                else if ( m == 'S' || m == 's' ) // if moves down
                {
                        floor[x][y] = tile;
                            x += 1;
                            for ( unsigned short int x = 0; x < 3; x++ )
                        {
                                if ( floor[x][y] == wall && floor[x][y] == trap[x] )
                                {
                                    x -= 1;
                                    trapsMove();
                                }
                            }
                            floor[x][y] = unit;
                }
               else if ( m == 'D' || m == 'd' )  // if moves to right
               {
                        floor[x][y] = tile;
                            y += 1;
                            for ( unsigned short int x = 0; x < 3; x++ )
                            {
                                if ( floor[x][y] == wall && floor[x][y] == trapx] )
                                {
                                    y -= 1;
                                    trapsMove();
                                }
                            }
                            floor[x][y] = unit;
               }
               else 
                        control(); 
               return;
 }

ai移动功能

void cave::trapsMove() 
{
                    int r[3]; // each index will hold the movement of traps 
                    for ( unsigned short int x = 0; x < 3; x++ )
                    {
                            r[x] = rand() % 4 + 1;
                            if ( r[x] == 1 ) // moves up
                                move(traps_positionX[x],traps_positionY[x],'w',trap[x]);
                            else if ( r[x] == 2 ) // moves to left
                                move(traps_positionX[x],traps_positionY[x],'a',trap[x]);
                            else if ( r[x] == 3 ) // moves down
                                move(traps_positionX[x],traps_positionY[x],'s',trap[x]);
                            else if ( r[x] == 4 ) // moves to right
                                move(traps_positionX[x],traps_positionY[x],'d',trap[x]);
                    }
                    return;
}

检查是否碰撞

bool cave::collision()
{
                    for ( unsigned short int x = 0; x < 3; x++ )
                    {
                            if ( floor[character_positionX][character_positionY] == trap[x] )
                                return true;
                    }
                    return false;
}

Answer 1

从您的代码中可以看出，Trap和Player切换了它们的位置。但是，陷阱（Q）的位置切换会用地砖覆盖玩家（L）。

本质上就是这样：

（1） .QL.

（2） .L.//L和Q位于同一块中

（3） 。。Q.//Q用"."覆盖L瓷砖

---

您的代码受到卷积的影响，因为您试图在一个函数中做太多，或者在几个不同的地方做。例如，这会导致您错过上面示例中所需的碰撞检查。

更好的策略是尝试构建程序流程。例如，您当前有：

（1） 提示方向

（2） 移动玩家磁贴

（2a）如果玩家击中了一个陷阱或墙的瓦片（代码中的拼写错误，必须是||而不是&&）移动陷阱

<--（2a）是陷阱可以连续移动两次的另一个陷阱。

（3） 移动陷阱

（4） 检查碰撞

（5） 从（1）重复

我想你想要的是：

（1） 提示方向

（2） 移动玩家磁贴

（2a）检查碰撞

（3） 移动陷阱

（3a）检查碰撞

（6） 从（1）重复

---

至于重构，试着回收你的代码。例如，在move（）函数中，字符'w'（…）只影响变量'x'或'y'。因此你也可以把它写成

move(...) {
    if((m=='w')||(m=='W')) { y = y+1 }
    else if((m=='a')||(m=='A')) { x = x-1 } // same for SD
    if(floor[x][y] != wall) {
        // set new position to object, if it can't move, just don't set it
    }
}

这样你就不必复制&粘贴你的for循环与最小的改动。