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

非常に見づらいので整形
投稿時に図表モードを選ぶだけで元の形を保てますよ


public class Db_common
{
	public OleDbDataReader executeReader(string StrCmd,string db)
	{
		try
		{
			//コマンドプロパティセット
			if (db == "db2")
			{
				commanddb2.Connection = conndb2;
				commanddb2.CommandText = StrCmd;
				readerdb2 = commanddb2.ExecuteReader();
				return readerdb2;
			}
			else
			{
				commandora.Connection = connora;
				commandora.CommandText = StrCmd;
				readerora = commandora.ExecuteReader();
				return readerora;
			}
		}
		catch (Exception e)
		{
			throw(e);
		}
	}


	public OleDbDataReader executeReader(string StrCmd,string db)
	{
		try
		{
			//コマンドプロパティセット
			if (db == "db2")
			{
				commanddb2.Connection = conndb2;
				commanddb2.CommandText = StrCmd;
				readerdb2 = commanddb2.ExecuteReader();
				return readerdb2;
			}
			else
			{
				commandora.Connection = connora;
				commandora.CommandText = StrCmd;
				readerora = commandora.ExecuteReader();
				return readerora;
			}
		}
		catch (Exception e)
		{
			throw(e);
		}
	}
}

■No61303 (逆運の覇者 さん) に返信
> まったく同じ処理をするメソッドを２つに分ける意味はありません。
> それに同じクラスに同じメソッド名で引数も同じだとメソッドの衝突（重複）が起きますね。
> オーバーロードできません。
> あと、変数宣言が見当たらず突然出てくるconndb2、connoraはどこからくるのか。
> という疑問もあります。
> 
> 
> ソースコードを見た私の感想としてはわけがわからない、です。

すみません。ＤＬＬ側のソースの貼り付けを間違ってました。
    public class Db_common
    {
      public Boolean connect()
        {
         　
            connectionString = ConfigurationSettings.AppSettings["conStringOLEORACLE"];

                    // 接続されていない場合
            if( connectedora == false )
            {
            // コネクションのインスタンス化

              connora = new OleDbConnection(connectionString);
            // コネクションオープン
              connora.Open();
            // 接続済み
             connectedora = true;
            // コマンドのインスタンス化
            commandora = new OleDbCommand();
           // コマンドにコネクションを設定
             commandora.Connection = connora;
            // データリーダの初期化
                 //OleDbDataReader reader;
                return connectedora;

           }
            else 
            {
                return connectedora;
            
            }

        }

　       public Boolean connect_db2()
        {

            connectionString = ConfigurationSettings.AppSettings["conStringOLEDB2"];

            // 接続されていない場合
            if (connecteddb2 == false)
            {
                // コネクションのインスタンス化

                conndb2 = new OleDbConnection(connectionString);
                // コネクションオープン
                conndb2.Open();
                // 接続済み
                connecteddb2 = true;
                // コマンドのインスタンス化
                commanddb2 = new OleDbCommand();
                // コマンドにコネクションを設定
                commanddb2.Connection = conndb2;
                // データリーダの初期化
                //OleDbDataReader reader;
                return connecteddb2;

            }
            else
            {
                return connecteddb2;

            }

        }
       public string[] executeReader(string StrCmd,string db)
    {
     int i = 0;      string[] r_db =null;
        try
        {
            //コマンドプロパティセット
            if (db == "db2")
            {
                commanddb2.Connection = conndb2;
                commanddb2.CommandText = StrCmd;
                readerdb2 = commanddb2.ExecuteReader();
            while (readerdb2.Read() == true)
                {
                  i= i + 1;
             
                     r_db[i]= readerdb2[1].ToString().TrimEnd();
            }
            return r_db;
            //return readerdb2;

            }
            //else
            //{
            //    commandora.Connection = connora;
            //    commandora.CommandText = StrCmd;
            //    readerora = commandora.ExecuteReader();
            //    return readerora;

            //}
            return r_db;

        }
        catch (Exception e)
        {     
            throw(e);
        }


    }
}

ちょっと作ってみた。動作テストはしてないし、まだ抜けはあると思います。

class Db_common:IDisposable
{
	private DbConnection _con = null;
	private DbDataReader _rd = null;

	public Db_common(string db)
	{
		string connectionString = string.Empty;

		switch (db)
		{
			case "ora":
				connectionString = ConfigurationManager.AppSettings["conStringOLEORACLE"];
				break;
			case "db2":
				connectionString = ConfigurationManager.AppSettings["conStringOLEDB2"];
				break;
		}

		if (connectionString.Length > 0)
		{
			_con = new OleDbConnection(connectionString);
		}
	}

	public DbDataReader executeReader(string CmdText)
	{
		if ((_rd != null) && (!_rd.IsClosed))
		{
			_rd.Close();
		}

		if (_con != null) {
			if (_con.State != ConnectionState.Broken)
			{
				_con.Close();
			}
			if (_con.State != ConnectionState.Open)
			{
				_con.Open();
			}
		}
		else 
		{
			_rd = null;
			return null;
		}

		DbCommand cmdrd = new OleDbCommand();
		cmdrd.Connection = _con;
		cmdrd.CommandText = CmdText;
		_rd = cmdrd.ExecuteReader();
		return _rd;
	}

	public void Dispose()
	{
		if ((_con != null) && (_con.State==ConnectionState.Open))
		{
			if ((_rd != null) && (!_rd.IsClosed))
			{
				_rd.Close();
			}
			_con.Close();
		}
	}
}