Alterar Cadastro C# com Sql Server 2008

23/06/2010

Pessoal, Sou iniciante em C# e estou desenvolvendo uma aplica��o Windows Form para cadastro, com 4 campos, bem b�sico, gravando os dados no Sql Server 2008. A parte de inser��o e exclus�o est� Ok, mas n�o estou conseguindo fazer com que o bot�o "Alterar" funcione, segue o fonte:

private void btnAlterar_Click(object sender, EventArgs e)

{
try
{
if (txtCodigo.Text != "")
{
SqlConnection conn = new SqlConnection(@"Data Source=DAMASCENO-PC\SQLEXPRESS1;
Initial Catalog=cadcd;Integrated Security=True;Pooling=False");
SqlCommand comm = new SqlCommand();
comm.Connection = conn;
comm.CommandText = "UPDATE cadcd SET TIPO = '" + ddlTipo.Text + "', TITULO = '" + txtTitulo.Text + "', DESCRICAO = '" + richTxtDescr.Text +
"WHERE REGISTRO = " + (txtCodigo.Text) + "";
// UPDATE cadcd set TIPO='-DIVERSOS', TITULO='TESTE1', DESCRICAO='TESTE1' where REGISTRO=007
comm.Parameters.AddWithValue("@REGISTRO", txtCodigo.Text);
comm.Parameters.AddWithValue("@TIPO", ddlTipo.Text);
comm.Parameters.AddWithValue("@TITULO", txtTitulo.Text);
comm.Parameters.AddWithValue("@DESCRICAO", richTxtDescr.Text);
conn.Open();
conn.Close();
MessageBox.Show("Midia alterada com sucesso!");
}
else
{
MessageBox.Show("Informe o c�digo da m�dia");
}
}

Estou usando o Visual Studio 2010 RC, eu debugo a aplica��o passo a passo, n�o da erro algum, mas nao vai, aparentemente roda, mas verifico no Banco e o Campo "Registro" que edito n�o alterar. Esse � meu Form, coloco os Campos. O campo "C�digo" � a, digamos assim, a referencia para altera��o no banco. O Sistema n�o d� "Try/Catch", aparentemente roda... mas... ... no Banco nada muda... Algu�m pode dar um help... No aguardo!

Bruno Damasceno

Curtir t�pico + 0

Responder

Posts

23/06/2010

Pedro Maitan

Antes desse comando : conn.Close();

adiciona isso: conn.ExecuteNonQuery();

Responder

Gostei + 0

23/06/2010

Gustavo Oliveira

Ol� amigo.
Ent�o como diz o Pedro esse comando voc� tem que chamar.

Observer esse comando porem utilizando o Firebird como SGBD.

[CODE]public static void updateChequeStatus()
[CODE]        {
[CODE]            try
[CODE]            {
[CODE]                FbConnection fbConectar = new FbConnection(Class.classBd.ClassStringConexao.stringConexao);
[CODE]                fbConectar.Open();
[CODE]                FbDataAdapter fbUpdate = new FbDataAdapter("update cheque set idstatus = @idstatus where [CODE]num = '" + Class.ClassCheque.ClassChequeVAR.num + "'", fbConectar);
[CODE]                fbUpdate.SelectCommand.Parameters.Clear();
[CODE]                fbUpdate.SelectCommand.Parameters.Add("@idstatus", [CODE]Class.ClassCheque.ClassChequeVAR.idstatus);
[CODE]                fbUpdate.SelectCommand.ExecuteNonQuery();
[CODE]                fbConectar.Close();
[CODE]            }
[CODE]            catch (FbException exe)
[CODE]            {
[CODE]                Class.classBd.ClassDbError.MensagemErro(exe.Errors.Count);
[CODE]            }
[CODE]        }

Observer que a conexao foi aberta antes e o EceuteNomQuery logo apos que o sql recebe os parametros.

espero ter ajudado.

Responder

Gostei + 0

24/06/2010

Bruno Damasceno

Obrigado pessoal. A noite vou testar o que foi passado e dou um parecer aqui. Att.

Responder

Gostei + 0

25/06/2010

Bruno Damasceno

Pessoal, Coloquei


conn.Open();
conn.ExecuteNonQuery();
conn.Close();

O programa n�o compila, da o seguinte erro "Error 1 'System.Data.SqlClient.SqlConnection' does not contain a definition for 'ExecuteNonQuery' and no extension method 'ExecuteNonQuery' accepting a first argument of type 'System.Data.SqlClient.SqlConnection' could be found (are you missing a using directive or an assembly reference?) C:\Users\Damasceno\documents\visual studio 2010\Projects\cadastroCD\cadastroCD\Form1.cs 192 26 cadastroCD"
Fiz um teste, coloquei o seguinte c�digo:


conn.Open();
comm.ExecuteNonQuery();
conn.Close();

S� que logo ap�s o "comm.ExecuteNonQuery();" o compilador vai para o Catch. N�o sei onde est� o erro... Nos outros bot�es o c�digo...


conn.Open();
comm.ExecuteNonQuery();
conn.Close();

... Funciona de boa. Alguma outra alternativa? Obrigado pela ajuda.

Responder

Gostei + 0

25/06/2010

Erickson Nascimento

Qual a mensagem que aparece no Catch, ou seja, qual o erro que d� ?

Responder

Gostei + 0

25/06/2010

Bruno Damasceno

Qual a mensagem que aparece no Catch, ou seja, qual o erro que d� ?






catch
{
MessageBox.Show("Erro na altera��o do cadastro", "Mensagem do Sistema",
MessageBoxButtons.OK, MessageBoxIcon.Warning);
}

Ou meu fonte de Bot�o "Alterar" est� assim:


private void btnAlterar_Click(object sender, EventArgs e)
{
try
{
if (txtCodigo.Text != "")
{
SqlConnection conn = new SqlConnection(@"Data Source=DAMASCENO-PC\SQLEXPRESS1;
Initial Catalog=cadcd;Integrated Security=True;Pooling=False");
SqlCommand comm = new SqlCommand();
comm.Connection = conn;
comm.CommandText = "UPDATE cadcd SET TIPO = '" + ddlTipo.Text + "', TITULO = '" + txtTitulo.Text + "', DESCRICAO = '" + richTxtDescr.Text +
"WHERE REGISTRO = " + (txtCodigo.Text) + "";
// UPDATE cadcd set TIPO='-DIVERSOS', TITULO='TESTE1', DESCRICAO='TESTE1' where REGISTRO=007
comm.Parameters.AddWithValue("@REGISTRO", txtCodigo.Text);
comm.Parameters.AddWithValue("@TIPO", ddlTipo.Text);
comm.Parameters.AddWithValue("@TITULO", txtTitulo.Text);
comm.Parameters.AddWithValue("@DESCRICAO", richTxtDescr.Text);
conn.Open();
comm.ExecuteNonQuery();
conn.Close();
MessageBox.Show("Midia alterada com sucesso!");
}
else
{
MessageBox.Show("Informe o c�digo da m�dia");
}
}
catch
{
MessageBox.Show("Erro na altera��o do cadastro", "Mensagem do Sistema",
MessageBoxButtons.OK, MessageBoxIcon.Warning);
}
}

Responder

Gostei + 0

26/06/2010

Gustavo Oliveira

o seu cod esta assim.


"WHERE
 REGISTRO = " + (txtCodigo.Text) + "";

tenta colocar assim.

"WHERE

 REGISTRO = '" + txtCodigo.Text + "'";

NO FIREBIRD quando se trata de string nesta situa��o ficaria assim.

Responder

Gostei + 0

26/06/2010

Bruno Damasceno

Pessoal, deu certo!

comm.CommandText = "UPDATE cadcd SET TIPO = '" + ddlTipo.Text + "', TITULO = '" + txtTitulo.Text + "', DESCRICAO = '" + richTxtDescr.Text + "' " +
"WHERE REGISTRO = " + (txtCodigo.Text) + "";

Era um erro na instru��o UPDATE, ap�s o "richTxtDescr.Text +" faltavam as aspas de fechamento. Valeu a todos! At� a pr�xima d�vida. rsrs

Responder

Gostei + 0

26/06/2010

Jusc�lio Reis

Uma pergunta, pq vc utiliza:
comm.Parameters.AddWithValue("@REGISTRO", txtCodigo.Text);comm.Parameters.AddWithValue("@TIPO", ddlTipo.Text);comm.Parameters.AddWithValue("@TITULO", txtTitulo.Text);comm.Parameters.AddWithValue("@DESCRICAO", richTxtDescr.Text);
se vc utiliza concatena��o de string
comm.CommandText = "UPDATE cadcd SET TIPO = '" + ddlTipo.Text + "', TITULO = '" + txtTitulo.Text + "', DESCRICAO = '" + richTxtDescr.Text +"WHERE REGISTRO = " + (txtCodigo.Text) + "";
Uma boa pratica � utilizar:comm.CommandText = "UPDATE cadcd SET TIPO = @TIPO, TITULO = @TITULO, DESCRICAO = @DESCRICAO WHERE REGISTRO = @REGISTRO";comm.Parameters.AddWithValue("@REGISTRO", txtCodigo.Text);comm.Parameters.AddWithValue("@TIPO", ddlTipo.Text);comm.Parameters.AddWithValue("@TITULO", txtTitulo.Text);comm.Parameters.AddWithValue("@DESCRICAO", richTxtDescr.Text);Percebe a diferen�a? Isso ajuda at� na quest�o de seguran�a. Da outra forma, vc tem codigo q tecnicamente n�o esta utilizando.

Responder

Gostei + 0

27/06/2010

Bruno Damasceno

Obrigado pela dica Amigo, por ser iniciante em Desenvolvimento .Net ainda n�o tenho essas manhas de boa pr�tica.... muito obrigado.

Responder

Gostei + 0

28/06/2010

Netasper

Crie um metodo tipo este:

    public DataTable BuscarDownloads(string palavraChave)
    {
        StringBuilder sql = new StringBuilder();
        sql.Append("select * from downloads where cod_turma is null and ");
        sql.Append("nom_download like '%%' ");
        sql.Append("order by nom_download");

        cmd.CommandText = String.Format(sql.ToString(), palavraChave);
        acesso.ExecuteQuery(ref cmd, ref resultado);
        return resultado;
    }

e chame-o de seu Form

[]s

Responder

Gostei + 0