eliminar registro de la BD desde java EE

jmm87 · #1 (**permalink**) 14/09/2013, 15:17

Hola amigos! estoy intentando borrar un registro de la tabla desde jee y no me funciona. mi idea es borrarlo con respecto al ID de la tabla. se donde esta el error pero no como solucionarlo. se que es algo muy bobo pero recien empiezo. les muestro mi codigo:

clienteDao.java:
aca esta el metodo

Código Java:

Ver originalpackage dao;
 
import clases.cliente;
import conexion.Conexion;
import java.sql.Connection;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.util.ArrayList;
 
public class clienteDao {
 
    //INSERTAR
    public void insertarCliente(Connection con, cliente c) throws SQLException{
        String sql="INSERT INTO cliente(nombre,apellido,dni) values('"+c.getNombre()+"','"+c.getApellido()+"','"+c.getDni()+"')";
        Statement st=con.createStatement();
        st.execute(sql);
        st.close();
 
    }
 
    //BORRAR
    public void borrarCliente(Connection con, cliente c) throws SQLException{
        String sql="DELETE FROM cliente WHERE idCliente="+c.getIdCliente();
        Statement st=con.createStatement();
        st.execute(sql);
        st.close();
    }
 
 
    //MOSTRAR
    public ArrayList<cliente> traerClientes() throws Exception {
 
        String sql= "SELECT * FROM cliente";
 
        Connection con = Conexion.getConnection();
        Statement st=con.createStatement();
        ResultSet rs=st.executeQuery(sql);
 
        ArrayList <cliente> al= new ArrayList<cliente>();
 
 
        while(rs.next()){
            cliente c= new cliente();
            c.setIdCliente(rs.getInt("idCliente"));
            c.setNombre(rs.getString("nombre"));
            c.setApellido(rs.getString("apellido"));
            c.setDni(rs.getString("dni"));
            //agrega todos los clientes al arrayList
            al.add(c);
        }
        //retorna el array lista de clientes
        return al;
    }
 
 
    //MODIFICAR
    public void modificarCliente(){
    }
 
 
    //BUSCAR
    public void buscarCliente(){
 
    }
}

clientes.jsp
listado de cliente (aca es donde funciona el traerCLientes q es otro metodo)

Código Java:

Ver original<%@page contentType="text/html" pageEncoding="UTF-8"%>
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
   "http://www.w3.org/TR/html4/loose.dtd">
 
<html>
    <head>
        <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
        <title>JSP Page</title>
        <link rel="stylesheet" type="text/css" href="estilo.css" />
        <%@page import="dao.clienteDao"%>
        <%@page import="clases.cliente"%>
        <%@page import="java.util.ArrayList"%>
    </head>
    <body>
        <div id="contenido">
            <table width=100%"" border="1">
                <tr>
                    <th><b>NOMBRE</b></th>
                    <th><b>APELLIDO</b></th>
                    <th><b>DNI</b></th>
                    <th><b>accion</b></th>
                </tr>
                <!--Completo tabla con registros de nombre, apellido y dni de los clientes-->
                <%
                clienteDao dao = new clienteDao();
                ArrayList<cliente> a = dao.traerClientes();
                for(int i=0;i<a.size();i++){
                %>
                <tr>
                    <td><%= a.get(i).getNombre()%></td>
                    <td><%= a.get(i).getApellido()%></td>
                    <td><%= a.get(i).getDni()%></td>
                    <td width="5%">
 
                        <form method="POST" action="clienteServlet">
                            <input type="hidden" value="<%= a.get(i).getIdCliente()%>" name="forParametros">
                            <input type="hidden" value="borrarCliente" name="accion">
                            <input type="submit" value="Eliminar" name="accion">
                        </form>
                    </td>
                </tr>
 
                <% } %>
            </table>
        </div>
    </body>
</html>

y el servlet que es el que hace q esto responda(clienteServlet):

Código Java:

Ver originalpackage servlet;
 
import clases.cliente;
import conexion.Conexion;
import dao.clienteDao;
import java.io.IOException;
import java.io.PrintWriter;
import java.sql.Connection;
import java.sql.SQLException;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
 
public class clienteServlet extends HttpServlet {
 
    @Override
    protected void doGet(HttpServletRequest request, HttpServletResponse response)
    throws ServletException, IOException {
 
        response.setContentType("text/html;charset=UTF-8");
        PrintWriter out = response.getWriter();
 
        out.println("<head>");
        out.println("<link rel='stylesheet' type='text/css' href='estilo.css' />");
        out.println("</head>");
 
        out.println("<div id='contenido'>");
 
        out.println("<table width='100%'>");
            //COMIENZA EL FORM DE CONTACTO
            out.println("<form method='POST' action='clienteServlet'>");
 
            out.println("<tr>");
 
                out.println("<td>");
                    out.println("<label>NOMBRE:</label>");
                    out.println("<input name='nombre' type='text'>");
                out.println("</td>");
 
                out.println("<td>");
                    out.println("<label>APELLIDO:</label>");
                    out.println("<input name='apellido' type='text'>");
                out.println("</td>");
 
                out.println("<td>");
                    out.println("<label>DNI:</label>");
                    out.println("<input name='dni' type='text'>");
                out.println("</td>");
 
                out.println("<td>");
                    out.println("<input type='hidden' value='borrarCliente' name='idCliente'>");
                    out.println("<input type='hidden' value='insertarCliente' name='accion'>");
                    out.println("<input type='submit' name='Enviar'>");
                out.println("</td>");
 
            out.println("</form>");
            //TERMINA EL FORM--------------------
 
            out.println("</tr>");
            out.println("</table>");
 
            out.println("<iframe src='clientes.jsp' width='970px' height='500' align='center' frameborder='0'>");
            out.println("</div>");
    }
 
    @Override
    protected void doPost(HttpServletRequest request, HttpServletResponse response)
    throws ServletException, IOException {
 
            response.setContentType("text/html;charset=UTF-8");
            PrintWriter out = response.getWriter();
 
 
            String formParametros = request.getParameter("accion");
            Connection con=null;
 
            if(formParametros !=null){
 
                try {
                    con = Conexion.getConnection();
                } catch (Exception ex) {
 
                    ex.printStackTrace();
                }
 
                clienteDao dao=new clienteDao();
 
                cliente c=new cliente();
 
                c.setIdCliente(request.getParameter("idCLiente"));
                c.setNombre(request.getParameter("nombre"));
                c.setApellido(request.getParameter("apellido"));
                c.setDni(request.getParameter("dni"));
 
                if(formParametros.equals("insertarCliente")){
                    try {
                        dao.insertarCliente(con, c);
                    } catch (SQLException ex) {
                        ex.printStackTrace();
                    }
 
                    if (c == null) {
                        response.sendRedirect("error.jsp");
                    } else {
                        response.sendRedirect("clienteServlet");
                    }
                }
 
                if(formParametros.equals("borrarCliente")){
                    try {
                        dao.borrarCliente(con, c);
                    } catch (SQLException ex) {
                        ex.printStackTrace();
                    }
                }
            }
    }
 
    @Override
    public String getServletInfo() {
        return "Short description";
    }// </editor-fold>
 
}

jmm87 · #2 (**permalink**) 14/09/2013, 15:19

para mi el error esta primero en la linea 36 de clientes.jps y el otro problema que tengo es en la linea 89 del servlet que instancion un cliente pero no le seteo un ID. cuando lo quiero setear me da un error. probe con el debbug y evidentemente cuando pongo eliminar llenar los datos del cliente pero no trae ningun ID.. desde ya al que pueda ayudar muchas gracias!!!

cazador1992 · #3 (**permalink**) 15/09/2013, 12:33

Para eliminar o modificar tienes que usar executeUpdate(), estas usando execute().

El request.getParameter() devuelve una String, si el campo ID del cliente no es String, te dará un error.

jmm87 · #4 (**permalink**) 15/09/2013, 13:15

entonces como hago para traer el ID del cliente?? lo transformo en un string?

cazador1992 · #5 (**permalink**) 15/09/2013, 15:30

Si el campo idCliente es int, lo tienes que pasar a int

c.setIdCliente(Integer.parseInt(request.getParamet er("idCLiente")));

jmm87 · #6 (**permalink**) 15/09/2013, 16:14

si ya habia probado de esa manera pero sigue sin eliminar el registro. evidentemente hay un error en otro lado

jmm87 · #7 (**permalink**) 15/09/2013, 17:23

el problema es que cuando llega al metodo de borrarCliente no esta tomando el ID. porque segun el debbuger llega asi: "DELETE FROM cliente WHERE idCliente=0" cualquier registro que pongo eliminar llega como ID 0

cazador1992 · #8 (**permalink**) 15/09/2013, 18:17

91. c.setIdCliente(request.getParameter("idCLiente"));

¿No sera idCliente?

jmm87 · #9 (**permalink**) 15/09/2013, 19:09

si eso ya lo corregi pero nada.. me estoy volviendo loco y se q es una bobada

cazador1992 · #10 (**permalink**) 15/09/2013, 19:57

37 <input type="hidden" value="<%= a.get(i).getIdCliente()%>" name="forParametros">

Si no me confundo le estas diciendo que el idCliente se llamara forParametros, y en el servlet estas buscando un parámetro llamado idCliente

91. c.setIdCliente(request.getParameter("idCLiente"));

prueba con forParametros.

jmm87 · #11 (**permalink**) 15/09/2013, 20:39

java.lang.UnsupportedOperationException: Not yet implemented
clases.cliente.setIdCliente(cliente.java:61)
servlet.clienteServlet.doPost(clienteServlet.java: 89)
javax.servlet.http.HttpServlet.service(HttpServlet .java:643)
javax.servlet.http.HttpServlet.service(HttpServlet .java:723

lo de formParametros creo q tienes razon. pero sigue habiendo un problema al setear el id y no se porque:S

chuidiang · #12 (**permalink**) 16/09/2013, 03:15

Hola:

Lo de UnsupportedOperationException "Not yet implemented" es una excepción que sueltan algunas clases java cuando el código todavía no se ha implementado en ese método. Revisa el código de clases.cliente a ver qué tiene en ese método setIdCliente.

Se bueno.

jmm87 · #13 (**permalink**) 16/09/2013, 07:04

Clase cliente

Código Java:

Ver originalpackage clases;
 
public class cliente {
 
    int idCliente;
    String Nombre;
    String Apellido;
    String Dni;
 
    //constructores
    public cliente(int idCliente, String Nombre, String Apellido, String Dni) {
        this.idCliente = idCliente;
        this.Nombre = Nombre;
        this.Apellido = Apellido;
        this.Dni = Dni;
    }
 
    public cliente() {
 
    }
 
    
 
    
 
    //get a set
    public String getApellido() {
        return Apellido;
    }
 
    public void setApellido(String Apellido) {
        this.Apellido = Apellido;
    }
 
    public String getDni() {
        return Dni;
    }
 
    public void setDni(String Dni) {
        this.Dni = Dni;
    }
 
    public String getNombre() {
        return Nombre;
    }
 
    public void setNombre(String Nombre) {
        this.Nombre = Nombre;
    }
 
    public int getIdCliente() {
        return idCliente;
    }
 
    public void setIdCliente(int idCliente) {
        this.idCliente = idCliente;
    }
 
 
 
 
 
 
 
}

chuidiang · #14 (**permalink**) 16/09/2013, 09:12

Hola:

El error habla de línea 61 en cliente.java

clases.cliente.setIdCliente(cliente.java:61)

¿Esa versión que has puesto de cliente.java corresponde con la del error? La línea 61 no corresponde al método setIdCliente.

En la excepción que te sale, ¿hay algo relativo al error delante de "java.lang.UnsupportedOperationException: Not yet implemented"?

Se bueno.

jmm87 · #15 (**permalink**) 16/09/2013, 10:45

muchachos lo soluciones! gracias a que ustedes me fueron orientando donde estaba el error. lo volvi a hacer el proyecto y funciono..
aparentemente uno de los errores esta en :

Código Java:

Ver originalc.setIdCliente(Integer.parseInt(request.getParameter("idCliente")));

que nop lo estaba tomando como un INT

y el otro error estaba en

Código Java:

Ver original<input type="text" value="<%= a.get(i).getIdCliente()%>" name="idCliente" readonly>

que en name decia ACCION

LES AGRADEZCO A TODOS Y ESPERO QUE SIRVA PARA OTRA GENTE TAMBIEN

Isakmtz · #16 (**permalink**) 18/09/2013, 18:12

Solo como nota, coincido con Cazador1922, para eliminar o modificar se debe usar executeUpdate() en lugar del execute().