C#でBitmapをDisposeできずにメモリが増えていくのを対処したい

Question

C#のBitmapのDisposeについて質問です。

pictureBoxに画像を表示して、Button1を押すと1pixel拡大するというコードを記述しました。

// Button1クリックイベント
private void button1_Click(object sender, EventArgs e) {
	Bitmap img = (Bitmap)pictureBox1.Image;
	pictureBox1.Image = ScaleBmp(img);
}

// 元のBitmapを引き数に渡すと1pixel拡大したBitmapを出力する
private Bitmap ScaleBmp(Bitmap src){
	Bitmap bmp = new Bitmap(src.Width + 1, src.Height + 1);
	using (Graphics g = Graphics.FromImage(bmp)) {
		g.DrawImage(src, 0, 0, src.Width + 1, src.Height + 1);
	}
	return bmp;
}

このButton1を連続で何回も押すとプロセスメモリがどんどん増えていきます。
ScaleBmp関数の中でBitmapを生成してreturnしているので使用メモリが増えるのは当然なのですが巨大な画像の場合はものすごい勢いで増えていきます。

BitmapをpictureBox1.Imageに渡している場合、DisposeするとエラーになるためにDisposeすることができず、メモリが増えてしまいます。

しばらく放置すると.NetFrameworkが勝手にどこかでGCを実行するので不要になったBitmapが解放されるのですが、Button1を連打すると解放される前にメモリリークを起こしそうで困っています。

button1_Click関数の中の最後にGC.Collect()を実行すればいいのかとは思いますが、Button1を連打する度にGCを呼ぶのもよくないと思ってしまいます。

何か対策はありますでしょうか？
どうぞよろしくお願い致します。

Windows10 + VisualStudio2015 C#

Comments

[hihijiji]

pictureBox1.Image = ScaleBmp(img); //の後に
img.Dispose(); // ←追加
するだけじゃダメですか？

[退会済みユーザー]

ありがとうございます、それで大丈夫でした！

Answer 1

これで改善しませんかね？
var oldImage = pictureBox1.Image;
pictureBox1.Image = ScaleBmp(img);
oldImage.Dispose();

Comments

[退会済みユーザー]

ありがとうございます。
てっきりScaleBmp関数内のBitmap bmp = new Bitmapで生成しまくっているほうがメモリを増やしているとばかり思っていました・・・。
そちらでメモリは増えなくなりました・・・！

[ishi9]

いや、それで合ってます。メモリの増大分はそれです。
というか考え方的にも手順的にも概ね合ってます。

唯一の問題はここで
>>BitmapをpictureBox1.Imageに渡している場合、DisposeするとエラーになるためにDisposeすることができず、メモリが増えてしまいます。

これのせいで開放処理ができなくなってしまっていたのが問題です。

pictureBox1.Image = ScaleBmp(img);
これだけだと古いBitmapは参照から外れるだけで開放されません。（いつかはGCによって開放されますが）

[退会済みユーザー]

再度ありがとうございます。

var oldImage = pictureBox1.Image;
pictureBox1.Image = ScaleBmp(img);
oldImage.Dispose();

ですが、2行目の引数は

pictureBox1.Image = ScaleBmp(oldImage);

ですよね・・・？

それといまいちわからないのですが、ScaleBmp関数内でnewで生成されたBitmapがどこで開放されているのかわかりません・・・。
ScaleBmp関数内で生成されたBitmapはpictureBox.Imageに渡りますが、ボタンを再び押すとまた生成されます。つまりその前に生成されたBitmapはどこで開放されているのでしょうか？

[ishi9]

>>pictureBox1.Image = ScaleBmp(oldImage);
すいません、そうでした。

>>ScaleBmp関数内で生成されたBitmapはpictureBox.Imageに渡りますが、ボタンを再び押すとまた生成されます。つまりその前に生成されたBitmapはどこで開放されているのでしょうか？

以前のコードだと一切開放されていません。完全放置状態です。
そして、それが今回のメモリ増大の原因です。

例えば、例を挙げると、こんな感じでしょうか

void Func () {
Bitmap image1 = (Bitmap)pictureBox1.Image;
var image2 = ScaleBmp(image1);
pictureBox1.Image = image2;
var image3 = ScaleBmp(image2);
pictureBox1.Image = image3;
・
・
以下繰り返し
}

こんな感じでimage1～10000000と処理を増やしていったらどうなるでしょうか？
当然、すぐにメモリオーバーしますよね？image2も3もそれ以降も全部
実際にBitmap持ってるわけですから。

前の処理はまさにこんな感じでした。
それを今回はこのように直しました。

void Func () {
Bitmap image1 = (Bitmap)pictureBox1.Image;
var image2 = ScaleBmp(image1);
pictureBox1.Image = image2;
image1.Dispose(); // ※１
var image3 = ScaleBmp(image2);
pictureBox1.Image = image3;
image2.Dispose(); // 上に同じ
・
・
以下繰り返し
}

※１
image1.Dispose(); // pictureBox1.Imageはimage2に新しく置き換わってimage1を使わなくなったので、古いイメージであるimage1は開放する

[退会済みユーザー]

非常にわかりやすいご説明をありがとうございます！
とても納得できました！
Idisposableのあるオブジェクトは開放をきちんとしないとどんどんメモリが増えていきますね・・・。
今回の件で理解できましたので本当に助かりました。
ありがとうございます！