glfwGetTime();関数を使ってフレームレートを固定する方法が知りたい。

Question

提示コードですが1000倍することによってミリ秒を取得してそれを使ってフレームレート制御してwait変数の数値の分プログラムを停止するコードを作りたいのですがなぜかint t の値が段々増えてしまし困っています。何が原因なのでしょうか？

//待機フレームを計算
void FrameWork::Window::FrameUpdate()
{
	
	if (count == 60)
	{
		int t = ((int)(glfwGetTime() * 1000.0f)) - startCount;



//		std::cout << "glfwGetTime(): " << (int)(glfwGetTime() * 1000.0f) << std::endl;
//		std::cout << "startCount: " << startCount << std::endl;
	//	std::cout <<"t: "<< t << std::endl;

		wait = t / 60;
		//std::cout << wait << std::endl;
		//startCount = 0;
		count = 0;
		//glfwSetTime(0);
	}



	if (count == 0)
	{
		startCount = ((int)(glfwGetTime() * 1000.0f));
		std::cout << "startCount: " << startCount << std::endl;


	}


	count++;

}



//待機
void FrameWork::Window::Wait()
{
	std::this_thread::sleep_for(std::chrono::milliseconds(wait)); // 3 ミリ秒

}

Comments

[itagagaki]

デバッグプリントの痕跡が見えますが、それではわからなかったのでしょうか？

[退会済みユーザー]

わからないので質問しました。なぜtの値が増え続けるのでしょうか？

[itagagaki]

startCountがおかしくはなかったですか？

[退会済みユーザー]

どんどん値が増えていく。これがおかしいのでしょうか？

[退会済みユーザー]

void FrameWork::Window::FrameUpdate()
{
std::cout << "glfwGetTime(): " << (int)(glfwGetTime() * 1000.0f) << std::endl;

と、ログだすとどんな値がでるのでしょうか？10回ぐらい分ぐらい値を見てみたいです。コメントアウトしたログ用の値も一緒にみれると何かわかるかもしれません。

glfwGetTime()のreturn値は、秒なので、*1000してもmsec単位の時刻がとれず、FrameUpdate()がコールしても時間の更新がないのでは。と想像しています。

[退会済みユーザー]

ログありがとうございます。このログだと、FrameUpdateの関数は想定どおりで、waitにはだいたい16がはいってそうです。

どのあたりを直したいのか教えてくれるとアドバイスできるかもしれません。

[退会済みユーザー]

waitの値がどんどん増えてしまっているのでt変数にどんどん大きな値が入っているのですがこれはどうすればいいのでしょうか？startCount 変数でその分の値を引いてるはずなのでこんな値になるはずないのですが

[itagagaki]

質問に貼られた画像では t: 999 前後でほぼ一定ですよね。
「int t の値が段々増えて」とはなっていないようですが。
何が問題なのですか。

[退会済みユーザー]

そう、ログだとtは一定みたい。FrameUpdateは、正しく実装されてそうー。
wait()はいつ、どう言う時に呼ばれるのでしょうか？

[退会済みユーザー]

毎フレームです。

[退会済みユーザー]

すいません、ちょっと間違っていました。
userisgodさんの言うとおり、実装するのがよいと思います。

[退会済みユーザー]

例でFrameUpdate()を実装してみました。
1秒単位で調整するのは難しそうだったので、毎Frame時間を調整するようにしました。

void UpdataFrame(void)
{
static double lasttime = 0;
double nowtime = 0;
double deltatime = 0;
double limittime = 1.0/60;

if (count == 0) {
lasttime = glfwGetTime();
} else {
nowtime = glfwGetTime();
deltatime = nowtime - lasttime;
}
if (deltatime > limittime) {
wait--;
} else {
wait++;
}
lasttime = nowtime;
count++;
return;
}
これより良い実装があるような気がしますが。。。

こちらの内容も参考になると思います。
https://stackoverflow.com/questions/20390028/c-using-glfwgettime-for-a-fixed-time-step

tの時間が増えるのは、毎フレーム処理何か処理をしていると思いますが、その処理時間が考慮されていないので、tがだんだん増えていくと思いますー。

[退会済みユーザー]

毎フレームの処理が一定でなければ、以下の方がよいかもしれません。

void UpdataFrame(void)
{
static double lasttime = 0;
double nowtime = 0;
double deltatime = 0;
double limittime = 1.0/60;

if (count == 0) {
lasttime = glfwGetTime();
} else {
nowtime = glfwGetTime();
deltatime = nowtime - lasttime;
}
wait = (2 * limittime) - deltatime;
lasttime = nowtime;
count++;
return;
}

[退会済みユーザー]

質問と矛盾する画像を貼られても困ると思いますが。
せめて、現象が再現している画像を貼るべきかと。

1000の次に999と増えていないのは、1フレームで行われる処理が十分に軽く、
「wait」の計算時に切り捨てられた部分によって減ったという感じでしょうか。

[退会済みユーザー]

なるほど。　
//待機
void FrameWork::Window::Wait()
{
if ((int)(wait * 1000.0f) > 0)
{
std::this_thread::sleep_for(std::chrono::milliseconds((int)(wait * 1000.0f))); // 3 ミリ秒
std::cout << wait * 1000.0f << std::endl;
}

}
こうしましたがこれは正しいのでしょうか？

[退会済みユーザー]

何が「なるほど」なのかわかりませんが、該当の「Wait()」だけを修正したのであれば、
意味の分からない修正を行っているのではないでしょうか。
誰も、「Wait()」に関して、指摘は行っていないと思います。
私の回答も理解できなかったということでしょうか。

> こうしましたがこれは正しいのでしょうか？

あなたの想定した通りに動くのであれば正しいのではないでしょうか。

[退会済みユーザー]

ミリ秒の取得にwait * 1000.0fという処理は正しいのでしょうか？

[退会済みユーザー]

それは動かしてみた上で聞いてるのですか？
私の回答は読んでないんですか？

[退会済みユーザー]

Kesoki4さんのコードで対応したなら、自己解決で質問を閉じたらどうでしょうか。

Answer 1

waitの計算が、「経過時間 / 60」となっているからです。

現状、60周期で経過時間分スリープするので、それ以外の処理に時間がかかっていれば、
経過時間は「前回の経過時間 + 処理にかかった時間」分となり増えていくことになります。

int t = ((int)(glfwGetTime() * 1000.0f)) - startCount;
wait = t / 60;

原因について理解できなかったようなので、追記

// 1000msから経過時間を引き、スリープが必要な時間を計算する
int t = (int)(1000 - ((glfwGetTime() * 1000.0) - startCount));
wait += t / 60;

提示コードですが1000倍することによってミリ秒を取得してそれを使ってフレームレート制御してwait変数の数値の分プログラムを停止するコードを作りたいのですがなぜかint t の値が段々増えてしまし困っています。何が原因なのでしょうか？

上記、質問自体については以上となります。

glfwGetTime();関数を使ってフレームレートを固定する方法が知りたい。

タイトルについては、毎フレーム処理が一定なんてことはないでしょうし、
この方法自体、筋が悪いと思うので組み直した方がいいでしょう。

Comments

[退会済みユーザー]

タイトルの「glfwGetTime();関数を使ってフレームレートを固定する方法が知りたい。」は丸投げなので無視し、本文にある問題に対する回答のみ行っています。

[退会済みユーザー]

質問ですが23.3ミリ秒　待機するみたいなのですがこれは正しいのでしょうか？

[退会済みユーザー]

それはKesoki4さんのコードで対応した結果でしょうか？
私の方法であれば、そのような数字にならない気がしますが。

フレームレートを60に固定するのであれば、1フレーム16.67msなので、
それを超えてる時点で自明だと思いますが。

[退会済みユーザー]

しかし毎フレーム負荷かが違う場合その分長く待機する場合などあるのでしょうか？
9.80453
21.333
17.9422
14.1985
10.2187
22.5987
17.5847
13.4182
といった待機時間が発生していますミリ秒

[退会済みユーザー]

「Wait()」の時に出力している数字でしょうか。

それと、Kesoki4さんのコードで対応した結果でしょうか？
私の方法であれば、60フレームごとに変わるので、そのような数字にならない気がしますが。

[退会済みユーザー]

そうです。間違えているのでしょうか？

[退会済みユーザー]

なら、聞く相手を間違えていないでしょうか。

あのコードに関してであれば、waitの計算の意味がわからないですし、
countを残している理由もわからないですね。

それと「Wait()」で出力されている数字だけ並べても意味ないですよね。
実際に処理にかかった時間と合わせて、1フレームの時間で調整されるべきなので。