[SOLVED] White square problem using vector<Texture>

[ondrej008]

Hi guys,
I've attempted to make a TextureManager class, which loads and stores the textures in a std::vector, however for some reason drawing the textures makes a white square.

Here's the code:

TextureManager

(click to show/hide)

class TextureManager
{
public:
        sf::Texture* getTexture(unsigned int ID)
        {
                if (textures.size() <= ID)
                {
                        return &textures[0];
                }
                else
                {
                        return &textures[ID];
                }
        }

        TextureManager()
        {
                sf::Texture text;

                text.loadFromFile("resources/null.png");

                textures.push_back(text);

                std::cout << &textures[0] << "\n";
        }

private:
        std::vector<sf::Texture> textures;
};

Tile

(click to show/hide)

class Tile
        {
        public:
                float X = 0;
                float Y = 0;

                sf::Texture* texture;

                void setTexture(sf::Texture* texture)
                {
                        this->texture = texture;
                }

                Tile(sf::Texture* texture = 0, float X = 0, float Y = 0)
                        : texture(texture)
                        , X(X)
                        , Y(Y)
                {

                }

                void draw(sf::RenderWindow& window)
                {
                        sf::Sprite sprite;
                        sprite.setTexture(*texture);
                        sprite.setPosition(X, Y);
                        window.draw(sprite);
                }

        };

I use tile.draw(window); and it just makes white squares.
Does the vector change the memory location or something?

[ondrej008]

I'm an idiot, the texture manager was in the wrong scope so the textures got deleted.

[fallahn]

I think it's worth noting that if you're using a vector of textures a call to push_back() may cause it to resize. When this happens the existing textures will be moved around in memory (probably using some fairly heavy copy operations due to the way sf::Texture is implemented) and invalidate existing pointers, returning you to your white square problem. Take a look at how the resource manager is implemented in the sfml game development book, it has served me well for many years

https://github.com/SFML/SFML-Game-Development-Book/blob/master/02_Resources/Include/Book/ResourceHolder.hpp