ヒープ変数かどうか| *新しいObject（）||オブジェクト（）？

1

を両ベクトルに格納し、ベクター自体がどのように自由ですそれらを保存したいが、通常フリーストアを使用する。大きな違いはしかしあり

：

vector.push_back(* new object());

これは、動的にobjectオブジェクトを割り当て、ベクター中のオブジェクトのコピーを保存し、新たに割り当てられたobjectを指し示すポインタは、その後すぐに破棄されます。

deleteobjectに割り当てたアドレスがnewではないため、後で使用するためにアドレスを保存しないため、この方法ではメモリリークが発生します。

vector.push_back(object());

これはあなたのベクトルにオブジェクトを追加する「正しい」方法である、コピーして保存されたベクトルにして、破壊され、一時的なobjectを作成します。

2017-05-16 13:38:03

+0

プラス1つ。これは私にとって正しいと思われる。どちらの場合もオブジェクトは作成されコピーされますか？ – Bathsheba

+0

違いは、*新しいオブジェクト（）は一時的ではないので削除されないということですか？ – DrTosh

+0

@DrToshまさに。 –

1

new object()ため

vector.push_back(* new object()); 
vector.push_back(object());

tyが動的な記憶域期間を持つ匿名の一時変数を作成します。

* new object()は、new object()からのポインタを保存しないので、単に参照解除してpush_backに渡すだけなので、恐ろしいメモリリークが発生します。

so do notそうです。

一方、object()は、匿名の暫定の自動保存期間のです。

2017-05-16 13:35:22 Bathsheba

4

これは間違っている：

vector.push_back(* new object());

はそれをしないでください。 newはすべてdeleteと一致させる必要があり、あなたはそれを呼び出さないので、メモリがリークしています。あなたができる最良のことは決してnewに電話することではありません。

これはOKです：

vector.push_back(object());

しかし、C++ 11には、あなたがより簡単に行うことができます。

vector.emplace_back(/*any constructor args go here*/);

2017-05-16 13:35:56

+1

C++ 11のアプローチに加えて1つ。 – Bathsheba

+0

私はdownvotesを理解していません。私たちはどうしたの？ – Bathsheba

-1

は、彼らが

object* temp = new object(); 
vector.push_back(*temp); 

object temp; 
vector.push_back(object);

それはvectorとは何の関係もありませんが、ちょうど何deleteが呼び出されていない、次のコードに似ています（そして、あなたがすることはできません）

2017-05-16 14:03:04

+0

なぜdownvote？ –

答えて