ポインタを作るゲッターを使用するunique_ptr

Question

このゲッターの後にメモリを解放する必要がありますか？はいの場合、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/classのメンバ変数であり、上記のメンバ関数がその一部であると仮定すると、その寿命は、その関数を呼び出したオブジェクトの存続期間に縛られます。これは悪い考えであることを意味し、特に

：sf::RectangleShapeを想定し

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?

は、コピーコンストラクタを持っている、あなたは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; 
    } 
}; 

上記の仮定が間違っている場合は、完全なクラス定義を表示して、正しい解決策を形成するための詳細情報を入手する必要があります。