OleDbDataReader 코드를 단순화

Question

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 ....)에 무엇을 읽고 또 다시 반복 할 필요가 나는 또 다시이 코드를 반복하지 않도록하는 방법은 없나요 ? . 감사합니다 :)

Answer 1

당신은 이런 식으로 그 방법을 리팩토링 할 수

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<string> 대신에 List<SomeObject> 저장).). 가끔 Person의 목록을 반환하고자하고 Building의 목록을 읽고 싶다고 가정 해 봅시다.

Func<OleDbDataReader, Person> selector = x => new Person { Name = x["Person"].ToString() }; 
Load("People", selector); 

Answer 2

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(); 
     } 

Answer 3

그냥 매개 변수로 테이블 이름을 전달 : 좋아

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; 
} 

당신은 그것을 호출 할 수 있습니다 : 그럼 당신은 이런 식으로 뭔가를 (이을 테스트하지 & 컴파일) 쓸 수

을

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(); 
    } 

Answer 4

"밀크"유형이 확실하지 않습니다. 시도 :

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(); 
     }