Jump to content

Erro no update


Juca
 Share

Go to solution Solved by FreiNando,

Recommended Posts

Boa tarde a todos,

Sou iniciante e estou desenvolvendo um sistema. A minha tabela esta em sql e tenho um campo com o nome usu_DataCadastro do tipo date. Quando faço o update, aparece a mensagem "Conversion failed when converting date and/or time from character string". Abaixo esta meu comando update.

cmd.CommandText = "Update tbl_Usuario Set usu_NomeCompleto = '" & txtUsuNomeCompleto.Text & "' , usu_CPF = '" & txtUsuCPF.Text & "' , usu_Cargo = '" & cmbUsuCargo.Text & "' , usu_Email = '" & txtUsuEmail.Text &_
"' , usu_Usuario = '" & txtUsuUsuario.Text & "' , usu_Nivel = '" & cmbUsuNivel.Text & "' , usu_SenhaAtual = '" & txtUsuSenhaAtual.Text & "' , usu_SenhaNova = '" & txtUsuSenhaNova.Text & _
"' , usu_SenhaConfirma = '" & txtUsuSenhaConfirma.Text & "' , usu_DataCadastro = '" & txtUsuDataCadastro.Text & "' , usu_Situacao = '" & cmbUsuSituacao.Text & "' Where usu_Id = " & ausuId(dgvUsuarios.CurrentRow.Index + 1)

Obrigado.

Edited by Juca
geshi
Link to comment
Share on other sites

  • Solution

O ideal será usar parâmetros, principalmente por causa dos campos de texto:

	cmd.CommandText = "Update tbl_Usuario Set usu_NomeCompleto =?, usu_CPF =?, usu_Cargo =?" _
		& ", usu_Email =?, usu_Usuario =?, usu_Nivel =?, usu_SenhaAtual =?, usu_SenhaNova =?" _
		& ", usu_SenhaConfirma =?, usu_DataCadastro =?, usu_Situacao =? Where usu_Id =?"
	cmd.Parameters.Add("@P1", OleDbType.VarChar).Value = txtUsuNomeCompleto.Text
	cmd.Parameters.Add("@P2", OleDbType.VarChar).Value = txtUsuCPF.Text
	cmd.Parameters.Add("@P3", OleDbType.VarChar).Value = cmbUsuCargo.Text
	cmd.Parameters.Add("@P4", OleDbType.VarChar).Value = txtUsuEmail.Text
	cmd.Parameters.Add("@P5", OleDbType.VarChar).Value = txtUsuUsuario.Text
	cmd.Parameters.Add("@P6", OleDbType.VarChar).Value = cmbUsuNivel.Text
	cmd.Parameters.Add("@P7", OleDbType.VarChar).Value = txtUsuSenhaAtual.Text
	cmd.Parameters.Add("@P8", OleDbType.VarChar).Value = txtUsuSenhaNova.Text
	cmd.Parameters.Add("@P9", OleDbType.VarChar).Value = txtUsuSenhaConfirma.Text
	'pode verificar se é uma data válida
	If IsDate(txtUsuDataCadastro.Text) Then
		cmd.Parameters.Add("@P10", OleDbType.Date).Value = CDate(txtUsuDataCadastro.Text)
	Else
		cmd.Parameters.Add("@P10", OleDbType.Date).Value = DBNull.Value
	End If
	cmd.Parameters.Add("@P11", OleDbType.VarChar).Value = cmbUsuSituacao.Text
	cmd.Parameters.Add("@P12", OleDbType.Integer).Value = ausuId(dgvUsuarios.CurrentRow.Index + 1)
	cmd.ExecuteNonQuery()

Se não usar parâmetros, dá erro se o utilizador colocar num dos texto uma plica ('), para não falar de Injeção SQL.

Edited by FreiNando

O caminho mais curto para conseguir fazer muitas coisas é fazer uma de cada vez. Samuel Smiles

Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
 Share

×
×
  • Create New...

Important Information

By using this site you accept our Terms of Use and Privacy Policy. We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.