ExecuteNonQuery：接続プロパティが初期化されていません

Question

ExecuteNonQueryを解決する方法：接続プロパティが初期化されていません。私はすでに私のcmd.Connection = conを作った。これは、あなたのコードを修正するための基準として、コードの下に使用することができます

Private void button1_Click(object sender, EventArgs e) 
{ 
    if (img_file != null) 
    { 

FileStream fs = new FileStream(img_file, System.IO.FileMode.Open, System.IO.FileAccess.Read); 
     byte[]image = new byte[fs.Length]; 

     fs.Read(image,0,Convert.ToString(fs.Length)); 
     fs.Close(); 
     SqlCommand cmd = new SqlCommand("INSERT INTO member_details (name,address,email,phone_number,picture) VALUES('"+textBox1.Text+"', '"+textBox2.Text+"', '"+textBox3.Text+"', @pic)", con); 
     SqlParameter prm = new SqlParameter("@pic", SqlDbType.VarBinary, image.Length, ParameterDirection.Input, false, 0,0, null, DataRowVersion.Current, image); 
     cmd.Parameters.Add(prm); 
     cmd.Connection = con; 
     cmd.ExecuteNonQuery(); 
     con.Close(); 
    } 
} 

Answer 1

を助けてください私のコードです：

string connetionString = null; 
SqlConnection cnn ; 
SqlCommand cmd ; 
string sql = null; 

connetionString = "Data Source=ServerName;Initial Catalog=DatabaseName;User ID=UserName;Password=Password"; 
sql = "Your SQL Statemnt Here"; 

cnn = new SqlConnection(connetionString); 
try 
{ 
    cnn.Open(); 
    cmd = new SqlCommand(sql, cnn); 
    cmd.ExecuteNonQuery(); 
    cmd.Dispose(); 
    cnn.Close(); 
    MessageBox.Show (" ExecuteNonQuery in SqlCommand executed !!"); 
} 
catch (Exception ex) 
{ 
    MessageBox.Show("Can not open connection ! "); 
} 

あなたは単に

これとは別に

using(SqlConnection con = new SqlConnection(connectionstring)) 
{ 
--write all your command n execution code here---; 
} 

による接続を初期化し、クローズすることができます、1つの提案として、SQLInjectionを避けるためにパラメタ付きクエリまたはストアドプロシージャを使用する必要があります。

SQLInjectionへの参照：https://www.veracode.com/security/sql-injection