Question

我的程序将处理器加载了80％。前段时间我在gpu上遇到了同样的问题，我通过计时器解决了它。 Cpu负载约为50-60％，而现在为80％。我做错了什么？我解决不了这个问题。

#include <fstream>
#include <SFML/Graphics.hpp>
#include <ctime>
using namespace std;
char * readFile(char * filePath, unsigned int &lengthBuffer, fstream &f) {
    f.open(filePath, ios::in | ios::app | ios::binary);
    f.seekg (0, f.end);
    lengthBuffer = f.tellg();
    f.seekg (0, f.beg);
    char * fileBuffer = new char[lengthBuffer];
    f.read(fileBuffer, lengthBuffer);
    f.close();
    return fileBuffer;
}
char * writeFile(char * fileBuffer, char * filePath, unsigned int &lengthBuffer, fstream &f, bool &fileCreated){
    filePath[23] += 1;
    f.open(filePath, ios::out | ios::app | ios::binary);
    f.write(fileBuffer, lengthBuffer);
    filePath[23] -= 1;
    fileCreated = 1;
    f.close();
    return fileBuffer;
}

void removeFile(char * filePath, bool &fileCreated) {
    filePath[23] += 1;
    remove(filePath);
    filePath[23] -= 1;
    fileCreated = 0;
}
unsigned int mouse(unsigned int &funcSelector, bool &mouseLeft, sf::RenderWindow &window) {
    mouseLeft = sf::Mouse::isButtonPressed(sf::Mouse::Left);
    sf::Vector2i mouse = sf::Mouse::getPosition(window);
    if (mouseLeft & mouse.y < 100) {
        funcSelector = 1 + mouse.x/100;
    }
    return funcSelector;
}
int main(){
    sf::RenderWindow window(sf::VideoMode(500, 400), "COT++", sf::Style::Titlebar);
    sf::VertexArray points(sf::Points, 3000);
    sf::Event event;
    fstream f;
    bool mouseLeft, fileCreated = 0;
    unsigned int n = 0, funcSelector = 0, lengthBuffer = 0;
    float start = 0, now = 0, x = 0.f, y = 1.f, pos = 0.f;
    char * fileBuffer, filePath[30] = "c:/users/79994/desktop/a.exe";
    while (x < 500.f){
        points[n].position = sf::Vector2f(x + pos, y + pos);
        points[n].color = sf::Color::Green;
        x += 1.f;
        n += 1;
        if (x == 500.f & y < 100.f){
            x = 0.f;
            y += 100.f;
        }
    }
    x = 100.f, y = 1.f;
    while (x < 600.f){
        points[n].position = sf::Vector2f(x + pos, y + pos);
        points[n].color = sf::Color::Green;
        y += 1.f;
        n += 1;
        if (y > 101.f){
            x += 100.f;
            y = 1.f;
        }
    }
    while (window.isOpen())
    {
        while (window.pollEvent(event)) {
            if((clock()-start) > 50){
            start = clock();
            switch(funcSelector){
                case 5: window.close();
                break;
                case 1: if (lengthBuffer == 0){
                    fileBuffer = readFile(filePath, lengthBuffer, f);
                }    
                break;
                case 2: if (lengthBuffer > 0 & fileCreated == 0) {
                    writeFile(fileBuffer, filePath, lengthBuffer, f, fileCreated);
                }
                break;
                case 3: removeFile(filePath, fileCreated);  
                break;
            }
            mouse(funcSelector, mouseLeft, window);
            window.clear();
            window.draw(points);
            window.display();
        }
    }
    }
    return 0;
}

P.S。 “看来您的帖子大部分是代码；请添加更多详细信息”-我认为我描述了足够的详细信息。

Answer 1

这取决于您想做什么。对于您给出的简单示例，可以使用waitEvent代替pollEvent。我建议检查waitEvent是否适合您的需求，但是看起来会合适。

但是，如果您坚持使用pollEvent，请继续阅读！

您的while(window.pollEvent(event))循环100％地全速运行，不停地轮询，即使您的目标似乎是每50个时钟仅“工作”一次。（除了CLOCKS_PER_SEC，which is implementation defined ...和std::clock可能与挂钟时间...和CLOCKS_PER_SEC { {3}} ...但我离题了。无论您如何跟踪时间，对于编写的代码，您的问题都会存在，尽管您可能需要其他计时机制。

这是对您的代码进行最少更改的一种可能的解决方案。替换：

    while (window.pollEvent(event)) {
        if((clock()-start) > 50){

具有以下内容：

    while (window.pollEvent(event)) {
        const auto delta = clock() - start;
        if (delta < 50) {
          std::this_thread::sleep_for(std::chrono::microseconds(
            static_cast<int>(1E6 * (50 - delta) / CLOCKS_PER_SEC)));
        } else {

这是做什么的，而不是不断地调用window.pollEvent(event)，它会调用它，并在需要时睡一会儿，然后做一些工作。

这种方法也有一些缺点，但是它应该使您开始思考问题。

您还需要#include <chrono>（或者，如果您不使用C++11或更高版本，则可以在几乎正确的时间内找到其他睡眠方式）。

Answer 2

仅通过查看代码来尝试猜测就或多或少是困难和不精确的。您应该尝试使用探查器对代码进行探查。

PS：您似乎在分配缓冲区但从未释放过缓冲区的“ readFile”和“ writeFile”中出现内存泄漏。

char * readFile(char * filePath, unsigned int &lengthBuffer, fstream &f) {
    // [...]
    char * fileBuffer = new char[lengthBuffer];
    // here you allocated memory on the heap, but you'll never free it.
    // [...]
    return fileBuffer;
}

我也认为完全不使用那些动态Char-Arrays和C-Files。只需使用std :: string和std :: ostream。 ostream可以与更多的STL和字符串一起使用，您不必在意它的内存使用情况和潜在的内存泄漏。它会自行清除，而不是字符数组。

高CPU负载。 C ++ / sfml

2 个答案: