我试图找出我的问题一个小时。
我要画出正在发生的事情。
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;
}
最佳答案
好吧,从您的代码看来,陷阱和播放器交换了位置。
但是,陷阱 (Q) 的位置开关会用地砖覆盖玩家 (L)。
基本上会发生这种情况:
(1) .QL。
(2) .L..//L 和 Q 位于同一个图块
(3) ...问//Q 用 '.' 覆盖 L瓦
您的代码受到卷积的影响,因为您试图在一个函数中做太多事情,或者在几个不同的地方做。例如,这会导致您在上面的示例中缺少必要的碰撞检查。
更好的策略是尝试构建您的程序流。例如,您目前拥有:
(1) 提示方向
(2) 移动玩家牌
(2a) 如果玩家击中了一个陷阱或墙壁(你的代码中的错别字,必须是 || 而不是 &&)移动陷阱
(3) 移动陷阱
(4) 检查碰撞
(5) 从 (1) 重复
我想你想要的是:
(1) 提示方向
(2) 移动玩家牌
(2a) 检查碰撞
(3) 移动陷阱
(3a) 检查碰撞
(6) 重复 (1)
至于重构尝试回收你的代码。例如,在您的 move() 函数中,char '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 循环。
关于c++ - 我的程序中关于碰撞的一个小错误,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/28606275/