C++ 標準ライブラリで処理をミリ秒待機させたい。std::this_thread::sleep_forは待機する関数として正しいのか？

Question

#現状
極端に動作重たい

質問内容

提示コードのコメント部のコードですがこれは処理を待機する関数とした正しいのでしょうか？実際下記のテキストファイル等に待機するミリ秒が表示されるのですがこれは６０フレームでの待機ミリ秒のため問題なと思うんですが動作が極端に遅くなります。
std::this_thread::sleep_for(std::chrono::milliseconds(wait));　関数はミリ秒待機する関数ですがこれが悪いのでしょうか？
原因がわかりません。

参考サイトA：　https://marycore.jp/prog/objective-c/sleep-process/#sleep%E9%96%A2%E6%95%B0

参考サイトB：　https://cpprefjp.github.io/reference/chrono.html

17
17
16
17
17
15
17
17
16
17

#ifndef ___FPS_H_
#define ___FPS_H_


#include <iostream>
#include <chrono>
#include <thread>
#include <time.h>
#include <fstream>



class Fps
{
public:
      Fps();
      ~Fps();

      void Wait();      
      void Update();    

private:

      long long getMilliSeconds();


      long long count;
      long long start;
      long long wait;
      long long fps = 60;
      
};



#endif

#include <iostream>
#include <chrono>
#include "../header/Fps.hpp"
#include "../lib/ncurses/include/curses.h"
#include "../header/Log.hpp"



Fps::Fps()
{
      wait = 0;
      start = 0;
      count = 0;
}

//ミリ秒を取得
long long Fps::getMilliSeconds()
{
      std::chrono::system_clock::time_point time = std::chrono::system_clock::now();
      std::chrono::milliseconds tp_msec = std::chrono::duration_cast<std::chrono::milliseconds>(time.time_since_epoch());      
      long long all_msec = tp_msec.count();//経過時間を総ミリ秒(整数値)に変換
      return all_msec;
}


void Fps::Wait()
{
      long long t = getMilliSeconds() - start;

      //Print("t                                    %lld",t);
      //Print("(count * 1000 / fps)                 %lld",(count * 1000 / fps));

      wait = (count * 1000 / fps) - t;
      //Print("%lld\n",wait);

      if(wait > 0)
      {
            //Print("%lld\n",wait);
           std::this_thread::sleep_for(std::chrono::milliseconds(wait));
      }
}


void Fps::Update()
{
      if(count == 0)
      {
            start = getMilliSeconds();                        
      }
      else if(count == fps)
      {
            start = getMilliSeconds();
            count = 0;

      }

      count++;
}


Fps::~Fps()
{
      
}

//Main.cpp
	
	while (true)
	{
		
		fps->Update();
		entry->Update();		//更新
		entry->Renderer();	//描画	

		if(entry->getChangeScene() == Scene::SceneType::Exit)
		{
			break;
		}

		fps->Wait();
	}

Answer 1

std::this_thread::sleep_forは関数が呼ばれてから指定秒数sleepします。

fpsを固定する目的の場合、1frame分の処理を始める前から指定秒数sleepしてほしいと思うので不適当です。代わりにstd::this_thread::sleep_untilを用います。これはあらかじめsleepを解除したいtimepointを計算しておきそれを指定します。

つまり、1frame分の処理を始める前に現在時刻を取得し、処理を開始し、処理を終えたらstd::this_thread::sleep_untilでsleepします。

C++11からC++17を駆け抜けるC++講座 - Qiita#リテラル演算子

Comments

[fana]

最も肝心な Fps::Wait() の呼び出し部分が示されていないので実情は謎ですが，例えば，

1frame分の処理 を終えた後で，
count をインクリメントし，
次frameの処理を開始すべき時刻までsleepする目的で Fps::Wait() を呼ぶ
（Fps::Wait() は，sleep_forの引数に sleep すべき時間を指定していると見える）

のであれば，sleep_for でも良いように思えます．

[退会済みユーザー]

提示コードを修正しました。
質問ですがtimepointを用いる場合どうやって計算を行えばいいのでしょうか？

[yumetodo]

単に足し算すれば良いです。operatorがoverloadされてます。

[yumetodo]

using clock = std::chrono::steady_clock;
using namespace std::chrono_literals;
auto t0 = clock::now();
//do something
std::this_thread::sleep_until(t0 + 1.0s / 60);

多分こんなかんじ

[退会済みユーザー]

質問ですが以下ように挑戦したのですがオペレーターがありません等のエラーでなかなかコンパイル出来ませんどうしたらいいのでしょうか？
void Fps::Wait()
{
std::chrono::_V2::steady_clock::time_point t = std::chrono::_V2::steady_clock::now() - start;
// std::this_thread::sleep_until();

using clock = std::chrono::steady_clock;
using namespace std::chrono_literals;
auto t0 = clock::now();
//do something
std::this_thread::sleep_until(t0 + 1.0s / 60);

}

void Fps::Update()
{
start = std::chrono::_V2::steady_clock::now();
}

Fps::~Fps()
{

}

[yumetodo]

エラーメッセージを提示してください。手元では通っているので
https://wandbox.org/permlink/9B8J7VYFXa5Jnhmo

[yumetodo]

`std::chrono::_V2::steady_clock`

あとこれいったい何をつかおうとしているんですか、やめましょう