Re[3]: インスタンスの生成と破棄

分類:[C#]　

　以下のコードは一応意図したとおり動くのですが、気になるのはファイルを読み込むたびに Bitmap のインスタンスが生成される（つまり複数生成されるのではないか？）のでちょっと不安ですし、これでなぜまともに動くのが不思議です。
　特に問題はないのでしょうか？

　①を
　　Bitmap bmp = new Bitmap(); //これはエラーになる
としてインスタンスを 1 回だけ生成し、後はそれを使い回して②では
　　bmp を使って dialog.FileName を読み込む
のようにできるものならばすっきりするのですが、そういったことはできそうにもありません。
　
public partial class Form1 : Form
{
　　Bitmap bmp; //①各イベントハンドラで共有して使うためここで宣言

　　private void BtnFileOpen_Click(object sender, EventArgs e)
　　{
　　　　OpenFileDialog dialog = new OpenFileDialog();
　　　　if (dialog.ShowDialog() == DialogResult.OK)
　　　　{
　　　　　　try
　　　　　　{
　　　　　　　　bmp = new Bitmap(dialog.FileName); //②インスタンス生成
　　　　　　　　PictureBox1.Image = bmp;
　　　　　　}
　　　　　　catch
　　　　　　{
　　　　　　　　MessageBox.Show("これは画像ファイルではありません", "エラー",
　　　　　　　　　　　　　　　 MessageBoxButtons.OK, MessageBoxIcon.Error);
　　　　　　　　return;
　　　　　　}
　　　　}
　　}

　　private void Rotation_Click(object sender, EventArgs e)
　　{
　　　　if (Pb.Image != null)
　　　　{
　　　　　　bmp = (Bitmap)Pb.Image;
　　　　　　bmp.RotateFlip(RotateFlipType.Rotate90FlipNone);　// 90°回転
　　　　　　PictureBox1.Image = bmp;
　　　　}
　　}

　　private void BtnInversion_Click(object sender, EventArgs e)
　　{
　　　　if (Pb.Image != null)
　　　　{
　　　　　　bmp = (Bitmap)Pb.Image;
　　　　　　bmp.RotateFlip(RotateFlipType.RotateNoneFlipX);　// 水平反転
　　　　　　PictureBox1.Image = bmp;
　　　　}
　　}

　　private void Form1_FormClosing(object sender, FormClosingEventArgs e)
　　{
　　　　bmp.Dispose(); //解放
　　}
}

■No98330 (星は昴さん) に返信 Pb.Image ってなんでしょうか？ PictureBox1.Image の間違いですか？ > 　以下のコードは一応意図したとおり動くのですが、気になるのはファイルを読み込むたびに Bitmap のインスタンスが生成される（つまり複数生成されるのではないか？）のでちょっと不安ですし、これでなぜまともに動くのが不思議です。参照されなくなった Bitmap オブジェクトは、そのうち GC が回収しますが、気になるのであれば bmp = new Bitmap(dialog.FileName); if (PictureBox1.Image != null) { PictureBox1.Image.Dispose(); } PictureBox1.Image = bmp; のようにすればいいと思います。インスタンス共有のために①の変数を用意しているのであれば、 Rotation_Click/BtnInversion_Click は private void Rotation_Click(object sender, EventArgs e) { if (bmp != null) { bmp.RotateFlip(RotateFlipType.Rotate90FlipNone); // 90°回転 PictureBox1.Invalidate(); } } private void BtnInversion_Click(object sender, EventArgs e) { if (bmp != null) { bmp.RotateFlip(RotateFlipType.RotateNoneFlipX);　// 水平反転 PictureBox1.Invalidate(); } } でいいでしょう。 > 　　private void Form1_FormClosing(object sender, FormClosingEventArgs e) > 　　{ > 　　　　bmp.Dispose(); //解放 > 　　} > } BtnFileOpen をクリックしないでフォームを閉じると異常終了しますね。

C# と VB.NET の質問掲示板

ASP.NET、C++/CLI、Java 何でもどうぞ

C# と VB.NET の入門サイト

Re[3]: インスタンスの生成と破棄

過去ログには書き込み不可