フォルダ内を回しているときに進捗率を表示させたい

Question

前提・実現したいこと

フォルダ内の容量を調べるプログラムを作っているのですが、foreach文で回しているときに
その進捗をプログレスバーで表示させようとして、countという変数を設けて
コードを組んでみたのですがエラーになってしまいます。
どのようなコードを組めばよいでしょうか？
ご教示お願いします。

該当のソースコード

        public static long GetDirectorySize(DirectoryInfo dirInfo, ProgressBar pb)
        {
            int dircount = dirInfo.GetDirectories().Length;
            int filecount = dirInfo.GetFiles().Length;
            int allcount = dircount + filecount;
            pb.Minimum = 0;
            pb.Maximum = allcount;

            int count = 0;
            long size = 0;

            //フォルダ内の全ファイルの合計サイズを計算する。
            foreach (FileInfo fi in dirInfo.GetFiles())
            {
                size += fi.Length;
                count++;
                pb.Value = count;
            }

            //サブフォルダのサイズを合計していく。
            foreach (DirectoryInfo di in dirInfo.GetDirectories()){
                size += GetDirectorySize(di, pb);
                count++;
                pb.Value = count;  //ここでエラー。
            }
            //結果を返す
            return size;
        }

エラーメッセージ

System.ArgumentOutOfRangeException: ''5' の値は 'Value' に対して有効ではありません。'Value' は 'minimum' と 'maximum' の間でなければなりません。
パラメーター名:Value'

Answer 1

現状のプログラムですと、サブフォルダ内のファイルを検索する際に再帰呼び出しを行っていますので、その時点でプログレスバーのMaximumが再設定されていまいます。
再帰での呼び出しから帰ってきたときに、countがMaximumを超えているのだと思います。

フォルダ以下のファイル容量を調べるのみであれば

string[] files = System.IO.Directory.GetFiles(@"C:\test", "*", System.IO.SearchOption.AllDirectories);

とすれば、上記例ですとC:\test配下のファイル全てが列挙されますので、再帰呼び出しする必要もなくなります。

どのみち、プログレスバーが再描画されるころにはMax値になっているはずなので、容量計算でループしている間は0%、計算終了で100%の表示になるだけだと思います。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
皆様のご回答を見させていただいた所、進捗率表示が仕様的に無理な気がしてきました。
一応、まだこの質問は開いておこうと思います。

Answer 2

ディレクトリに対して再帰的に呼び出しをかけているのにpbが一つだからです。

仮に最初の改装にディレクトリが一つファイルが4つで、子ディレクトリにファイルが一つしかないと過程します。
この関数の最初にmaximumが5になりますが、子に下降すると1に再セットされます。そして親に戻ってきたときにカウントアップされると5になりますがoutofrangeですよね。

でも、これを頑張っても、下の階層が読めない以上、昔のwindowsのコピーとかで出てた信用ならないプログレスバーができあがる気がします。

Comments

[papinianus]

頑張る頑張りかたとしては、pb.maximumに代入(=)するのではなく、加算(+=)することが考えられます。

[退会済みユーザー]

ご回答ありがとうございます。
皆様のご回答を見させていただいた所、進捗率表示が仕様的に無理な気がしてきました。
一応、まだこの質問は開いておこうと思います。

Answer 3

ProgressBarのValueプロパティの範囲外エラーですね。
maximumとminimumとの間の値かをチェックしてください