无法尝试打电话阅读器关闭时读取？

Question

我在查询上运行DbDataReader以删除下拉列表中的项目（如果它们已附加到特定提交中），并且我不断收到错误，告诉我该阅读器已关闭。看不出为什么我的读者在这里被视为封闭。我错过了什么？

protected void Page_Load(object sender, EventArgs e) 
{ 

    string x = Request.QueryString["SubId"]; 
    string connectionString = System.Configuration.ConfigurationManager. 
     ConnectionStrings["MyConnectionString"].ConnectionString; 
    string displayQuery = "SELECT CustName, CustAdd, CustCity, CustState, " + 
     "CustZip FROM Customer WHERE SubId =" + x; 
    string broQuery = "SELECT EntityType FROM Broker WHERE SubId =" + x; 
    string ddlQuery = "SELECT ProductId FROM SubmissionProducts " + 
     "WHERE SubmissionId =" + x; 
    using (SqlConnection displayConn = new SqlConnection(connectionString)) 
    { 
     displayConn.Open(); 
     SqlCommand DlistCmd = new SqlCommand(ddlQuery, displayConn); 

     using (SqlDataReader Ddldr = DlistCmd.ExecuteReader()) 
     { 
      while (Ddldr.Read()) 
      { 

       switch (Ddldr.GetInt32(0)) 
       { 
        case 1: 
         DdlProductList.Items.RemoveAt(1); 
         break; 
        case 2: 
         DdlProductList.Items.RemoveAt(2); 
         break; 
        case 3: 
         DdlProductList.Items.RemoveAt(3); 
         break; 
        case 4: 
         DdlProductList.Items.RemoveAt(4); 
         break; 
        case 5: 
         DdlProductList.Items.RemoveAt(5); 
         break; 
        case 6: 
         DdlProductList.Items.RemoveAt(6); 
         break; 
        case 7: 
         DdlProductList.Items.RemoveAt(7); 
         break; 
        default: 
         break; 
       } 
       Ddldr.Close(); 
      } 

     } 

Answer 1

不要叫Ddldr.Close();，尤其是while内。这样，您正在进行第一次迭代，关闭阅读器，第二次迭代当然会在阅读器关闭时进行。 using声明将照顾它。只需从代码中删除此行。

所以：

using (SqlDataReader Ddldr = DlistCmd.ExecuteReader()) 
{ 
    while (Ddldr.Read()) 
    { 
     switch (Ddldr.GetInt32(0)) 
     { 
      ... your cases here 
      default: 
       break; 
     } 
    } 
} 

而且下面几行：

string x = Request.QueryString["SubId"]; 
string displayQuery = "SELECT CustName, CustAdd, CustCity, CustState, CustZip FROM Customer WHERE SubId =" + x; 
string broQuery = "SELECT EntityType FROM Broker WHERE SubId =" + x; 
string ddlQuery = "SELECT ProductId FROM SubmissionProducts WHERE SubmissionId =" + x; 

臭像一堆s..t的。你应该使用参数化查询，绝对不要写这样的代码，否则你的应用程序将很容易被SQL注入。每次在编写SQL查询时使用字符串连接时，警报都会响起，告诉您您做错了。

所以在这里谈到这样做的正确方法：

protected void Page_Load(object sender, EventArgs e) 
{ 
    string x = Request.QueryString["SubId"]; 
    string connectionString = System.Configuration.ConfigurationManager.ConnectionStrings["MyConnectionString"].ConnectionString; 
    using (var conn = new SqlConnection(connectionString)) 
    using (var cmd = conn.CreateCommand()) 
    { 
     conn.Open(); 
     cmd.CommandText = "SELECT ProductId FROM SubmissionProducts WHERE SubmissionId = @SubmissionId"; 
     cmd.Parameters.AddWithValue("@SubmissionId", x) 

     using (var reader = cmd.ExecuteReader()) 
     { 
      while (Ddldr.Read()) 
      { 
       switch (reader.GetInt32(reader.GetOrdinal("ProductId"))) 
       { 
        ... your cases here 
        default: 
         break; 
       } 
      } 

     } 
    } 
} 

Answer 2

删除此行：

Ddldr.Close();