Java - JAVA WEB - AYUDA POR FAVOR

<div class="card  col-sm-4 mt-4" style=" margin-left:6px;border:2px solid cadetblue;">

    <div class="card-body">

        <form method="POST" action="Controladores" >

            <div class="form-group">

            <lable>IdCliente</lable>

            <input type="text"  name="id" class="form-control">

            </div>

            <div class="form-group">

             <lable>Dni</lable>

             <input type="text"  name="dni" class="form-control">

            </div>

            <div class="form-group">

            <lable>Nombre</lable>

            <input type="text"  name="nom" class="form-control">

            </div>

            <div class="form-group">

            <lable>Direccion</lable>

            <input type="text"  name="direc" class="form-control"></input

            </div>

            <div class="form-group">

            <lable>Estado</lable>

            <input type="text" name="esta" class="form-control">

            </div>

            <div class="form-group">

            <input type="submit" value="Registros" name="accion" class="btn btn-outline-success" class="form-control">

            </div>

        </form>

       </div>

   </div>

</div>

protected void doPost(HttpServletRequest request, HttpServletResponse response)

    throws ServletException, IOException {

         String accion=request.getParameter("accion");

     // String menu=request.getParameter("menu");

     if(accion.equalsIgnoreCase("Registros")){

       id=Integer.parseInt(request.getParameter("id"));

       String dni=request.getParameter("nomb");

       String nombre=request.getParameter("nom");

       String direcc=request.getParameter("direc");

       String estado=request.getParameter("esta");

       cl.setIdcli(id);

       cl.setDni(dni);

       cl.setNombre(nombre);

       cl.setDireccion(direcc);

       cl.setEstado(estado);

       cld.ingresarcli(cl);

      request.getRequestDispatcher("Cliente.jsp").forward(request, response);

      }

}

public int ingresarcli(Cliente c){

    String sql="insert into cliente (idcliente,dni,nombre,direccion,estado)values(?,?,?,?,?)";

        try {

            con=cn.Conexion();

            ps=con.prepareStatement(sql);

            ps.setInt(1,c.getIdcli());

            ps.setString(2,c.getDni());

            ps.setString(3,c.getNombre());

            ps.setString(4,c.getDireccion());

            ps.setString(5,c.getEstado());

            ps.executeUpdate();

        } catch (Exception e) {

        }

        return r;

    }