質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

WPF

Windows Presentation Foundation (WPF) は、魅力的な外観のユーザー エクスペリエンスを持つ Windows クライアント アプリケーションを作成するための次世代プレゼンテーション システムです

Q&A

解決済

1回答

9618閲覧

【C#】WPFのメモリ使用量の増加

退会済みユーザー

退会済みユーザー

総合スコア0

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

WPF

Windows Presentation Foundation (WPF) は、魅力的な外観のユーザー エクスペリエンスを持つ Windows クライアント アプリケーションを作成するための次世代プレゼンテーション システムです

0グッド

0クリップ

投稿2017/03/29 12:43

編集2017/03/29 12:52

###現在の状況
WPFを使ってデスクトップマスコットを作っています。
タスクマネージャーを見ていると、ウィンドウに表示する画像を
切り替えた時にメモリの使用量が増えていました。

これは「メモリ・リーク」という現象なのでしょうか。

追記
デスクトップマスコット自体はこれです。
https://drive.google.com/open?id=0B5ATtnO9WMT0V1dJQXJvVi1SRE0
このデスクトップマスコットを起動したときのメモリの使用量は40MBくらいだと思います。
デスクトップマスコットのメニューの「フォルダ移動」を使って一周するとメモリの使用量が
2倍程度になります。

###該当のソースコード

1.characterフォルダの中にあるフォルダを取得して配列に入れる。
2.それぞれのフォルダの中にある画像を取得して、BitmapImageの配列を作る。
3.BitmapImageの配列を表示。

C#

1 private void Order_GC() 2 { 3 GC.Collect(); 4 GC.WaitForPendingFinalizers(); 5 GC.Collect(); 6 } 7 public int Folders_Count = 0; 8 public string[] Folders = null; 9 private void Get_Folders() 10 { 11 Folders = System.IO.Directory.GetDirectories("character"); 12 } 13 public BitmapImage[] imgs; 14 public int flag = 0; 15 private void Get_Imgs() 16 { 17 Order_GC(); 18 try 19 { 20 string[] files = System.IO.Directory.GetFiles(Folders[Folders_Count], "*.png"); 21 if (files.Length != 0) 22 { 23 imgs = new BitmapImage[files.Length]; 24 for (int x = 0; x < files.Length; x++) 25 { 26 var tmp = new BitmapImage(); 27 tmp.BeginInit(); 28 tmp.UriSource = new Uri(files[x], UriKind.RelativeOrAbsolute); 29 tmp.CacheOption = BitmapCacheOption.OnLoad; 30 tmp.CreateOptions = BitmapCreateOptions.None; 31 try 32 { 33 tmp.EndInit(); 34 } 35 catch 36 { 37 tmp = new BitmapImage(new Uri("default.png", UriKind.Relative)); 38 } 39 tmp.Freeze(); 40 imgs[x] = tmp; 41 } 42 } 43 else 44 { 45 imgs = new BitmapImage[1]; 46 imgs[0] = new BitmapImage(new Uri("default.png", UriKind.Relative)); 47 } 48 } 49 catch 50 { 51 MessageBox.Show("2.エラーが発生しました。", "ERROR", MessageBoxButton.OK, MessageBoxImage.Error, MessageBoxResult.None, MessageBoxOptions.DefaultDesktopOnly); 52 Environment.Exit(0); 53 54 } 55 } 56 public int Files_Count = 0; 57 public string Load_Mode = "folder"; 58 private void Show_Image() 59 { 60 try 61 { 62 if (Load_Mode == "folder") 63 { 64 DataContext = imgs; 65 Width = imgs[0].PixelWidth; 66 Height = imgs[0].PixelHeight; 67 } 68 if (Load_Mode == "file") 69 { 70 BitmapImage[] array = new BitmapImage[1]; 71 array[0] = imgs[Files_Count]; 72 DataContext = array; 73 Width = array[0].PixelWidth; 74 Height = array[0].PixelHeight; 75 array = null; 76 } 77 } 78 catch 79 { 80 MessageBox.Show("3.エラーが発生しました。", "ERROR", MessageBoxButton.OK, MessageBoxImage.Error, MessageBoxResult.None, MessageBoxOptions.DefaultDesktopOnly); 81 Environment.Exit(0); 82 83 } 84 } 85 private void Next() 86 { 87 switch(Load_Mode) 88 { 89 case "folder": 90 Folders_Count++; 91 if (Folders_Count > Folders.Length - 1) 92 { 93 Folders_Count = 0; 94 } 95 Files_Count = 0; 96 Get_Imgs(); 97 Show_Image(); 98 break; 99 case "file": 100 Files_Count++; 101 if (Files_Count > imgs.Length - 1) 102 { 103 Files_Count = 0; 104 } 105 Show_Image(); 106 break; 107 108 } 109 }

###試したこと
ガベージ・コレクタを明示的に動作させるには?
俺が遭遇したWPFイメージコントロールのメモリーリークと回避法(?)の1つ
これらのページに書いてあったことをおまじない的に取り入れました。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

動かしてみた感じだと大丈夫だと思います。
windbgで見てもBitmapImageは21とか22とかで特に増えてないですよ。

ただ、GC.Collect()はあまり意味がないからやらないほうがいいです。
.Netのガベージコレクタは世代別GCになっています。
ガベージコレクションを実施してしまうと、その時に参照されているオブジェクトは
古い世代に移動して回収されにくくなります。

例えば今回の処理だとGet_Imgsのすぐ後にOrder_GCを実施していますが
行うのであればGet_Imgsの最後に行ったほうがましです。
前回imgsに格納されていたオブジェクトがGet_Imgs内で置き換えられて参照がなくなります。
そのため、置き換えた後に行えば前回imgsに格納されていたオブジェクトの回収が期待できます。

投稿2017/03/29 14:41

hmmm

総合スコア818

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2017/03/29 16:02 編集

ご回答ありがとうございます。安心しました。GC.Collect()は書き直します。 教えてくださってありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問