using文の使い方について

Question

visual C# 2010 Express　を使用しています。
MySqlConnectionのデータベースのコネクション部分だけを次のようなメソッドとして、
そのオブジェクトだけを得たいと思いまして、次のような感じにしたのですが、

class Program
    {
        static void Main(string[] args)
        {
            MySqlConnection conn = GetConnection();
            using (MySqlCommand cmd = new MySqlCommand("SELECT * FROM tbl where id = '1'", conn))
            {
                conn.Open();
                MySqlDataReader reader = null;
                reader = cmd.ExecuteReader();
                reader.Read();
                int i = reader.GetInt32(reader.GetOrdinal("id"));
                Console.WriteLine(i.ToString());
                Console.ReadLine();
                conn.Close();
            }
        }

static MySqlConnection GetConnection()
        {
            string connStr = "server=localhost;Database=test;Uid=root;Pwd=";
            using (MySqlConnection conn = new MySql.Data.MySqlClient.MySqlConnection(connStr))
            {
                return conn;
            }
        }
    }

一応これで表示できたのですが、
これで生成したMySqlConnectionはちゃんと開放（消滅）されているのでしょうか？
このようなやり方でデータを取得しているのを見たことがないで
大丈夫なのかな？と思いました。。

Yune-Kichi · Accepted Answer

IDisposable.Disposeは，大元はUnmanaged Resource，
つまりは.NET Frameworkが管理しないリソース，
ぶっちゃければメモリ以外のリソースの統一した解放手段を提供するための物です。

で，IDisposable自体は自分できっちり実装する必要があります。

class Class1 : IDisposable
{
  private bool _disposed = false;
  private int _i = 2;

~class1 () { Dispose(false); }

public int
  {
     get
     {
       if (_disposed) throw new ObjectDisposedException(); // 破棄されたオブジェクト
       return _i;
     }
     set
     {
       if (_disposed) throw new ObjectDisposedExecption();
       _i = value;
     }
  }

protected virtual void Dispose (bool disposing)
  {
    if (_disposed) return; // 多重Disposeしても安全であること
    if (disposing)
    {
      // フィールドにあるIDisposableの解放等
     }
    _disposed = true;
  }

public void Dispose ()
  {
    Dispose(true);
    GC.SuppressFinalize(this); // ファイナライザの呼び出しを抑制する
  }
}

で，本題ですが，トランザクションとかが関係しないのであれば，
Queryメソッドでコネクションを開き，DataSetなりにデータを読んで，コネクションを閉じるまでしてしまいます。
IDisposableなオブジェクトは，他のオブジェクト以上に所有権を考えないといけないため，
できるだけIDisposableなオブジェクトはusingで囲っておきたいです。

ASP.NETだとまた別の解があって，
HttpContext.Current.ApplicationInstance.EndRequestイベントで解放するように構築してしまう，というのも手です。
ASP.NETだと短い時間で処理が終わるので，終了イベントを捕まえてそこで解放しても十分，という判断です。
# 実際にこういう作りにしたことがあります。

Yune-Kichi · Answer

個人的には，「えっ，これで動くの？」という感じですが……。

using文は，そのブロックから出たタイミングでオブジェクトをDisposeします。
このため，GetConnectionメソッドの戻り値は「破棄されたオブジェクト」になり，
本来はOpenメソッドがObjectDisposedExceptionを発生させなければなりません。

書き換えるなら，こうでしょうか。
class Program
{
  static void Main(string[] args)
  {
    using (MySqlConnection conn = GetConnection())
    using (MySqlCommand cmd = new MySqlCommand("SELECT * FROM tbl where id = '1'", conn))
    {
      conn.Open();
      using (reader = cmd.ExecuteReader()) // IDataReaderはIDisposableなのでDisposeを呼ぶ必要がある
      {
        reader.Read();
        int i = reader.GetInt32(reader.GetOrdinal("id"));
        Console.WriteLine(i.ToString());
      } // ここでreaderが解放される
      conn.Close(); // 書いた方が行儀が良いが，省略可能
    } // ここでcmdとconnが解放される
    Console.ReadLine();
  }

static MySqlConnection GetConnection()
  {
    const string connStr = "server=localhost;Database=test;Uid=root;Pwd="; // constを付けておくと，変更不可能になる
    return new MySql.Data.MySqlClient.MySqlConnection(connStr);
  }
}

using文の使い方について

個人的には，「えっ，これで動くの？」という感じですが……。

IDisposable.Disposeは，大元はUnmanaged Resource，

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング