正確な指示にもかかわらずコードが機能しない

Question

私はJavascriptでプログラミングするのが初めてで、ビデオの助けを借りて簡単なピンポンゲームを構築しています。私はビデオのすべての指示に従ってきましたが、それは動作していないようです。 ballXは、キャンバスの幅を過ぎると中央にリセットされるはずですが、Xは右のパドルに当たった場合には反転することになっています。

if (ballX < 0) { 
    if (ballY > paddle1Y && 
    ballY < paddle1Y + paddleHeight) { 
    ballXSpeed = -ballXSpeed; 
    } else { 
    ballReset(); 
    } 
    if (ballX > canvas.width) { 
    if (ballY > paddle2Y && 
     ballY < paddleHeight + paddle2Y) { 
     ballXSpeed = -ballXSpeed; 
    } else { 
     ballReset(); 
    } 
    } 
}; //ends movement(); 

Answer 1

私はあなたがあなたの場合は構造を修正すべきだと思う：

// if ballX is less than 0, expected to use is paddle1 
if (ballX < 0) { 
    if (ballY > paddle1Y && 
    ballY < paddle1Y + paddleHeight) { 
    ballXSpeed = -ballXSpeed; 
    } else { 
    ballReset(); 
    } 
} 
// else if ballX is in the other side of the table, expected to use is paddle2 
else if (ballX > canvas.width) { 
    if (ballY > paddle2Y && 
    ballY < paddleHeight + paddle2Y) { 
    ballXSpeed = -ballXSpeed; 
    } else { 
    ballReset(); 
    } 
} 

あなた};が間違っていました。

Answer 2

は}を移動 - それはballXが0よりも小さい場合、それは非常に単純にコードのその部分は無用作り、キャンバスの幅より大きくなることはありません

if (ballX < 0) { 
    if (ballY > paddle1Y && 
    ballY < paddle1Y + paddleHeight) { 
    ballXSpeed = -ballXSpeed; 
    } else { 
    ballReset(); 
    } 
} // added 
if (ballX > canvas.width) { // this probably should be an else if but it really doesn't matter 
    if (ballY > paddle2Y && 
    ballY < paddleHeight + paddle2Y) { 
    ballXSpeed = -ballXSpeed; 
    } else { 
    ballReset(); 
    } 
    // a } removed from here 
} //ends movement(); 

Answer 3

に役立つかどうかを確認します。論理は単に意味をなさない。

Answer 4

括弧の問題があると思われます。

if (ballX < 0) { 
    if ((ballY > paddle1Y) && 
    ((ballY < paddle1Y) + paddleHeight))) { 
    ballXSpeed = -ballXSpeed; 
    } else { 
    ballReset(); 
    } 
} // added 
if (ballX > canvas.width) { // this probably should be an else if but it really doesn't matter 
    if ((ballY > paddle2Y) && 
    ((ballY < paddleHeight) + paddle2Y))) { 
    ballXSpeed = -ballXSpeed; 
    } else { 
    ballReset(); 
    } 
    // a } removed from here 
} //ends movement();