2013-04-25 1 views
2

Мой вопрос, который похож на this one, как я могу использовать OracleDataReader для извлечения всех полей для данной записи? В настоящее время я использую этот метод, который возвращает только одно значение столбца в то время:Как получить все поля для данной записи с помощью OracleDataReader?

public string Select_File(string filename, string subdirectory, string envID) 
{ 
    Data_Access da = new Data_Access(); 
    OracleConnection conn = da.openDB(); 

    OracleCommand cmd = new OracleCommand(); 
    cmd.Connection = conn; 
    cmd.CommandText = "SELECT * FROM EIP_Deployment_Files" 
     + " WHERE Filename ='" + filename + "'" 
     + " AND Subdirectory = '" + subdirectory + "'" 
     + " AND Environment_ID = '" + envID + "'"; 
    cmd.CommandType = CommandType.Text; 

    string x; 
    OracleDataReader dr = cmd.ExecuteReader(); 
    if (dr.HasRows) // file exists in DB 
    { 
     dr.Read(); 
     x = dr.GetString(2).ToString(); // return baseline filename (index 2) 
    } 
    else 
    { 
     x = "New File"; 
    } 

    cmd.Dispose(); 
    da.CloseDB(conn); 
    return x; 
} 

Я уверен, что этот метод далек от совершенства и госзакупки будут быстры, чтобы указать, что из (я был в основном, данный мой руководитель, поскольку у меня не было опыта в ASP.NET), но все, что мне действительно волнует, это то, что он работает. Мой вопрос: как его можно изменить, чтобы вернуть все поля для данной записи?

Поля будут либо типами данных VARCHAR2, CHAR, либо DATE (если это имеет значение), а некоторые из этих значений могут быть пустыми. Я думаю, что могу преобразовать их в строки и вернуть их в список?

+0

Примечание: 'cmd.CommandText' уязвим для инъекционных атак – rdev5

ответ

6

Чтобы прочитать все данные из столбцов текущей строки в DataReader, вы можете просто использовать GetValues ​​() и извлечь значения из массива - они будут Объектами, типами баз данных.

Object[] values; 
int numColumns = dr.GetValues(values); //after "reading" a row 
for (int i = 0; i < numColumns; i++) { 
    //read values[i] 
} 

MSDN - «Для большинства приложений, метод GetValues ​​обеспечивает эффективное средство для извлечения всех столбцов, вместо извлечения каждого столбца по отдельности.»

+0

Извините, я смущен. Получает ли GetValues ​​() фактическое значение для каждого поля записи? – user1985189

+0

ДА. Он заполняет массив объектов, в который вы проходите, и возвращаете количество столбцов в качестве возвращаемого значения. Вот ссылка на документы для него: http://msdn.microsoft.com/en-us/library/system.data.oracleclient.oracledatareader.getvalues.aspx – Jasmine

+0

Хорошо, спасибо, что происходит под '// read values ​​[ i] 'часть? – user1985189

6

если и хотят что-то вроде этого:

List<User> lstUser = new List<User>(); 
      string sqlQuery = "Select * from User_T where User_Name='" + oUser.UserName + "' And Password='" +oUser.Password + "' AND IsActive='"+1+"' AND IsDelete='"+0+"'"; 
      string connectionString = "Data Source=ORCL;User Id=ACCOUNTS;Password=ACCOUNTS"; 
      using (DBManager dbManager = new DBManager(connectionString)) 
      { 
       try 
       { 

        dbManager.Open(); 
        OracleDataReader dataReader = dbManager.ExecuteDataReader(sqlQuery); 
        while (dataReader.Read()) 
        { 
         oUser = new User(); 
         oUser.Id = Convert.ToInt32(dataReader["ID"]); 
         oUser.CompanyId = Convert.ToInt32(dataReader["Company_ID"]); 
         oUser.BranchId = Convert.ToInt32(dataReader["Branch_ID"]); 
         oUser.UserName = Convert.ToString(dataReader["User_Name"]); 
         lstUser.Add(oUser); 
        } 
        dataReader.Close(); 
        dataReader.Dispose(); 

       } 
       catch 
       (Exception) 
       { 


       } 
       finally 
       { 
        dbManager.Close(); 
        dbManager.Dispose(); 
       } 
+0

Хмм, может быть! Мне нужно будет заглянуть в него tmw, но спасибо за ваш ответ. – user1985189

+1

Это каждый столбец * по имени *, который не совсем то же самое, что и чтение всех столбцов, независимо от их имен. – Jasmine

+1

Примечание: 'sqlQuery' уязвим для инъекционных атак. – rdev5

0

Извините за размещение ответа на очень старый вопрос. Поскольку ни один из ответов не является правильным (либо они имеют проблемы с безопасностью, либо не проверяют DBNull), я решил опубликовать свои собственные.

public async Task<StringBuilder> FetchFileDetailsAsync(string filename, string subdirectory, string envId) 
    { 
     var sb = new StringBuilder(); 
     //TODO: Check the parameters 

     const string connectionString = "user id=userid;password=secret;data source=" + 
             "(DESCRIPTION=(ADDRESS=(PROTOCOL=tcp)(HOST=10.0.0.8)(PORT=1521))(CONNECT_DATA=(SERVICE_NAME=xe)))"; 

     const string selectQuery = "SELECT * FROM EIP_Deployment_Files" 
           + " WHERE Filename = :filename" 
           + " AND Subdirectory = :subdirectory" 
           + " AND Environment_ID = :envID" 
           + " AND rownum<=1"; 

     using (var connection = new OracleConnection(connectionString)) 
     using (var cmd = new OracleCommand(selectQuery, connection) {BindByName = true, FetchSize = 1 /*As we are expecting only one record*/}) 
     { 

      cmd.Parameters.Add(":filename", OracleDbType.Varchar2).Value = filename; 
      cmd.Parameters.Add(":subdirectory", OracleDbType.Varchar2).Value = subdirectory; 
      cmd.Parameters.Add(":envID", OracleDbType.Varchar2).Value = envId; 

      //TODO: Add Exception Handling 
      await connection.OpenAsync(); 
      var dataReader = await cmd.ExecuteReaderAsync(CommandBehavior.CloseConnection); 

      var rowValues = new object[dataReader.FieldCount]; 
      if (dataReader.Read()) 
      { 
       dataReader.GetValues(rowValues); 
       for (var keyValueCounter = 0; keyValueCounter < rowValues.Length; keyValueCounter++) 
       { 
        sb.AppendFormat("{0}:{1}", dataReader.GetName(keyValueCounter), 
         rowValues[keyValueCounter] is DBNull ? string.Empty : rowValues[keyValueCounter]) 
         .AppendLine(); 
       } 

      } 
      else 
      { 
       //No records found, do something here 
      } 
      dataReader.Close(); 
      dataReader.Dispose(); 
     } 
     return sb; 
    }