try / catch +使用、正しい構文

どれ：

using (var myObject = new MyClass())
{
   try
   {
      // something here...
   }
   catch(Exception ex)
   {
      // Handle exception
   }
}

または

try
{
   using (var myObject = new MyClass())
   {
      // something here...
   }
}
catch(Exception ex)
{
   // Handle exception
}

私は2番目のものを好みます。オブジェクトの作成に関連するエラーもトラップする場合があります。

usingブロックは、try / finally（MSDN）の構文を単純化したものにすぎないので、個人的には次のようにしますが、2番目のオプションとはかなり異なると思います。

MyClass myObject = null;
try {
  myObject = new MyClass();
  //important stuff
} catch (Exception ex) {
  //handle exception
} finally {
  if(myObject is IDisposable) myObject.Dispose();
}

場合によります。Windows Communication Foundation（WCF）を使用しusing(...) { try... }ている場合、usingステートメントのプロキシが例外状態にあると、は正しく機能しません。つまり、このプロキシを破棄すると、別の例外が発生します。

個人的には、最小限の処理アプローチ、つまり実行時に気付いている例外のみを処理することを信じています。つまり、変数の初期化でusing特定の例外がスローされることがわかっている場合は、でラップしtry-catchます。同様に、usingbody 内で何かが発生する可能性があり、それがの変数に直接関連していない場合はusing、tryその特定の例外のために別の変数でラップします。ESで使用することExceptionはほとんどありませんcatch。

しかし、私は好きですがIDisposable、using偏見があるかもしれません。

catchステートメントがusingステートメントで宣言された変数にアクセスする必要がある場合は、insideが唯一のオプションです。

catchステートメントで、破棄する前にusingで参照されるオブジェクトが必要な場合は、insideが唯一のオプションです。

catchステートメントがユーザーへのメッセージの表示など、不明な期間のアクションを実行し、それが発生する前にリソースを破棄したい場合は、outsideが最善のオプションです。

私がこれに似たscenerioを持っているときはいつでも、try-catchブロックは通常、コールスタックを使用していない別のメソッドにあります。このようにメソッド内で発生する例外を処理する方法をメソッドが知ることは一般的ではありません。

したがって、私の一般的な推奨事項は外にあり、外にあります。

private void saveButton_Click(object sender, EventArgs args)
{
    try
    {
        SaveFile(myFile); // The using statement will appear somewhere in here.
    }
    catch (IOException ex)
    {
        MessageBox.Show(ex.Message);
    }
}

どちらも有効な構文です。それは本当に何をしたいかにかかっています。オブジェクトの作成/破棄に関連するエラーをキャッチしたい場合は、2番目を使用します。そうでない場合は、最初のものを使用します。

ここで重要なことが1つあります。最初のものは、コンストラクターの呼び出しから発生する例外をキャッチしませんMyClass。

C＃8.0から、私はこのように2番目のものを使用することを好みます

public class Person : IDisposable
{
    public Person()
    {
        int a = 0;
        int b = Id / a;
    }
    public int Id { get; set; }

    public void Dispose()
    {
    }
}

その後

static void Main(string[] args)
    {

        try
        {
            using var person = new Person();
        }
        catch (Exception ex) when
        (ex.TargetSite.DeclaringType.Name == nameof(Person) &&
        ex.TargetSite.MemberType == System.Reflection.MemberTypes.Constructor)
        {
            Debug.Write("Error Constructor Person");
        }
        catch (Exception ex) when
       (ex.TargetSite.DeclaringType.Name == nameof(Person) &&
       ex.TargetSite.MemberType != System.Reflection.MemberTypes.Constructor)
        {
            Debug.Write("Error Person");
        }
        catch (Exception ex)
        {
            Debug.Write(ex.Message);
        }
        finally
        {
            Debug.Write("finally");
        }
    }

Using（）ブロックで初期化しているオブジェクトが例外をスローする可能性がある場合は、2番目の構文を使用する必要があります。

私のシナリオでは、ファイルを開く必要があり、Using（）ブロックで初期化しているオブジェクトのコンストラクターにfilePathを渡していたため、filePathが間違っている/空の場合は例外がスローされる可能性があります。したがって、この場合、2番目の構文は意味があります。

私のサンプルコード：-

try
{
    using (var obj= new MyClass("fileName.extension"))
    {

    }
}
catch(Exception ex)
{
     //Take actions according to the exception.
}

以下からのC＃8.0には、簡素化することができusing、ネストされたブロックを取り除くためにいくつかの条件の下でのステートメントを、そしてそれはちょうど外側のブロックに適用されます。

したがって、2つの例は次のように削減できます。

using var myObject = new MyClass();
try
{
   // something here...
}
catch(Exception ex)
{
   // Handle exception
}

そして：

try
{
   using var myObject = new MyClass();
   // something here...
}
catch(Exception ex)
{
   // Handle exception
}

どちらもかなり明確です。次に、2つの間の選択を、オブジェクトのスコープをどのようにしたいか、インスタンス化エラーを処理する場所、およびいつ破棄するかという問題に減らします。