C++は、私がこれに似た質問がある知っているが、それらのどれも私の質問に明確な答えを与えていない新しいオブジェクト

Question

への参照を返す...

は、ベストプラクティスの両方の観点からこれら大丈夫か？それともポインタを返すべきですか？ ベストプラクティスに従うためにどのように変更する必要がありますか？

関数から新しいオブジェクトへの参照を返したいとします。

MyClass& doSomething() { 
    return *(new MyClass()); 
} 

MyClass a = doSomething(); 

MyClassの新しいインスタンスがヒープに新しいもので割り当てられるため、これは問題ありませんか？

これを一定にする必要がありますか（これを行うかどうかはわかりません）。

const MyClass& doSomething() { 
    return *(new MyClass()); 
} 

これらの両方が間違っている場合は、新しいオブジェクトへのポインタを戻すだけでいいですか？

ありがとうございました。

Answer 1

これは間違いではありませんが、それは良い考えではありません。あなたが返された後

MyClass& doSomething() { 
    return *(new MyClass()); 
} 

、誰もが元のポインタを持っていないので、誰もがそれ史上deleteます。*だから、メモリリークです。

対応するdeleteまたはそれ以上のスマートポインタコンストラクタがない限り、newを書くことはほとんどありません。

---

一方、元のコードでこの行：

MyClass a = doSomething(); 

...とにかく値のコピーを作成しようとしています。それが修正されなければならない別のバグではないと仮定すると、なぜヒープ割り当てをオブジェクトに割り当て、コピーとリークの参照を返しますか？オブジェクトを値で返すだけです。

MyClass doSomething() { 
    return MyClass(); 
} 

ヒープ上に何も作成していないので、何も削除することを心配する必要はありません。

---

ベストプラクティスは、通常、RAII：Resource Acquisition Is Initializationの4文字で集計できます。 （そしてその結果、その破壊は解放されます。）あなたが値段で回ることは不可能な、あるいは高価な何かを持っているなら、価値によってそれにいくらかのハンドルを渡します。例：

unique_ptr<MyClass> doSomething() { 
    return unique_ptr<MyClass>(new myClass()); 
} 

unique_ptr<MyClass> a = doSomething(); 

これは単なるポインタのコピーです。オブジェクトはdoSomething内に作成され、aがスコープ外になると削除されます（または、別の変数に渡した場合は、がスコープ外になるなど）。

一方、MyClassが簡単にコピーできる値のほんの一握り**である場合は、コピーしてください。

---

*それが今までそれを削除する不可能ではありません。あなたはいつもリファレンスへのポインタをとることができ、それはdeleteです。あなたがそうすることはほとんどありません。それは厄介なように見えます。ポインタを渡す場合は、ポインタを渡します。ポインタを渡す必要がない場合は、クラス内で所有権をラップし、値でクラスを渡します。

**簡単にコピーできるとは、を安全ににコピーして、実際にそうすることを意味します。例えば、生のポインタやファイルハンドルは数バイトで、デフォルトのコピーコンストラクタはあなたを喜んでコピーしますが、同じヒープオブジェクトやファイルへの複数の参照が終わることになります。誰がそれを削除または閉鎖するかを担当しています。