連想配列に格納されたオブジェクトのdelete

次のコードをgcc5.4.0でコンパイルすると、

-----------------------------------------
#include <iostream>
#include <map>
using namespace std;

class samp{
};


int main(){

std::map<int, samp> sampls;


for(int i=0;i<3;i++){
samp *s = new samp();
sampls[i]=*s;
}

std::map<int, samp>::iterator it = sampls.begin();
while(it!=sampls.end()){
delete it->second;
sampls.erase(it);
it = sampls.begin();
}

return 0;
}
-----------------------------------------

以下のエラーメッセージが出力されるのですが、
どのように修正したらよいでしょうか？
test.cpp:21:16: error: type ‘class samp’ argument given to ‘delete’, expected pointer
delete it->second;

No.3 ベストアンサー 回答者： hitomura 回答日時： 2017/03/14 00:42

samp クラスをコピーして問題ないなら

for(int i=0;i<3;i++){
// samp *s = new samp();
samp s; // 追加
sampls[i]=s;
}

std::map<int, samp>::iterator it = sampls.begin();
while(it!=sampls.end()){
// delete it->second;
sampls.erase(it);
it = sampls.begin();
}

諸般の事情でコピーできないのでポインタで管理したいなら

//std::map<int, samp> sampls; // 以下に変更
std::map<int, samp*> sampls;


for(int i=0;i<3;i++){
samp *s = new samp();
// sampls[i]=*s; // 以下に変更
sampls[i]=s;
}

//std::map<int, samp>::iterator it = sampls.begin(); // 以下に変更
std::map<int, samp*>::iterator it = sampls.begin();

No.2 回答者： aozakana_dha 回答日時： 2017/03/14 00:33

deleteが何をするステートメントかは把握していますか？

deleteする対象はポインタでないといけません。

質問の例では it->second の型は samp なのでポインタではありません。

No.1 回答者： Tacosan 回答日時： 2017/03/14 00:15

本質的に

void foo()
{
int *x = new int(3);
delete *x;
}
がおかしいのと同じなんだけど, とりあえずどのような動作を期待しているんでしょうか?