Low framerate with only map and minimap drawing (SFML)

Question

I'm working on a small "game" like project as a practice, and I've managed to get my framerate down to not even 3 FPS. While the only thing that is being drawn is screen filling tiles and a minimap. Now I've found that the problem is with the minimap, without it caps at 60 FPS. But unfortunately I'm not experienced enough to find out what the real problem is with it...

My draw function:

void StateIngame::draw()
{
    m_gui.removeAllWidgets();
    m_window.setView(m_view);

    // Frame counter
    float ct = m_clock.restart().asSeconds();
    float fps = 1.f / ct;
    m_time = ct;

    char c[10];
    sprintf(c, "%f", fps);
    std::string fpsStr(c);
    sf::String str(fpsStr);

    auto fpsText = tgui::Label::create();
    fpsText->setText(str);
    fpsText->setTextSize(16);
    fpsText->setPosition(15, 15);
    m_gui.add(fpsText);

    //////////////
    // Draw map //
    //////////////
    int camOffsetX, camOffsetY;
    int tileSize = m_map.getTileSize();
    Tile tile;

    sf::IntRect bounds = m_camera.getTileBounds(tileSize);

    camOffsetX = m_camera.getTileOffset(tileSize).x;
    camOffsetY = m_camera.getTileOffset(tileSize).y;

    // Loop and draw each tile
    // x and y = counters, tileX and tileY is the coordinates of the tile being drawn
    for (int y = 0, tileY = bounds.top; y < bounds.height; y++, tileY++)
    {
        for (int x = 0, tileX = bounds.left; x < bounds.width; x++, tileX++)
        {
            try {
                // Normal view
                m_window.setView(m_view);
                tile = m_map.getTile(tileX, tileY);
                tile.render((x * tileSize) - camOffsetX, (y * tileSize) - camOffsetY, &m_window);
            } catch (const std::out_of_range& oor)
            {}
        }
    }


    bounds = sf::IntRect(bounds.left - (bounds.width * 2), bounds.top - (bounds.height * 2), bounds.width * 4, bounds.height * 4);

    for (int y = 0, tileY = bounds.top; y < bounds.height; y++, tileY++)
    {
        for (int x = 0, tileX = bounds.left; x < bounds.width; x++, tileX++)
        {
            try {
                // Mini map
                m_window.setView(m_minimap);
                tile = m_map.getTile(tileX, tileY);
                sf::RectangleShape miniTile(sf::Vector2f(4, 4));
                miniTile.setFillColor(tile.m_color);
                miniTile.setPosition((x * (tileSize / 4)), (y * (tileSize / 4)));
                m_window.draw(miniTile);
            } catch (const std::out_of_range& oor)
            {}
        }
    }

    // Gui
    m_window.setView(m_view);
    m_gui.draw();
}

The Tile class has a variable of type sf::Color which is set during map generating. This color is then used to draw the minimap instead of the 16x16 texture that is used for the map itself.

So when I leave out the minimap drawing, and only draw the map itself, it's more fluid than I could wish for...

Any help is appreciated!

Answer 1

You are generating the view completly new for every frame. Do this once on startup should be enought.