[WPF][C#] ImageResourceのBindingによるメモリリーク

Question

初投稿です。何かと不備があるかと思いますが、よろしくお願いします。

現在、WPF（Livet）を用いたアプリを開発しております。WPFは触ったばかりであまり深い理解ができていません。
現状、問題となっていますのが、ImageのSourceにBindingを使用した結果起こる深刻なメモリリークです。

ImageのSourceはSliderの動きと共に、BitmapImageのListのメンバの参照先を変更しています。
ListはメンバをあらかじめDBからバイナリー形式で取得し、BitmapImageに変換してListを生成しています。

Window.xaml

                <Image 
                    x:Name="photographicImage"
                    Source="{Binding CurrentImage}"
                    Grid.Row="0"
                    Margin="5,10,10,5">
                    
                </Image>

WindowViewModel.cs

        public void Initialize()
        {
            this.PhotographicImages = PhotographicImage.CreatePhotographicImages();

            DisplayIndex = 0;
            CurrentImage = PhotographicImages[0].ImageSource;
        }

        #region PhotographicImages変更通知プロパティ
        private List<PhotographicImage> _PhotographicImages;

        public List<PhotographicImage> PhotographicImages
        {
            get
            { return _PhotographicImages; }
            set
            { 
                if (_PhotographicImages == value)
                    return;
                _PhotographicImages = value;
                RaisePropertyChanged();
            }
        }
        #endregion

        #region CurrentImage変更通知プロパティ
        private BitmapImage _CurrentImage;

        public BitmapImage CurrentImage
        {
            get
            { return _CurrentImage; }
            set
            { 
                if (_CurrentImage == value)
                    return;

                _CurrentImage = null;
                GC.Collect();

                _CurrentImage = value;
                
                RaisePropertyChanged();
            }
        }
        #endregion

PhotographicImage.cs

        public static List<PhotographicImage> CreatePhotographicImages(int tripId)
        {
            var ret = new List<PhotographicImage>();

            StringBuilder query = new StringBuilder();
       // query生成

            DataTable pictureTable;
            pictureTable = DatabaseAccesser.GetResult(query.ToString());

            BitmapImage bitmapImage = new BitmapImage();
            bitmapImage.BeginInit();
            bitmapImage.UriSource = new Uri("no-image.jpg", UriKind.Relative);


            for (int i = 0; i < pictureTable.Rows.Count; i++)
            {
                ret.Add(new PhotographicImage()
                {
                   ImageSource = (pictureTable.Rows[i]["picture"] == DBNull.Value ? bitmapImage : byteToImageSource( (Byte[])pictureTable.Rows[i]["picture"]))
                });

                ret[i].ImageSource.Freeze();
            }

            return ret;
        }

        public static BitmapImage byteToImageSource(byte[] byteAttay)
        {
            BitmapImage bitmapImage = new BitmapImage();
            MemoryStream memoryStream = new MemoryStream(byteAttay);
            bitmapImage.BeginInit();
            //bitmapImage.CreateOptions = BitmapCreateOptions.PreservePixelFormat;
            //bitmapImage.CacheOption = BitmapCacheOption.OnLoad;
            bitmapImage.StreamSource = memoryStream;
            bitmapImage.EndInit();

            bitmapImage.Freeze();

            return bitmapImage;
        }

解決策を検索して見つけたBitmapCreateOptions.PreservePixelFormat;などは試してみましたが、List生成時に大量のメモリを占有していました（この問題の原因は理解できていません）
また、Freeze()も試していますが、メモリリークが解消されません。

達成したいことは、ImageSourceを動的に変えることができ、かつメモリリークが起こらない設計です。できるだけMVVMに沿ったコーディングスタイルで実現できるほうが望ましいです。

よろしくお願いします。

Answer 1

こんにちは。
ソースが長くて追うのを途中で辞めましたが、まず最初に試すならばbyteToImageSourceメソッド。
MemoryStreamが開きっぱなしになってますが、念のためFreeze後にDisposeしてみては。

他に気づいた点があれば追記します。

Comments

[Daiki-Kawanuma]

ご回答いただき、ありがとうございます。
ご助言頂いた通り、Freeze後にDisposeを挿入した結果、メモリリークは起こらなくなりましたが、今度は画像が表示されなくなってしまいました。BitmapImageのインスタンスのHash値を表示してみた結果どれもnullではなく、値が異なっていたためBitmapImageのインスタンスは正しく生成されているようでした。

MemoryStreamの使い方に問題があるということでしょうか・・・？
よろしくお願いします。

[Tak1wa]

>//bitmapImage.CacheOption = BitmapCacheOption.OnLoad;
これコメントアウトしたままでしたか？

[Daiki-Kawanuma]

自己解決致しましたので経緯をお伝えしたいと思います。
まず、DBから取得していた画像ですが、1000*800[px]程度の画像を3000枚程取得していました。この時点でそもそもOOMの可能性に気づくべきでした。
質問に書きました私の手法ではListのあるメンバーを参照するごとに画像がデコードされていたようで、そのため参照先が変わるごとにメモリを圧迫していました。

これをSliderのIndexが変わるごとに画像を取得するように変更したところ、メモリリークも起こらず、画像も正常に表示されるようになりました。

有効な手法への気づきを与えてくださったTak1wa様には大変感謝しております。
ありがとうございました。