Home/ Questions/Q 6684799
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-26T04:58:55+00:00

I am aware that this is an easy problem but I don’t know how

  • 0

I am aware that this is an easy problem but I don’t know how to do properly what I have in mind. I am using SFML but you don’t need to understand how it works.
Here is my simple code snippet that generate tiles and assign them to a 2D-vector of tiles:
http://www.pastie.org/2665489

During the construction of the TilePlane object, each operation (tilesMap[i]).push_back(tile) takes far too long and I feel that I am copying data instead of using a more elegant method. So in this case, what is the good method to solve this problem?

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    You have a lot of performance issues in your code. If the bottleneck indeed comes from push_back, you should take Masoud’s suggestion and push pointers onto the array and not values.

    Another suggestion is that you declare your functions and constructors to use references – for example:

    Tile::Tile(const Tileset& tileset, int tileId, int posX, int posY)

    instead of

    Tile::Tile(Tileset tileset, int tileId, int posX, int posY)

    Passing the parameter by value will force a copy to be created, which can be expensive. Passing by const reference guarantees the object is not copied or modified.

    • 0