二値化処理を繰り返した際のメモリー増加を防ぐ方法

Question

実現したいこと

画像を二値化するプログラムを作成しました。
繰り返して二値化処理をするとメモリーがどんどん増加してしまうため
usingステートメントにてメモリーの開放を試みました。
ところが以下のエラーが出てしまいます。

■エラー内容
System.ArgumentException: '使用されたパラメーターが有効ではありません。
というエラーが表示され、以下の文がハイライトされます。
Application.Run(new Form1());

自分の考えでは以下の文にてpictureBox1.Image を開放しているのが悪さをしているのではないかと考えています。しかし、こうしなければ二値化処理を繰り返すたびにメモリーがどんどん増加してしまうため、どう対処すればよいかわからない状態です。
どなたかご教示願います。

using (image_Clone = new Threshold().Apply(image_Clone))
{
pictureBox1.Image = image_Clone;
}

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using AForge.Imaging.Filters;

namespace Aforge_メモリーリーク
{
    public partial class Form1 : Form
    {
        Bitmap image;
        Bitmap image_Clone;

        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {

        }

        private void BtnLoad_Click_1(object sender, EventArgs e)
        {
            var ofd = new OpenFileDialog();
            //ファイルフィルタ  
            ofd.Filter = "Image File(*.bmp,*.jpg,*.png,*.tif)|*.bmp;*.jpg;*.png;*.tif|Bitmap(*.bmp)|*.bmp|Jpeg(*.jpg)|*.jpg|PNG(*.png)|*.png";
            //ダイアログの表示 
            if (ofd.ShowDialog() == DialogResult.OK)
            {
                image = (Bitmap)Image.FromFile(ofd.FileName);
                pictureBox1.Image = image;
            }

        }

        private void BtnStart_Click(object sender, EventArgs e)
        {
                //コピーを作成
                using (image_Clone = (Bitmap)image.Clone())
                //グレー化
                using (image_Clone = new Grayscale(0.2125, 0.7154, 0.0721).Apply(image_Clone))
                //二値化
                using (image_Clone = new Threshold().Apply(image_Clone))
                {
                   pictureBox1.Image = image_Clone;
                }
       
        }

    }
}

バージョン

Comments

[SurferOnWww]

https://teratail.com/questions/aaykzrhr3rytbo ←このスレッドは話はどうした？　過去スレッドで多々回答をもらっていながら放置するような人は相手にされなくなるよ。

[moko]

お世話になっております。そちら回答しましたのでご確認願います。なかなか解決せず、本気で悩んでいるものになります。どうか知見をお借りできればと思います。

[SurferOnWww]

↓ これら全部コメントや回答をもらってるけどあなたは放置・放棄してるのですよ。
https://teratail.com/questions/aaykzrhr3rytbo
https://teratail.com/questions/qqz739zdtofls9
https://teratail.com/questions/496qcybpc4s1s8
https://teratail.com/questions/jw08czzbhwru2j
https://teratail.com/questions/i5wdg8q70tws4x
全部きちんと対応してクローズしてください。話はそれからにしましょう。

[moko]

過去の質問はすべて解決しましたので感謝をお伝えしクローズいたしました。
よろしくお願いいたします。

[SurferOnWww]

相変わらずだなぁ・・・　昨日 14:14 に YAmaGNZ さんから回答をもらったのに未だに無言。

[SurferOnWww]

それから、前のスレッド https://teratail.com/questions/aaykzrhr3rytbo の回答で「だた、質問者さんのオリジナルのコード、追記版のコード、さらに上のように pic も開放するコードのいずれも 1,000 回実行する意味はない（1 回でも結果は同じ）ということは理解されているでしょうか？」と書きましたがその意味を理解してますか？　そこを考えて修正するなりしないと、やってることに意味がないのです。

Answer 1

必要なくなったら解放です。使っている時に解放してはいけません。

    using (image_Clone = new Threshold().Apply(image_Clone))
    {
        pictureBox1.Image = image_Clone;
    }

これだとpictureBox1.Imageでimage_Cloneを使っているのに解放となります。
なのでpictureBox1.Imageにセットするのであればusingを使用すべきではありません。

var previmage = pictureBox1.Image;
pictureBox1.Image = pic;
previmage.Dispose();

と新しいものをセットする前に一旦退避し、使用しなくなってから解放しましょう。

また

                //コピーを作成
                using (image_Clone = (Bitmap)image.Clone())
                //グレー化
                using (image_Clone = new Grayscale(0.2125, 0.7154, 0.0721).Apply(image_Clone))
                //二値化
                using (image_Clone = new Threshold().Apply(image_Clone))
                {
                   pictureBox1.Image = image_Clone;
                }

このように同じ変数を使うべきではありません。
ただでさえ理解していないのに、各オブジェクトの資源がどこまで必要でどこで必要なくなるのかが分からなくなります。
それぞれ別の変数に入れるなどして何がどこまで必要でどこで必要なくなり開放すべきなのかを理解しましょう。