Sqlserver
 sql >> Database >  >> RDS >> Sqlserver

Errore durante l'inserimento di data e ora in SQL Server 2005 datetime c#?

Dovresti SEMPRE usa parametrizzato query:previene attacchi SQL injection, migliora le prestazioni ed evita conversioni non necessarie di dati in stringhe solo per inserirli nel database.

Prova qualcosa come questo:

// define your INSERT query as string *WITH PARAMETERS*
string insertStmt = "INSERT into survey_Request1(sur_no, sur_custname, sur_address, sur_emp, sur_date, sur_time, Sur_status) VALUES(@Surname, @SurCustName, @SurAddress, @SurEmp, @SurDate, @SurTime, @SurStatus)";

// put your connection and command into "using" blocks
using(SqlConnection conn = new SqlConnection("-your-connection-string-here-"))
using(SqlCommand cmd = new SqlCommand(insertStmt, conn))
{
    // define parameters and supply values
    cmd.Parameters.AddWithValue("@Surname", textBox9.Text.Trim());
    cmd.Parameters.AddWithValue("@SurCustName", textBox8.Text.Trim());
    cmd.Parameters.AddWithValue("@SurAddress", textBox5.Text.Trim());
    cmd.Parameters.AddWithValue("@SurEmp", textBox1.Text.Trim());
    cmd.Parameters.AddWithValue("@SurDate", dateTimePicker2.Value.Date);
    cmd.Parameters.AddWithValue("@SurTime", dateTimePicker2.Value.Time);
    cmd.Parameters.AddWithValue("@SurStatus", "Active");

    // open connection, execute query, close connection again
    conn.Open();
    int rowsAffected = cmd.ExecuteNonQuery();
    conn.Close();
}

Sarebbe anche consigliabile dare un nome alle caselle di testo con nomi più espressivi. textbox9 in realtà non mi dice quale casella di testo sia - textboxSurname sarebbe MOLTO meglio!