どのように私はブール値が真のままであることを確認して衝突をチェックするのですか？

Question

自分のキャラクターが私自身のSDL2ベースのフレームワークのプラットフォームに着陸しようとしています。現在のところ、私は複数のプラットフォームを持っている場合、私は1つを生成する場合、デフォルトであるために動作していません。キャラクターは最後に追加されたプラットフォームにのみ留まります。残りは徐々に落ち、groundedブール値はtrueからfalseに切り替わります。この機能は、すべてのdeltatime更新

void NHTVScene::EntitiesGrounded() 
{ 
    std::vector<Entity*>::iterator it = platformVector.begin(); 
    while (it != platformVector.end()) { 
     if (player->isColliding((*it))) { 
     player->velocity = Vector2(0, 0); 
     player->grounded = true; 
    } 
    else if (!player->isColliding((*it))) { 
     player->grounded = false; 
    } 

    it++; 
    } 
} 

と呼ばれる：

は、ここに私のコードです。

このコードは、ブール値がtrueに設定されているのと衝突していないすべてのプラットフォームで、最新のエントリでは実行しないという奇妙なものだと思います。これを正しく行う方法を手がかりにしてください。

私はこれが私の問題を洞察するのに十分な情報であることを望みます。そうでない場合は、私に知らせてください、私はより多くを供給します。

Answer 1

1. 適切な字下げを使ってwhileとif else句を区別しやすくなり、少し明確にネストされたインデントレベルの書式設定を検討してください。
2. 私は、isColliding(Entity)がブール値であることを前提にしているので、すでにtrueをチェックしてからfalseをチェックする必要はありません。このロジックを単純化することを検討してください。
3. 衝突を検出した後にプレーヤを接地した状態にしたい場合は、終了基準にする必要があります。

何かお探しのものがありますか？

void NHTVScene::EntitiesGrounded() 
{ 
    std::vector<Entity*>::iterator it = platformVector.begin(); 
    while (it != platformVector.end()) 
    { 
     if (player->isColliding((*it))) 
     { 
      player->velocity = Vector2(0, 0); 
      player->grounded = true; 
      break; // exit criteria met 
     } 
     else 
     { 
      player->grounded = false; 
     } 
     it++; 
    } 
}