0

Obs: Várias pessoas não conseguiram resolver essa pergunta e marcaram minha pergunta como erroneamente duplicada por causa de uma resposta em uma outra pergunta parecida com essa mas a resposta não me ajudo em nada.

Estou desenvolvendo um software para uma videolocadora com design pattern MVC (Model-view-controller) e está ocorrendo o problema de não alterar os atributos das pessoaa. Quando vou alterar clicando no meu botão salvar para salvar minhas alterações exibe a mensagem "alterado com sucesso" mas não altera realmente!

Classe VideoPessoa incluindo os botoes de salvar e de alterar, da camada view:

 public VideoPessoa() {
            initComponents();

            new Conexao();
            pessoaController = new PessoaController();
            pessoa = new Pessoa();
            this.carregarPessoas();
             this.novaPessoa();
            this.habilitarCampos();

        }

    public boolean alterarPessoa() {
            pessoa.setCodigo( Integer.parseInt(this.txtCodigo.getText()));
            pessoa.setNome(this.txtNome.getText());
            pessoa.setEndereco(this.txtEndereco.getText());
            pessoa.setBairro(this.txtBairro.getText());
            pessoa.setCPF(this.txtCPF.getText());
            pessoa.setSexo(this.txtSexo.getText());
            pessoa.setUf(this.txtUF.getText());
            pessoa.setCelular(this.txtCelular.getText());
            pessoa.setTelefone(this.txtTelefone.getText());
            pessoa.setCidade(this.txtCidade.getText());

            if (pessoaController.alterar(pessoa)) {

                JOptionPane.showMessageDialog(this, "Registro alterado com sucesso!");
                 this.desabilitarCampos();
                this.carregarPessoas();
            } else {

                JOptionPane.showMessageDialog(this, "Erro ao alterar os dados!", "ERRO", JOptionPane.ERROR_MESSAGE);

            }

            return true;

        }

private void btnSalvarActionPerformed(java.awt.event.ActionEvent evt) {                                          
        // TODO add your handling code here:

      if (!alterarPessoa){
              salvarPessoa();
          } else {
              alterarPessoa();
          }

    }                                         

    private void btnAlterarActionPerformed(java.awt.event.ActionEvent evt) {                                           
        // TODO add your handling code here:
       habilitarCampos();
     recuperarPessoas();
     alterarPessoa = true;

    }                     

Classe PessoaController:

public class PessoaController {

    private final PessoaDAO pessoaDAO;

    public PessoaController() {
        pessoaDAO = new PessoaDAO();

    }

 public boolean alterar( Pessoa pessoa ) {
        boolean retorno;

        retorno = pessoaDAO.alterar(pessoa);

        System.out.println("Pessoa: "+pessoa); 

        return retorno;
    }

Classe PessoaDAO, incluindo o método de alterar pessoas:

public class PessoaDAO {

    private Connection con;

    private final String SQLSELECT = " SELECT  codigo, nome, endereco, bairro, sexo, telefone, celular, CPF,  uf, cidade FROM PESSOA";

    private final String SQLUPDATE = " UPDATE pessoa"
            + " SET nome = ?, "
            + " endereco = ?, "
            + " bairro   = ?, "
            + " sexo     = ?, "
            + " telefone = ?, "
            + " celular  = ?, "
            + " CPF      = ?, "
            + " uf       =?, "
            + " cidade   =? "
            + " WHERE codigo = ?";

    private PreparedStatement sqlSelect, sqlUpdate;

    public PessoaDAO() {

        con = Conexao.getConnection();
        try {
            sqlSelect = con.prepareStatement(SQLSELECT);
            sqlUpdate = con.prepareStatement(SQLUPDATE);
        } catch (SQLException ex) {
            Logger.getLogger(PessoaDAO.class.getName()).log(Level.SEVERE, null, ex);
        }

    }

    public Pessoa getPessoaPeloCodigo(int codigo) {

        Pessoa pessoa = null;
 try (
            PreparedStatement ps = con.prepareStatement(SQLPESSOAPELOCODIGO)
        ) {
            ps.setInt(1, codigo);
            try (ResultSet rs = ps.executeQuery()) {
                if (!rs.next()) return null; // Não encontrou.
                // Instancia a nova pessoa.
                //Instancia o novo filme
                pessoa = new Pessoa();

                //Seta as informações no filme
                pessoa.setCodigo(rs.getInt("codigo"));
                pessoa.setNome(rs.getString("nome"));
                pessoa.setEndereco(rs.getString("endereco"));
                pessoa.setBairro(rs.getString("bairro"));
                pessoa.setSexo(rs.getString("sexo"));
                pessoa.setTelefone(rs.getString("telefone"));
                pessoa.setCelular(rs.getString("celular"));
                pessoa.setCPF(rs.getString("CPF"));

                pessoa.setUf(rs.getString("uf"));
                pessoa.setCidade(rs.getString("cidade"));

            }
        } catch (SQLException ex) {
            Logger.getLogger(PessoaDAO.class.getName()).log(Level.SEVERE, null, ex);
        }
        return pessoa;
    }

public boolean alterar(Pessoa pessoa) {
        boolean retorno = false;
        try {
            sqlUpdate.setInt(1, pessoa.getCodigo());
            sqlUpdate.setString(2,pessoa.getNome());
            sqlUpdate.setString(3, pessoa.getEndereco());
            sqlUpdate.setString(4, pessoa.getBairro());
            sqlUpdate.setString(5, pessoa.getSexo());
            sqlUpdate.setString(6, pessoa.getTelefone());
            sqlUpdate.setString(7, pessoa.getCelular());
            sqlUpdate.setString(8, pessoa.getCPF());
            sqlUpdate.setString(9, pessoa.getUf());
            sqlUpdate.setString(10, pessoa.getCidade());


            sqlUpdate.executeUpdate();

            retorno = true;

        } catch (SQLException ex) {
            Logger.getLogger(PessoaDAO.class.getName()).log(Level.SEVERE, null, ex);
        }
        return retorno;
    }
9
  • 5
    Verificou o autocommit do banco se está ativado?, Relacionada: prepared statement não funciona ou como habilitar no workbench
    – rray
    27/10/2015 às 18:52
  • @rray verifiquei sim e está ativado! Esses dois links não resolveu minha pergunta. 28/10/2015 às 16:37
  • @rray Verifiquei e estava ligado sim e esses seus dois links não resolveu meu problema! 28/10/2015 às 16:42
  • No seu alterar, mude sqlUpdate.executeUpdate(); por : sqlUpdate.executeUpdate(SQLUPDATE);
    – rray
    28/10/2015 às 16:50
  • Deu erro : com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: 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 '?, endereco = ?, bairro = ?, sexo = ?, telefone = ?, celular = ?, C' at line 1 28/10/2015 às 17:07

1 Resposta 1

0

Parece que os parâmetros passados não estão na ordem certa para realizar o update, veja:

UPDATE pessoa SET
    nome = ?,    <--- 1
    endereco = ?,<--- 2
    bairro   = ?,<--- 3
    sexo     = ?,<--- 4
    telefone = ?,<--- 5
    celular  = ?,<--- 6
    CPF      = ?,<--- 7
    uf       = ?,<--- 8
    cidade   = ? <--- 9
WHERE codigo = ? <--- 10

Olhe os binds agora, o codigo deve ser o número 10 e não número 1, pois a primeira interrogação faz refêrencia ao campo nome

sqlUpdate.setInt(1, pessoa.getCodigo());
sqlUpdate.setString(2,pessoa.getNome());
sqlUpdate.setString(3, pessoa.getEndereco());
sqlUpdate.setString(4, pessoa.getBairro());
sqlUpdate.setString(5, pessoa.getSexo());
sqlUpdate.setString(6, pessoa.getTelefone());
sqlUpdate.setString(7, pessoa.getCelular());
sqlUpdate.setString(8, pessoa.getCPF());
sqlUpdate.setString(9, pessoa.getUf());
sqlUpdate.setString(10, pessoa.getCidade());

O código final deve ficar assim:

sqlUpdate.setString(1,pessoa.getNome());
sqlUpdate.setString(2, pessoa.getEndereco());
sqlUpdate.setString(3, pessoa.getBairro());
sqlUpdate.setString(4, pessoa.getSexo());
sqlUpdate.setString(5, pessoa.getTelefone());
sqlUpdate.setString(6, pessoa.getCelular());
sqlUpdate.setString(7, pessoa.getCPF());
sqlUpdate.setString(8, pessoa.getUf());
sqlUpdate.setString(9, pessoa.getCidade());
sqlUpdate.setInt(10, pessoa.getCodigo());
int linhasAfetadas = sqlUpdate.executeUpdate(SQLUPDATE);

if(linhasAfetadas > 0){
   return true;
}else{
   return false;
} 
8
  • Fiz como você falou e não deu certo! 28/10/2015 às 17:48
  • @IgorContini, Você trocou todos os indíces? qual erro apareceu?
    – rray
    28/10/2015 às 17:48
  • Caiu a internet aqui.. Coloquei o código final e o sqlUpdate.executeUpdate(SQLUPDATE); ficou sublinhado com linha vermelha indicando erro e exibindo a mensagem: unreachable statement! 28/10/2015 às 18:08
  • @IgorContini, veja agora, qualquer coisa remova o if-else para testar.
    – rray
    28/10/2015 às 18:21
  • Troquei todos os índices a 1 até o 10 e o código ficou com o índice 10. 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 '?, endereco = ?, bairro = ?, sexo = ?, telefone = ?, celular = ?, C' at line 1 ` 28/10/2015 às 18:24

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