C# throw文について

Question

意図的に例外を起こす為のthrowの使い方として、
参考書には下記の例が挙げられているのですが、３つ質問があります。
※「～省略～」としている箇所は参考書上でも実際に記載が省略されています。

①内側のtryの「エラーを含んだ処理」とは具体的にどんなものが入るのでしょうか。
②下記の例ではtry内に更にtry-catchを記載していますが、普通のtry-catchとの違いがよく分かりません。
③e1.Messageは読み取り専用とのエラーが出てしまいます。

よろしくお願いいたします。

try{
    ～省略～
    try{
        ～省略～　←エラーを含んだ処理  
    }
    catch (Exception e1) {
        Eception e1 = new Exception();
        e1.Message = "エラー";
        throw.e1;
    }
}
catch(Exception e2){
    Console.WriteLine(e2.Message);
}

2022/5/1 追記
上記のコードは実際に参考書に書いてあります・・・笑
随分ひどいようなので他の本を使おうと思います。

Comments

[Zuishin]

① System.IO.File.ReadAllLines("aaaaa");
② 内側で throw したものを catch するため
③ e1.Message には代入できないから

とりあえずこの参考書はやめましょう。
本当にこんなのが売られていましたか？
ろくでもないコードです。

[退会済みユーザー]

> 参考書には下記の例が挙げられているのですが

参考書の書籍名を教えてください。

[SHOSHINSHA]

みなさま回答ありがとうございます。
使用した参考書はこちらです。↓のURLはアマゾンですが、私は図書館で借りました。
C#の絵本　(株)アンク著
https://www.amazon.co.jp/C-%E3%81%AE%E7%B5%B5%E6%9C%AC-%E6%A0%AA%E5%BC%8F%E4%BC%9A%E7%A4%BE%E3%82%A2%E3%83%B3%E3%82%AF-ebook/dp/B00NV5L802

[退会済みユーザー]

こういう本を売っているんですね。特に以下の部分は信じがたいほどメチャクチャなんですが、何かも勘違いとかはないですか？

catch (Exception e1) {
Eception e1 = new Exception();
e1.Message = "エラー";
throw.e1;
}

[退会済みユーザー]

> 2022/5/1 追記
> 上記のコードは実際に参考書に書いてあります・・・笑
> 随分ひどいようなので他の本を使おうと思います。

画像を拝見しました。それは例外処置の一般的・基本的な方法の説明とかではなくて、何か別のことを説明したくて、それが本文や赤い矢印で説明してあるのではないですか？　たぶん、そこを質問者さんが理解でされてないような気がしますけど、違います？

・・・と思いましたけど、それでもやっぱり e1.Message = "エラー"; が変ですね。

[SHOSHINSHA]

画像差し替えました。
ご確認お願いいたします。

Answer 1

参考書には下記の例が挙げられているのですが、

どこの誰が書いたかもわからないネットの記事ならともかく、一般に販売されている書籍にそのようなこと書いてあることはないはずなので、このスレッドは釣りを疑うレベルですが・・・

質問に書いてあるコードは、はっきり言ってデタラメでそれをベースに議論できるレベルには無いので、自分が持っている Microsoft の本に質問のコードとよく似たものがあったのでそれをベースに説明します。

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;
}

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

①内側のtry

余計なものを catch しないよう、PK 制約違反が発生するコード一行のみを囲む。

②下記の例ではtry内に更にtry-catchを記載していますが

外側の try はどこで例外が発生しても必ず connection.Close(); に制御が飛んで接続リークを防止するため。内側の try は上に説明した通り。

③e1.Messageは読み取り専用とのエラーが出てしまいます。

質問のコードの内側の catch 内のコードはメチャクチャで議論の対象になるようなものではありません。Exception を catch することがあるとすると、例えば、原因不明の問題が発生していてその調査のためログを取るためぐらいでしょうか。ログを取ったら即 throw し、それをランタイムに拾わせてアプリケーションを停止させます。

参考に、.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

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

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

Comments

[SHOSHINSHA]

詳しく回答頂きありがとうございます。

SurferOnWwwさんの例のようなコードであれば、
「プログラム上問題がある動き(配列の範囲外など)」と、
「プログラム上は問題ないが、作成者側がエラーにしたい条件」を
分けて扱いたい為の処理ということが理解できます。

私の挙げた例は、参考書にあるぐらいだから何か意図があってこの形になっているのだろうと思いましたが、
みなさまの回答を見る限りそうではなさそうですね・・・。
その事が分かって良かったです。

ありがとうございました。

[退会済みユーザー]

本は「例外を意図的に起こしたいとき」の説明のようですけど、画像を見る限りでは説明が不適切だと思います。（百歩譲っても e1.Message = "エラー"; という間違いは問題外）

例として書くなら、あるインターフェイスを継承してクラスを書いたが、一部未実装なので、それを呼ばれたら NotImplementedException を throw して自爆するとかいうのはよく目にするので、それを書くとか。

public Task<IdentityResult> DeleteAsync(Role role, CancellationToken cancellationToken)
{
throw new NotImplementedException();
}

Answer 2

1. そりゃ例外を出す処理ですね
2. 同じです。try節内の処理で例外が出ると、それに対応したcatch が呼び出されます
3. そういうもんです

Comments

[退会済みユーザー]

回答になってないのでマイナス評価。もしこれにベストアンサーを付けたら質問者さんもマイナス評価。