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

初めまして。

タイトルのように、メインフォームから呼び出した進捗フォームのマーキータイプのプログレスバーが動きません。
MainWorkは複数の関数を呼び出したりファイルの入出力を行っているため結構重い処理になっています。

原因はどこにあるのかご教授下さい。宜しくお願いします。環境は、.NET2.0C#です。


#メインフォーム
private void button1_Click(object sender, EventArgs e)
{
    ProgressBar pb = new ProgressBar("処理中...",MainWork,"");
    DialogResult result = pb.ShowDialog(this);

    if (result == DialogResult.Cancel)
    {
         //キャンセル処理
    }
    else if (result == DialogResult.OK)
    {
        //OK処理
    }
}


#進捗フォーム
キャンセルボタンとプログレスバーのみ配置しています。

    public partial class ProgressBar : Form
    {
        public ProgressBar (string caption, DoWorkEventHandler doWork, object argument)
        {
            InitializeComponent();
            this.Text = caption;
            this.progressBar.Style = ProgressBarStyle.Marquee;
            this.progressBar.MarqueeAnimationSpeed = 200;
            this.CancelButton = this.cancelAsyncButton;
            this.progressBar.Minimum = 0;
            this.progressBar.Maximum = 100;
            this.cancelAsyncButton.Text = "キャンセル";
            this.cancelAsyncButton.Enabled = true;
            this.backgroundWorker1.WorkerReportsProgress = true;
            this.backgroundWorker1.WorkerSupportsCancellation = true;
            workerArgument = argument;
            //イベント
            this.Shown += new EventHandler(ProgressDialog_Shown);
            this.cancelAsyncButton.Click += new EventHandler(cancelAsyncButton_Click);
            this.backgroundWorker1.DoWork += doWork;
            this.backgroundWorker1.RunWorkerCompleted +=
                new RunWorkerCompletedEventHandler(backgroundWorker1_RunWorkerCompleted);
        }

        private object workerArgument = null;

        private void ProgressDialog_Shown(object sender, EventArgs e)
        {
            this.backgroundWorker1.RunWorkerAsync(this.workerArgument);
        }

        //キャンセルボタンが押されたとき
        private void cancelAsyncButton_Click(object sender, EventArgs e)
        {
            if (!cancelAsyncButton.Enabled) return;
            DialogResult result = MessageBox.Show("キャンセルしますか？", "処理中", MessageBoxButtons.OKCancel,MessageBoxIcon.Exclamation);
            if (result == DialogResult.OK)
            {
                cancelAsyncButton.Enabled = false;
                backgroundWorker1.CancelAsync();
            }
        }

        //バックグラウンド処理が終了したとき
        private void backgroundWorker1_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
        {
            if (e.Cancelled)
            {
                this.DialogResult = DialogResult.Cancel;
            }
            else
            {
                this._result = e.Result;
                this.DialogResult = DialogResult.OK;
            }
            this.Close();
        }
    }

■No67164 (魔界の仮面弁士 さん) に返信
> ■No67163 (げんろく さん) に返信
>>マーキータイプのプログレスバーが動きません。
> 
> すくなくとも当方では、
> void MainWork(object sender, DoWorkEventArgs e) { System.Threading.Thread.Sleep(10000); }
> に対して、プログレスバーが作動していました。
> 
> Visual Style は有効になっているかを確認してみてください。
> （XP のクラシックスタイルなどでは、マーキータイプは動作しません）
> 

そのMainWorkに変更したら私のほうでも動作しました。

> 
>>MainWorkは複数の関数を呼び出したりファイルの入出力を行っているため結構重い処理になっています。
> MainWork はスレッドセーフになっていますか?
> 

スレッドセーフ・・・ここかもしれません。
あまり理解していないのですが・・・（すいません、結構初心者なもので。。）
フォーム上のコントロールを操作しない場合でも関係ありますか？


> 
>>public partial class ProgressBar : Form
> フォーム名も ProgressBar なのでしょうか? 名前空間が異なるとはいえ、
> System.Windows.Forms.ProgressBar と競合する名前は混乱の元なので、避けた方が無難かと思います。
> 
> 
>> this.progressBar.Style = ProgressBarStyle.Marquee;
> Form 名だけでなく、コントロール名も progressBar なのですか?
> 大文字小文字を混在させたとしても、コントロールに重複する名前を付けるべきではありません。
> (この場合、Form と ProgressBar の両方が、"ProgressBar" という名前になります)
> 
> 重複させても動かないわけではありませんが、Visual Studio のデザイナが解釈できなくなるので、
> 何か別の名前を付ける事をお奨めします。
> 
>>    ProgressBar pb = new ProgressBar("処理中...",MainWork,"");
>>    DialogResult result = pb.ShowDialog(this);
> pb を using ブロックで囲みましょう。
> ShowDialog で呼び出したフォームは Dispose せねばなりません。

ありがとうございます。ここ気をつけます。

> 
> 
>>this._result = e.Result;
> this._result とは?

特に気にしないで下さい。