1

使用するデータベースのデータを読み取るプログラムを作成しましたOleDbDataReaderが、問題は異なるテーブルがあることです。このコードは完全に機能しますが、少し「ハードコード」されていることがわかりました。または、recursiveここに私のサンプルコードがあります

        private void loadMilk()
        {
            cn.Open();
            OleDbDataReader reader = null;
            OleDbCommand cmd = new OleDbCommand("select* from Milk", cn);
            reader = cmd.ExecuteReader();
            while (reader.Read())
            {
                Milk.Add(reader["Product"].ToString());
            }
            cn.Close();
        }

他のテーブルにあるものを読み取るためだけに、これを何度も繰り返す必要があります (たとえば、"select* from Fruitその後"select* from Classics....) このコードを何度も繰り返さないようにする方法はありますか? ありがとう。：）

4

4 に答える 4

4

そのメソッドを次のようにリファクタリングできます。

private IList<string> Load(string tableName, string columnName)
{
    var result = new List<string>();
    cn.Open();
    OleDbDataReader reader = null;
    OleDbCommand cmd = new OleDbCommand(string.Format("select* from {0}", tableName), cn);
    reader = cmd.ExecuteReader();
    while (reader.Read())
    {
        result.Add(reader[columnName].ToString());
    }
    cn.Close();

    return result;
}

コード サンプルは次のようになります。

var milkItems = Load("Milk", "Product");
var classicItems = Load("Classics", "..."); //Enter the column here.

編集：

もう少し具体的なものが必要になる場合があります (たとえばList<SomeObject>、 だけではなくa を格納するなどList<string>)。のリストを返したい場合がありPerson、また のリストを読みたいとしますBuilding。次に、次のようなものを書くことができます（コンパイルおよびテストされていません）：

private IList<T> Load<T>(string tableName, Func<OleDbDataReader, T> selector)
{
    IList<T> result = new List<T>();
    cn.Open();
    OleDbDataReader reader = null;
    OleDbCommand cmd = new OleDbCommand(string.Format("select* from {0}", tableName), cn);
    reader = cmd.ExecuteReader();
    while (reader.Read())
    {
        result.Add(selector(reader));
    }
    cn.Close();

    return result;
}

次のように呼び出すことができます。

Func<OleDbDataReader, Person> selector = x => new Person { Name = x["Person"].ToString() };
Load("People", selector);
于 2013-02-26T08:41:56.820 に答える
3 
private void loadMilk(string TableName, string itemValue)
        {
            string SQLString = String.Format("select * from {0}",TableName);

            cn.Open();
            OleDbDataReader reader = null;
            OleDbCommand cmd = new OleDbCommand(SQLString, cn);
            reader = cmd.ExecuteReader();
            while (reader.Read())
            {
                Milk.Add(reader[ItemValue].ToString());
            }
            cn.Close();
        }
于 2013-02-26T08:42:04.760 に答える
1

「ミルク」の種類がわかりません。試す：

private void loadObjectsFrom(string tableName, object obj, string column)
        {
            cn.Open();
            OleDbDataReader reader = null;
            OleDbCommand cmd = new OleDbCommand("select* from " + tableName, cn);
            reader = cmd.ExecuteReader();
            while (reader.Read())
            {
                obj.Add(reader[column].ToString());
            }
            cn.Close();
        }
于 2013-02-26T08:43:48.347 に答える
0

テーブル名をパラメーターとして渡すだけです。

    private void loadMilk(string tableName)
    {
        cn.Open();
        OleDbDataReader reader = null;
        OleDbCommand cmd = new OleDbCommand(string.Format("select* from {0}",tableName), cn);
        reader = cmd.ExecuteReader();
        while (reader.Read())
        {
            Milk.Add(reader["Product"].ToString());
        }
        cn.Close();
    }
于 2013-02-26T08:43:17.640 に答える