3

Tenho uma tabela com o nome Produtos em que eu inseri 3 produtos para testes. Agora estou montando o formulário de cadastro de produtos e preciso que sempre gere um código de produto sequencial a partir do último id já cadastrado + 1.

O código que fiz é esse, segui a minha lógica mas n está funcionando.

    private void GerarNovoCodigo()
    {
        string strCon = "Data Source=MEUPC; Initial Catalog=MEUDB; Integrated Security=SSPI";
        SqlConnection sqlCon = new SqlConnection(strCon);
        SqlCommand sqlCom = new SqlCommand();
        sqlCom.Connection = sqlCon;
        sqlCon.Open();

        sqlCom.CommandText = "Select max(prod_codigo) + 1  from Produtos";

        txt_prod_cod.Text = Convert.ToString(sqlCom.ExecuteNonQuery());
    }
6
  • A não ser pelo fato que isto é um perigo em ambiente concorrente e que provavelmente deve ter muito código repetido na aplicação, pelo descrito não consegui ver problema. Precisar adicionar mais detalhes.
    – Maniero
    Commented 14/07/2016 às 13:33
  • 1
    Bem vindo ao Stack Overflow em Português. Por favor, leia o guia Como perguntar e o faça um tour, para aprender um pouco mais sobre o funcionamento do site para assim aumentar suas chances de conseguir uma boa resposta.
    – Randrade
    Commented 14/07/2016 às 13:35
  • Aparentemente, o código executa corretamente, qual é o problema enfrentado? Commented 14/07/2016 às 13:43
  • @bigown Como voce faria para sempre gerar um novo código de forma simples? É uma aplicação básica por enquanto, de cadastro, consulta e movimentação de produtos diversos. Há sim código repetido na aplicação, mas por hora quero fazer funcionar o básico para enxugar depois, visto que estou na fase de aprendizagem ainda. Obrigado pela resposta. Commented 14/07/2016 às 13:45
  • @JulioBorges O textBox não está recebendo o novo código. Quando executo a aplicação ele carrega em branco. Commented 14/07/2016 às 13:46

3 Respostas 3

4

Elaborei essa resposta apenas para englobar e tentar sanar todas as dúvidas que encontrei na pergunta sobre SQL, inclusive.

Vamos falar sobre SQL primeiro...

O que você quer o próprio banco de dados já faz. Para isso, basta marcar o campo da Key como Auto Increment (Identity). Feito isso, você não precisa inserir o código no INSERT, que o banco faz tudo automático para você.

Um exemplo seria:

CREATE TABLE Produtos
(  
 Id int IDENTITY(1,1),  
 Nome varchar (100) 
); 

INSERT INTO Produtos VALUES ('Produto1')
INSERT INTO Produtos VALUES ('Produto2')

Feito isso, o banco irá salvar os dados como:

Id  | Nome
----------------
1   | Produto1
2   | Produto2

Caso queira saber qual será o próximo código, você pode utilizar o código abaix:

SELECT isnull(IDENT_CURRENT('Produtos') + IDENT_INCR('Produtos'),1)

Esse código irá buscar o último id IDENT_CURRENT() e somar com o valor incremental, com o IDENT_INCR().

Em seu exemplo você está aumentando mais 1, mas se fosse 2, 3, etc... O IDENT_INCR() obtém esse valor da Key automaticamente.

Solução do problema...

Como já apontado nas demais respostas, basta alterar o ExecuteNonQuery() pelo ExecuteScalar() que resolverá o seu problema.

Caso queira entender um pouco mais a diferença entre os métodos, este link explica um pouco sobre eles.

2
  • Muito bom cara! Vou usar identity, visto que parece ser mais correto e seguro, certo? Commented 14/07/2016 às 18:08
  • @LeonardoCoelho Essa é a forma correta de se fazer isso, tanto os outros bancos de dados além do SQL Server possuem essa função. Existem várias perguntas sobre isso no site, basta ver este link
    – Randrade
    Commented 14/07/2016 às 18:34
4

O problema específico é que o método usado retorna o número de linhas afetadas e não o código gerado. O certo é usar o ExecuteScalar().

Em casos onde há acesso concorrente (quase sempre tem) pegar um código assim é problemático. Tem que mandar gravar os dados e aí gerar um novo código, caso contrário tem uma condição de corrida.

Além disso parece que a aplicação tem bastante código repetido. Não é uma boa ideia enxugar depois, mas isso é com você.

1
  • Obrigado pela resposta! Commented 14/07/2016 às 16:26
2

O método que você está utilizando para executar o SQL (ExecuteNonQuery()) é um método utilizado para inserts e updates e o retorno dele é a quantidade de linhas afetadas conforme descrito na documentação.

Utilize o método ExecuteScalar(), que o mesmo irá retornar o que você deseja.

Deverá ficar desta forma:

txt_prod_cod.Text = Convert.ToString(sqlCom.ExecuteScalar());

Documentação do ExecuteScalar().

Isto pode resolver o seu problema, mais conforme sugerido pelo Maniero essa não é uma boa prática, e provavelmente você deve estar repetindo muito código em sua aplicação, seria bom assim que possível reavaliar a forma de trabalho e verificar melhorias.

5
  • Qual seria uma boa prática para gerar códigos sequenciais? O código repetido na aplicação é só na parte de Conexão com o Banco, vou reavaliar sim, obrigado! Estou praticando POO e C#. Commented 14/07/2016 às 14:24
  • 1
    No meu caso, na maioria das vezes utilizo o auto incremento do próprio SQL Server, criando o campo chave com auto incremento, desta forma, o próprio SQL gerencia essa criação de índices. Caso não seja possível trabalhar com auto incremento, vc pode iniciar a aplicação sem o código e quando o usuário realizar a gravação, no momento em que grava, você busca o novo código. Após a gravação recarrega este novo código na tela. Commented 14/07/2016 às 14:32
  • @LeonardoCoelho, Já que você está praticando POO e C#, seria uma boa também dar uma estudada em um framework ORM como o EntityFramework (msdn.microsoft.com/pt-br/data/aa937709.aspx). Commented 14/07/2016 às 14:34
  • Muito obrigado Júlio, vou pesquisar sobre esse auto incremento, o SQL Server a partir do 2012 faz isso certo? Estou usando-o! Vou estudar sobre entity framework também, agradecido! @JulioBorges Commented 14/07/2016 às 16:30
  • @LeonardoCoelho, sempre suportou... vide exemplo: devmedia.com.br/… Commented 14/07/2016 às 17:12

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .