Сделать указатель Getter use unique_ptr

Question

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

} // RectangleShape Getter 

Нужно ли мне освобождать память после этого геттера? Если да, как это сделать с unique_ptr?

Я попытался

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

} // RectangleShape Getter 

Но он говорит, что нет функции соответствия для оператора скобки. Как еще это сделать?

Answer 1

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

} // RectangleShape Getter 

Do I need to free memory after this getter?

Все, что вам сделать, это вернуть указатель на player. Предполагая, что это переменная-член структуры/класса, указанная выше функция-член является частью, то ее время жизни привязано к времени жизни объекта, с которым вы вызываете эту функцию.

В частности, это означает, что это плохая идея:

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 имеет конструктор копирования, то вы можете сделать копию player:

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

Копия вы получаете этот путь «ваш», то есть вы можете управлять жизнью, как хотите.

Answer 2

Похоже, что player является членом класса, и вы пытаетесь передать указатель на него для его изменения вне класса?

В этом случае класс, которому он принадлежит, принадлежит к памяти, и до этого класса он обрабатывает освобождение данных при их уничтожении. Указатель на этот элемент не должен быть полностью освобожден вне класса.

Это поможет получить дополнительную информацию об владеющем классе, но если я могу предположить, что player является членом данных, то ваш первый пример технически хорош. Однако часто бывает более идиоматично возвращать ссылку, чем указатель.

class SomeClass 
{ 
    sf::RectangleShape player; 
    sf::RectangleShape& operator()() 
    { 
     return player; 
    } 
}; 

Если приведенное выше допущение неверно, вы должны показать свое полное определение класса, чтобы иметь более подробную информацию для формирования правильного решения.