Make Pointer Getter verwenden unique_ptr

Question

sf::RectangleShape* operator()() 
{ 
    return &player; 

} // RectangleShape Getter 

Muss ich Speicher nach diesem Getter freigeben? Wenn ja, wie würde man das mit unique_ptr machen?

versuchte ich

std::unique_ptr<sf::RectangleShape> operator()() 
{ 
    return std::unique_ptr<sf::RectangleShape>(player); 

} // RectangleShape Getter 

Aber es sagt, dass es für die Klammer Operator keine passende Funktion ist. Wie sonst sollte das gemacht werden?

Answer 1

sf::RectangleShape* operator()() 
{ 
    return &player; 

} // RectangleShape Getter 

Do I need to free memory after this getter?

Alles, was Sie tun, ist ein Zeiger auf player zurückzukehren. Angenommen, dies ist eine Membervariable der Struktur/Klasse, zu der die obige Elementfunktion gehört, dann ist ihre Lebensdauer an die Lebensdauer des Objekts gebunden, mit dem Sie diese Funktion aufrufen.

Insbesondere bedeutet dies, dass dies eine schlechte Idee ist:

std::unique_ptr<sf::RectangleShape> operator()() { 
    return make_unique<sf::RectangleShape>(player); 
} 

Die Kopie:

struct Foo { 
    sf::RectangleShape player; 
    // ... 
    // insert your operator here 
}; 

sf::RectangleShape * some_function(void) { 
    Foo f; 
    return f(); // UB, returning pointer to object with automatic memory whose lifetime has ended 
} 

[..] how would one do this with unique_ptr?

sf::RectangleShape eine Kopie Konstruktor Unter der Annahme, dann könnte man eine Kopie player machen Du bekommst den Weg "dein", dh du kannst dein Leben so gestalten, wie du es willst.

Answer 2

Es scheint wie player Mitglied einer Klasse ist und Sie versuchen, einen Zeiger darauf zu verteilen, damit es außerhalb der Klasse geändert wird?

In diesem Fall besitzt die Klasse, zu der sie gehört, den Speicher, und es liegt an dieser Klasse, die Daten freizugeben, wenn sie zerstört werden. Ein Zeiger auf dieses Member sollte absolut nicht außerhalb der Klasse freigegeben werden.

Es würde helfen, mehr Informationen über die besitzende Klasse zu haben, aber wenn ich annehmen kann, dass player ein Datenmitglied ist, dann ist Ihr erstes Beispiel technisch in Ordnung. Es ist jedoch oft idiomatischer, eine Referenz als Zeiger zurückzugeben.

Wenn die obige Annahme falsch ist, sollten Sie Ihre vollständige Klassendefinition anzeigen, damit wir mehr Informationen haben, um eine korrekte Lösung zu bilden.