C# 例外処理について

Question

C#(プログラミング)の勉強中です。
visual studio 2017を使用して
タイマー(Windows.Forms.Timer)を利用して下記のloopを定期的に実行させるプログラムを作成しています。
例外処理のところでよく理解できておらず苦戦していますのでご教授お願いします。

webbrowserでページが表示出来ない・DownloadFileでダウンロード出来ない場合
例外処理が発生すると思いますが、例外処理が発生したときの処理は終了して
再度タイマーの時間がきた際には新規でloopを実行したいです。

現状、下記のソースで可能かと思っていましたが、疎通が取れるようになっても
lavel1が開始にならず、接続エラーの状態になっています。

原因・理由や改善方法等、出来るだけ初心者向けに回答していただけますと幸いです。
お手数おかけしますが回答お願いいたします。

追記

どんな例外処理が発生しても、その処理は終了して再度時間がきた際には１から「loop」の処理を開始したいです。
例えば、物理的にLANケーブルが抜けていた場合、webbrowserコントロールでもDownloadFileでも例外が発生すると思います。その処理は停止します。
途中でLANケーブルを挿しなおせば再度繋がるようになり、その後からタイマーの時間が来た際は例外処理が発生しないと思いますのでその際は普通に処理するという流れが良いです。

現状では挿しなおした後も、例外処理が発生する状態です。

# using は省略

namespace WindowsFormsApp
{
    public partial class Form1 : Form
    {
        // キャンセレーショントークンソース
        private CancellationTokenSource CancellationTokenSource;
        //タイマーの宣言
        System.Windows.Forms.Timer timer2 = new System.Windows.Forms.Timer();
        public Form1()
        {
            InitializeComponent();
            //ボタン
            button1.Enabled = true;
            button2.Enabled = false;
            //タイマー
            loop(Main_loop);
        }
        public void button1_Click(object sender, EventArgs e)
        {
            //開始ボタンは有効・停止ボタンは無効
            button1.Enabled = false;
            button2.Enabled = true;

            //タイマーのインターバル設定
            timer2.Interval = intVal * 60000;
            //タイマー有効化
            timer2.Enabled = true;
        }
        private void button2_Click(object sender, EventArgs e)
        {
            //タイマーの停止
            timer2.Enabled = false;

            //開始ボタンは有効・停止ボタンは無効
            button1.Enabled = true;
            button2.Enabled = false;
        }
        
        public bool OpenWebWait()
        {
            try
            {
                //読み込み完了まで待つ
                while (webBrowser1.IsBusy || webBrowser1.ReadyState != WebBrowserReadyState.Complete)
                {
                    //無処理
                    System.Windows.Forms.Application.DoEvents();
                    System.Threading.Thread.Sleep(300);
                }

                return true;
            }
            catch (Exception)
            {
                return false;
            }
        }
        
        
        //numericUpDown1 * 60秒 でループするタイマー
        public void loop(EventHandler eventHandler)
        {
            timer2.Tick += new EventHandler(eventHandler);
            timer2.Enabled = false;
        }
        private void Main_loop(object sender, EventArgs e)
        {
            try
            {
                //TLS1.2を使用・SSL自己証明書のスルー
                ServicePointManager.ServerCertificateValidationCallback = new RemoteCertificateValidationCallback(OnRemoteCertificateValidationCallback);
                ServicePointManager.SecurityProtocol = SecurityProtocolType.Tls12;
                //ファイルの保存先指定
                string filepath = (保存先);  

                //label1の表示変更
                label1.Text = "接続中";
                label1.Refresh();


                //接続
                webBrowser1.Navigate(接続先URL);

                //ページの読み込み完了まで待つ
                OpenWebWait();

                // クッキーを取得する。
                string cookieStr = webBrowser1.Document.Cookie;

                // WebClientを生成する。
                WebClient wc = new WebClient();
                Encoding enc = Encoding.UTF8;

                // WebClientのヘッダ設定を行う。
                wc.Headers[HttpRequestHeader.Cookie] = cookieStr;

                //label1の表示変更
                label1.Text = "ダウンロード中";
                label1.Refresh();

                //ダウンロード
                wc.DownloadFile(ダウンロードURL);
                wc.Dispose();



            }
            catch (Exception w) when (w is ObjectDisposedException || w is InvalidOperationException || w is ArgumentOutOfRangeException || w is COMException)
            {
                //label1の表示変更
                label1.Text = "接続エラー";
                label1.Refresh();

                //Console.WriteLine(w.Message);
                //1秒待つ
                Task.Delay(1000).Wait();
            }
            catch (Exception w) when (w is COMException)
            {
                Console.WriteLine(w.Message);
                Application.Exit();
            }
        }
        

        }
    }
}

Comments

[coco_bauer]

「lavel1が開始にならず」のlavel1は何で、どのように定義されているのでしょうか？ webBrower1,wcなど定義不明な変数が散見されて、プログラムの理解を妨げています。

[y_waiwai]

それに加え、どのような例外が出て、どういう表示がされたんでしょうか

[warks1]

System.Windows.Forms.FormにSystem.Windows.Forms.lavelを設置しています。

[izmktr]

OpenWebWait は何をする関数なんでしょう？ これ内部でブロッキングしてませんか？

[warks1]

OpenWebWait　を追記しました。ご確認よろしくお願いします。

[Wind]

lavel・・・

[Wind]

Timerを使って、もっとシンプルに例外を再現出来るソースコードを書いてみた方がいいのでは無いでしょうか？

[warks1]

webbrowserコントロールを設置してもらってグーグルでもなんでも指定してLANケーブルを抜いてやっていただければ再現は出来るかと思いますが、、、

Answer 1

基礎的なことを学んでみて今ちょうど欲しい物を付け焼刃の知識で書いてる感じですかね
大抵の人はForm1に実装処理書いてるところから作り直したくなる衝動に駆られるような気がします

全部作るというよりはその中の小さい処理からちゃんと動くかどうかを書いて確認したり、クラス分けをできるようにしていったほうがいいような気がします

ついでに言えばWindowsFormなので全体的に何をしようとしているのかコードだけで判断できない人が多数のようにも感じます

Comments

[warks1]

回答ありがとうございます。
クラス分けについても検索してみたりはしたのですが、どうも継承のところが理解しきれておらずうまく出来ませんでした。検索していると簡単なソースであればクラス分けする必要もないとも見ましたが、実際よく分かっていません。ソースが確認しやすくなる他には何か動作的に変わるものがあるのでしょうか。

[DeadEndShoot666]

クラス分けをするとインスタンスを作成して呼び出す必要があるのでよりオブジェクト指向らしい動作になりますが、ソフトの見た目やどういう挙動をするかは何を書くか次第です。可読性が上がればメンテナンス性も向上し、結果的に作業が効率化されるという利点は確実にあります。完成品の予定が掲載しているコードからせいぜい10行程度しか増えないのであれば必要はありません。個人的に今後ボタンを更に追加したり、他の機能を追加したりするんでないかと思ったのでクラス分けを推奨しました。継承は今別に理解しなくてもいいんでないでしょうか。継承がなにかは必要になったとき割とすぐ理解できるので

[warks1]

ご丁寧に回答していただき、ありがとうございます。
すぐに理解して反映させることは難しいですが、クラス分けも今後の課題としてより良いプログラム作成が出来るようにしていきます。

Answer 2

以下のTimer.Tickイベントの使い方、正しいのでしょうか？

        //numericUpDown1 * 60秒 でループするタイマー
        public void loop(EventHandler eventHandler)
        {
            timer2.Tick += new EventHandler(eventHandler);
            timer2.Enabled = false;
        }

そもそも、タイマースタート（timer2.Enabled = true;）後、タイマーの時間が来たら（つまりtimer2.Tickのイベントが発生したら）どの処理が走るのでしょうか？
まずはそこを確認するのが先決ではないでしょうか？

Comments

[warks1]

一部切り取りしてしまったので、そこは問題ないです。
Main_loopを指定しています。
すみません。。

[kenshirou]

eventHandlerはMain_loopなのですね。
ところで、タイマーの使い方ですが、例外が発生したときにタイマーをスタートして、リトライした方がよいのではないかと思います。（もし正常に終了したらtimer2.Enabled = false;で止める）
理由は、例外発生する前にタイマー時間が来てしまったら、DL処理等に支障が出るように思ったからです。
また、タイムアウト用のタイマーが必要であれば、上記とは別に用意した方がよいでしょう。

Answer 3

例外の話ではないんですが、設計がまずいという話です

OpenWebWait で待っている間にフォームを閉じようとすると
「アプリケーションが応答していません」ってのが出ませんか？

基本的に、Formsから呼び出された処理は、出来るだけ早く関数を終了しないといけません
それをせずに内部で無限ループを作った場合、いろいろ問題が置きます
ラベルが「開始」にならないのもそれが原因です

なぜなら、ラベルを開始と表示するためには、ラベルを書き換える必要がありますが、
そのためには、ラベルクラスに一旦処理を渡さないといけません
しかし、Windows.Forms.Timerが内部で無限ループを作って、処理を終わらせないので、
ラベルを更新する処理に飛んできません

このような関数の途中でSleepを挟みたい、みたいなことをする場合は、
async/awaitの機能を使う必要があります

Comments

[warks1]

回答ありがとうございます。
ループ間隔を遅らせればよいでしょうか。
それとも、呼び出すまでループするという考え方自体に問題があるということでしょうか。
理解が出来ず申し訳ありません。
async/await機能を使うとのことですが、具体的な例などいただけないでしょうか。
非同期処理を実施する際に使うものだと認識しているのですが、どう書けば良いかわからない状況です。

[izmktr]

処理が終わるまでループするのが駄目です
Sleepするのではなく、returnして、もう一度Forms.Timerから呼び出されたときに続きの処理を行ってください

[warks1]

丁寧に回答していただきありがとうございます。returnの使い方が正直まだ理解できていない為。すぐに対応できそうにないので試行錯誤してみます。

Answer 4

タイマーを再設定してないからでは？

Comments

[warks1]

質問文のソースを具体的に変更しました。
このソースでは再設定できていないでしょうか。

[Zuishin]

質問を読み間違っていたのかもしれません。
ですが、これは最初から作り直したいですね。
全部一度に作ろうとせず、まずはタイマーによる繰り返し処理と例外による停止処理から作ってみてはどうでしょうか？
非同期処理について学ぶのが先決です。
ループ内で CancellationToken を使えば停止できます。
ざっと見たところ、タイマーで時間のたびに最初から処理をしているように見えるので、問題はループが終了しないということでいいんですよね？

[warks1]

タイマーで時間のたびに最初から処理しています。
↑その動作で問題ありません。原因が自身では分かっていないのですが、例外処理後のタイマー処理の動作が正常に出来ないところが問題に思っています。
今まで、タイマー処理であったりをわけ、以前は非同期処理で行ったりしていたのですが、
理解不足で思った動作が出来ず、現在のソースコードになっています。
どこのサイトが分かりやすいですとか教えていただけないでしょうか。

[Zuishin]

正常にできないというのはどうなるのですか？

[warks1]

返答が遅れてしまい申し訳ありません。
例外が発生した後のmain_loopが動作しない場合があります。
動作している場合もありますししていない場合もあります。

100％動作していないわけではない為、事象を起こそうとしても出来ていない状況です。

[Zuishin]

例外が発生しない場合は必ず動作するのですか？

[warks1]

例外が発生しない場合には、問題なく動作が実行されています。
例外が発生した後の動作だけが正常に動作したりしなかったりしている状況です。

[Zuishin]

失敗する場合、例外はどこで起きていますか？
Main_loop の一行目で F9 を押してブレークポイントを設定し、実行して止まったところで F10 を繰り返し押して一行ずつ実行して確認してください。

[Zuishin]

私の予想としては、WebClient を短時間の間にいくつも生成していることでソケットが食いつぶされているのではないかと思っています。

[Zuishin]

その場合、例外が起こったことが原因で動作しなくなるのではなく、動作しないことを示すために例外が起こっているのでしょう。

[warks1]

動作しないことを示すために例外が起こっている場合、どのように対策すれば改善されますでしょうか。

[Zuishin]

まずは原因を特定してください。