DirectXでなんフレーム経過などという処理の書き方が正しいのか知りたい

Question

別のファイルで毎秒60フレームに設定しています。
////コメント部のmb++ を使ってそのブロックが落下してからのフレームを加算してそれが60を超えたら
その処理をするというプログラムを作っているのですがこれはやり方として正しいのでしょうか？
このような処理を書くの初めてなので質問しました。

/*ブロック 種類、回転　ｘ、ｙ*/
static const int gblock[8][4][4][4]
{//文字数の関係で省略以下例
			
		{
			{0,1,0,0},
			{1,1,0,0},
			{0,1,0,0},
			{0,0,0,0},

		},

	},
	
};

/*
画面サイズ: 1280,720
ステージサイズ: 288,504
*/
struct block_pos
{
public:
	
	int x;
	int y;
};

enum key_state
{
	Left,
	Right,
	Down,
	Rotation,
	eHold,
	Invalid,
};



static int stage[STAGE_X][STAGE_Y];					//ブロック記録 


static int downspeed = DOWN_SPEED;					//落下スピード
static int block_hold = -1;							//ホールドしてるブロックを記録
static int nexts[4] = {-1};							//ブロックのネクストを表示
static int nowblock = 0;

static key_state key = Invalid;							//キー入力管理
static block_pos   bpos;									//ブロック座標
static int  block_r = 0;							//回転

static bool isdown = false;							//落下して止まったかどうか
static uint cidx[7] = { 5,3,4,1,6,2,7 };			//カラーインデックス

#define NEXT_LAST 3
#define NEXT 1

/*一個詰めて最後の値を設定*/
void randb_last()
{
	int la = rand() % 6;
	for (int i = 0; i < 4; i++)
	{
		if (i == NEXT_LAST)
		{
			nexts[i] = la;
			break;
		}
		int a = nexts[i + 1];
		nexts[i] = a;
	}
}


/*初期化*/
void Player::Initialize()
{
	frame_hendle = LoadGraph("Asset/Frame_Block_24px.png");
	LoadDivGraph("Asset/Block_tmp_24px.png", 8, 4, 2, 24, 24, block_tmp);
	back_ground = LoadGraph("Asset/BG.png");


	
	bpos.x = 4;
	bpos.y = 0;

	/*-1ですべての盤面を初期化*/
	for (int y = 0; y < STAGE_Y; y++)
	{
		for (int x = 0; x < STAGE_X; x++)
		{
			stage[x][y] = STAGE_EMPTY;
		}
	}

	/*盤面初期化*/
	
	srand((unsigned int)time(NULL));

	/************ブロック乱数記録配列の初期化***************/
	nowblock = rand() % 6;
	for (int i = 0; i < 4; i++)
	{
		nexts[i] =  rand() % 6;
	}

	
	
}



// -------------------------------------------------------------------------------
// 計算
// -------------------------------------------------------------------------------

static int mb = 0;//固定時の直前移動

static int t = 0;//4ラインのテトリスの数
static int c3 = 0;//3ラインの数
static int c2 = 0;//2ラインの数
static int c1 = 0;//1ラインの数


static bool d = true;//isdown中でも落下する場合


/*ブロックを生成*/
////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
void Player::Create_block()
{

	//落下中のブロックが落ちてないときはfalse 落ちたらtrue
	//落下したとき
	if (isdown == true) {
		mb++;
		if (mb > 60)//固定までの待ち時間
		{
			/*ブロック座標を盤面配列に値を設定*/
			for (int dy = 0; dy < 4; dy++)
			{
				for (int dx = 0; dx < 4; dx++)
				{

					if (gblock[nowblock][block_r][dx][dy] != 0)
					{
						stage[bpos.x + dx][bpos.y + dy] = nowblock;
					}
				}
			}

			/*新しいブロックの乱数を再設定してブロック座標を初期*/
			//randb();//新しいブロック乱数を再設定
			bpos.x = 4;
			bpos.y = 0;
			isdown = false;
			mb = 0;
			/*ネクスト設定*/
			nowblock = nexts[0];
			randb_last();

		}
//////////////////////////////////////////////////////////////////////////////////////////////////////////////		
		///
		//Block_Down();
		for (int y = 0; y < 4; y++)
		{
			for (int x = 0; x < 4; x++)
			{
				if (gblock[nowblock][block_r][x][y] == 1)
				{
					if (//stage[bpos.x + x][bpos.y + y + 1] == STAGE_FRAME || stage[bpos.x + x][bpos.y + y + 1] == STAGE_LOWER
						stage[bpos.x + x][bpos.y + y + 1] == STAGE_EMPTY)
					{
						isdown = false;
					}
				}
			}
		}


	}

	
}

/*ブロック削除*/
void Player::Block_Clear()
{
	bool clear = false;
	int c = 0;
	int cy[20] = { 99 };
	int num = 0;//削除ライン
	int line[22] = { 0 };//上から4ラインあったらテトリス

	/*ライン削除できるかどうか判定*/
	for (int y = 0; y < STAGE_Y - 1; y++)
	{
		for (int x = 1; x <= STAGE_X - 2; x++)
		{
			if (stage[x][y] != -1)
			{
				c++;
			}
			 
			if (c == 10)
			{
				line[y] = 1;//削除ライン
				num = y;
				clear = true;
			}
		}

		/**/
		if (clear == true)//ラインを消す場合
		{
			int dy = 0;//一行前のデータ
			int dyy = 0;
			int tmp[13] = { -10 };

			for (int y = num; y > 0; y--)
			{
				dy = y - 1;
				
				/*一つ上の行の配列をコピー*/
				for (int x = 1; x <= STAGE_X - 2; x++)
				{
					tmp[x] = stage[x][dy];
				}

				/*コピー*/
				for (int x = 1; x <= STAGE_X - 2; x++)
				{
					stage[x][y] = tmp[x];
				}

				/*配列再初期化*/
				for (int i = 0; i < 11; i++)
				{
					tmp[i] = -10;
				}
				/*ずらした行をすべてゼロ*/
				for (int x = 1; x <= STAGE_X - 2; x++)
				{
					stage[x][dy] = -1;
				}

			}
		}
		clear = false;
		c = 0;
	}
	/*要修正　　　　ライン処理ばバグってる*/
	/************スコア計算******************/
	int t = 0;
	int i = 0;
	//for (int i = 0; i < 22; i++)
	

	/***************************************/


}




/*ブロック落下
ブロックが落下できな場合 isdown = true;
*/
void Player::Block_Down()
{


	for (int y = 0; y < 4; y++)
	{
		for (int x = 0; x < 4; x++)
		{
			if (gblock[nowblock][block_r][x][y] == 1)
			{
				if (stage[bpos.x + x][bpos.y + y + 1] == STAGE_FRAME || stage[bpos.x + x][bpos.y + y + 1] == STAGE_LOWER
					|| stage[bpos.x + x][bpos.y + y + 1] != STAGE_EMPTY)
				{
					isdown = true;
					
				}
			}
		}
	}

	/*落下中のみ*/
	if (isdown == false)
	{
		/*落下速度*/
		if (fCount % downspeed == 0)
		{
			bpos.y += 1;
		}
	}

}

/*操作*/
void Player::Move()
{	
	
}


/*当たり判定 ぶつかったらtrue			→方向*/
bool Player::Right_Collision()
{	
	for (int y = 0; y < 4; y++)
	{
		for (int x = 0; x < 4; x++)
		{
			if (gblock[nowblock][block_r][x][y] == 1)
			{
				if ( (stage[bpos.x + x + 1][bpos.y + y] == STAGE_FRAME //|| stage[bpos.x + x + 1][bpos.y + y + 1] == STAGE_LOWER 
					|| stage[bpos.x + x + 1][bpos.y + y] != STAGE_EMPTY ) )
				{
					return true;//ぶつかったらtrue
				}
			}			
		}
	}

	return false;
}

/*当たり判定 ぶつかったらtrue	 ←方向*/
bool Player::Left_Collision()
{
	for (int y = 0; y < 4; y++)
	{
		for (int x = 0; x < 4; x++)
		{
			if (gblock[nowblock][block_r][x][y] == 1)
			{
				if (   (stage[bpos.x + x -1][bpos.y + y] == STAGE_FRAME  //stage[bpos.x + x -1][bpos.y + y + 1] == STAGE_LOWER
					|| stage[bpos.x + x - 1][bpos.y + y] != STAGE_EMPTY) )
				{
					return true;//ぶつかったらtrue
				}
			}
		}
	}

	return false;

}


static long f = 0;//長押し管理変数

/*キー入力受付処理*/
void Player::Control()
{
	
}

/*回転処理*/
void Player::Rotate()
{
	int tr = block_r;
	tr++;
	if (tr > 3)
	{
		tr = 0;
	}
	int c = 0;
	int cb = 0;
	int bc[4] = { 0 };

	for (int y = 0; y < 4; y++)
	{
		for (int x = 0; x < 4; x++)
		{
			if (gblock[nowblock][tr][x][y] == 1)
			{
				if (stage[bpos.x + x][bpos.y + y] == STAGE_FRAME || stage[bpos.x + x][bpos.y + y] == STAGE_LOWER 
					|| stage[bpos.x + x][bpos.y + y] != STAGE_EMPTY)
				{
					bpos.x--;
				}
			}
		}
	}

	block_r++;
	if (block_r > 3)
	{
		block_r = 0;
	}

}


//***********************************************
void Player::Update()
{	
	for (int i = 0; i < 4; i++)
	{
		DrawFormatString(0, 100 * i,Color,"%d",nexts[i]);
	}
	
	
	Create_block();
	Block_Down();
	Move();	
	Control();
	Block_Clear();
}
//***********************************************






// -------------------------------------------------------------------------------
// 描画
// -------------------------------------------------------------------------------



/*ブロック描画 (落下中)*/
void Player::Draw_Block()
{
	int x = bpos.x;
	int y = bpos.y;	

	for (int dy = 0; dy < 4; dy++)
	{
		for (int dx = 0; dx < 4; dx++)
		{			
			if (gblock[nowblock][block_r][dx][dy] != 0)
			{				
				DrawGraph( (dx + x) * block + DRAW_AREA_X , (dy + y) * block + DRAW_AREA_Y, block_tmp[cidx[nowblock]] , true);
			}
		}
	}	
}

/*ブロック固定描画*/
void Player::Draw_PutBlock()
{
	for (int y = 0; y < STAGE_Y; y++)
	{
		for (int x = 0; x < STAGE_X; x++)
		{
			if ( (stage[x][y] != STAGE_FRAME) && (stage[x][y] != -1) && (stage[x][y] != STAGE_LOWER) )
			{				
				DrawGraph(x * block + DRAW_AREA_X, y * block + DRAW_AREA_Y,block_tmp[ cidx[stage[x][y]] ],true);
			}
		}
	}
}

void Player::Draw_Score()
{
	long score = (t * LINE_4) + (c3 * LINE_3) + (c2 * LINE_2) + (c1 * LINE_1);

	DrawFormatString(50, 100, Color, "4ライン: %d", t);
	DrawFormatString(50, 200, Color, "3ライン: %d", c3);
	DrawFormatString(50, 300, Color, "2ライン: %d", c2);
	DrawFormatString(50, 400, Color, "1ライン: %d", c1);

	DrawFormatString(800, 720 / 3, Color, "SCORE: %ld ", score);


}
/*ホールド描画*/
void Player::Draw_Hold()
{
	const int pos_x = 100;
	const int pos_y = 100;

	for (int y = 0; y < 4; y++)
	{
		for (int x = 0; x < 4; x++)
		{
			if (block_hold != -1)
			{
				if (gblock[block_hold][0][x][y] != 0)
				{
					DrawGraph((block * x) + pos_x, (block * y) + pos_y, block_tmp[cidx[block_hold]], true);
				}
			}
		}
	}
}

void Player::Draw_Next()
{
	int cvy = 0;
	int addx = DRAW_AREA_X + block * 14;
	int addy = DRAW_AREA_Y + block * 2;

	for (int i = 0; i < 4; i++) {
		for (int y = 0; y < 4; y++) {
			for (int x = 0; x < 4; x++) {
				if (gblock[nexts[i]][0][x][y] != 0)
				{
					DrawGraph((block * x) + addx, (block * y) + (cvy)+addy, block_tmp[cidx[nexts[i]]], true);
				}
			}
		}
		cvy += 100;
	}
}

//***********************************************
void Player::DrawUpdate()
{	
	//DrawGraph(0,0,back_ground,true);
	Draw_Hold();
	Draw_Score();
	Draw_PutBlock();
	frame();
	Draw_Block();
	Draw_Next();

}

//***********************************************


/*枠組み*/
void Player::frame()
{

	/*枠の表示*/
	for (uint y = 0; y < STAGE_Y; y++)
	{
		for (uint x = 0; x < STAGE_X; x++)
		{
			if (stage[x][y] == STAGE_FRAME || stage[x][y] == STAGE_LOWER)
			{
				DrawGraph(x * block + DRAW_AREA_X , y * block + DRAW_AREA_Y, frame_hendle, true);
			}
		}
	}
}




/*終了*/
void Player::EndProcess()
{

}

Answer 1

「正しい」かどうかはプログラム全体としてちゃんと期待した動作をするかどうか、という話になるのでなんとも言えませんが、手法としてはありでしょう。

画面のリフレッシュは、ハード的に発生する定時呼び出しになるでしょう。その回数を数えるのですから、計時の手法としてはありです。
また、画面の表示を行うプログラムですから、画面の更新より細かい時間には意味はありません。次の画面更新までに次の画面の描画が出来ていればいいわけですから、それは必要にして十分な更新周期、と言えるでしょう。

Comments

[退会済みユーザー]

いろんなものがアニメーションする場合はゲーム起動時からのフレーム数を数えてそれを % 60 == 0
するなりすればいいのでしょうか？