'='付近の構文が正しくない

Question

このエラーをどのように訂正できますか？この行のエラー：readK.Fill(dt1);。ありがとうございました！このコードはFormatExceptioncomboBox5.SelectedValueが整数でない原因となる場合

SqlCommand selK = new SqlCommand("Select * from Kafedra where id_facultet=@id_facultet, conn); 
sqlK.Parameters.Add("@id_facultet", SqlDbType.Int).Value = int.Parse(comboBox5.SelectedValue.ToString()); 

：

private void comboBox5_SelectedValueChanged(object sender, EventArgs e) 
{ 
    comboBox6.SelectedValueChanged -= comboBox6_SelectedValueChanged; 
    DataTable dt1 = new DataTable(); 
    BindingSource bd = new BindingSource(); 
    conn.Open(); 
    bd.DataSource = dt1; 
    SqlCommand selK = new SqlCommand("Select * from Kafedra where id_fcultet=" + comboBox5.SelectedValue, conn); 
    SqlDataAdapter readK = new SqlDataAdapter(selK); 
    readK.Fill(dt1); 
    comboBox6.DataSource = bd; 
    comboBox6.DisplayMember = "name"; 
    comboBox6.SelectedIndex = -1; 
    comboBox6.ValueMember = "id_kafedra"; 
    textBox2.Text = i + comboBox5.SelectedValue; 
    conn.Close(); 
    comboBox6.SelectedValueChanged += comboBox6_SelectedValueChanged; 
} 

Answer 1

この問題を解決する最良の方法も可能で、SQLインジェクションを修正するSQLパラメータを使用することです。最初の場所にある整数の場合（デバッガを使用する場合）は、キャストを使用することもできます。... .Value = (int)comboBox5.SelectedValue;

Answer 2

クエリを作成するための文字列連結アプローチは、SQLインジェクションの大きな扉を開きます。パラメータ化クエリを使用することを強くお勧めします。 SQLインジェクションを避ける。ここで同じことを行うための一例である：

SqlCommand selK = new SqlCommand("Select * from Kafedra where id_facultet=@someID", conn); 
selK.Parameters.Add("@someID",SqlDbType.Varchar).value= comboBox5.SelectedValue; 

あなたは表内のその特定のフィールドの種類ごとにSqlDbType.Varcharを選択することができます。