[SOLVED] Spritesheet algorithm not working

[gustav9797]

This is the current code we use to convert one image full of blocks to seperated ones, but we only get a white block. Using sfml 1.6.

How we use it:

Code: [Select]

Initializer:
if(AddSpriteSheet(std::string("BlockSolid.png"), 16, 16))
{
std::cout << "its working, cheese-pie.";
}
PlayState.cpp:

Code: [Select]

sf::Sprite* tempSprite = tc.getTexture(std::string("BlockSolid.png"));
playerSprite = &(tempSprite[0]);
TextureContainer.cpp:

Code: [Select]

bool TextureContainer::AddTexture(std::string &fileName)
{
sf::Image image;
bool success = image.LoadFromFile(fileName);
sf::Sprite *sprite = new sf::Sprite();
sprite->SetImage(image);
textureMap.emplace(fileName, sprite);

return success;
}

bool TextureContainer::AddSpriteSheet(std::string &fileName, int spriteWidth, int spriteHeight)
{
sf::Image image;
bool success = image.LoadFromFile(fileName);

int width = image.GetWidth()/spriteWidth;
int height = (image.GetHeight()/spriteHeight);

sf::Sprite *sprite = new sf::Sprite[width*height];

sf::Image tempImage(16, 16);

for (int x = 0; x+1 <= width; x++)
{
for (int y = 0; y+1 <= height; y ++)
{
tempImage.Copy(image, 16, 0, sf::IntRect(x*spriteWidth, y*spriteHeight, (x + 1)*spriteWidth, (y + 1)*spriteHeight), true);
sprite[x+y*width].SetImage(tempImage);
}
}
textureMap.emplace(fileName, sprite);
return success;
}

[G.]

Keep your image alive.
It's asked A LOT in the forums...

[ostkaka]

Keep your image alive.
It's asked A LOT in the forums...

TY. -.-
[EDIT]
But no... It didn't really help. Why doesn't this work?

Code: [Select]

tempImage.Copy(image, 16, 0, sf::IntRect(x*spriteWidth, y*spriteHeight, (x + 1)*spriteWidth, (y + 1)*spriteHeight), true);

[G.]

There are at least 2 flaws in TextureContainer.cpp. (btw, who are you lol?)
The tutorial says this: "a sprite only points to an external image it doesn't own one".
Which means that:
- when your image is deleted (end of scope where you declared it) your sprite doesn't have an image anymore, hence the white box
- if your image is modified, it affects every sprites pointing to this image

[ostkaka]

There are at least 2 flaws in TextureContainer.cpp. (btw, who are you lol?)
The tutorial says this: "a sprite only points to an external image it doesn't own one".
Which means that:
- when your image is deleted (end of scope where you declared it) your sprite doesn't have an image anymore, hence the white box
- if your image is modified, it affects every sprites pointing to this image

oh, ty... I actually know that.

[G.]

So what do you actually want? 
Maybe I'm wrong, I'll let other people try.