spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

mein vorschlag: http://pastebin.com/QWj4hnXE
so kannst du das image mehrmals verwenden ohne es mehrmals im speicher zu haben.
das unterteilen in draw und update macht sinn weil du das oft nicht direkt hintereinander machen darfst/willst und zum beispiel bei einer pause nur die draw methode aufrufen kannst. dann wird das schiff gezeichnet aber es bewegt sich nicht.
versuch auch immer den konstruktor zum initialisieren zu benutzten. eine load methode kann man schnell mal vergessen, besonders wenn man die klasse nicht selbst geschrieben hat, aber den konstruktor kann man überhaupt nicht vergessen.

sf::Image ist eine recht speicherintensive Klasse. Darum gibt es auch sf::Sprite, die für die Verwaltung des angezeigten Bildes zuständig ist. Du solltest also auf jeden Fall das sf::Image aus deiner Klasse auslagern. Evtl. schreibst du dir auch einen kleinen Manager, der alle vorhanden Bilder verwaltet und bei Anforderung erst mal prüft, ob das Bild schon im Speicher ist und ggf. eine Referenz auf dieses Bild liefert.