データベースに挿入

Question

すべてがデータベースの列型に関係していることを確認しましたが、引き続きSQLCeExceptionが発生します。誰もこのコードの何が間違っているのか教えていただけますか？

private void ANDPaddDriverButton_Click(object sender, EventArgs e) 
{ 
    string first = ANDPfirstNametextBox.Text; 
    string last = ANDPlastNametextBox.Text; 
    string mid = textBox5.Text; 
    string phone = ANDPphonetextBox.Text; 
    string social = ANDPsSNtextBox.Text; 
     // EmployeeType="Employee" 
    string city = ANDPCityTextbox.Text; 
    string state = ANDPStatetextBox.Text; 
    string zip = ANDPzipCodetextbox.Text; 
    string email = ANDPemailtextBox.Text; 
    string address = ANDPaddressTextBox.Text; 
    string user = userName.Text; 

    DBConn.Open(); 
    SqlCeCommand cmd = new SqlCeCommand("INSERT INTO [Employee Table] VALUES (" + 
      first + "," + last + "," + mid + "," + address + "," + phone + "," + social + "," 
       + "Employee" + "," + city + "," + state + "," + zip + "," + email + "," + userName + ")", DBConn); 
    cmd.ExecuteNonQuery(); 
    DBConn.Close(); 
} 

Answer 1

フィールドの文字列/ varcharは、一重引用符で囲む必要があります。

SqlCeCommand cmd = new SqlCeCommand("INSERT INTO [Employee Table] VALUES (" + 
    "'" + first + "'," 

のように...

また、他の誰かが既にあなたが大幅にSQLインジェクション攻撃にあなたのコードを公開するつもりだコメントとして

Answer 2

ロレンソが言ったように、文字列値でなければなりません一重引用符で囲まれていますが、このようにクエリを作成すべきではない理由を説明するthis pageを読んで、パラメータを使用してクエリを実行する方法を示してください。

Answer 3

使用するパラメータは、SQLインジェクションを防ぐために、列の名前、あなたは数量に頼って、そしてあなたのテーブルの列の順序と、それはおそらく将来的に変更されますされているので、（私は、カラム名を推測している）：

ところで

SqlCeCommand cmd = new SqlCeCommand("INSERT INTO [Employee Table] (First, Last, Mid, Address, Phone, Social, Employee, City, State, Zip, Email, UserName) VALUES (@First, @Last, @Mid, @Address, @Phone, @Social, @Employee, @City, @State, @Zip, @Email, @UserName)", DBConn); 
cmd.Parameters.AddWithValue("@First", first); 
cmd.Parameters.AddWithValue("@Last", last); 
cmd.Parameters.AddWithValue("@Mid", mid); 
cmd.Parameters.AddWithValue("@Address", address); 
cmd.Parameters.AddWithValue("@Phone", phone); 
// etc. each column 

は、それがSQLインジェクション穴、と貧しいテーブル名を持っているテーブルと列の名前;-)