質問編集履歴

文章を修正しました。

2021/02/22 11:18

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

	@@ -1 +1 @@
1	- 素早く動かすと当たり判定がおかしくなる原因が知りたい。　~~dxl~~ib
1	+ [Dxlib]素早く動かすと当たり判定がおかしくなる原因が知りたい。GitHub全ソースあり

test CHANGED Viewed

@@ -6,6 +6,16 @@
+**試したこと**
+キー入力は問題りませんでした。
+**その他**
+ゆっくりと動かすと問題なのですが。　素早くキーを押し変えると問題が発生します。

提示文章をコードを修正

2021/02/22 11:17

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -1,10 +1,14 @@
+提示コードの///コメント部内部のコードですが当たり判定が以下の画像のような挙動をしてしまい上手く実装出来ません。これはどうしたらいいのでしょうか？
-提示画像のように壁に当たってキーを離してすぐに上下に動かすとおかしくはなります。提示コードの////コメント部のコードですが
+「左側に衝突して補正されて左キーを離してすぐに下キーを押押すと二枚目のように少しめり込み、キーを離すと上に補正されてしまいます。」
-**キーを左に押して離してすぐ上下に動かすと当たり判定おかしくなりわけのわからない方向に移動してしまいます**。これは何が原因なのでしょうか？**素早く**というところで意味がわかりません。
+これはどうしたらいいのでしょうか？おそらくコメント内部のコードが悪いと思うのですが何度見ても正しいです。どうすればいいのでしょうか？
-順序は確認しました。またキーは同時押し出来ないです。確認しました。まためり込み量を修正する関数はvoid Player::FixPos(glm::ivec2 pos)部です。
@@ -14,9 +18,15 @@
-![イメージ説明](b511b295fe7cbc122072793d4c3be871.jpeg)
+![イメージ説明](5b1ab883189c55ddf8fbd3066519b86d.jpeg)
+![イメージ説明](1f7f8545a351bb0d4f1d18289175b6e6.jpeg)
+![イメージ説明](764e45264fe105b173f68fc8a6cc74cb.jpeg)
 2Dアクションゲーム

文章を修正しました。

2021/02/22 10:16

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -1,6 +1,12 @@
 提示画像のように壁に当たってキーを離してすぐに上下に動かすとおかしくはなります。提示コードの////コメント部のコードですが
-**キーを左に押して離してすぐ上下に動かすと当たり判定おかしくなりわけのわからない方向に移動してしまいます**。これは何が原因なのでしょうか？**素早く**というところで意味がわかりません。順序などは確認しました。またキーは同時押し出来ないです。
+**キーを左に押して離してすぐ上下に動かすと当たり判定おかしくなりわけのわからない方向に移動してしまいます**。これは何が原因なのでしょうか？**素早く**というところで意味がわかりません。
+順序は確認しました。またキーは同時押し出来ないです。確認しました。まためり込み量を修正する関数はvoid Player::FixPos(glm::ivec2 pos)部です。

文章を編集しました。

2021/02/22 09:02

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -1,10 +1,12 @@
 提示画像のように壁に当たってキーを離してすぐに上下に動かすとおかしくはなります。提示コードの////コメント部のコードですが
-**キーを左に押して離してすぐ上下に動かすと当たり判定おかしくなりわけのわからない方向に移動してしまいます**。これは何が原因なのでしょうか？
+**キーを左に押して離してすぐ上下に動かすと当たり判定おかしくなりわけのわからない方向に移動してしまいます**。これは何が原因なのでしょうか？**素早く**というところで意味がわかりません。順序などは確認しました。またキーは同時押し出来ないです。
-**素早く**というところで意味がわかりません。順序などは確認しました。またキーは同時押し出来ないです。
+Github: https://github.com/Shigurechan/ActionGame
@@ -16,6 +18,12 @@
 ```cpp
 #include "Input.hpp"

提示コードを修正

2021/02/22 05:27

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -1,887 +1,803 @@
-提示画像のように壁に当たってキーを離して上下に動かすとおかしくはならないのですが
+提示画像のように壁に当たってキーを離してすぐに上下に動かすとおかしくはなります。提示コードの////コメント部のコードですが
 **キーを左に押して離してすぐ上下に動かすと当たり判定おかしくなりわけのわからない方向に移動してしまいます**。これは何が原因なのでしょうか？
 **素早く**というところで意味がわかりません。順序などは確認しました。またキーは同時押し出来ないです。
+![イメージ説明](b511b295fe7cbc122072793d4c3be871.jpeg)
+2Dアクションゲーム
-**#define CELL ((int)48)**です。
+**#define CELL ((int)48)**
-![イメージ説明](f5f059168215bd6c467b5d71509a14a0.jpeg)
 ```cpp
+#include "Input.hpp"
+// コンストラクタ
+Input::Input()
+{
+    //キー状態配列を初期化
+    for (int i = 0; i < 256; i++)
+    {
+        Key[i] = 0;
+    }
+}
+//キー入力状態を更新
+void Input::Update()
+{
+    char tmpKey[256];   // 現在のキーの入力状態を格納する
+    GetHitKeyStateAll(tmpKey);  // 全てのキーの入力状態を得る
+    for (int i = 0; i < 256; i++) {
+        if (tmpKey[i] != 0) {   // i番のキーコードに対応するキーが押されていたら
+            Key[i]++;   // 加算
+        }
+        else {
+            // 押されていなければ
+            Key[i] = 0;   // 0にする
+        }
+    }
+}
+//キーを押した時だけ
+bool Input::getKeyDown(unsigned char KeyCode)
+{
+    if (Key[KeyCode] == 1)
+    {
+        return true;
+    }
+    else
+    {
+        return false;
+    }
+}
+//キーを押している時
+int Input::getKeyDownHold(unsigned char KeyCode)
+{
+    //printf("あああ\n");
+    return Key[KeyCode];
+}
+```
+```cpp
-#include "Game_Scene.hpp"
+#include "Player.hpp"
 //コンストラクタ
-Game_Scene::Game_Scene(Scene_Type t,Entry *e) : Scene_base(t,e)
+Player::Player(Entry* e) : Actor(e)
-{
+{
-	Owner = e;
-	//文字数の関係で省略
-}
+}
+//座標を修正
+void Player::FixPos(glm::ivec2 pos)
+{
+	if (mVector == VECTOR_UP)
+	{
+		printf("UP\n");
+		mPosition.y = pos.y + CELL + CELL / 2;
+	}
+	else if (mVector == VECTOR_DOWN)
+	{
+		printf("DOWN\n");
+		mPosition.y = pos.y - (CELL / 2);
+	}
+	else if (mVector == VECTOR_LEFT)
+	{
+		printf("LEFT\n");
+		mPosition.x = pos.x + (CELL + CELL / 2);
+	}
+	else if (mVector == VECTOR_RIGHT)
+	{
+		printf("RIGHT\n");
+		mPosition.x = pos.x - (CELL / 2);
+	}
+	else {
+		printf("None\n");
+	}
+}
 //更新
-void Game_Scene::Update()
-{
-	//バトル
-	if (player->getIsMenu() == false)
-	{
-		stage->ScrollMap(*player);				//画面スクロール
-		player->Update();						//プレイヤー更新
-		stage->ColPlayer(*player);				//プレイヤーとマップとの当たり判定
-		stage->Update();						//ステージ更新
-		enemy_mng->Update();					//エネミー更新
-		stage->ColBullet(player->getBullet());	//バレットとマップとの当たり判定
-		stage->ColEnemy(enemy_mng->getEnemy());
-	}
-	else
-	{
-		//ショップ画面
-		if (shop->getState() == true)
+void Player::Update()
+{
+	DrawFormatString(0,0,GetColor(0,255,0),"Position: %d , %d ",mPosition.x,mPosition.y);
+	//printf("%d\n",mSpeed_Max);
+	Bullet_Update();	//バレットを更新
+	Player_Update();	//プレイヤー更新
+}
+//ステータスを設定
+void Player::set_Bulid(ItemData data)
+{
+	mSpeed_Max += data.mSpeed_Max;
+	HP_Max += data.HP_Max;
+	HP += data.HP_Rec;
+	HP_autoRec += data.HP_autoRec;
+	Attack += data.Attack;
+	Coin += data.Coin;
+}
+//キー入力部
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+//プレイヤー　更新
+void Player::Player_Update()
+{
+	//printf("いいいい\n");
+	mSpeed = mSpeed_Max;
+	//キー入力
+	if (Owner->InputKey->getKeyDownHold(KEY_INPUT_LEFT) > 0)
+	{
+		mVector = VECTOR_LEFT;	//方向
+	}
+	else if (Owner->InputKey->getKeyDownHold(KEY_INPUT_RIGHT) > 0)
+	{
+		mVector = VECTOR_RIGHT;	//方向
+	}
+	else if (Owner->InputKey->getKeyDownHold(KEY_INPUT_UP) > 0)
+	{
+		mVector = VECTOR_UP;	//方向
+	}
+	else if (Owner->InputKey->getKeyDownHold(KEY_INPUT_DOWN) > 0)
+	{
+		mVector = VECTOR_DOWN;	//方向
+	}
+	else {
+		mSpeed = 0;
+	}
+	//当たり判定
+	//スプライトの中心からのため修正
+	glm::ivec2 pos = mPosition;
+	pos.x += -(CELL / 2);
+	pos.y += -(CELL / 2);
+	mCol.setPosition(pos);
+	mCol.setSize(mSize);
+	mCol.setTrigger(false);
+	mCol.setVector(mVector);
+}
+/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+//描画
+void Player::Draw()
+{
+	Bullet_Draw();	//バレット描画
+	Player_Draw();	//プレイヤー描画
+}
+//プレイヤー　描画
+void Player::Player_Draw()
+{
+	//方向
+	if (mVector == VECTOR_UP)
+	{
+		DrawRotaGraph(mPosition.x, mPosition.y, 1.0, 0, mSprite, true, false);
+	}
+	else if (mVector == VECTOR_DOWN)
+	{
+		DrawRotaGraph(mPosition.x, mPosition.y, 1.0, PI, mSprite, true, false);
+	}
+	else if (mVector == VECTOR_LEFT)
+	{
+		DrawRotaGraph(mPosition.x, mPosition.y, 1.0, -(PI * 2) / 4, mSprite, true, false);
+	}
+	else if (mVector == VECTOR_RIGHT)
+	{
+		DrawRotaGraph(mPosition.x, mPosition.y, 1.0, (PI * 2) / 4, mSprite, true, false);
+	}
+	DrawPixel(mPosition.x, mPosition.y,GetColor(0,255,0));
+}
+//バレット　描画
+void Player::Bullet_Draw()
+{
+	for (std::vector<Bullet>::iterator itr = mBullet->begin(); itr != mBullet->end(); itr++)
+	{
+		itr->Draw();	//バレット描画
+	}
+}
+//デストラクタ
+Player::~Player()
+{
+}
+```
+```cpp
+#include "Stage.hpp"
+//コンストラクタ
+Stage::Stage(Entry* e)
+{
+}
+//更新
+/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+//プレイヤーとの当たり判定
+void Stage::ColPlayer(Player &player)
+{
+	for (std::vector<MapChip>::iterator itr = mStage.begin(); itr != mStage.end(); itr++)
+	{
+		if (Box_Collision::Intersect(itr->mCol,player.mCol) == true)
 		{
-			shop->Update(*player);
+			switch (itr->mCol.getObjectType())
+			{
+				//ショップ
+				case StageObjectType::Shop:
+				{
+					if (Owner->InputKey->getKeyDown(KEY_INPUT_P) == true)
+					{
+						player.setIsMenu(true);
+						player.FixPos(itr->mCol.getPosition());
+					}
+				}
+				break;
+				//レンガとの当たり判定
+				case StageObjectType::Brick:
+				{
+					player.FixPos(itr->mCol.getPosition());
+				}
+				break;
+				//ブロックとの当たり判定
+				case StageObjectType::Block:
+				{
+					player.FixPos(itr->mCol.getPosition());
+				}
+				break;
+			}
 		}
+	}
+}
+///////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+//画面スクロール
+void Stage::ScrollMap(Player& player)
+{
+	//左右移動
+	if (player.getPosition().x > (SCREEN_WIDTH - 100) + player.getSpeed() && player.getVector() == VECTOR_RIGHT) {
+		printf("Right\n");
+		for (std::vector<MapChip>::iterator itr = mStage.begin(); itr != mStage.end(); itr++)
+		{
-		else {
+			glm::ivec2 pos;
+			pos = itr->getPosition();
+			pos.x += -player.getSpeed();
-			player->setIsMenu(false);
+			itr->setPosition(pos);
-			shop = std::make_shared <Shop>(Owner);
 		}
 	}
-}
-//描画
-void Game_Scene::Draw()
-{
-	player->Draw();	//プレイヤー描画
-	stage->Draw();	//ステージ描画
-	//enemy_mng->Draw();
-	//ショップ画面
-	if (player->getIsMenu() == true)
-	{
-		shop->Draw();
-	}
-}
+	else if (player.getPosition().x < ( 100 ) + player.getSpeed() && player.getVector() == VECTOR_LEFT)
+	{
+		printf("Left\n");
+		for (std::vector<MapChip>::iterator itr = mStage.begin(); itr != mStage.end(); itr++)
+		{
+			glm::ivec2 pos;
+			pos = itr->getPosition();
+			pos.x += player.getSpeed();
+			itr->setPosition(pos);
+		}
+		//上下移動
+	}else if (player.getPosition().y > (SCREEN_HEIGHT - 100) + player.getSpeed() && player.getVector() == VECTOR_DOWN) {
+		printf("Dwon\n");
+		for (std::vector<MapChip>::iterator itr = mStage.begin(); itr != mStage.end(); itr++)
+		{
+			glm::ivec2 pos;
+			pos = itr->getPosition();
+			pos.y += -player.getSpeed();
+			itr->setPosition(pos);
+		}
+	}
+	else if (player.getPosition().y < (100) + player.getSpeed() && player.getVector() == VECTOR_UP)
+	{
+		printf("Up\n");
+		for (std::vector<MapChip>::iterator itr = mStage.begin(); itr != mStage.end(); itr++)
+		{
+			glm::ivec2 pos;
+			pos = itr->getPosition();
+			pos.y += player.getSpeed();
+			itr->setPosition(pos);
+		}
+	}
+	else {
+		//移動
+		int speed = player.getSpeed();
+		glm::ivec2 vec = player.getVector();
+		glm::ivec2 pos = player.getPosition();
+		pos.x += vec.x * speed;
+		pos.y += vec.y * speed;
+		player.setPosition(pos);
+	}
+}
+//エネミーとの当たり判定
+void Stage::ColEnemy(std::shared_ptr<std::vector<Enemy>> enemy)
+{
+	for (std::vector<MapChip>::iterator itr = mStage.begin(); itr != mStage.end(); itr++)
+	{
+		MapChip chip = *itr;
+		for (std::vector<Enemy>::iterator itr = enemy->begin(); itr != enemy->end(); itr++)
+		{
+			//交差判定
+			if (Box_Collision::Intersect(chip.mCol, itr->mCol) == true )
+			{
+				switch (chip.mCol.getObjectType())
+				{
+					//ブロック
+				case StageObjectType::Block:
+				{
+					itr->FixPos(chip.mCol.getPosition());
+					itr->setMove_Rand();
+				}
+				}
+			}
+		}
+	}
+}
 //デストラクタ
-Game_Scene::~Game_Scene()
+Stage::~Stage()
-{
+{
-}
+}
 ```
-```cpp
-#include "MapChip.hpp"
-//コンストラクタ
-MapChip::MapChip(StageObjectType t, glm::ivec2 pos, glm::ivec2 size,int handle) : Actor(nullptr)
-{
-	//文字数の関係で省略
-}
-void MapChip::Update()
-{
-	//当たり判定
-	mCol.setPosition(mPosition);
-	mCol.setSize(mSize);
-	mCol.setTrigger(true);
-}
-void MapChip::Draw()
-{
-	DrawGraph(mPosition.x, mPosition.y,mSprite,true);
-}
-//デストラクタ
-MapChip::~MapChip()
-{
-}
-```
-```cpp
-#include "Player.hpp"
-//コンストラクタ
-Player::Player(Entry* e) : Actor(e)
-{
-	//文字数の関係で省略
-}
-//座標を修正
-void Player::FixPos(glm::ivec2 pos)
-{
-	if (mVector == VECTOR_UP)
-	{
-		printf("UP\n");
-		mPosition.y = pos.y + CELL + CELL / 2;
-	}
-	else if (mVector == VECTOR_DOWN)
-	{
-		printf("DOWN\n");
-		mPosition.y = pos.y - (CELL / 2);
-	}
-	else if (mVector == VECTOR_LEFT)
-	{
-		printf("LEFT\n");
-		mPosition.x = pos.x + (CELL + CELL / 2);
-	}
-	else if (mVector == VECTOR_RIGHT)
-	{
-		printf("RIGHT\n");
-		mPosition.x = pos.x - (CELL / 2);
-	}
-	else {
-		printf("None\n");
-	}
-}
-int Player::getSpeed()
-{
-	return mSpeed;
-}
-//更新
-void Player::Update()
-{
-	DrawFormatString(0,0,GetColor(0,255,0),"Position: %d , %d ",mPosition.x,mPosition.y);
-	//printf("%d\n",mSpeed_Max);
-	Bullet_Update();	//バレットを更新
-	Player_Update();	//プレイヤー更新
-}
-//ステータスを設定
-void Player::set_Bulid(ItemData data)
-{
-	mSpeed_Max += data.mSpeed_Max;
-	HP_Max += data.HP_Max;
-	HP += data.HP_Rec;
-	HP_autoRec += data.HP_autoRec;
-	Attack += data.Attack;
-	Coin += data.Coin;
-}
-//プレイヤー　更新
-void Player::Player_Update()
-{
-	//printf("いいいい\n");
-	mSpeed = mSpeed_Max;
-	//キー入力
-	if (Owner->InputKey->getKeyDownHold(KEY_INPUT_LEFT) > 0)
-	{
-		mVector = VECTOR_LEFT;	//方向
-	}
-	else if (Owner->InputKey->getKeyDownHold(KEY_INPUT_RIGHT) > 0)
-	{
-		mVector = VECTOR_RIGHT;	//方向
-	}
-	else if (Owner->InputKey->getKeyDownHold(KEY_INPUT_UP) > 0)
-	{
-		mVector = VECTOR_UP;	//方向
-	}
-	else if (Owner->InputKey->getKeyDownHold(KEY_INPUT_DOWN) > 0)
-	{
-		mVector = VECTOR_DOWN;	//方向
-	}
-	else {
-		mSpeed = 0;
-	}
-	//当たり判定
-	//スプライトの中心からのため修正
-	glm::ivec2 pos = mPosition;
-	pos.x += -(CELL / 2);
-	pos.y += -(CELL / 2);
-	mCol.setPosition(pos);
-	mCol.setSize(mSize);
-	mCol.setTrigger(false);
-	mCol.setVector(mVector);
-}
-//描画
-void Player::Draw()
-{
-	Bullet_Draw();	//バレット描画
-	Player_Draw();	//プレイヤー描画
-}
-//プレイヤー　描画
-void Player::Player_Draw()
-{
-	//方向
-	if (mVector == VECTOR_UP)
-	{
-		DrawRotaGraph(mPosition.x, mPosition.y, 1.0, 0, mSprite, true, false);
-	}
-	else if (mVector == VECTOR_DOWN)
-	{
-		DrawRotaGraph(mPosition.x, mPosition.y, 1.0, PI, mSprite, true, false);
-	}
-	else if (mVector == VECTOR_LEFT)
-	{
-		DrawRotaGraph(mPosition.x, mPosition.y, 1.0, -(PI * 2) / 4, mSprite, true, false);
-	}
-	else if (mVector == VECTOR_RIGHT)
-	{
-		DrawRotaGraph(mPosition.x, mPosition.y, 1.0, (PI * 2) / 4, mSprite, true, false);
-	}
-	DrawPixel(mPosition.x, mPosition.y,GetColor(0,255,0));
-}
-```
-```cpp
-#include "Stage.hpp"
-//コンストラクタ
-Stage::Stage(Entry* e)
-{
-}
-//更新
-void Stage::Update()
-{
-	for (std::vector<MapChip>::iterator itr = mStage.begin(); itr != mStage.end(); itr++)
-	{
-		itr->Update();
-	}
-}
-//描画
-void Stage::Draw()
-{
-	for (std::vector<MapChip>::iterator itr = mStage.begin(); itr != mStage.end(); itr++)
-	{
-		itr->Draw();
-	}
-}
-/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-//プレイヤーとの当たり判定
-void Stage::ColPlayer(Player &player)
-{
-	for (std::vector<MapChip>::iterator itr = mStage.begin(); itr != mStage.end(); itr++)
-	{
-		if (Box_Collision::Intersect(itr->mCol,player.mCol) == true)
-		{
-			switch (itr->mCol.getObjectType())
-			{
-				//ショップ
-				case StageObjectType::Shop:
-				{
-					if (Owner->InputKey->getKeyDown(KEY_INPUT_P) == true)
-					{
-						player.setIsMenu(true);
-						player.FixPos(itr->mCol.getPosition());
-					}
-				}
-				break;
-				//レンガとの当たり判定
-				case StageObjectType::Brick:
-				{
-					player.FixPos(itr->mCol.getPosition());
-				}
-				break;
-				//ブロックとの当たり判定
-				case StageObjectType::Block:
-				{
-					player.FixPos(itr->mCol.getPosition());
-				}
-				break;
-			}
-		}
-	}
-}
-///////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-//画面スクロール
-void Stage::ScrollMap(Player& player)
-{
-	//左右移動
-	if (player.getPosition().x > (SCREEN_WIDTH - 100) + player.getSpeed() && player.getVector() == VECTOR_RIGHT) {
-		printf("Right\n");
-		for (std::vector<MapChip>::iterator itr = mStage.begin(); itr != mStage.end(); itr++)
-		{
-			glm::ivec2 pos;
-			pos = itr->getPosition();
-			pos.x += -player.getSpeed();
-			itr->setPosition(pos);
-		}
-	}
-	else if (player.getPosition().x < ( 100 ) + player.getSpeed() && player.getVector() == VECTOR_LEFT)
-	{
-		printf("Left\n");
-		for (std::vector<MapChip>::iterator itr = mStage.begin(); itr != mStage.end(); itr++)
-		{
-			glm::ivec2 pos;
-			pos = itr->getPosition();
-			pos.x += player.getSpeed();
-			itr->setPosition(pos);
-		}
-		//上下移動
-	}else if (player.getPosition().y > (SCREEN_HEIGHT - 100) + player.getSpeed() && player.getVector() == VECTOR_DOWN) {
-		printf("Dwon\n");
-		for (std::vector<MapChip>::iterator itr = mStage.begin(); itr != mStage.end(); itr++)
-		{
-			glm::ivec2 pos;
-			pos = itr->getPosition();
-			pos.y += -player.getSpeed();
-			itr->setPosition(pos);
-		}
-	}
-	else if (player.getPosition().y < (100) + player.getSpeed() && player.getVector() == VECTOR_UP)
-	{
-		printf("Up\n");
-		for (std::vector<MapChip>::iterator itr = mStage.begin(); itr != mStage.end(); itr++)
-		{
-			glm::ivec2 pos;
-			pos = itr->getPosition();
-			pos.y += player.getSpeed();
-			itr->setPosition(pos);
-		}
-	}
-	else {
-		//移動
-		int speed = player.getSpeed();
-		glm::ivec2 vec = player.getVector();
-		glm::ivec2 pos = player.getPosition();
-		pos.x += vec.x * speed;
-		pos.y += vec.y * speed;
-		player.setPosition(pos);
-	}
-}
-//デストラクタ
-Stage::~Stage()
-{
-}
-```