C# try catchの変数

Question

下記コードで変数宣言は外でしていますが、try内の変数を外で操作できません。
switchの終わりまでtryにすると実行できるのですが、宣言場所に問題があるのでしょうか？

using 省略

namespace trycatch
{
public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();
}

    private void button1_Click(object sender, EventArgs e)
    {
        string mt1;
        string st1;
        string strTime1;
        string strTime2;

        try   //StreamReaderだけをtryにすると外でmt1をいじれない
        {
            StreamReader sr1 = new StreamReader(@"d:\1sample.txt", 　　　　　　　　　　　　　　　　　　　　　　　　　　　　　Encoding.GetEncoding("Shift_JIS"));

            mt1 = sr1.ReadToEnd(); 
            sr1.Close();
        }

        catch (Exception ex)
        {
            label1.Text = ex.Message;
        }
        StreamReader sr2 = new StreamReader(@"d:\2sample.txt", 　　　　　　　　　　　　　　　　　　　　　　　　　　　Encoding.GetEncoding("Shift_JIS"));
            st1 = sr2.ReadToEnd(); 
            sr2.Close();


            
            strTime1 = mt1.Substring(10, 28);　//ここでエラー
            strTime2 = st1.Substring(10, 28);
            

            switch (strTime1.CompareTo(strTime2))
            {
                case -1:
                    
                    textBox1.Text = "strTime1はstrTime2より前の日時です";

                    break;
                case 0:
                    
                    textBox1.Text = "strTime1はstrTime2は同じ日時です";

                    break;
                case 1:

                    
                    textBox1.Text = "strTime1はstrTime2より後の日時です";

                    break;
            }
        
        

    }

}

}

Comments

[YAmaGNZ]

エラーというのは、コンパイル時のエラーですか？実行時のエラーですか？
また、エラーメッセージは何ですか？

Answer 1

mt1のところでエラーと書いているのがコンパイルエラーで
「未割り当てのローカル変数 'mt1'が使用されました」
というものであれば、try-catchの外だから使用できないというわけではありません。

try
{
    StreamReader sr1 = new StreamReader(@"d:\1sample.txt",Encoding.GetEncoding("Shift_JIS"));

    mt1 = sr1.ReadToEnd(); //ファイルがない場合などでこの行が実行されずmt1が初期化されない
    sr1.Close();
}
catch (Exception ex)
{
    label1.Text = ex.Message;
}

ここで、d:\1sample.txtがない場合などの例外が発生した場合、mt1は初期化されない状態となります。
このような初期化されないルートがあるので、ダメですよとエラーが発生している状態です。

Comments

[onGu]

すべての変数を="";で初期化したら赤波線「未割り当てのローカル変数 'mt1'が使用されました」が消えました。低レベルの質問に丁寧に回答いただき感謝です。ありがとうございました。

Answer 2

try catch に限らず、スコープを意識したプログラミングが必要です。

私が検索した限り一番良い文書は↓です。
[雑記] 識別子のスコープとオブジェクトの寿命

理解しにくい場合は、"C# 変数 スコープ"で検索して片っ端から読んでみてください。

Answer 3

今回の問題は他の回答者の方が言われているようにスコープを意識してないからですね。

今回の問題とは関係ない余計なお世話かもしれませんが、例外処理に関して気になることを一言・・・（こういうコードを見ると黙ってられないのですみません）

とにかくまず Exception をキャッチするのは止めた方がいいです。その他例外処置について、自分的に一般的と思うことを書きます。

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

(2) 「例外」はランタイムに拾わせてアプリケーションを停止させる。

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

(4) キャッチせざるを得ない場合でも Execption はキャッチしない。

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

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

詳しくは以下の記事を見てください。

NETの例外処理 Part.1
https://blogs.msdn.microsoft.com/nakama/2008/12/29/net-part-1/

.NETの例外処理 Part.2
https://blogs.msdn.microsoft.com/nakama/2009/01/02/net-part-2/

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

破損状態例外を処理する
https://msdn.microsoft.com/ja-jp/magazine/dd419661.aspx

Comments

[onGu]

try catch の使い方ご教授ありがとうございます。何かエラーおきたら表示させたいと言うだけで使っしまいました。ありがとうございました。

[退会済みユーザー]

開発時点でしたらそれでもいいと思います。

Answer 4

スコープの問題ではないでしょうか?
tryの中で宣言した変数は、tryの中だけでのみ使えるはずです。