As Gambit said, be specific when you ask a question. Please read this carefully:
http://en.sfml-dev.org/forums/index.php?topic=5559.0And please provide real code that we can copy, not a screen capture.
To answer the question: no, this code is not correct. What your constructor does is to create its own copy the texture, but I assume that you just want to use the original one.
const sf::Texture& m_texture;
Player::Player(const sf::Texture& texture) : m_texture(texture)
{
...
}
Or
const sf::Texture* m_texture;
Player::Player(const sf::Texture& texture)
{
m_texture = &texture;
}
This comment and solution are based on assumption about what you want to do, so no guarantee that it matches what you were asking.