イベントを正しく処理していないと思います。ここで行っているのは、各イベント(キーボード入力であるかどうかに関係なく)でsf :: Key :: Downキーが押されているかどうかをチェックすることです(sf :: Key :: Leftでも同じです)。
第一に、あなたが望む結果が得られないので、それは効果的ではありません。次に、イベントがマウスの移動、マウスのクリックなどである可能性があることを認める無駄なチェックを実行します。このような場合にこれらのキーが押されているかどうかをチェックすることは、プログラムにとって無意味です。
私はあなたのコード全体を見ることができませんが、あなたはあなたのメインループとしてこの味の何かを試してみるべきです:
bool isMovingLeft = false;
bool isMovingDown = false;
sf::Event event;
while (win.IsOpen())
{
// While window catches events...
while(win.GetEvent(event))
{
// If the caught event is a click on the close button, close the window
if (event.Type == sf::Event::Closed)
win.Close();
// If it's a key press, check which key and move consequently
else if (event.Type == sf::Event::KeyPressed)
{
if(event.Key.Code == sf::Key::Left)
isMovingLeft = true;
else if(event.Key.Code == sf::Key::Down)
isMovingDown = true;
}
// If it's a key release, stop moving in the following direction
else if (event.Type == sf::Event::KeyReleased)
{
if(event.Key.Code == sf::Key::Left)
isMovingLeft = false;
else if(event.Key.Code == sf::Key::Down)
isMovingDown = false;
}
}
// Now that we have caught events, we move the lil' thing if we need to.
if(isMovingLeft)
x = x - SPEED;
if(isMovingDown)
y = y - SPEED;
win.Clear();
// Draw things on the screen...
win.Display();
}
このコードでは、プロセス全体が2つの部分に分割されています。
- まず、ユーザー入力をインターセプトして、モノの移動状態を変更する必要があるかどうかを確認します。
- 次に、すべてのイベントをキャッチして徹底的に分析したら、必要に応じて移動します。これは2つのブール値を介して行われます(4方向の制御が必要な場合は4つに増やす必要があります。斜めの方向を処理する場合は、8つのブール値よりも列挙型を使用する方が賢明です。 -そのような単純なタスクのために消費します。)
注:「Speed」を「SPEED」に変更したことに気付くかもしれません。それがdefineなのか、const varなのか、それとも単にあなたが与えたコードからのvarなのかはわかりませんが、最良のオプションは最初の2つのうちの1つです。私はそのようなことに#defineを使用することを好みます。これは、定数に簡単に到達できるようにするため(プリプロセッサに配置されるため)、完全にキャップされた書き込みにより、コード内の従来の変数との差別化が容易になります。しかし、それは私たちがここで話しているコーディングスタイルです:)