エラーをキャッチした後

Question

try&catchでthrow new Exceptionでエラーをキャッチした場合でもそのまま止まらずにつぎに進むことは可能でしょうか？？　たとえば

try｛
                string[] data1 = System.IO.File.ReadAllLines(@"filepath1");
                string[] data2 = System.IO.File.ReadAllLines(@"filepath2");
                string[] data3 = System.IO.File.ReadAllLines(@"filepath3");

                int[] array1 = Array.ConvertAll(file1, int.Parse);
                if (array1 == null || array1.Length == 0)
                {
                    throw new Exception("Empty file");
                }
                //なにかする。。。

                int[] array2 = Array.ConvertAll(file2, int.Parse);
                if (array2 == null || array2.Length == 0)
                {
                    throw new Exception("Empty file");
                }
　　　　　　　　　　//なにかする。。

                int[] array3 = Array.ConvertAll(file3, int.Parse);
                if (array3 == null || array3.Length == 0)
                {
                    throw new Exception("Empty file");
                }
　　　　　　　　　　//なにかする。。
｝
            catch (FileNotFoundException ex)
            {
                Console.WriteLine(ex.Message);
            }

file1が仮に空のファイルで、エラーをキャッチしたとしても、そのままarray2に進むことは可能なのでしょうか？？
質問がわかりづらくて申し訳無いですが、どなたか回答お願いします。

Comments

[dodox86]

ちょっと謎なご質問なようです。"throw new Exception..."とご自身でコードを書いているのですから、それすなわちそこで例外処理に移行すべく、catch節で捉えられるよう自分で制御していることになるのですが。

> そのままarray2に進むことは可能なのでしょうか？？

進めたいなら、そうなるようにコードを書くのです。

[YAmaGNZ]

それぞれtry－catchすればいいのでは？

Answer 1

try&catchでthrow new Exceptionでエラーをキャッチした場合でもそのまま止まらずにつぎに進むことは可能でしょうか？？

質問のコードの catch (FileNotFoundException ex) を catch (Exception ex) にすれば try 句の中で発生した throw new Exception("Empty file"); を含むすべての例外を握りつぶしてなかったことにできるので可能なはずです。

ただし、catch (Exception ex) とするのは悪手ですが。

.NET アプリの例外処置について Microsoft Bolg に書いてあったことを要約して紹介しておきます（Bolg は今はリンク切れです）。自分としては以下のようにするのをお勧めします。

(1) 予測可能で正しい業務フローに戻すことができる「業務エラー」（例：ユーザーの入力間違い）と、予測できないもしくは予測はできても何の対応もできない「例外」（例：DB サーバーダウン）を区別して対処。

(2) 「例外」はランタイムに拾わせてアプリケーションを停止させる。無かったことにして、ユーザが作業を続けられるようにすると、強制的に停止させるより好ましからざる状況に陥るかも（ユーザーが大事なデータを壊したりとか）。

(3) よほどのことがない限り try-catch は書かない。

(4) キャッチせざるを得ない場合でも Execption はキャッチしない。キャッチせざるを得ないとしても範囲を絞る。例えば DB 関係の例外が予測されるなら SqlException に限定して catch し、Number プロパティなどでエラーの内容を調べて対処するとか。

(5) 間違って補足してしまった例外は throw する。（注：catch ブロックでキャッチした例外を throw するとスタックトレースが途切れるので単に throw と書く）

(6) ユーザーへの通知が必要なら、集約的例外処置を利用する。

それから、.NET 4 からは破損状態例外は catch できなくなっているそうですが、「それでも Catch (Exception e) を使用するのはよくない」ということについては以下の記事を見てください。

破損状態例外を処理する
https://docs.microsoft.com/ja-jp/archive/msdn-magazine/2009/february/clr-inside-out-handling-corrupted-state-exceptions

自分が持っている Microsoft の本に、上に書いた原則に則ったサンプルが載っていたので紹介しておきます。

データベースに INSERT する際に PK 制約違反で発生する例外のみ catch して「業務エラー」（2 重登録）としてユーザーに再入力を促し、その他は再 throw してランタイムに拾わせてアプリケーションを停止させるというものです。

public bool InsertAuthors()
{
    var connection = new SqlConnection("接続文字列");
    var command = new SqlCommand("INSERT INTO authors VALUES ('172-32-1176', 'White', ...)", connection);
    try
    {
        connection.Open();
        try
        {
            command.ExecuteNonQuery();
        }
        catch(SqlException sqle)
        {
            if (sqle.Number == 2627)
            {
                return false;
            }
            else
            {
                throw;
            }
        }
    }
    finally
    {
        connection.Close();
    }

    return true;
}

以下の記事も参考になると思います。

例外の推奨事項
https://docs.microsoft.com/ja-jp/dotnet/standard/exceptions/best-practices-for-exceptions

Comments

[fruitpunch]

すごく勉強になりました！！
丁寧にご説明ありがとうございます！！
破損状態例外処理のサイトはとてもためになりました！！

Answer 2

file1が仮に空のファイルで、エラーをキャッチしたとしても、そのままarray2に進む

file1 に関する処理だけの try ブロックを作ればよいのではないでしょうか．

try
{
  //file1の処理
}
catch( 略 ){}

//array2の処理
//必要なのであれば，また別途 try ブロックにすればいい
try
{
  //array2の処理
}
catch( 略 ){}

Comments

[fruitpunch]

アドバイスありがとうございます。
全くそのとおりでした！！
ありがとうございます！！