Dxlib アニメーションの切り替え実装について

Question

１，Animation関数の部分なのですがMain.cppで60フレームに固定していてif(s_anime == ewalk){}部で左右キーを押している間だけフレーム数を足しているのですがこの切り替え方法はわかりやすいのでしょうか？自分ではこの実装ほうしか思いつかんずもっとわかりやすい方法などありますでしょうか？

２、ある一定のフレーム数で変数を再初期化？１に戻さないと値がおかしくなる？なんか不安なので１に戻しているのですがこれはいいのでしょうか？

３，なんか変わりづらく作ってる気がしてならないのですが慎重にわかりやすく作りたいのですが何かヒントがあったら教えてもらいたいです。

#include <iostream>
#include <fstream>
#include "Input.h"
#include "Player.h"
#include "DxLib.h"
#include "Map.h"

#define MOVE_SPD 5
/*コンストラクタ*/
Player::Player(const char* str, int xx, int yy) :pos{xx,yy}
{
	LoadDivGraph(str,7,7,1,64,64,g_handle);

	
	
}

/*計算更新*/
void Player::Update()
{
	input_key();
	//gravity();

}
bool isCollision = false;
bool jump_c = false;

/*描画更新*/
void Player::Draw_Update()
{
	/*
	0　待機
	1,2,3//走る
	*/
	//DrawFormatString(100,100,GetColor(255,255,255),"Debug");

	/*←左*/
	if (key == -1)
	{
		//DrawTurnGraph(gPos_x(), gPos_y(), g_handle[0], true);
		DrawTurnGraph(gPos_x(), gPos_y(), g_handle[Animation()], true);

	}
	/*右→*/
	if (key == 1)
	{
		DrawGraph(gPos_x(), gPos_y(), g_handle[Animation()], true);
	}

}


bool isJump = false;//ジャンプボタンを押したかどうか
bool isJumpNow = false;//ジャンプ中かどうか

/*キー入力*/
void Player::input_key()
{
	if (keybord(KEY_INPUT_LEFT) > 0)
	{
		key = -1;
		s_anime = ewalk;

		sPos_x(-MOVE_SPD);

	}else if (keybord(KEY_INPUT_RIGHT) > 0)
	{
		key = 1;
		s_anime = ewalk;

		sPos_x(+MOVE_SPD);

	}else if (keybord(KEY_INPUT_UP) > 0)
	{
		sPos_y(+MOVE_SPD);

	}else if (keybord(KEY_INPUT_DOWN) > 0)
	{
		sPos_y(-MOVE_SPD);
	}
	else if (keybord(KEY_INPUT_SPACE) > 0)
	{
		/*ジャンプできる時isJump false*/
		if (isCollision == true)
		{
			isJump = true;
		}		
	}
	else//何も押していない時
	{
		s_anime = ewait;
	}



	jump_up();
	gravity();

}
/*アニメーション管理*/
int Player::Animation()
{
#define ANIME_SPD 20

	anime_f++;
	if (s_anime == ewait)
	{
		anime_f = 0;
		return 0;

	}

	if (s_anime == ewalk)
	{
		if (anime_f % ANIME_SPD == 0)
		{
			Nanime += 1;
			if (Nanime > 3)
			{
				Nanime = 1;
			}
		}


		if (anime_f > 120)
		{
			anime_f = 1;
		}
		return Nanime;

	}
	
}

/*ジャンプ管理*/
void Player::jump_up()
{
	if (isJump == true)
	{
		DrawFormatString(0, 0, GetColor(255, 255, 255), "%d ", jump_c);

		n = (n - 2);	

		if (n <= 0)
		{
			isJump = false;
			n = 0;
		}

		//j = n;

		sPos_y(n);

	}
	else if(isJump == false && n == 0) {
		n = jump_range;
	}


}


/*重力効果*/
void Player::gravity()//ジャンプしてるかどうかと当たり判定を比較
{
	
	isCollision = Map::col(pos.x,pos.y);
	const char filename[100] = { "Debug_Log.txt" };
	//std::fstream ofs(filename, std::ios::app);
	std::fstream ofs(filename, std::ios::app);


	if (!ofs)
	{

	}
	else {
		DrawFormatString(0, 100, GetColor(255, 255, 255), "isCollision %d", isCollision);

		//ofs << "aaaa"<< std::endl;

		char b[100];//バッファ
		char str[100];
		sprintf_s(str,sizeof(b), "[x: %d , y: %d] : isCollsion %d\n", pos.x,pos.y,isCollision);

		//= "x: \n" + (string)pos.x;//+ "y: " + pos.y + "\n";
		ofs << str;
		//ofs << "x: " + pos.x << "y: " + pos.y<<std::endl;
	}

	if (isJump == false) {

		

		gn = (gn + 2);
		if (gn >= 6) {
			gn = 10;
		}
		
		//sPos_y(-gn);
		//sPos_y(-2);
		/*当たっていない時ジャンプボタンを押した時*/
		if (isCollision == false)
		{

			sPos_y(-gn);
		}
		else if (isCollision == true)
		{
			sPos_y(1);


			//	isJump = false;
		}
	}
	if (jump == true)
	{
		
	
	}

}

/*座標の取得と設定*/
void Player::sPos_x(int xx)//加算
{
	pos.x += xx;
}

void Player::sPos_y(int yy)//加算
{
	pos.y += -yy;
}

int& Player::gPos_x()
{
	return pos.x;
}

int& Player::gPos_y()
{
	return pos.y;
}

#ifndef ___PLAYER_H
#define ___PLAYER_H
#include <iostream>
#include <fstream>
using namespace std;
#include "Chr.h"
#include "Map.h"

#define JUMP_RANGE 30
class Player : Chr
{
private:
	
	enum anime_s
	{
		ewait,
		ewalk,
		ejump,
		egame_over,

	};
	
	anime_s s_anime = ewait;//アニメ管理
	position pos;
	int g_handle[8];//画像格納ハンドル

	/*座標の取得と設定*/
	void Chr::sPos_x(int xx);
	void Chr::sPos_y(int yy);

	int& Chr::gPos_x();
	int& Chr::gPos_y();

	void input_key();//キー入力
	void gravity();//重力落下
	void jump_up();//ジャンプ上昇

	int prev = 0;//前の座標
	int gn = 0;//落下速度
	int Nanime = 0;
	int anime_f = 0;
	bool jump = false;
	const int jump_range = 30;//ジャンプ最大値
	int n = jump_range;
	int j = 1;
	int key = 1;//向き管理

	int Chr::Animation();

public:

	Player(const char* str,int xx,int yy);

	
	void Chr::Update();
	void Chr::Draw_Update();

};

#endif

Answer 1

１，Animation関数の部分なのですがMain.cppで60フレームに固定していてif(s_anime == ewalk){}部で左右キーを押している間だけフレーム数を足しているのですがこの切り替え方法はわかりやすいのでしょうか？自分ではこの実装ほうしか思いつかんずもっとわかりやすい方法などありますでしょうか？

キーを押している間だけ、フレームを動かすのは考え方として間違っていません。むしろそれしかありません。しかし作り方がどうも、丁寧な作り方ではないように思えます。
Draw_Update()にこの処理を置くのではなく、Update()にこの処理を置いた方がいいように思えます。

DrawTurnGraph(gPos_x(), gPos_y(), g_handle[Animation()], true);
//1.自分のクラス内でget関数を呼ぶ必要がない。pos.xで良い。
//2.g_handle[Animation()],ではなく、csv等の外部データにアニメパターンのデータを置いておくのがベスト。
//3.trueのみでは、後に読み直した時なんのことか分からなくなる可能性がある。

２、ある一定のフレーム数で変数を再初期化？１に戻さないと値がおかしくなる？なんか不安なので１に戻しているのですがこれはいいのでしょうか？

ここの事でしたら全体的に不安しかありません。

int Player::Animation()
{
#define ANIME_SPD 20//関数内で値を動かしたくない場合は、constexprまたはstatic constを使う。

    anime_f++;
    if (s_anime == ewait)
    {
        anime_f = 0;
        return 0;

    }

    if (s_anime == ewalk)
    {
        if (anime_f % ANIME_SPD == 0)
        {
            Nanime += 1;
            if (Nanime > 3)
            {
                Nanime = 1;//上で0を返してしまっている以上仕方ない気がする。
            }
        }


        if (anime_f > 120)
        {
            anime_f = 1;
        }
        return Nanime;

    }
    //一番最後のreturnがないのが超不安。
}

３，なんか変わりづらく作ってる気がしてならないのですが慎重にわかりやすく作りたいのですが何かヒントがあったら教えてもらいたいです。

それは「慣れろ」です。LEGOで上手く作るコツを聞くのと同じ質問です。
ヒントなんかありません。正解がないのが楽しいものに正解を求めてどうするのですか？
仕事ではそうは行きませんが、個人で作る場合は自由にやっていいと思います。
気に食わないならまた書き直せば良い。余力がないならそれはそれでいいではないかと思います。

Answer 2

animationの状態を直接Playerが持つのはちょっと汚いという思いがあります。クラスに出してしまいたいところです。anime_sはenum classにするほうがいいでしょう。

あとカウンター変数はintじゃなくてstd::uint64_tとかを使うほうがいいでしょう。少なくともunsignedであるほうが適切でしょう。

---

追記

時間不足で中保半端な解答になってしまったのでもう少し追記します。

まずanimationの状態を直接Playerが持つのは責任がぼやけているというのはすでに指摘したとおりです。なので新たにアニメーションクラスを作るわけですが、今回どういうアニメーションかというと、状態がサイクリックですよね？1->2->3->1みたいな。

とするとまずそういうイテレータを作るべきでしょう。つまり任意の閉区間[n1...n2]をサイクリックするイテレータです。イテレータの作り方は

イテレータの解説をするなんて今更佳代

を参照してください。

次に結局これらの数字は何だったかと言うと、画像ハンドルの配列の要素番号として使うのでしたね。

とするとこのイテレータをラップしたアニメーションクラスをつくりましょう。以下の物を少なくとも持つクラスです。

1. 今は走っているのか待機しているのかのフラグ
2. 走っている状態を表すイテレータ
3. 画像ハンドルの配列

ところで現状Player::Draw_Update()では左右にキーが入力されているときのみ描画するようですが、これだと静止しているときに何も描画されないのでは？つねに何かは描画する必要があるでしょう。なぜなら通常

1. 画面クリア
2. なんか描画
3. 裏画面を表画面に転送

を繰り返すはずだからです。

まあ何れにせよPlayer::Draw_Update()ではキーの状態によってアニメーションクラスの状態を更新し画像ハンドルをもらい、描画する処理になりますね。

さて、そう考えるとおかしなことがあります。なぜPlayerがキーの状態を持つのでしょうか？責任が広すぎます。動くものが一つだけならぎりぎりわかりますがちょっと読みにくいです。別クラスに追い出して、必要なキー状態はメンバ関数の引数経由で教えることにしましょう。

Comments

[退会済みユーザー]

質問ですがキーの情報とはキー入力でしょうか？Animeの切り替えのためのキー情報でしょうか？