2014-04-20 47 views
-1

我有了功能的SqlDB.dll:SqlDataReader的出现InvalidOperationException

  public SqlDataReader getEnumValues(int enumId) 
    { 
     SqlDataReader reader = null; 
     using (SqlConnection connection = new SqlConnection(connectionString)) 
     { 
      connection.Open(); 
      SqlCommand command = 
       new SqlCommand(
        "SELECT * FROM [EnumValue] WHERE enumId LIKE '" + enumId + "';", 
        connection); 
      reader = command.ExecuteReader(); 
      //if(reader.Read()) 
      // Debug.WriteLine("Inside sqlDb->getEnumValues command = " + command.CommandText + " reader[name] = " + reader["name"].ToString() + " reader[value] = " + reader["value"].ToString() + " reader[description] = " + reader["description"].ToString()); 
     } 
     //reader.Close(); 
     return reader; 
    } 

正如你可以看到我已经尝试返回之前关闭的读者,也是我读出的数据内,它的确定。 我使用的功能是这样的:

using (SqlDataReader getEnumValuesReader = (SqlDataReader)getEnumValues.Invoke(sqlDB, getEnumValuesForEnumParam)) 
        { 
         Debug.WriteLine("Success getEnumValues -- "); 

         if (getEnumValuesReader.HasRows) 
         { 
          while (getEnumValuesReader.Read())  //Loop throw all enumValues and add them to current enum 
          { 
           try 
           { 
            values.Add(new Model.EnumValue(getEnumValuesReader["name"].ToString(), getEnumValuesReader["value"].ToString(), getEnumValuesReader["description"].ToString())); 
            Debug.WriteLine("Value[0].name = " + values[0].Name); 
           } 
           catch (Exception ex) 
           { 
            Debug.WriteLine("Error in building new EnumValue: " + ex.Message); 
           } 
          } 
         } 

        } 

和我越来越类型的异常“System.InvalidOperationException”

我猜它是与SqlDataReader的传递。

+1

[SQL注入警报](http://msdn.microsoft.com/en-us/library/ms161953%28v=sql.105%29.aspx) - 您应该**不**将您的SQL语句连接在一起 - 使用**参数化查询**来代替以避免SQL注入 –

+0

我根本没有安全问题,它没有任何用户\其他开发人员\未来的计划,但感谢您的答案 – Yogevnn

+0

如果此代码用于网站 - **任何这些网站都有被SQL注入攻击的高度危险。只是让我知道这些网站是什么,所以我可以确保永远不会访问他们中的任何一个.... –

回答

2

SQL读取器不能存在于SQL连接的上下文之外。您的连接正在处理您的方法,因此您返回的读者无法在调用方法中获取任何数据。

最好的选择是从阅读器而不是阅读器返回值。传阅读者并不是一个好习惯,这意味着底层的联系等是开放的和不存在的。

+0

谢谢你的回答,我的问题是如何返回所有的值?因为不是所有的值都来自相同的类型,有些是字符串,有些是int – Yogevnn

+0

,你只需要用你所有的类型(string,int等)声明一个类,然后返回一个List 。称为DTO。数据传输对象。 –

相关问题