C sharp - No guarda Insert DDL

Estimados, buenas tardes tengo un botón enviar el cual toma datos de un DropDownList y los guarda en una Base de Datos, bueno al menos eso debería hacer, pero no guarda la información que le envío, quisiera saber que estoy haciendo mal, el proyecto esta hecho en Visual Studio C#, los DropDownList no los relleno mediante código si no que por el SQLDataSource que me entrega el VisualStudio, la base de datos a la que conecto es a SQLServer 2008, el codigo es el siguiente:

protected void Button1_Click(object sender, EventArgs e)
{
SqlConnection conexion = new SqlConnection();
SqlCommand comando = new SqlCommand();
String querysql;
conexion.ConnectionString = "Server=localhost;Database=db;User Id=dbuser;Password=Sq2013+;";
comando.Connection = conexion;
comando.CommandType = CommandType.Text;
querysql = "Insert Into Respuestas (IDP, Respuesta, Estado, observaciones) Values (1," + DropDownList1.SelectedItem.ToString() + ", 'VIGENTE', 'observ')";
comando.CommandText = querysql;
conexion.Open();
comando.Dispose();
conexion.Close();
Response.Write("<script>alert;('Gracias por Contestar Nuestra Encuesta');</script>");


}

Si alguien pudiese ayudarme se lo agradecería.


Saludos Cordiales

Te saca algún mensaje de error??? Después de conexion.Open(); te falta un comando.ExecuteNonQuery() que es el que ejecuta el insert en la base de datos

Saludos

Hola, Te recomiendo que hagas un debug de tu metodo linea por linea y veas la data que tiene en ese momento, prueba de esta manera el query

String item = DropDownList1.Selected.item.ToString();

querysql = @"Insert Into Respuestas (IDP, Respuesta, Estado, observaciones) Values (1," + item + ", 'VIGENTE', 'observ')";

cuando lo debugees (osea colocar un breakpoint al inicio del metodo) mira que hay en Selecteditem si es el valor que deseas.

Otra cosa, mira el tipo de dato de la tabla Respuestas, si coincide con lo que le estas mandando y el tamaño tmb.

utiliza el metodo de conexion, executenonquery. agregale esa linea :
...
...
conexion.ExecuteNonQuery();
conexion.Dispose(); // ---> (Libera todos los recursos utilizados por el objeto Component. , en tu caso el objeto conexion.)
conexion.Close();
.....
....

Te recomiendo encerrar todo tu codigo en un Try Catch para que el sistema no te truene.

Buena Suerte.