asp.net數據庫連接

Question

請看下面的代碼。

protected void Button2_Click(object sender, EventArgs e) 
{ 
    SqlConnection con = new SqlConnection(constring); 

    SqlCommand cmd = new SqlCommand(); 

    if (DropDownList3.SelectedItem.Text == "Economy") 
    { 
     seats = Convert.ToInt32(DropDownList1.SelectedItem.Text); 

     cmd.Connection = con; 
     con.Open(); 
     cmd.CommandText = "select easeats from flight where fno='" + fn + "'"; 
     int eds = Convert.ToInt32(cmd.ExecuteScalar()); 

     if (eds > seats) 
     { 
      Panel2.Visible = true;    //seats available 
      cl = DropDownList3.SelectedItem.Text; 
      seat = seats.ToString(); 
      seats = eds; 
     } 
     else 
     { 
      Panel3.Visible = true;   // seats not available 
     } 
     con.Close(); 
    } 
} 

我收到錯誤行：int eds = Convert.ToInt32(cmd.ExecuteScalar());

和錯誤是

誤差轉換爲varchar值定義了int類型

什麼不對這個代碼？

Answer 1

首先，永遠不要像這樣構造你的查詢。改用參數化查詢。

其次，該錯誤信息是很清楚 - 你想轉換爲int一些VARCHAR列，這可能包含一些文字字符，以及不僅數...

第三，喜歡「用」通過明確關閉連接的語句。這樣更安全。

Answer 2

嘗試更換：

int eds = Convert.ToInt32(cmd.ExecuteScalar()); 

有：

int eds = 0; 

int.TryParse(cmd.ExecuteScalar(), out eds); 

中，如果轉換失敗，你有沒有問題，繼續編= 0這樣

...

側面說明，您對異常和連接壽命的處理非常差，請考慮以這種方式替換整個區塊：

protected void Button2_Click(object sender, EventArgs e) 
{ 
    using(var con = new SqlConnection(constring)) 
    using(var cmd = con.CreateCommand()) 
    { 
     if (DropDownList3.SelectedItem.Text == "Economy") 
     { 
      seats = Convert.ToInt32(DropDownList1.SelectedItem.Text); 

      con.Open(); 
      cmd.CommandText = "select easeats from flight where fno='" + fn + "'"; 

      int eds = 0; 
      object result = cmd.ExecuteScalar(); 
      int.TryParse(result, out eds); 

      if (eds > seats) 
      { 
       Panel2.Visible = true;    //seats available 
       cl = DropDownList3.SelectedItem.Text; 
       seat = seats.ToString(); 
       seats = eds; 
      } 
      else 
      { 
       Panel3.Visible = true;   // seats not available 
      } 
     } 
    } 
} 

當然，無論如何，您還應該考慮重構整個事物並將數據庫和業務邏輯從UI邏輯中分離出來，一般而言，您絕不應直接處理UI內部的連接和查詢，甚至不會在ButtonClick事件處理程序。