C# と VB.NET の質問掲示板

DBのトランザクションに関するご質問です。

開発環境：VisualStadio2005(.NET2.0), WindowsXP
使用言語：C#

DBTransaction クラスを使用して
トランザクションの管理をしておりますが
DBへの処理途中でエラーで終了や意図的に終了した場合に（未COMMIT状態）
必ずRollBack処理を行う良い方法はありますでしょうか？

現在、下記の例のように
boolのフラグを設けて無理やり判定を行っておりますが
フラグを使わずにもう少しスッキリとした方法を探しております。

他言語ですが、DelphiのDB接続を管理するオブジェクト（TDataBase）の
InTransactionメソッド（トランザクション中の判定）のような
メソッドがDBTransactionにあれば・・と思ったのですが

DBTransaction クラスのメンバには
トランザクション中かどうかを判定するメソッド・プロパティは
無いようでした・・

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

-------例-------------------------------
＊コネクションは既に存在するとする

//トランザクションの開始
using (DBTransaction trans = コネクション.BeginTransaction())
{
    //トランザクション判定フラグ初期化
    bool transFlg = false;
    try
        //////////////
        //DBへの処理(DML)//
        //省略//
        //////////////

        //コミット
        trans.Commit;

        //トランザクション判定フラグをtrue(コミット完了)に変更する
        transFlg = true;
    }
    finally
    {
        //トランザクション判定フラグがfalse(未コミット)の場合はロールバックする
        if (!transFlg)
        {
            trans.RollBack();
        }
    }
}
    
-----------------------------------

■No1537 (HIRO さん) に返信
> 2007/02/27(Tue) 14:51:29 編集(投稿者)
> 
> ■No1535 (緑茶庵 さん) に返信
>>DBへの処理途中でエラーで終了や意図的に終了した場合に（未COMMIT状態）
> 
> エラー発生時のRollBackなら
> catchの中で行えば良いのではないでしょうか？
> 
> tryに対応するcatchが無いのは何か意図していますか？
> 
> 下記はVB.NETですが参考になるかと思います
> http://www.microsoft.com/japan/msdn/thisweek/step7/ADONET_Transaction/

HIRO　さん　ご返信をありがとうございます。

catch を使用しなかったのは例外発生時以外に
意図的に終了する場合があるためでした。

try
    //DML文処理１（省略）

    int execCount = コマンド.execNonQuery();

    if (execCount == 0)
    {
        MessageBox.Show("更新件数が０件のため処理を中止します");
        return;
    }

    //DML文処理２（省略）

    //コミット
    trans.Commit();
finally
{
    //ロールバック処理（省略）
}


ただ、この処理のようにCommitを行う前にreturnを行った場合でも
DBConnectionのお陰なのか、実際にDBへの更新は行われませんでした。

finallyでロールバック処理を書いたのは
保険の意味あいがとても強く、Commitしていない処理は
とにかく全てロールバック処理を行いたかったからです。

＊return の前にロールバックを書いても良いかも知れませんが
　書き忘れもあるのでやりたくありません


commitせずに且つcatch を通らない処理を行う場合は
普通どうされているのでしょうか？
ロールバックを書かないのが普通なのでしょうか？
逆に catch句では遭えてロールバックを行っているので
ロールバックを行わないのは不安です。。

勉強不足で申し訳ございませんが
よろしくお願いいたします。

finally の件大変失礼しました。

以下のように例外をスローさせ、catchで処理をするというのはどうでしょう？

try
{
    // 省略

    if (execCount == 0)
    {
        //catchで受け取れるよう例外をスロー
        throw new Exception("更新件数が０件のため処理を中止します"); 
        return;
    }

    //Commit処理
    trans.Commit();
      
}
catch ( Exception ex )
{
    // ロールバック処理
    trans.RollBack();
}

はじめまして。まさると申します。

■No1582 (HIRO さん) に返信
> finally の件大変失礼しました。
> 
> 以下のように例外をスローさせ、catchで処理をするというのはどうでしょう？
> 
> try
> {
>     // 省略
> 
>     if (execCount == 0)
>     {
>         //catchで受け取れるよう例外をスロー
>         throw new Exception("更新件数が０件のため処理を中止します"); 
>         return;
>     }
> 
>     //Commit処理
>     trans.Commit();
>       
> }
> catch ( Exception ex )
> {
>     // ロールバック処理
>     trans.RollBack();
> }

これだと例外を握りつぶしてしまっているため、危険な実装ですよ。

明示的に発生させた例外ならば、

try
{
    // 処理１

    if ( 条件 )
    {
        throw new ApplicationException("〜");
    }

    // 処理２
}
catch ( ApplicationException apex )
{
    // ロールバック処理
    trans.RollBack();

    // 想定された例外の処理
    MessageBox.Show(apex.Message);
}
catch ( Exception )
{
    // ロールバック処理
    trans.RollBack();

    // 発生した例外をthrow
    throw;
}

のようにしたほうが良いのではないでしょうか。

＃ただ、明示的にRollBackしなくとも、Commitしていなければ、
＃usingステートメントを抜ける際（もしくはDisposeが呼ばれた際）に
＃自動的にRollBackされるので、それでよい気がしますが。