Ajuda com sintaxe MySql

16 de julho de 2011 31 respostas

wesleypc 16 de julho de 2011

Olá,

gostaria de uma ajuda, estou tentando fazer um insert no MySql, e tem um checkbox entre o formulario porem quando mando compilar ele da o seguinte erro " Incorrect integer value: ‘false’ for column ‘ativo’ at row 1 " .

O meu insert esta da seguinte maneira:

String sql = "INSERT INTO "
                   + "fornecedor(cnpj, data, razao_social, nome_fantasia, insc_estadual, endereco,"
                   + "complemento, numero, bairro, cep, id_pais, id_uf, id_municipio,"
                   + "telefone, celular,"
                   + "fax, contato, nro_banco, nro_agencia, conta_corrente, email, site, ativo, cpf, rg, nome)"
                   + "VALUES ('"+f.getCNPJ()+"', '"+f.getData()+"','"+f.getRazaoSocial()+"','"+f.getNomeFantasia()+"','"+f.getInscEstadual()+"','"+f.getEndereco()+"',"
                   + "'"+f.getComplemento()+"','"+f.getNumero()+"','"+f.getBairro()+"','"+f.getCEP()+"','"+f.getIdPais()+"','"+f.getIdUF()+"','"+f.getIdMunicipio()+"',"
                   + "'"+f.getTelefone()+"','"+f.getCelular()+"','"+f.getFAX()+"','"+f.getContato()+"','"+f.getNroBanco()+"','"+f.getNroAgencia()+"','"+f.getContaCorrente()+"','"+f.geteMail()+"',"
                   + "'"+f.getSite()+"','"+f.getAtivo()+"','"+f.getCPF()+"','"+f.getRG()+"','"+f.getNome()+"');";

            Statement stmt = conexao.createStatement();
            stmt.execute(sql,Statement.RETURN_GENERATED_KEYS);

Grato.

31 Respostas

jweibe 16 de jul. de 2011

Caro wesleypc prefira o uso de PreparedStatement ao invés de Statement. link
O e o seu problema e que vc ta passando um valor false direito para o MySQL.
Substitua:
Quando for true a coluna deve receber 1
Quando for false a coluna deve receber 0

ViniGodoy 16 de jul. de 2011

Ou use PreparedStatement, como sugerido, e nunca mais tenha esse tipo de problema. Com o PreparedStatement, é o Java que conhece o formato do banco. Ele que se preocupa se o valor deve vir entre aspas simples, em que formato a data deve ser fornecida, se um boolean deve ter o valor 0 ou 1 no banco.

Se você quiser ver o quão frágil fica seu sistema sem o PreparedStatement, tente cadastrar como fornecedor o MacDonald’s, ou o Bob’s.
As aspas simples vão destruir sua SQL em dois tempos.

Sem falar que um usuário mal intencionado que conheça programação poderia dar o nome do seu fornecedor assim:

thiagodelgado 16 de jul. de 2011

Só ilustrando o que os amigos disseram acima:

-se você usar mySQL não esqueça*

import com.mysql.jdbc.PreparedStatement;

Formando uma consulta:

PreparedStatement pst = seuObjetoDeConexao.prepareStatement("SELECT campo1,campo2,campoN FROM suaTabela WHERE campo1=?");

/*Agora, dependendo do tipo do campo (int, String, double, etc...) você usa um método específico para passar o valor que substituirá o '?'
nesse exemplo, vamos imaginar que é um valor do tipo inteiro*/

pst.setInt(1,777);


ResultSet rs = pst.executeQuery();

Dai pra frente acho que você se vira, pra usar delete, insert ou update, você deve alterar pst.executeQuery() pra pst.executeUpdate()

wesleypc 18 de jul. de 2011

Olá,

consegui consertar o erro que estava dando daseguinte maneira:

‘"+(f.getAtivo()?1:0)+"’

Agora estou com um erro de sintaxe, segue o erro:

java.sql.SQLException: Column Index out of range, 0 < 1.

Alguem poderia me dizer o que é esse erro?

Grato.

ViniGodoy 18 de jul. de 2011

Isso provavelmente é na hora de ler os dados da sua coluna.

Você deve fazer algo como:

No Java, o primeiro índice de uma coluna é 1, e não 0. Mas melhor ainda é você usar o nome do campo:

Assim você não precisará revisar todos os gets caso uma coluna seja acrescentada ou removida do select.

Você mudou para o PreparedStatement? No PreparedStatement, os índices também começam em 1.

wesleypc 18 de jul. de 2011

Olá viniGodoy,

coloquei o PreparedStatement, mais deu o seguinte erro: java.lang.ClassCastException: com.mysql.jdbc.StatementImpl cannot be cast to com.mysql.jdbc.PreparedStatement

Segue meu código abaixo, poderia verificar se esta incorreto alguma coisa?

try {
            String sql = "INSERT INTO "
                   + "fornecedor(cnpj, data, razao_social, nome_fantasia, insc_estadual, endereco,"
                   + "complemento, numero, bairro, cep, id_pais, id_uf, id_municipio,"
                   + "telefone, celular,"
                   + "fax, contato, nro_banco, nro_agencia, conta_corrente, email, site, ativo, cpf, rg, nome)"
                   + "VALUES ('"+f.getCNPJ()+"', '"+f.getData()+"','"+f.getRazaoSocial()+"','"+f.getNomeFantasia()+"','"+f.getInscEstadual()+"','"+f.getEndereco()+"',"
                   + "'"+f.getComplemento()+"','"+f.getNumero()+"','"+f.getBairro()+"','"+f.getCEP()+"','"+f.getIdPais()+"','"+f.getIdUF()+"','"+f.getIdMunicipio()+"',"
                   + "'"+f.getTelefone()+"','"+f.getCelular()+"','"+f.getFAX()+"','"+f.getContato()+"','"+f.getNroBanco()+"','"+f.getNroAgencia()+"','"+f.getContaCorrente()+"','"+f.geteMail()+"',"
                   + "'"+f.getSite()+"','"+(f.getAtivo()?1:0)+"','"+f.getCPF()+"','"+f.getRG()+"','"+f.getNome()+"');";

//            Statement stmt = conexao.createStatement();
//            stmt.execute(sql,Statement.RETURN_GENERATED_KEYS);

            PreparedStatement stmt = (PreparedStatement) conexao.createStatement();
            stmt.execute(sql,Statement.RETURN_GENERATED_KEYS);

            ResultSet rs = stmt.getGeneratedKeys();
            int idGerado = 0;
            if(rs.next()){
                idGerado = rs.getInt(1);
            }
            String ret = "";
            if (idGerado != 1){

                ret = this.salvaVendedores(f.getListaVendedores(), idGerado);
            }

            stmt.close();
            return "true";
        } catch (Exception e) {
            e.printStackTrace();
            return e.getMessage();
        }

ViniGodoy 18 de jul. de 2011

É porque vc fez errado:

try {
            String sql = "INSERT INTO fornecedor("
                   + " cnpj, data, razao_social, nome_fantasia, insc_estadual,"
                   + " endereco, complemento, numero, bairro, cep,"
                   + " id_pais, id_uf, id_municipio, telefone, celular,"
                   + " fax, contato, nro_banco, nro_agencia, conta_corrente,"
                   + " email, site, ativo, cpf, rg, nome)"
                   + "VALUES (?, ?, ?, ?, ?,"
                   + " ?, ?, ?, ?, ?,"
                   + " ?, ?, ?, ?, ?,"
                   + " ?, ?, ?, ?, ?,"
                   + " ?, ?, ?, ?, ?, ?)";

            PreparedStatement stmt = conexao.prepareStatement(sql);
            stmt.setString(1, f.getCNPJ());
            stmt.setDate(2, f.getData());
            stmt.setString(3, f.getRazaoSocial());
            //Complete aqui o resto dos campos
            stmt.executeUpdate();
            ResultSet rs = stmt.getGeneratedKeys();
            int idGerado = 0;
            if(rs.next()){
                idGerado = rs.getInt(1);
            }
            String ret = "";
            if (idGerado != 1){
                ret = this.salvaVendedores(f.getListaVendedores(), idGerado);
            }
            rs.close();
            stmt.close();
            return "true";
        } catch (Exception e) {
            e.printStackTrace();
            return e.getMessage();
        }

brenowbc 18 de jul. de 2011

eu uso o MySQL como banco, e não importo o com.mysql.jdbc.PreparedStatement não,
importo simplesmente o java.sql.; e nunca tive erros não.
Ao autor do tópico ae, tenta simplesmente importar o java.sql. e vê se da certo.

so pra encargo de consciência, o que diferencia o com.mysql.jdbc.PreparedStatement do java.sql.* ?

um abraço

ViniGodoy 18 de jul. de 2011

É melhor importar só o do java.sql, nunca o do mysql.

A diferença é que o do java.sql é a interface, e o do mysql é a implementação da interface. Quem é responsável por obter essa implementação de forma transparente é o driver. Se você vincular a java.mysql direto, você terá muito mais difículdades se um dia decidir trocar de banco de dados.

brenowbc 18 de jul. de 2011

justamente, éra isso que eu tava pensando.
só outra coisa, eu escuto falar muito no Hibernate, só que ja vi e dei uma olhada rapida, parece que ele
pode sofrer SQLInjection, então já não achei uma boa opção, mas desculpe o desvio de assunto, mas qual
a opinião de vocês, acham melhor utilizar o PreparedStatement ou o Hibernate?
um abraço

wesleypc 18 de jul. de 2011

Olá ViniGodoy,

alterei para o jeito que vc flw, porem esta dando o seguinte erro: You need to specify Statement.RETURN_GENERATED_KEYS to Statement.executeUpdate() or Connection.prepareStatement()

Segue o Código:

try {

             String sql = "INSERT INTO "
             + "fornecedor(cnpj, data, razao_social, nome_fantasia, insc_estadual,"
             + "endereco, complemento, numero, bairro, cep,"
             + "id_pais, id_uf, id_municipio, telefone, celular,"
             + "fax, contato, nro_banco, nro_agencia, conta_corrente,"
             + " email, site, ativo, cpf, rg, nome)"
             + "VALUES (?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?, ?)";

      PreparedStatement stmt = (PreparedStatement) conexao.prepareStatement(sql);
      stmt.setString(1, f.getCNPJ());
      stmt.setString(2, f.getData());
      stmt.setString(3, f.getRazaoSocial());
      stmt.setString(4, f.getNomeFantasia());
      stmt.setString(5, f.getInscEstadual());
      stmt.setString(6, f.getEndereco());
      stmt.setString(7, f.getComplemento());
      stmt.setString(8, f.getNumero());
      stmt.setString(9, f.getBairro());
      stmt.setString(10, f.getCEP());
      stmt.setString(11, f.getIdPais());
      stmt.setInt(12, f.getIdUF());
      stmt.setInt(13, f.getIdMunicipio());
      stmt.setString(14, f.getTelefone());
      stmt.setString(15, f.getCelular());
      stmt.setString(16, f.getFAX());
      stmt.setString(17, f.getContato());
      stmt.setString(18, f.getNroBanco());
      stmt.setString(19, f.getNroAgencia());
      stmt.setString(20, f.getContaCorrente());
      stmt.setString(21, f.geteMail());
      stmt.setString(22, f.getSite());
      stmt.setBoolean(23, f.getAtivo());
      stmt.setString(24, f.getCPF());
      stmt.setString(25, f.getRG());
      stmt.setString(26, f.getNome());
      
      
      stmt.executeUpdate();
      ResultSet rs = stmt.getGeneratedKeys();
      int idGerado = 0;
      if(rs.next()){
          idGerado = rs.getInt(1);
      }
      String ret = "";
      if (idGerado != 1){

          ret = this.salvaVendedores(f.getListaVendedores(), idGerado);
      }

      stmt.close();
      return "true";
}

brenowbc 18 de jul. de 2011

olha, vou mandar um método basico que uso para inserção em BD, eu faço com que o banco gere automaticamente
a codigo, ou seja, ele é AUTO_INCREMENT, e uso ele como a chave primária, por isso que ele nem aparece aqui no exemplo.

aqui vai o código:

public void adicionarCliente(Cliente cliente)
	{
		// COMANDO SQL QUE SERÁ EXECUTADO
		String comandoSQL = 
"INSERT INTO CLIENTE (NOME, ENDEREÇO, NUMERO, BAIRRO, CIDADE, UF, TELEFONE, CELULAR, EMAIL) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)"; // aqui devem entrar os devidos valores
		
		try // executa os comandos com o banco de dados
		{
			
			// prepared statement para a inserção no banco
			
			executarSQL = conexao.prepareStatement(comandoSQL); // insere o comando no statement, onde conexao é a 
                       //  variavel de conexão com o BD, no caso, uso uma classe exclusivamente para isso.
			
			// seta os valores, de acordo com a ordem da interrogação
			
			executarSQL.setString(1, cliente.getNome()); // coloca o nome no banco de dados
			
                        executarSQL.setString(2, cliente.getEndereco()); // coloca o endereço
                        executarSQL.setInt(3, cliente.getNumero()); // coloca o numero da residencia
                        executarSQL.setString(4, cliente.getBairro()); // coloca o bairro
                        executarSQL.setString(5, cliente.getCidade()); // coloca a cidade
                        executarSQL.setString(6, cliente.getUF()); // coloca o UF
                        
			executarSQL.setString(7, cliente.getTelefone()); // coloca o telefone
			executarSQL.setString(8, cliente.getCelular());  // coloca o celular
                        executarSQL.setString(9, cliente.getEmail()); // coloca o email
			
			// executa o statement
			executarSQL.execute();
			
			JOptionPane.showMessageDialog(null, "Dados Gravados Com Sucesso",
                                "Resultado", JOptionPane.INFORMATION_MESSAGE);
			
			executarSQL.close(); // fecha o statement do comando sql
			
			conexao.close(); // fecha a conexão com o banco de dados
			
			System.out.println("\nConexão Fechada!\n");
		}
		catch(SQLException e)
		{
			System.out.println("Cliente Não Adicionado!");
			
			throw new RuntimeException(e);
		}

tente adequa-lo a sua necessidade, um abraço.

brenowbc 18 de jul. de 2011

e para colocar, por exemplo, num ArrayList, so uso o seguinte método:

public List<Cliente> getContatos()
	{
		try
		{
			System.out.println("\nPesquisando...\n\n");
			List<Cliente> contatosList = new ArrayList<Cliente>(); // cria o arrayList
			
			PreparedStatement pesquisarSQL = this.conexao.prepareStatement( // gera o codigo sql para a consulta, logo abaixo
				"SELECT * FROM CLIENTE" );
				
			ResultSet resultadoQuery = pesquisarSQL.executeQuery(); // coloca em resultadoQuery o resultado da consulta ao BD
			
			while (resultadoQuery.next())
			{
				Cliente contato = new Cliente(); // cria o objeto Cliente
				
                              //cod_cliente, nome, endereço... são os campos na tabela.

                                contato.setCodigo(resultadoQuery.getInt("cod_cliente"));
				contato.setNome(resultadoQuery.getString("nome")); // captura o nome
				contato.setEndereco(resultadoQuery.getString("endereço")); // captura o endereço
                                contato.setNumero(resultadoQuery.getInt("numero"));
                                contato.setBairro(resultadoQuery.getString("bairro"));
                                contato.setCidade(resultadoQuery.getString("cidade"));
                                contato.setUF(resultadoQuery.getString("uf"));
                                contato.setTelefone(resultadoQuery.getString("telefone"));
				contato.setCelular(resultadoQuery.getString("celular")); // captura o celular
				contato.setEmail(resultadoQuery.getString("email"));  // captura o email
				
				contatosList.add(contato); // adiciona o contato ao ArrayList
					
			}			
			
			resultadoQuery.close(); // fecha o resultado da pesquisa
			pesquisarSQL.close(); // fecha o codigo de pesquisaSQL
                        
                        System.out.println("\nPesquisa Concluída");
			
			return contatosList;	
		}
		catch(SQLException e)
		{
			System.out.println("\nPesquisa Errada!\n");
			throw new RuntimeException(e);
		}
		
	}

novamente, tente adapta-la as suas necessidades, só mais uma demonstração rsrs.
um abraço

brenowbc 18 de jul. de 2011

E só para finalizar, esse exemplo mostra melhor o uso de um PreparedStatement, por exemplo, para editar um
determinado Cliente:

public void editarCliente(Cliente cliente) // recebe o Objeto que será usado no PreparedStatement
        {
            String comandoSQL =                                 // todo esse bloco, é o comando SQL que será executado
                    "UPDATE CLIENTE SET " +                   // as interrogações serão substituidas pelo que será colocado no PreparedStatement
                    "NOME = ?, " +
                    "ENDEREÇO = ?, " +
                    "NUMERO = ?, " +
                    "BAIRRO = ?, " +
                    "CIDADE = ?, " +
                    "UF = ?, " +
                    "TELEFONE = ?, " +
                    "CELULAR = ?, " +
                    "EMAIL = ? " +
                    "WHERE COD_CLIENTE = ?";
                      
            try
            {
                PreparedStatement editarSQL = conexao.prepareStatement(comandoSQL); // coloca o comandoSQL no preparedStatement, para serem substituidas as interrogações
                
                // seta por ordem os valores que serão colocados nas interrogações
                editarSQL.setString(1, cliente.getNome());
                editarSQL.setString(2, cliente.getEndereco());
                editarSQL.setInt(3, cliente.getNumero());
                editarSQL.setString(4, cliente.getBairro());
                editarSQL.setString(5, cliente.getCidade());
                editarSQL.setString(6, cliente.getUF());
                editarSQL.setString(7, cliente.getTelefone());
                editarSQL.setString(8, cliente.getCelular());
                editarSQL.setString(9, cliente.getEmail());
                editarSQL.setInt(10, cliente.getCodigo());
                
                editarSQL.execute(); // executa o preparedStatement
                editarSQL.close();  // fecha o preparedStatement
                
                JOptionPane.showMessageDialog(null, "Cliente Alterado com Sucesso!");
            }
            catch(SQLException e)
            {
                JOptionPane.showMessageDialog(null, "Cliente Não Alterado");
                throw new RuntimeException(e);
            } 
        }

está bem comentado, tomara que entenda, uso tantos comentários, mas é pelo fato de cursar Licenciatura em computação, ae quando futuramente
eu for dar aula, é muito bom ser comentado para o bom entendimento dos futuros alunos… rsrsrs

um abraço

wesleypc 18 de jul. de 2011

Olá,

Acho que o meu INSERT eu consegui arrumar graças a ajuda de vcs, o meu unico problema agora é para manipular o ResultSet, não sei o q colocar na linha 48 abaixo, se eu coloco
ResultSet rs = stmt.executeQuery(); ele da o seguinte erro = Can not issue data manipulation statements with executeQuery()

E se eu coloco ResultSet rs = stmt.getGeneratedKeys(); ele da o seguinte erro = You need to specify Statement.RETURN_GENERATED_KEYS to Statement.executeUpdate() or Connection.prepareStatement()

Gostaria de uma ajuda para resolver isso…

try {

             String sql = "INSERT INTO "
             + "fornecedor(cnpj, data, razao_social, nome_fantasia, insc_estadual,"
             + "endereco, complemento, numero, bairro, cep,"
             + "id_pais, id_uf, id_municipio, telefone, celular,"
             + "fax, contato, nro_banco, nro_agencia, conta_corrente,"
             + " email, site, ativo, cpf, rg, nome)"
             + "VALUES (?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?, ?)";

      PreparedStatement stmt = (PreparedStatement) conexao.prepareStatement(sql);

      stmt.setString(1, f.getCNPJ());
      stmt.setString(2, f.getData());
      stmt.setString(3, f.getRazaoSocial());
      stmt.setString(4, f.getNomeFantasia());
      stmt.setString(5, f.getInscEstadual());
      stmt.setString(6, f.getEndereco());
      stmt.setString(7, f.getComplemento());
      stmt.setString(8, f.getNumero());
      stmt.setString(9, f.getBairro());
      stmt.setString(10, f.getCEP());
      stmt.setString(11, f.getIdPais());
      stmt.setInt(12, f.getIdUF());
      stmt.setInt(13, f.getIdMunicipio());
      stmt.setString(14, f.getTelefone());
      stmt.setString(15, f.getCelular());
      stmt.setString(16, f.getFAX());
      stmt.setString(17, f.getContato());
      stmt.setString(18, f.getNroBanco());
      stmt.setString(19, f.getNroAgencia());
      stmt.setString(20, f.getContaCorrente());
      stmt.setString(21, f.geteMail());
      stmt.setString(22, f.getSite());
      stmt.setBoolean(23, f.getAtivo());
      stmt.setString(24, f.getCPF());
      stmt.setString(25, f.getRG());
      stmt.setString(26, f.getNome());
      
      
      //stmt.executeUpdate();
      stmt.execute();

      ResultSet rs = stmt.executeQuery();
      int idGerado = 0;
      if(rs.next()){
          idGerado = rs.getInt(1);
      }
      String ret = "";
      if (idGerado != 1){

          ret = this.salvaVendedores(f.getListaVendedores(), idGerado);
      }

      stmt.close();
      return "true";

ViniGodoy 18 de jul. de 2011

Você deve usar o executeUpdate() para INSERT, UPDATE e DELETE. Ele retorna um int com o número de registros afetados.
E você deve usar o executeQuery() para SELECTS.

Para poder usar o getGeneratedKeys, crie o preparedStatement assim:

wesleypc 18 de jul. de 2011

ViniGodoy,

Inseri a linha 49 no meu código, ele esta inserindo na tabela mais ele esta apresentando o seguinte erro agora: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ‘?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)’ at line 1

public String insere(Fornecedor f){
        try {

             String sql = "INSERT INTO "
             + "fornecedor(cnpj, data, razao_social, nome_fantasia, insc_estadual,"
             + "endereco, complemento, numero, bairro, cep,"
             + "id_pais, id_uf, id_municipio, telefone, celular,"
             + "fax, contato, nro_banco, nro_agencia, conta_corrente,"
             + " email, site, ativo, cpf, rg, nome)"
             + "VALUES (?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?, ?)";

      PreparedStatement stmt = (PreparedStatement) conexao.prepareStatement(sql);


      stmt.setString(1, f.getCNPJ());
      stmt.setString(2, f.getData());
      stmt.setString(3, f.getRazaoSocial());
      stmt.setString(4, f.getNomeFantasia());
      stmt.setString(5, f.getInscEstadual());
      stmt.setString(6, f.getEndereco());
      stmt.setString(7, f.getComplemento());
      stmt.setString(8, f.getNumero());
      stmt.setString(9, f.getBairro());
      stmt.setString(10, f.getCEP());
      stmt.setString(11, f.getIdPais());
      stmt.setInt(12, f.getIdUF());
      stmt.setInt(13, f.getIdMunicipio());
      stmt.setString(14, f.getTelefone());
      stmt.setString(15, f.getCelular());
      stmt.setString(16, f.getFAX());
      stmt.setString(17, f.getContato());
      stmt.setString(18, f.getNroBanco());
      stmt.setString(19, f.getNroAgencia());
      stmt.setString(20, f.getContaCorrente());
      stmt.setString(21, f.geteMail());
      stmt.setString(22, f.getSite());
      stmt.setBoolean(23, f.getAtivo());
      stmt.setString(24, f.getCPF());
      stmt.setString(25, f.getRG());
      stmt.setString(26, f.getNome());
      
      
      stmt.executeUpdate();

      stmt.execute(sql,Statement.RETURN_GENERATED_KEYS);

      ResultSet rs = stmt.getGeneratedKeys(); 
      int idGerado = 0;
      if(rs.next()){
          idGerado = rs.getInt(1);
      }
      String ret = "";
      if (idGerado != 1){

          ret = this.salvaVendedores(f.getListaVendedores(), idGerado);
      }

      stmt.close();
      return "true";

        } catch (Exception e) {
            e.printStackTrace();
            return e.getMessage();
        }
    }

ViniGodoy 18 de jul. de 2011

Dê um espaço depois do ) de nome, na linha 9. Caso contrário ficará grudado com o VALUES da linha 10.

wesleypc 19 de jul. de 2011

Ola,

ainda estou com um probleminha aqui, o insert ta funcionando e tudo mais, porem ele ainda acusa uma exception, segue a mesma: You need to specify Statement.RETURN_GENERATED_KEYS to Statement.executeUpdate() or Connection.prepareStatement()

Alguem poderia me ajudar a solucionar ess exception?

Segue o código:

try {

             String sql = "INSERT INTO "
             + "fornecedor(cnpj, data, razao_social, nome_fantasia, insc_estadual,"
             + "endereco, complemento, numero, bairro, cep,"
             + "id_pais, id_uf, id_municipio, telefone, celular,"
             + "fax, contato, nro_banco, nro_agencia, conta_corrente,"
             + " email, site, ativo, cpf, rg, nome) "
             + "VALUES (?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?, ?)";

      //PreparedStatement stmt = (PreparedStatement) conexao.prepareStatement(sql, Statement.RETURN_GENERATED_KEYS);
    PreparedStatement stmt = (PreparedStatement) conexao.prepareStatement(sql);

      stmt.setString(1, f.getCNPJ());
      stmt.setString(2, f.getData());
      stmt.setString(3, f.getRazaoSocial());
      stmt.setString(4, f.getNomeFantasia());
      stmt.setString(5, f.getInscEstadual());
      stmt.setString(6, f.getEndereco());
      stmt.setString(7, f.getComplemento());
      stmt.setString(8, f.getNumero());
      stmt.setString(9, f.getBairro());
      stmt.setString(10, f.getCEP());
      stmt.setString(11, f.getIdPais());
      stmt.setInt(12, f.getIdUF());
      stmt.setInt(13, f.getIdMunicipio());
      stmt.setString(14, f.getTelefone());
      stmt.setString(15, f.getCelular());
      stmt.setString(16, f.getFAX());
      stmt.setString(17, f.getContato());
      stmt.setString(18, f.getNroBanco());
      stmt.setString(19, f.getNroAgencia());
      stmt.setString(20, f.getContaCorrente());
      stmt.setString(21, f.geteMail());
      stmt.setString(22, f.getSite());
      stmt.setBoolean(23, f.getAtivo());
      stmt.setString(24, f.getCPF());
      stmt.setString(25, f.getRG());
      stmt.setString(26, f.getNome());
      
      
      stmt.executeUpdate();

     // stmt.execute(sql,Statement.RETURN_GENERATED_KEYS);

      ResultSet rs = stmt.getGeneratedKeys(); 
      int idGerado = 0;
      if(rs.next()){
          idGerado = rs.getInt(0);
      }
      String ret = "";
      if (idGerado != 1){

          ret = this.salvaVendedores(f.getListaVendedores(), idGerado);
      }

      stmt.close();
      return "true";

      } catch (Exception e) {
            e.printStackTrace();
            return e.getMessage();
            //return "";
        }

ViniGodoy 19 de jul. de 2011

Repetindo…

[quote=ViniGodoy]Para poder usar o getGeneratedKeys, crie o preparedStatement assim:

wesleypc 19 de jul. de 2011

ViniGodooy,

alterei conforme vc flw e rodei, e ele deu o seguinte erro na linha 54 : Column Index out of range, 0 < 1.

Na linha 47, eu não tenho q alterar para a seguinte maneira??

“stmt.executeUpdate(sql);”

Fiz isso, alterei inserindo o “sql” no “executeUpdate()” e ele deu o seguinte erro: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ‘?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)’ at line 1

try {

             String sql = "INSERT INTO "
             + "fornecedor(cnpj, data, razao_social, nome_fantasia, insc_estadual,"
             + "endereco, complemento, numero, bairro, cep,"
             + "id_pais, id_uf, id_municipio, telefone, celular,"
             + "fax, contato, nro_banco, nro_agencia, conta_corrente,"
             + " email, site, ativo, cpf, rg, nome) "
             + "VALUES (?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?, ?)";


       PreparedStatement stmt = (PreparedStatement) conexao.prepareStatement(sql, Statement.RETURN_GENERATED_KEYS);

      stmt.setString(1, f.getCNPJ());
      stmt.setString(2, f.getData());
      stmt.setString(3, f.getRazaoSocial());
      stmt.setString(4, f.getNomeFantasia());
      stmt.setString(5, f.getInscEstadual());
      stmt.setString(6, f.getEndereco());
      stmt.setString(7, f.getComplemento());
      stmt.setString(8, f.getNumero());
      stmt.setString(9, f.getBairro());
      stmt.setString(10, f.getCEP());
      stmt.setString(11, f.getIdPais());
      stmt.setInt(12, f.getIdUF());
      stmt.setInt(13, f.getIdMunicipio());
      stmt.setString(14, f.getTelefone());
      stmt.setString(15, f.getCelular());
      stmt.setString(16, f.getFAX());
      stmt.setString(17, f.getContato());
      stmt.setString(18, f.getNroBanco());
      stmt.setString(19, f.getNroAgencia());
      stmt.setString(20, f.getContaCorrente());
      stmt.setString(21, f.geteMail());
      stmt.setString(22, f.getSite());
      stmt.setBoolean(23, f.getAtivo());
      stmt.setString(24, f.getCPF());
      stmt.setString(25, f.getRG());
      stmt.setString(26, f.getNome());
      
      
      stmt.executeUpdate(sql);

     // stmt.execute(sql,Statement.RETURN_GENERATED_KEYS);

      ResultSet rs = stmt.getGeneratedKeys(); 
      int idGerado = 0;
      if(rs.next()){
          idGerado = rs.getInt(0);
      }
      String ret = "";
      if (idGerado != 1){

          ret = this.salvaVendedores(f.getListaVendedores(), idGerado);
      }

      stmt.close();
      return "true";

      } catch (Exception e) {
            e.printStackTrace();
            return e.getMessage();
            //return "";
        }

ViniGodoy 19 de jul. de 2011

Cara, sempre que tiver problemas:

Leia com atenção a mensagem de erro. Ela diz:
Column Index out of range, 0 < 1.
"Índice da coluna fora do intervalo. 0 < 1".

Vá até sua linha 54, você está tentando obter algum índice 0?

Veja a documentação do método onde dá erro. A documentação do getInt é essa aqui:

getInt

int getInt(int columnIndex)
throws SQLException

Retrieves the value of the designated column in the current row of this ResultSet object as an int in the Java programming language.

Parameters:
    columnIndex - [b]the first column is 1[/b], the second is 2, ... 
Returns:
    the column value; if the value is SQL NULL, the value returned is 0 
Throws:
    SQLException - if the columnIndex is not valid; if a database access error occurs or this method is called on a closed result set

http://download.oracle.com/javase/6/docs/api/java/sql/ResultSet.html#getInt(int)

Jamais tente corrigir coisas por tentativa e erro. Se o erro estava na linha 54, por que você mudou a linha do execute?
Para corrigir nesse caso, apenas faça o que a documentação está mandando. Use o índice 1 no lugar do 0:

Deixe o executeUpdate sem o parâmetro sql dentro.

Além disso, procure ler e entender o que seus colegas de fórum estão falando. Eu também já tinha falado que os índices começavam em 1 a poucos posts atrás.

wesleypc 19 de jul. de 2011

Olá,

desculpe pelas perguntas desnecessarias, consegui arrumar o erro.

Grato pela ajuda.

wesleypc 21 de jul. de 2011

Olá,

estou com uma duvida, tenho um formulario e no mesmo tem varias abas (dados, ICMS, PIS, …) , porem estou querendo somente cadastrar os dados de uma aba especifica, e ai ela ocorre o null Pointer de campos de outras abas, olhei no meu Mysql e as mesmas encontram desmarcada o campo not null, alguem poderia me dar uma mão?

Segue a parte do código do Insert abaixo, grato.

try {

           String sql = "INSERT INTO "
             + "produto(cod_prod,cod_barra,descricao,id_grupo,id_unidade,peso_bruto,peso_liquido,ncm,id_marca,"
                   + "lucro_minimo,lucro_atacado,lucro_promocao,lucro_varejo,valor_compra,valor_minimo,valor_atacado,"
                   + "valor_promocao,valor_venda,quant_estoque,estoque_minimo,frete,despesas,bloqueado,gera_estoque,"
                   + "orig_icms,cst_icms,mod_bc_icms,p_red_bc_icms,v_bc_icms,p_icms,v_icms,mod_bc_st_icms,p_mvast_icms,"
                   + "p_red_bc_st_icms,v_bc_st_icms,p_icms_st,v_icms_st,cst_pis,v_bc_pis,p_pis,v_pis,q_bc_prod_pis,v_aliq_prod_pis,"
                   + "cst_cofins,v_bc_cofins,p_cofins,v_cofins,q_bc_prod_cofins,v_aliq_prod_cofins,totalizador,cst_ipi,"
                   + "classe_enquad_ipi,cod_enquad_ipi,cnpj_prod_ipi,cod_selo_contr_ipi,quant_selo_contr_ipi,tipo_calculo,"
                   + "val_bc_ipi,aliquota_ipi,quant_total_un_padrao_ipi,valor_unidade_ipi,valor_ipi,preco_fabrica,referencia,"
                   + "numeracao,produto_composto,id_subgrupo,idcst_icms,idmod_bc,idmod_bc_st,idcst_cofins,idcst_ipi,idcst_pis"
                   + ") "
             + "VALUES (?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?,"
             + " ?, ?, ?, ?, ?, ?, ?)";


              PreparedStatement stmt = (PreparedStatement) conexao.prepareStatement(sql);


      stmt.setString(1, p.getCodProduto());
      stmt.setString(2, p.getCodBarra());
      stmt.setString(3, p.getDescricao());
      stmt.setInt(4, p.getIdGrupo());
      stmt.setInt(5, p.getIdUnidade());
      stmt.setBigDecimal(6, p.getPesoBruto());
      stmt.setBigDecimal(7, p.getPesoLiquido());
      stmt.setString(8, p.getNCM());
      stmt.setInt(9, p.getIdMarca());
      stmt.setBigDecimal(10, p.getLucroMinimo());
      stmt.setBigDecimal(11, p.getLucroAtacado());
      stmt.setBigDecimal(12, p.getLucroPromocao());
      stmt.setBigDecimal(13, p.getLucroVarejo());
      stmt.setBigDecimal(14, p.getValorCompra());
      stmt.setBigDecimal(15, p.getValorMinimo());
      stmt.setBigDecimal(16, p.getValorAtacado());
      stmt.setBigDecimal(17, p.getValorPromocao());
      stmt.setBigDecimal(18, p.getValorVenda());
      stmt.setBigDecimal(19, p.getQuantEstoque());
      stmt.setBigDecimal(20, p.getEstoqueMinimo());
      stmt.setBigDecimal(21, p.getFrete());
      stmt.setBigDecimal(22, p.getDespesas());
      stmt.setBoolean(23, p.isBloqsueado());
      stmt.setBoolean(24, p.isGerasEstoque());
      stmt.setInt(25, p.getOrigICMS());
      stmt.setString(26, p.getCSTICMS());
       stmt.setInt(27, p.getModBCICMS());
      stmt.setBigDecimal(28, p.getpRedBCICMS());
      stmt.setBigDecimal(29, p.getvBCICMS());
      stmt.setBigDecimal(30, p.getpICMS());
      stmt.setBigDecimal(31, p.getvICMS());
      stmt.setInt(32, p.getModBCSTICMS());
      stmt.setBigDecimal(33, p.getpMVASTICMS());
      stmt.setBigDecimal(34, p.getpRedBCSTICMS());
      stmt.setBigDecimal(35, p.getvBCSTICMS());
      stmt.setBigDecimal(36, p.getpICMSST());
      stmt.setBigDecimal(37, p.getvICMSST());
      stmt.setString(38, p.getCSTPIS());
      stmt.setBigDecimal(39, p.getvBCPIS());
      stmt.setBigDecimal(40, p.getpPIS());
      stmt.setBigDecimal(41, p.getvPIS());
      stmt.setBigDecimal(42, p.getqBCProdPIS());
      stmt.setBigDecimal(43, p.getvAliqProdPIS());
      stmt.setString(44, p.getCSTCOFINS());
      stmt.setBigDecimal(45, p.getvBCCOFINS());
      stmt.setBigDecimal(46, p.getpCOFINS());
      stmt.setBigDecimal(47, p.getvCOFINS());
      stmt.setBigDecimal(48, p.getqBCProdCOFINS());
      stmt.setBigDecimal(49, p.getvAliqProdCOFINS());
      stmt.setString(50, p.getTotalizador());
      stmt.setString(51, p.getCSTIPI());
      stmt.setString(52, p.getClasseEnquadIPI());
      stmt.setString(53, p.getCodEnquadIPI());
      stmt.setString(54, p.getCNPJProdIPI());
      stmt.setString(55, p.getCodSeloContrIPI());
      stmt.setString(56, p.getQuantSeloContrIPI());
      stmt.setString(57, p.getTipoCalculo());
      stmt.setBigDecimal(58, p.getValBCIPI());
      stmt.setBigDecimal(59, p.getAliquotaIPI());
      stmt.setBigDecimal(60, p.getQuantTotalUnPadraoIPI());
      stmt.setBigDecimal(61, p.getValorUnidadeIPI());
      stmt.setBigDecimal(62, p.getValorIPI());
      stmt.setBigDecimal(63, p.getPrecoFabrica());
      stmt.setString(64, p.getReferencia());
      stmt.setString(65, p.getNumeracao());
      stmt.setBoolean(66, p.isProdutoComposto());
      stmt.setInt(67, p.getIdSubGrupo());
      stmt.setInt(68, p.getIdcst_icms());
      stmt.setInt(69, p.getIdmod_bc());
      stmt.setInt(70, p.getIdmod_bc_st());
      stmt.setInt(71, p.getIdcst_cofins());
      stmt.setInt(72, p.getIdcst_ipi());
      stmt.setInt(73, p.getIdcst_pis());


            stmt.execute();
            stmt.close();
            return "true";
        } catch (Exception e) {
            e.printStackTrace();
            return e.getMessage();
        }

wesleypc 27 de jul. de 2011

OLá pessoal,

referente ao erro acima eu ja resolvi, porem estou com um outro probleminha aqui, estou tentando dar um update em uma tabela e a mesma retorna o seguinte erro:

“You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ‘idcst_icms=‘2’ idmod_bc=‘3’ idmod_bc_st=‘0’ idcst_cofins=‘0’ idcst_ipi=‘0’ idcst’ at line 1”

Os campos falados no erro e seus valores são valores que eu setei na mão caso eles sejam null.

Segue o código do update:

public String atualiza(Produto p){
        try {
            String sql = "UPDATE produto "
                    + "SET cod_prod='"+ p.getCodProduto() +"',"
                    + "cod_barra='"+ p.getCodBarra() +"',"
                    + "descricao='"+ p.getDescricao() +"',"
                    + "id_grupo='"+ p.getIdGrupo() +"',"
                    + "id_unidade='"+ p.getIdUnidade() +"',"
                    + "peso_bruto='"+ p.getPesoBruto() +"',"                    
                    + "peso_liquido='"+ p.getPesoLiquido() +"',"
                    + "ncm='"+ p.getNCM() +"',"
                    + "id_marca='"+ p.getIdMarca() +"',"
                    + "lucro_minimo='"+ p.getLucroMinimo() +"',"
                    + "lucro_atacado='"+ p.getLucroAtacado() +"',"
                    + "lucro_promocao='"+ p.getLucroPromocao() +"',"
                    + "lucro_varejo='"+ p.getLucroVarejo() +"',"
                    + "valor_compra='"+ p.getValorCompra() +"',"
                    + "valor_minimo='"+ p.getValorMinimo() +"',"
                    + "valor_atacado='"+ p.getValorAtacado() +"',"
                    + "valor_promocao='"+ p.getValorPromocao() +"',"
                    + "valor_venda='"+ p.getValorVenda() +"',"
                    + "quant_estoque='"+ p.getQuantEstoque() +"',"
                    + "estoque_minimo='"+ p.getEstoqueMinimo() +"',"
                    + "frete='"+ p.getFrete() +"',"
                    + "despesas='"+ p.getDespesas() +"',"
                    + "bloqueado='"+ (p.isBloqsueado()?1:0) +"',"
                    + "gera_estoque='"+ (p.isGerasEstoque()?1:0) +"',"
                    + "orig_icms='"+ p.getOrigICMS() +"',"
                    + "cst_icms='"+ p.getCSTICMS() +"',"
                    + "mod_bc_icms='"+ p.getModBCICMS() +"',"
                    + "p_red_bc_icms='"+ p.getpRedBCICMS() +"',"
                    + "v_bc_icms='"+ p.getvBCICMS() +"',"
                    + "p_icms='"+ p.getpICMS() +"',"
                    + "v_icms='"+ p.getvICMS() +"',"
                    + "mod_bc_st_icms='"+ p.getModBCSTICMS() +"',"
                    + "p_mvast_icms='"+ p.getpMVASTICMS() +"',"
                    + "p_red_bc_st_icms='"+ p.getpRedBCSTICMS() +"',"
                    + "v_bc_st_icms='"+ p.getvBCSTICMS() +"',"
                    + "p_icms_st='"+ p.getpICMSST() +"',"
                    + "v_icms_st='"+ p.getvICMSST() +"',"
                    + "cst_pis='"+ p.getCSTPIS() +"',"
                    + "v_bc_pis='"+ p.getvBCPIS() +"',"
                    + "p_pis='"+ p.getpPIS() +"',"
                    + "v_pis='"+ p.getvPIS() +"',"
                    + "q_bc_prod_pis='"+ p.getqBCProdPIS() +"',"
                    + "v_aliq_prod_pis='"+ p.getvAliqProdPIS() +"',"
                    + "cst_cofins='"+ p.getCSTCOFINS() +"',"
                    + "v_bc_cofins='"+ p.getvBCCOFINS() +"',"
                    + "p_cofins='"+ p.getpCOFINS() +"',"
                    + "v_cofins='"+ p.getvCOFINS() +"',"
                    + "q_bc_prod_cofins='"+ p.getqBCProdCOFINS() +"',"
                    + "v_aliq_prod_cofins='"+ p.getvAliqProdCOFINS() +"',"
                    + "totalizador='"+ p.getTotalizador() +"',"
                    + "cst_ipi='"+ p.getCSTIPI() +"',"
                    + "classe_enquad_ipi='"+ p.getClasseEnquadIPI() +"',"
                    + "cod_enquad_ipi='"+ p.getCodEnquadIPI() +"',"
                    + "cnpj_prod_ipi='"+ p.getCNPJProdIPI() +"',"
                    + "cod_selo_contr_ipi='"+ p.getCodSeloContrIPI() +"',"
                    + "quant_selo_contr_ipi='"+ p.getQuantSeloContrIPI() +"',"
                    + "tipo_calculo='"+ p.getTipoCalculo() +"',"
                    + "val_bc_ipi='"+ p.getValBCIPI() +"',"
                    + "aliquota_ipi='"+ p.getAliquotaIPI() +"',"
                    + "quant_total_un_padrao_ipi='"+ p.getQuantTotalUnPadraoIPI() +"',"
                    + "valor_unidade_ipi='"+ p.getValorUnidadeIPI() +"',"
                    + "valor_ipi='"+ p.getValorIPI() +"',"
                    + "preco_fabrica='"+ p.getPrecoFabrica() +"',"
                    + "referencia='"+ p.getReferencia() +"',"
                    + "numeracao='"+ p.getNumeracao() +"', "
                    + "produto_composto='"+ p.isProdutoComposto() +"', "
                    + "id_subgrupo='"+ p.getIdSubGrupo() +"' "
                    + "idcst_icms='"+ p.getIdcst_icms() +"' "
                    + "idmod_bc='"+ p.getIdmod_bc() +"' "
                    + "idmod_bc_st='"+ p.getIdmod_bc_st() +"' "
                    + "idcst_cofins='"+ p.getIdcst_cofins() +"' "
                    + "idcst_ipi='"+ p.getIdcst_ipi() +"' "
                    + "idcst_pis='"+ p.getIdcst_pis() +"' "

                    + "WHERE id_produto = '"+ p.getIdProduto() +"';" ;


            Statement stmt = conexao.createStatement();
            stmt.executeUpdate(sql);
            stmt.close();
            return "true";
        } catch (SQLException e) {
            e.printStackTrace();
            return e.getMessage();
        }
    }

wesleypc 27 de jul. de 2011

Alterei o código acima com o PreparedStatement, porem ainda esta acusando o mesmo erro, alguem poderia me ajudar??

String sql = "UPDATE produto "
                    + "SET cod_prod= ? ,"
                    + "cod_barra= ? ,"
                    + "descricao=?,"
                    + "id_grupo=?,"
                    + "id_unidade=?,"
                    + "peso_bruto=?,"
                    + "peso_liquido=?,"
                    + "ncm=?,"
                   + "id_marca=?,"
                   + "lucro_minimo=?,"
                   + "lucro_atacado=?,"
                   + "lucro_promocao=?,"
                   + "lucro_varejo=?,"
                   + "valor_compra=?,"
                    + "valor_minimo=?,"
                   + "valor_atacado=?,"
                    + "valor_promocao=?,"
                    + "valor_venda=?,"
                    + "quant_estoque=?,"
                    + "estoque_minimo=?,"
                    + "frete=?,"
                    + "despesas=?,"
                    + "bloqueado=?,"
                    + "gera_estoque=?,"
                    + "orig_icms=?,"
                    + "cst_icms=?,"
                    + "mod_bc_icms=?,"
                    + "p_red_bc_icms=?,"
                    + "v_bc_icms=?,"
                    + "p_icms=?,"
                    + "v_icms=?,"
                    + "mod_bc_st_icms=?,"
                    + "p_mvast_icms=?,"
                   + "p_red_bc_st_icms=?,"
                   + "v_bc_st_icms=?,"
                   + "p_icms_st=?,"
                    + "v_icms_st=?,"
                    + "cst_pis=?,"
                    + "v_bc_pis=?,"
                    + "p_pis=?,"
                    + "v_pis=?,"
                    + "q_bc_prod_pis=?,"
                    + "v_aliq_prod_pis=?,"
                    + "cst_cofins=?,"
                    + "v_bc_cofins=?,"
                    + "p_cofins=?,"
                    + "v_cofins=?,"
                    + "q_bc_prod_cofins=?,"
                    + "v_aliq_prod_cofins=?,"
                    + "totalizador=?,"
                    + "cst_ipi=?,"
                    + "classe_enquad_ipi=?,"
                    + "cod_enquad_ipi=?,"
                    + "cnpj_prod_ipi=?,"
                    + "cod_selo_contr_ipi=?,"
                    + "quant_selo_contr_ipi=?,"
                    + "tipo_calculo=?,"
                    + "val_bc_ipi=?,"
                    + "aliquota_ipi=?,"
                    + "quant_total_un_padrao_ipi=?,"
                    + "valor_unidade_ipi=?,"
                   + "valor_ipi=?,"
                    + "preco_fabrica=?,"
                    + "referencia=?,"
                    + "numeracao=?, "
                    + "produto_composto=?, "
                    + "id_subgrupo=? "
                    + "idcst_icms=? "
                    + "idmod_bc=? "
                    + "idmod_bc_st=? "
                    + "idcst_cofins=? "
                    + "idcst_ipi=? "
                    + "idcst_pis=? "
                     + "WHERE id_produto = ?";


   PreparedStatement stmt = (PreparedStatement) conexao.prepareStatement(sql);


                   
                   stmt.setString(1, p.getCodProduto());
      stmt.setString(2, p.getCodBarra());
      stmt.setString(3, p.getDescricao());
      stmt.setInt(4, p.getIdGrupo());
      stmt.setInt(5, p.getIdUnidade());
      stmt.setBigDecimal(6, p.getPesoBruto());
      stmt.setBigDecimal(7, p.getPesoLiquido());
      stmt.setString(8, p.getNCM());
      stmt.setInt(9, p.getIdMarca());
      stmt.setBigDecimal(10, p.getLucroMinimo());
      stmt.setBigDecimal(11, p.getLucroAtacado());
      stmt.setBigDecimal(12, p.getLucroPromocao());
      stmt.setBigDecimal(13, p.getLucroVarejo());
      stmt.setBigDecimal(14, p.getValorCompra());
      stmt.setBigDecimal(15, p.getValorMinimo());
      stmt.setBigDecimal(16, p.getValorAtacado());
      stmt.setBigDecimal(17, p.getValorPromocao());
      stmt.setBigDecimal(18, p.getValorVenda());
      stmt.setBigDecimal(19, p.getQuantEstoque());
      stmt.setBigDecimal(20, p.getEstoqueMinimo());
      stmt.setBigDecimal(21, p.getFrete());
      stmt.setBigDecimal(22, p.getDespesas());
      stmt.setBoolean(23, p.isBloqsueado());
      stmt.setBoolean(24, p.isGerasEstoque());
      stmt.setInt(25, p.getOrigICMS());
      stmt.setString(26, p.getCSTICMS());
       stmt.setInt(27, p.getModBCICMS());
      stmt.setBigDecimal(28, p.getpRedBCICMS());
      stmt.setBigDecimal(29, p.getvBCICMS());
      stmt.setBigDecimal(30, p.getpICMS());
      stmt.setBigDecimal(31, p.getvICMS());
      stmt.setInt(32, p.getModBCSTICMS());
      stmt.setBigDecimal(33, p.getpMVASTICMS());
      stmt.setBigDecimal(34, p.getpRedBCSTICMS());
      stmt.setBigDecimal(35, p.getvBCSTICMS());
      stmt.setBigDecimal(36, p.getpICMSST());
      stmt.setBigDecimal(37, p.getvICMSST());
      stmt.setString(38, p.getCSTPIS());
      stmt.setBigDecimal(39, p.getvBCPIS());
      stmt.setBigDecimal(40, p.getpPIS());
      stmt.setBigDecimal(41, p.getvPIS());
      stmt.setBigDecimal(42, p.getqBCProdPIS());
      stmt.setBigDecimal(43, p.getvAliqProdPIS());
      stmt.setString(44, p.getCSTCOFINS());
      stmt.setBigDecimal(45, p.getvBCCOFINS());
      stmt.setBigDecimal(46, p.getpCOFINS());
      stmt.setBigDecimal(47, p.getvCOFINS());
      stmt.setBigDecimal(48, p.getqBCProdCOFINS());
      stmt.setBigDecimal(49, p.getvAliqProdCOFINS());
      stmt.setString(50, p.getTotalizador());
      stmt.setString(51, p.getCSTIPI());
      stmt.setString(52, p.getClasseEnquadIPI());
      stmt.setString(53, p.getCodEnquadIPI());
      stmt.setString(54, p.getCNPJProdIPI());
      stmt.setString(55, p.getCodSeloContrIPI());
      stmt.setString(56, p.getQuantSeloContrIPI());
      stmt.setString(57, p.getTipoCalculo());
      stmt.setBigDecimal(58, p.getValBCIPI());
      stmt.setBigDecimal(59, p.getAliquotaIPI());
      stmt.setBigDecimal(60, p.getQuantTotalUnPadraoIPI());
      stmt.setBigDecimal(61, p.getValorUnidadeIPI());
      stmt.setBigDecimal(62, p.getValorIPI());
      stmt.setBigDecimal(63, p.getPrecoFabrica());
      stmt.setString(64, p.getReferencia());
      stmt.setString(65, p.getNumeracao());
      stmt.setBoolean(66, p.isProdutoComposto());
      stmt.setInt(67, p.getIdSubGrupo());
      stmt.setInt(68, p.getIdcst_icms());
      stmt.setInt(69, p.getIdmod_bc());
      stmt.setInt(70, p.getIdmod_bc_st());
      stmt.setInt(71, p.getIdcst_cofins());
      stmt.setInt(72, p.getIdcst_ipi());
      stmt.setInt(73, p.getIdcst_pis());
      stmt.setInt(74, p.getIdProduto());



            stmt.executeUpdate();
            stmt.close();
            return "true";
        } catch (SQLException e) {
            e.printStackTrace();
            return e.getMessage();
        }

ViniGodoy 27 de jul. de 2011

Esse erro é bem claro. Ele diz que há erro de sintaxe na sua SQL.
Sempre que tiver esse erro, revise sua query pois com certeza houve algum erro de digitação, ou problema na hora do ctrl+c ctrl+v.

Note que esses sete campos:

+ "id_subgrupo=? "  
+ "idcst_icms=? "  
+ "idmod_bc=? "  
+ "idmod_bc_st=? "  
+ "idcst_cofins=? "  
+ "idcst_ipi=? "  
+ "idcst_pis=? "

Estão sem a vírgula os separando.

Outra coisa. Agora que percebi, mas esse tópico nunca foi de Java Básico. Temos um fórum específico para dúvidas de bancos de dados, que é o fórum de persistência.
Vou mover o tópico para lá, mas procure ler com atenção a descrição dos fóruns antes de postar suas dúvidas, ok?

wesleypc 28 de jul. de 2011

Olá, desculpe pela postagem no lugar errado.

Estou com o erro de sintaxe abaixo, revisei meu código porem não consegui verificar onde está o erro, alguem poderia me ajudar?

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ‘WHERE id_produto =‘8’’ at line 1

public String atualiza(Produto p){
        try {
            String sql = "UPDATE produto "
                    + "SET cod_prod='"+ p.getCodProduto() +"',"
                    + "cod_barra='"+ p.getCodBarra() +"',"
                    + "descricao='"+ p.getDescricao() +"',"
                    + "id_grupo='"+ p.getIdGrupo() +"',"
                    + "id_unidade='"+ p.getIdUnidade() +"',"
                    + "peso_bruto='"+ p.getPesoBruto() +"',"
                    + "peso_liquido='"+ p.getPesoLiquido() +"',"
                    + "ncm='"+ p.getNCM() +"',"
                    + "id_marca='"+ p.getIdMarca() +"',"
                    + "lucro_minimo='"+ p.getLucroMinimo() +"',"
                    + "lucro_atacado='"+ p.getLucroAtacado() +"',"
                    + "lucro_promocao='"+ p.getLucroPromocao() +"',"
                    + "lucro_varejo='"+ p.getLucroVarejo() +"',"
                    + "valor_compra='"+ p.getValorCompra() +"',"
                    + "valor_minimo='"+ p.getValorMinimo() +"',"
                    + "valor_atacado='"+ p.getValorAtacado() +"',"
                    + "valor_promocao='"+ p.getValorPromocao() +"',"
                    + "valor_venda='"+ p.getValorVenda() +"',"
                    + "quant_estoque='"+ p.getQuantEstoque() +"',"
                    + "estoque_minimo='"+ p.getEstoqueMinimo() +"',"
                    + "frete='"+ p.getFrete() +"',"
                    + "despesas='"+ p.getDespesas() +"',"
                    + "bloqueado='"+ (p.isBloqsueado()?1:0) +"',"
                    + "gera_estoque='"+ (p.isGerasEstoque()?1:0) +"',"
                    + "orig_icms='"+ p.getOrigICMS() +"',"
                    + "cst_icms='"+ p.getCSTICMS() +"',"
                    + "mod_bc_icms='"+ p.getModBCICMS() +"',"
                    + "p_red_bc_icms='"+ p.getpRedBCICMS() +"',"
                    + "v_bc_icms='"+ p.getvBCICMS() +"',"
                    + "p_icms='"+ p.getpICMS() +"',"
                    + "v_icms='"+ p.getvICMS() +"',"
                    + "mod_bc_st_icms='"+ p.getModBCSTICMS() +"',"
                    + "p_mvast_icms='"+ p.getpMVASTICMS() +"',"
                    + "p_red_bc_st_icms='"+ p.getpRedBCSTICMS() +"',"
                    + "v_bc_st_icms='"+ p.getvBCSTICMS() +"',"
                    + "p_icms_st='"+ p.getpICMSST() +"',"
                    + "v_icms_st='"+ p.getvICMSST() +"',"
                    + "cst_pis='"+ p.getCSTPIS() +"',"
                    + "v_bc_pis='"+ p.getvBCPIS() +"',"
                    + "p_pis='"+ p.getpPIS() +"',"
                    + "v_pis='"+ p.getvPIS() +"',"
                    + "q_bc_prod_pis='"+ p.getqBCProdPIS() +"',"
                    + "v_aliq_prod_pis='"+ p.getvAliqProdPIS() +"',"
                    + "cst_cofins='"+ p.getCSTCOFINS() +"',"
                    + "v_bc_cofins='"+ p.getvBCCOFINS() +"',"
                    + "p_cofins='"+ p.getpCOFINS() +"',"
                    + "v_cofins='"+ p.getvCOFINS() +"',"
                    + "q_bc_prod_cofins='"+ p.getqBCProdCOFINS() +"',"
                    + "v_aliq_prod_cofins='"+ p.getvAliqProdCOFINS() +"',"
                    + "totalizador='"+ p.getTotalizador() +"',"
                    + "cst_ipi='"+ p.getCSTIPI() +"',"
                    + "classe_enquad_ipi='"+ p.getClasseEnquadIPI() +"',"
                    + "cod_enquad_ipi='"+ p.getCodEnquadIPI() +"',"
                    + "cnpj_prod_ipi='"+ p.getCNPJProdIPI() +"',"
                    + "cod_selo_contr_ipi='"+ p.getCodSeloContrIPI() +"',"
                    + "quant_selo_contr_ipi='"+ p.getQuantSeloContrIPI() +"',"
                    + "tipo_calculo='"+ p.getTipoCalculo() +"',"
                    + "val_bc_ipi='"+ p.getValBCIPI() +"',"
                    + "aliquota_ipi='"+ p.getAliquotaIPI() +"',"
                    + "quant_total_un_padrao_ipi='"+ p.getQuantTotalUnPadraoIPI() +"',"
                    + "valor_unidade_ipi='"+ p.getValorUnidadeIPI() +"',"
                    + "valor_ipi='"+ p.getValorIPI() +"',"
                    + "preco_fabrica='"+ p.getPrecoFabrica() +"',"
                    + "referencia='"+ p.getReferencia() +"',"
                    + "numeracao='"+ p.getNumeracao() +"', "
                    + "produto_composto='"+ p.isProdutoComposto() +"', "
                    + "id_subgrupo='"+ p.getIdSubGrupo() +"', "
                    + "idcst_icms='"+ p.getIdcst_icms() +"', "
                    + "idmod_bc='"+ p.getIdmod_bc() +"', "
                    + "idmod_bc_st='"+ p.getIdmod_bc_st() +"', "
                    + "idcst_cofins='"+ p.getIdcst_cofins() +"', "
                    + "idcst_ipi='"+ p.getIdcst_ipi() +"', "
                    + "idcst_pis='"+ p.getIdcst_pis() +"', "
                    + "WHERE id_produto ='"+ p.getIdProduto() +"'" ;



            Statement stmt = conexao.createStatement();
            stmt.executeUpdate(sql);

            
            stmt.close();
            return "true";
        } catch (SQLException e) {
            e.printStackTrace();
            return e.getMessage();
        }
    }

ViniGodoy 28 de jul. de 2011

Por que você voltou atrás no PreparedStatement?

O problema é que provavelmente id_produto é um int. Então, não vai entre aspas simples.
Note que se você tivesse mantido o PreparedStatement, esse problema não teria ocorrido. O PreparedStatement sabe exatamente o tipo do campo, e isso elimina a necessidade de aspas, ou ter que conhecer algum formato específico do banco (separador de decimais ter vírgula ou ponto, ordem dos campos de data, por exemplo).

Aliás, também não daria problema se algum produto seu tiver apostrofe no nome, como “Batatas do Mac Donald’s”, sem o PreparedStatement, sua query vai dar erro.

wesleypc 28 de jul. de 2011

sim o id_produto é um int, tirei as aspas simples como vc flw porem o erro ainda persiste.

“WHERE id_produto =”+ p.getIdProduto() +";" ;

Referente ao PreparedStatement, tirei ele pois estava dando o seguinte erro:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ‘=‘02’,cod_barra= ‘02’,descricao=‘pok21’,id_grupo=8,id_unidade=1,peso_bruto=321.0’ at line 1

public String atualiza(Produto p){
        try {


                     String sql = "UPDATE produto"
                    + "SET cod_prod=?,"
                    + "cod_barra= ?,"
                    + "descricao=?,"
                    + "id_grupo=?,"
                    + "id_unidade=?,"
                    + "peso_bruto=?,"
                    + "peso_liquido=?,"
                    + "ncm=?,"
                   + "id_marca=?,"
                   + "lucro_minimo=?,"
                   + "lucro_atacado=?,"
                   + "lucro_promocao=?,"
                   + "lucro_varejo=?,"
                   + "valor_compra=?,"
                    + "valor_minimo=?,"
                   + "valor_atacado=?,"
                    + "valor_promocao=?,"
                    + "valor_venda=?,"
                    + "quant_estoque=?,"
                    + "estoque_minimo=?,"
                    + "frete=?,"
                    + "despesas=?,"
                    + "bloqueado=?,"
                    + "gera_estoque=?,"
                    + "orig_icms=?,"
                    + "cst_icms=?,"
                    + "mod_bc_icms=?,"
                    + "p_red_bc_icms=?,"
                    + "v_bc_icms=?,"
                    + "p_icms=?,"
                    + "v_icms=?,"
                    + "mod_bc_st_icms=?,"
                    + "p_mvast_icms=?,"
                   + "p_red_bc_st_icms=?,"
                   + "v_bc_st_icms=?,"
                   + "p_icms_st=?,"
                    + "v_icms_st=?,"
                    + "cst_pis=?,"
                    + "v_bc_pis=?,"
                    + "p_pis=?,"
                    + "v_pis=?,"
                    + "q_bc_prod_pis=?,"
                    + "v_aliq_prod_pis=?,"
                    + "cst_cofins=?,"
                    + "v_bc_cofins=?,"
                    + "p_cofins=?,"
                    + "v_cofins=?,"
                    + "q_bc_prod_cofins=?,"
                    + "v_aliq_prod_cofins=?,"
                    + "totalizador=?,"
                    + "cst_ipi=?,"
                    + "classe_enquad_ipi=?,"
                    + "cod_enquad_ipi=?,"
                    + "cnpj_prod_ipi=?,"
                    + "cod_selo_contr_ipi=?,"
                    + "quant_selo_contr_ipi=?,"
                    + "tipo_calculo=?,"
                    + "val_bc_ipi=?,"
                    + "aliquota_ipi=?,"
                    + "quant_total_un_padrao_ipi=?,"
                    + "valor_unidade_ipi=?,"
                   + "valor_ipi=?,"
                    + "preco_fabrica=?,"
                    + "referencia=?,"
                    + "numeracao=?, "
                    + "produto_composto=?, "
                    + "id_subgrupo=?, "
                    + "idcst_icms=?, "
                    + "idmod_bc=?, "
                    + "idmod_bc_st=?, "
                    + "idcst_cofins=?, "
                    + "idcst_ipi=?, "
                    + "idcst_pis=?, "
                     + "WHERE id_produto = ?;";



             PreparedStatement stmt = (PreparedStatement) conexao.prepareStatement(sql);


                   
      stmt.setString(1, p.getCodProduto());
      stmt.setString(2, p.getCodBarra());
      stmt.setString(3, p.getDescricao());
      stmt.setInt(4, p.getIdGrupo());
      stmt.setInt(5, p.getIdUnidade());
      stmt.setBigDecimal(6, p.getPesoBruto());
      stmt.setBigDecimal(7, p.getPesoLiquido());
      stmt.setString(8, p.getNCM());
      stmt.setInt(9, p.getIdMarca());
      stmt.setBigDecimal(10, p.getLucroMinimo());
      stmt.setBigDecimal(11, p.getLucroAtacado());
      stmt.setBigDecimal(12, p.getLucroPromocao());
      stmt.setBigDecimal(13, p.getLucroVarejo());
      stmt.setBigDecimal(14, p.getValorCompra());
      stmt.setBigDecimal(15, p.getValorMinimo());
      stmt.setBigDecimal(16, p.getValorAtacado());
      stmt.setBigDecimal(17, p.getValorPromocao());
      stmt.setBigDecimal(18, p.getValorVenda());
      stmt.setBigDecimal(19, p.getQuantEstoque());
      stmt.setBigDecimal(20, p.getEstoqueMinimo());
      stmt.setBigDecimal(21, p.getFrete());
      stmt.setBigDecimal(22, p.getDespesas());
      stmt.setBoolean(23, p.isBloqsueado());
      stmt.setBoolean(24, p.isGerasEstoque());
      stmt.setInt(25, p.getOrigICMS());
      stmt.setString(26, p.getCSTICMS());
       stmt.setInt(27, p.getModBCICMS());
      stmt.setBigDecimal(28, p.getpRedBCICMS());
      stmt.setBigDecimal(29, p.getvBCICMS());
      stmt.setBigDecimal(30, p.getpICMS());
      stmt.setBigDecimal(31, p.getvICMS());
      stmt.setInt(32, p.getModBCSTICMS());
      stmt.setBigDecimal(33, p.getpMVASTICMS());
      stmt.setBigDecimal(34, p.getpRedBCSTICMS());
      stmt.setBigDecimal(35, p.getvBCSTICMS());
      stmt.setBigDecimal(36, p.getpICMSST());
      stmt.setBigDecimal(37, p.getvICMSST());
      stmt.setString(38, p.getCSTPIS());
      stmt.setBigDecimal(39, p.getvBCPIS());
      stmt.setBigDecimal(40, p.getpPIS());
      stmt.setBigDecimal(41, p.getvPIS());
      stmt.setBigDecimal(42, p.getqBCProdPIS());
      stmt.setBigDecimal(43, p.getvAliqProdPIS());
      stmt.setString(44, p.getCSTCOFINS());
      stmt.setBigDecimal(45, p.getvBCCOFINS());
      stmt.setBigDecimal(46, p.getpCOFINS());
      stmt.setBigDecimal(47, p.getvCOFINS());
      stmt.setBigDecimal(48, p.getqBCProdCOFINS());
      stmt.setBigDecimal(49, p.getvAliqProdCOFINS());
      stmt.setString(50, p.getTotalizador());
      stmt.setString(51, p.getCSTIPI());
      stmt.setString(52, p.getClasseEnquadIPI());
      stmt.setString(53, p.getCodEnquadIPI());
      stmt.setString(54, p.getCNPJProdIPI());
      stmt.setString(55, p.getCodSeloContrIPI());
      stmt.setString(56, p.getQuantSeloContrIPI());
      stmt.setString(57, p.getTipoCalculo());
      stmt.setBigDecimal(58, p.getValBCIPI());
      stmt.setBigDecimal(59, p.getAliquotaIPI());
      stmt.setBigDecimal(60, p.getQuantTotalUnPadraoIPI());
      stmt.setBigDecimal(61, p.getValorUnidadeIPI());
      stmt.setBigDecimal(62, p.getValorIPI());
      stmt.setBigDecimal(63, p.getPrecoFabrica());
      stmt.setString(64, p.getReferencia());
      stmt.setString(65, p.getNumeracao());
      stmt.setBoolean(66, p.isProdutoComposto());
      stmt.setInt(67, p.getIdSubGrupo());
      stmt.setInt(68, p.getIdcst_icms());
      stmt.setInt(69, p.getIdmod_bc());
      stmt.setInt(70, p.getIdmod_bc_st());
      stmt.setInt(71, p.getIdcst_cofins());
      stmt.setInt(72, p.getIdcst_ipi());
      stmt.setInt(73, p.getIdcst_pis());
      stmt.setInt(74, p.getIdProduto());


            stmt.executeUpdate();
            stmt.close();
            return "true";
        } catch (SQLException e) {
            e.printStackTrace();
            return e.getMessage();
        }
    }

ViniGodoy 28 de jul. de 2011

Então, o erro é bem claro, existem ainda problemas na sintaxe do seu SQL.

Verifique:

a) Os tipos estão certos? Cod barra é mesmo String? Note que o statemente tem que estar em conformidade com os tipos do banco de dados, não da sua aplicação;

b) Se não falta vírgulas, ou se não ficou nenhuma aspas simples sobrando;

c) Se não faltou espaço entre uma linha e outra (note que sua SQL, do jeito que está, está com a palavra produto grudada na palavra SET, pois não há espaço nem depois de produto, nem antes de SET);

Não tem nada a ver com o fato de você usar PreparedStatement.

Fazer um código concatenando Strings como vc fez, usando o Statement é muito porco. Fica pouco legível e susceptível a erros. Conserte o que está errado no PreparedStatement, sem voltar a uma alternativa ainda pior.

Criado 16 de julho de 2011

Ultima resposta 28 de jul. de 2011

Respostas 31

Participantes 5

31 Respostas

Topicos relacionados