我正在创建一堆需要在帧周期内工作的线程。我想控制一秒钟完成多少帧。我简化了我的代码，这样我就可以向你展示我写的内容

// setup the frame timer
std::chrono::time_point<std::chrono::system_clock> start = std::chrono::system_clock::now();
std::chrono::time_point<std::chrono::system_clock> end = std::chrono::system_clock::now();

while(running == true)
{
    // update timer
    start = std::chrono::system_clock::now();
    std::chrono::duration<double> elapsed_seconds = start - end;
    double frameTime = elapsed_seconds.count();

    this->Work(frameTime);

    // update timer
    std::chrono::time_point<std::chrono::system_clock> afterWork = std::chrono::system_clock::now();
    std::chrono::duration<double> elapsedWorkTime = afterWork - end ;

    const double minWorkTime = 1000 / this->timer.NumberOfFramePerSeconds;
    if(elapsedWorkTime.count() < minWorkTime)
    {
        double timeToSleep = minWorkTime - elapsedWorkTime.count();
        std::this_thread::sleep_for(std::chrono::milliseconds((int)timeToSleep));
    }

    // update fps
    end = start;
    timer.FrameCount += 1;
}

并非所有线程的工作量都相同，有些线程比其他线程多，因此如果不休眠，我会得到与此相关的结果 线程 1:150fps，线程 2:5000fps，线程 3:5000fps，线程 4:5000fps

我想要的是能够将每秒帧数设置为 60，因此使用上面的代码我会将 this->timer.NumberOfFramePerSeconds 设置为 60。我的问题是当我这样做最终得到这样的结果 Thread 1 : 30fps, Thread 2: 60fps, Thread 3: 60fps, Thread 4: 60fps

这表明我的代码有问题，因为当我注释掉 sleep 时线程 1 会愉快地执行超过 150 帧，但是当 sleep 存在时它会在我试图达到的 60 帧以下工作。

我的代码/算法是否正确？

最佳答案

在我看来，您可能遇到了单位转换错误。这通常是由处理 chrono 之外的时间单位引起的类型系统，并引入显式转换因子。例如:

double frameTime = elapsed_seconds.count();
this->Work(frameTime);

可以Work修改为取duration<double>而不是 double ？这将有助于确保 double不会被重新解释为 Work 内秒以外的其他内容.如果由于某种原因它不能，您至少可以通过以下方式减少错误风险:

this->Work(elapsed_seconds.count());

以下看起来很可疑:

std::chrono::duration<double> elapsedWorkTime = afterWork - end ;

const double minWorkTime = 1000 / this->timer.NumberOfFramePerSeconds;
if(elapsedWorkTime.count() < minWorkTime)

elapsedWorkTime显然有秒的单位。 minWorkTime 似乎以毫秒为单位。 if丢弃所有单位信息。这应该是这样的:

std::chrono::duration<double> minWorkTime(1./this->timer.NumberOfFramePerSeconds);
if(elapsedWorkTime < minWorkTime)

或者如果您真的想要minWorkTime表示毫秒(这不是必需的):

std::chrono::duration<double, std::milli> minWorkTime(1000./this->timer.NumberOfFramePerSeconds);
if(elapsedWorkTime < minWorkTime)

我推荐前者，因为1000的介绍只是另一个错误出现的机会。

    double timeToSleep = minWorkTime - elapsedWorkTime.count();
    std::this_thread::sleep_for(std::chrono::milliseconds((int)timeToSleep));

在这里，您再次不必要地离开了 chrono 的安全网和手动做事。这应该只是:

    std::this_thread::sleep_for(minWorkTime - elapsedWorkTime);

或者如果你想更详细一点:

    auto timeToSleep = minWorkTime - elapsedWorkTime;
    std::this_thread::sleep_for(timeToSleep);

.count()的使用应该很少见。目前打印出来需要用到duration秒。还可能有必要处理无法与 chrono 一起使用的遗留代码。 .如果您发现自己使用 .count()不仅如此，退后一步并尝试将其删除。 chrono是否有助于防止单位转换错误，以及当您通过 .count() 退出系统时，你颠覆chrono存在的理由。

更新

VS2013 有一个错误，它不会在基于 double 的持续时间内休眠。因此，作为解决方法...

std::this_thread::sleep_for(std::chrono::duration_cast
                <std::chrono::milliseconds>(minWorkTime - elapsedWorkTime));

这很丑陋，但仍然将所有转换因子保留为 chrono .

感谢Caesar为我测试这个解决方法。

关于c++ - 控制帧率，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/25120825/