Cで短縮されたコードの意味（int_wall関数内）

Question

これはMAZEアルゴリズムの一部です。機能に含まれるこのコードの短縮された部分はあまり明確ではありません。私の提案は正しいですか？

... 
#define UP 1 
#define RIGHT 2 
#define DOWN 4 
#define LEFT 8 
... 
int wall_ahead(int m[MAZE_SIZE][MAZE_SIZE], int x, int y, int dir){ 
    x = (dir == LEFT) ? --x : 
     (dir == RIGHT) ? ++x : x; 
    y = (dir == UP) ? --y : 
     (dir == DOWN) ? ++y : y; 

    return m[x][y]; 
} 

私の提案は次のとおりです。

if(dir == LEFT){ 
    x -= 1; 
    if(dir==RIGHT){ 
     x += 1; 
    } else { 
     x = x; 
    } 
} 

Answer 1

あなたはこのように、switchステートメントを使用したい：

switch (dir) { 
case UP: 
    --y; break; 
case DOWN: 
    ++y; break; 
case LEFT: 
    --x; break; 
case RIGHT: 
    ++x; break; 
} 

は、元のリストのような割り当てを行うには、これらの接頭演算子は、すでに値を変更し、平野間違っています。

OTOH、UPとDOWNは相互に排他的になるようだが、ないにRIGHTとLEFTと同じことが他の方法で回避を適用するので、これは、より正確になります：

x += !!(dir & DOWN) - !!(dir & UP); 
y += !!(dir & RIGHT) - !!(dir & LEFT); 

また、それほど重要ではありませんしかし、元のリストにはいくつかのタイプミスがあります。それらをチェックしてください。配列の次元を与える必要があります。そうしないと、コンパイラはアドレス計算を行うことができません。

Answer 2

if(dir == LEFT){ 
    x -= 1; 
    if(dir==RIGHT){ 
     x += 1 
    } else { 
     x = x; 
    } 
} 

が正しくありません。最初のコードブロックが行うことはしません。

それはする必要があります：

if(dir == LEFT){ 
    x -= 1; 
} 
else if(dir==RIGHT){ 
    x += 1 
} 

コードの意味が今従うことは簡単です。

dirがLEFTであれば、dirがRIGHTであれば、xで何もしないそうでなければ1
でxをインクリメント1.
でxをデクリメント。

最初のコードブロック内のコードの残りの部分の意味：

dirがUPある場合dirがDOWNであれば、y
1によりをデクリメント、行わないエルス1
でyをインクリメントyのもの