プログラマ専用SNS ミクプラ

deleteとnull初期化について聞きたいことがあります
座標（0,0）から右に向かって移動する円を管理するマネージャークラスを作ったのですが聞きたいことがあります。

コード:

//ヘッダー
Sphere sphere[10];

//cpp
SphereMgr::SphereMgr()
{
	for(int i = 0 ; i < 10 ; i++)
		sphere[i] = NULL;//null初期化
}
SphereMgr::~SphereMgr()
{
	delete[] sphere;
}
SphereMgr::create()
{
	static int count = 0;
	count++;
	if(count % 10 == 0)
	{
		for(int i = 0 ; i < 10 ; i++)
		{
			if(sphere[i] == NULL)
			{
				sphere[i] = new Sphere();
				break;
			}
		}
	}
}

SphereMgr::calc()
{
	for(int i = 0 ; i < 10 ; i++)
	{
		if(sphere[i] != NULL)
		{
			sphere[i]->calc();
			if(/*画面外に出たら*/)
			{
				delete sphere[i];
				sphere[i] = NULL;
			}
		}
	}
}

ここで４０行目のdeleteをせずに（コメントにして）４１行目でNULLで初期化したところ、しっかり１１コ目～が出てきました。

deleteとNULL初期化の決定的な違いがわかりません。
deleteだけやってはNULLではないので１１コ目～はでないと思いますが、NULLの前にdeleteを書く理由が知りたいです。
このdeleteがないとどのようなことが起きるのでしょうか？
龍神録C++でもやっていたので（それと「nullptr」とはなんなのでしょうか？）

それは基本的なことなので、いちどC++の入門書を読むことをお勧めします。

参考書を見てもイマイチ実感が湧かないから質問したのですが・・・・

参考書には delete は何をするものだと書いてありましたか？

もしかして回答された方は二重解放のことを調べて欲しいと思っているなら表現を変えます。
載せたコードでdeleteしないでnullで初期化（４１行目）だけやるとdeleteした場合とメモリとかどこか違ってくる要素があるのでしょうか？

deleteはnewで作成した物を捨てる行為ということになります。
NULL初期化は単なる数値の代入だけなのでこの捨てるという行為をしません。

よってdeleteを無視すると、捨てたことにしたにもかかわらず実際は捨てていないので、不要なゴミがメモリ上に溜まってしまうわけです。

簡潔に書くとこんなかんじですかねえ。

白い時空さんが書きました:deleteはnewで作成した物を捨てる行為ということになります。
NULL初期化は単なる数値の代入だけなのでこの捨てるという行為をしません。

よってdeleteを無視すると、捨てたことにしたにもかかわらず実際は捨てていないので、不要なゴミがメモリ上に溜まってしまうわけです。

返信ありがとうございます。
聞きたかったことはそれです。
ありがとうございました。

>delete[] sphere;
これもまずいですよね．

>載せたコードでdeleteしないでnullで初期化（４１行目）
これを”初期化”とは呼びません．

オフトピック

>いちどC++の入門書を読むことをお勧めします。
激しく同意です．
基礎的な部分をちゃんとやらずに
断片的な事柄だけを掲示板で訊きまくっていても碌に身につかないのではないかな，と危惧します．

あなたの参考書に白い時空さんの回答が書かれていなかったのなら
参考書選びから考えた方がいいですね

N.R さんが書きました:あなたの参考書に白い時空さんの回答が書かれていなかったのなら
参考書選びから考えた方がいいですね

おすすめの参考書とかありますか？（個人的にでいいので）

返信ありがとうございます（見逃していました）

usao さんが書きました:>delete[] sphere;
これもまずいですよね．

これはなぜいけないのでしょうか？参考書には配列の解放はこのように書いていたのですが・・・
newする前に呼ばれたらマズイから・・・でしょうか？

usao さんが書きました:>載せたコードでdeleteしないでnullで初期化（４１行目）
これを”初期化”とは呼びません．

失礼しました。

私が参考にした書籍は非常に良いものでしたがもう２０年以上前の話で
今はもっと良いものがWEBで参照できると思いますが私自身が参考にしたわけではないので紹介は控えます。
ただ delete C++ でぐぐっただけでも
http://www.geocities.jp/ky_webid/cpp/language/012.html
このくらいのものは出てきます。

ここだけでは不十分とか思うなら、１ページ見て自分の知りたい事が全てわかる都合のよいものは無いと知ってください。
掲示板で簡単に答えを得ようとした結果がusaoさんの指摘したような解釈を産んでいると思いますね。

>これを”初期化”とは呼びません
「ポインタをNULLで初期化している」という言葉自体に間違いはないですが、sqlさんの考える初期化とは挙動が違うでしょうね

>これはなぜいけないのでしょうか？
delete には遂になる new があるはずです。

N.Rさん返信ありがとうございます
自分なりの考えなのですが

N.R さんが書きました:>これはなぜいけないのでしょうか？
delete には遂になる new があるはずです。

はデストラクタに

コード:

for(int i = 0 ;  i < 10 ; i++)
{
	if(sphere[i] != NULL)
		delete sphere[i];
}

と書けば良いのでしょうか？

×遂に
○対に
ですね

new がないなら delete はいらない。
delete 使うなら new を使う。
(基本的に、ですが)

N.R さんが書きました:×遂に
○対に
ですね

new がないなら delete はいらない。
delete 使うなら new を使う。
(基本的に、ですが)

自分が初めに書いたコードの２４行目でnewしているポインタがまだ画面にある状態（deleteされていない）で不祥事によりデストラクタが呼ばれたらどのように解放するのでしょうか？
ひとつ前の自分のデストラクタの書き方ではダメなのでしょうか？

なるほど、それで全部の要素を開放していると考えていたのですね。
それはまた別の話です。

配列の要素１つ１つを開放する必要があります。
(２重開放にも対応する必要があります)

N.R さんが書きました:配列の要素１つ１つを開放する必要があります。
(２重開放にも対応する必要があります)

サイト（ここ）で見るとdeleteした後にポインタをNULLで初期化すれば二重解放を回避できると書いてありましたので、そのようにして、
メモリ未確保または画面には出たけど画面外に出てdeleteされたものもdeleteされてしまうと困るので直後にNULLで初期化をしています。（40,41行目）
デストラクタでは、メモリ解放されたorメモリ未確保はNULL値であるので、それ以外（画面に出ている）をdeleteしろ・・・という意味で書いたのですが・・・・

何回も尋ねるようで悪いのですが、自分の書いたデストラクタ（NO.13の）は間違っているのでしょうか？
間違っているか大丈夫（正しい（この場合だとですが））なのかわからないのでモヤモヤしています・・・

No.13で合っています。
No.1のデストラクタは、delete[] sphereに対応するnew[]がないので間違っています。
存在してはいけないdelete[]です。

N.Rさんも仰っていますが、
　・newで確保したものはdeleteで解放する
　・new[]で確保したものはdelete[]で解放する
　・mallocで確保したものはfreeで解放する
これを意識する必要があります。

Pocoさん返信ありがとうございます。。
大丈夫なようでホッとしています。

Poco さんが書きました:・newで確保したものはdeleteで解放する
　・new[]で確保したものはdelete[]で解放する
　・mallocで確保したものはfreeで解放する
これを意識する必要があります。

大切な助言ありがとうございます。

では、これにて。

オフトピック

こういうことを何度も言われるのは嫌だろうけども，これは文句とかではなく純粋な　助言　です．
ちゃんと勉強しませんか？
（まぁ，する気は最初からないのだろうな，みたいな邪推もできますけど，一応言ってみる）

少しでもまともにnewとdeleteあたりのことだけでも勉強されたのであれば，そもそも
>deleteをせずに（コメントにして）４１行目でNULLで初期化
みたいな謎状態には至らないはずです．
もし仮に
>・newで確保したものはdeleteで解放する
>・new[]で確保したものはdelete[]で解放する
>・mallocで確保したものはfreeで解放する
こういう基本を押さえていないような本で勉強されているのであれば，それは即刻焼却するなり地中奥深くに封印するなりして
本屋に「普通の」C++の入門書を買いにいくべき，とか言うくらいの「基礎」です．
（超個人的見解で言えば，「C++始めました（7日目）」とかいう人物でも余裕で知っているレベルです．）

掲示板にしても，あたなたがNo.17でリンクしているページにしても，
そういう前提知識すら無い状態で局所的な情報だけを見ていても仕方ないですよ．

急いでいたので肝心の事を書いてなかったですね。
No.13の対応で合っています。
失礼しました。

疑問に思った事をつきつめるのは非常に良い事です。
現場でも基本を飛ばして分かったつもりになっている者もザラにいます。
書籍を買わずお金をかけずに勉強したいという姿勢をとやかく言う事もしないです。

しかし疑問の解決は自分で見てみないと意味がないです。
ブレークをはって動きを見てみるなり、リソースモニターを使ってメモリ状況を確認するなり
質問するならその方法です。
掲示板で答えを求めてもWEBに書いてある通りの事か自分で確認しろとしか返しようがないです。

プログラマ専用SNS ミクプラ

deleteとNULL初期化

deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化

Re: deleteとNULL初期化