Sprite as attribute doesnt Draw but local sprite Draw

[prchakal]

Hi,

Im with a small problem here that my sprite that is in my class "Character" doesnt "draw" with "window.draw", but if i create a local sprite (the same sprite code and image) it draw.

You can see here (have both sprite - local and as attribute):
https://github.com/prsolucoes/sfml-cmake

I got no error and when debug, all sprite attributes is fine

[prchakal]

I need implement that methods for cast on every class that i send to v8? Example:

namespace v8wrap {
    v8::Handle<v8::Value> CastToJS(const Character &JS);
    v8::Handle<v8::Value> CastToJS2(const Character &JS);
    void AutoCastToCPP(v8::Handle<v8::Value> Value, Character &CPP);
    v8::Handle<v8::Value> CastToJS(Character *JS);
    v8::Handle<v8::Value> CastToJS2(const Character *JS);
    void AutoCastToCPP(v8::Handle<v8::Value> Value, Character *&CPP);
}

[eXpl0it3r]

Im with a small problem here that my sprite that is in my class "Character" doesnt "draw" with "window.draw", but if i create a local sprite (the same sprite code and image) it draw.

Do you call window.display()?

I need implement that methods for cast on every class that i send to v8? Example:

I don't think any/many will know how this V8 thing works with SFML, specificly with your code, also not many if any will go and read your full code just to understand what the problem actually is.
If you want us to help you need to provide more information and make your question more specific.

[prchakal]

Hi,

o.o

The problem is here:

window.clear(sf::Color(255, 255, 255));
        window.draw(*robot1->getSprite());  // it dont display
        window.draw(*sprite);   // it display
        window.display();

I think it is not related, because the sprite in another class out of v8.

https://github.com/prsolucoes/sfml-cmake/blob/master/main.cpp
and
https://github.com/prsolucoes/sfml-cmake/blob/master/Character.cpp

Look only this two files, is a short piece of code.

[Laurent]

void Character::loadSprite()
{
    sf::Texture texture;
    if (!texture.loadFromFile(resourcePath() + "images/" + type + ".png"))
    {
        throw new std::exception();
    }
    sprite = new sf::Sprite(texture);
    sprite->setPosition(100, 200);
}

The texture is destroyed when the function exists, so the sprite is left with a pointer to an invalid memory zone. The texture must live as long as the sprite uses it.

And don't dynmically allocate the exceptions that you throw.

[prchakal]

Humm...ok.

I think that texture is store in the sprite and the object is not destroyed.

And don't dynmically allocate the exceptions that you throw.

I dont understand it ^

[FRex]

I dont understand it ^

Good:

throw std::exception();

Bad:

throw new std::exception();

[prchakal]

Humm ok.

But, can you explain me why it is bad? Only to i understand, please

[FRex]

There is memory leak if you catch it without deleting it.
And new can throw too, I think that'd take you to nearest catch(...) or catch(std::exception& e) block or call terminate. I'm not sure.