[EDIT] PersonaSqlserverDAO - ResponseHelper

src/java/demojsoncrud/sqlserverdao/PersonaSqlserverDAO.java

@@ -22,11 +22,6 @@ import org.json.JSONObject;
  */
 public class PersonaSqlserverDAO implements PersonaDAO {
 
-    public static void main(String[] args) {
-        PersonaSqlserverDAO personaSqlserverDAO = new PersonaSqlserverDAO();
-        System.out.println(personaSqlserverDAO.listarPersonaTest());
-    }
-
     @Override
     public JSONObject listarPersonaTest() {
 
@@ -73,7 +68,7 @@ public class PersonaSqlserverDAO implements PersonaDAO {
             response.setStatus(true);
 
         } catch (SQLException e) {
-            response.setMensaje("Error -> " + e.getMessage() + " [" + e.getErrorCode() + "]");
+            response.setMessage("Error -> " + e.getMessage() + " [" + e.getErrorCode() + "]");
             response.setStatus(false);
         } finally {
             try {
@@ -87,12 +82,13 @@ public class PersonaSqlserverDAO implements PersonaDAO {
                     connection.close();
                 }
             } catch (SQLException e) {
-                response.setMensaje("Error -> " + e.getMessage() + " [" + e.getErrorCode() + "]");
+                response.setMessage("Error -> " + e.getMessage() + " [" + e.getErrorCode() + "]");
                 response.setStatus(false);
             }
         }
-        
+
         jsonReturn = new JSONObject(response);
+
         return jsonReturn;
 
     }
@@ -181,7 +177,7 @@ public class PersonaSqlserverDAO implements PersonaDAO {
                     connection.close();
                 }
             } catch (SQLException e) {
-                jsonReturn.put("Error", "Error inesperado -> " + e.getMessage() + " Code: " + e.getErrorCode());
+                jsonReturn.put("Error", "Error inesperado -> " + e.getMessage() + " [" + e.getErrorCode() + "]");
             }
         }
 
@@ -190,7 +186,7 @@ public class PersonaSqlserverDAO implements PersonaDAO {
 
     @Override
     public JSONObject registrarPersona(PersonaBean bean) {
-        JSONObject jsonReturn = null;
+        JSONObject jsonReturn = new JSONObject();
         int resultDni = 0;
         int resultPersona = 0;
         String base = "demojsoncrud";
@@ -220,25 +216,21 @@ public class PersonaSqlserverDAO implements PersonaDAO {
                 psInsertarPersona.setString(c++, bean.getCorreo());
                 resultPersona = psInsertarPersona.executeUpdate();
                 if (resultPersona == 1) {
-                    response.setMensaje("Se registro correctamente");
-                    response.setResultado("1"); // REGISTRO
+                    response.setMessage("Registro correcto!");
+                    response.setStatus(true);
                 } else {
-                    response.setMensaje("Error al registrar");
-                    response.setResultado("0"); // ERROR
+                    response.setMessage("Error al registrar");
+                    response.setStatus(false); // ERROR
                 }
 
             } else {
-                response.setMensaje("El número de documento ya existe");
-                response.setResultado("2"); // DNI REPETIDO
+                response.setMessage("El documento ya existe en mi base de datos. Prueba otro.");
+                response.setStatus(false); // DNI REPETIDO
             }
 
-            jsonReturn = new JSONObject(response);
-
         } catch (SQLException e) {
-
-            response.setResultado("-1");
-            response.setMensaje("Error al registrar -> " + e.getMessage() + "[" + e.getErrorCode() + "]");
-            jsonReturn = new JSONObject(response);
+            response.setStatus(false);
+            response.setMessage("Error al registrar -> " + e.getMessage() + "[" + e.getErrorCode() + "]");
         } finally {
             try {
                 if (connection != null) {
@@ -248,9 +240,13 @@ public class PersonaSqlserverDAO implements PersonaDAO {
                     psInsertarPersona.close();
                 }
             } catch (SQLException e) {
-                System.out.println("Error inesperado registrarPersona -> " + e.getMessage());
+                response.setStatus(false);
+                response.setMessage("Error al registrar -> " + e.getMessage() + "[" + e.getErrorCode() + "]");
             }
         }
+
+        jsonReturn = new JSONObject(response);
+
         return jsonReturn;
     }
 
@@ -274,28 +270,25 @@ public class PersonaSqlserverDAO implements PersonaDAO {
                     + "where codigo = ?";
 
             psEditarPersona = connection.prepareStatement(sql);
-            int c = 1; // CONTADOR PARA LAS COLUMNAS
-            psEditarPersona.setString(c++, bean.getApellidos());
-            psEditarPersona.setString(c++, bean.getNombres());
-            psEditarPersona.setString(c++, bean.getEdad());
-            psEditarPersona.setString(c++, bean.getCorreo());
-            psEditarPersona.setString(c++, bean.getCodigo());
+            int q = 1; // CONTADOR PARA LAS COLUMNAS
+            psEditarPersona.setString(q++, bean.getApellidos());
+            psEditarPersona.setString(q++, bean.getNombres());
+            psEditarPersona.setString(q++, bean.getEdad());
+            psEditarPersona.setString(q++, bean.getCorreo());
+            psEditarPersona.setString(q++, bean.getCodigo());
             resultPersona = psEditarPersona.executeUpdate();
 
             if (resultPersona == 1) {
-                response.setMensaje("Se actualizo correctamente");
-                response.setResultado("1"); // ACTUALIZO
+                response.setMessage("Se actualizo correctamente");
+                response.setStatus(true); // ACTUALIZO
             } else {
-                response.setMensaje("Error al actualizar");
-                response.setResultado("0"); // ERROR
+                response.setMessage("Error al actualizar");
+                response.setStatus(false); // ERROR
             }
 
-            jsonReturn = new JSONObject(response);
-
         } catch (SQLException e) {
-            response.setResultado("-1");
-            response.setMensaje("Error al actualizar -> " + e.getMessage() + "[" + e.getErrorCode() + "]");
-            jsonReturn = new JSONObject(response);
+            response.setStatus(false);
+            response.setMessage("Error al actualizar -> " + e.getMessage() + "[" + e.getErrorCode() + "]");
         } finally {
             try {
                 if (connection != null) {
@@ -305,10 +298,13 @@ public class PersonaSqlserverDAO implements PersonaDAO {
                     psEditarPersona.close();
                 }
             } catch (SQLException e) {
-                System.out.println("Error inesperado editarPersona -> " + e.getMessage());
+                response.setStatus(false);
+                response.setMessage("Error al actualizar -> " + e.getMessage() + "[" + e.getErrorCode() + "]");
             }
         }
 
+        jsonReturn = new JSONObject(response);
+
         return jsonReturn;
     }
 
@@ -327,26 +323,23 @@ public class PersonaSqlserverDAO implements PersonaDAO {
                     = "update persona "
                     + "set estado = 1 "
                     + "where codigo = ?";
-            System.out.println(sql);
+
             psEditarPersona = connection.prepareStatement(sql);
-            int c = 1; // CONTADOR PARA LAS COLUMNAS
-            psEditarPersona.setString(c++, bean.getCodigo());
+
+            psEditarPersona.setString(1, bean.getCodigo());
             resultPersona = psEditarPersona.executeUpdate();
 
             if (resultPersona == 1) {
-                response.setMensaje("Se actualizo correctamente");
-                response.setResultado("1"); // ACTUALIZO
+                response.setMessage("Se actualizo correctamente");
+                response.setStatus(true); // ACTUALIZO
             } else {
-                response.setMensaje("Error al actualizar");
-                response.setResultado("0"); // ERROR
+                response.setMessage("Error al actualizar");
+                response.setStatus(false); // ERROR
             }
 
-            jsonReturn = new JSONObject(response);
-
         } catch (SQLException e) {
-            response.setResultado("-1");
-            response.setMensaje("Error al actualizar -> " + e.getMessage() + "[" + e.getErrorCode() + "]");
-            jsonReturn = new JSONObject(response);
+            response.setStatus(false);
+            response.setMessage("Error al actualizar -> " + e.getMessage() + "[" + e.getErrorCode() + "]");
         } finally {
             try {
                 if (connection != null) {
@@ -356,10 +349,13 @@ public class PersonaSqlserverDAO implements PersonaDAO {
                     psEditarPersona.close();
                 }
             } catch (SQLException e) {
-                System.out.println("Error inesperado activarPersona -> " + e.getMessage());
+                response.setStatus(false);
+                response.setMessage("Error al actualizar -> " + e.getMessage() + "[" + e.getErrorCode() + "]");
             }
         }
 
+        jsonReturn = new JSONObject(response);
+
         return jsonReturn;
     }
 
@@ -378,26 +374,22 @@ public class PersonaSqlserverDAO implements PersonaDAO {
                     = "update persona "
                     + "set estado = 0 "
                     + "where codigo = ?";
-            System.out.println(sql);
+
             psEditarPersona = connection.prepareStatement(sql);
-            int c = 1; // CONTADOR PARA LAS COLUMNAS
-            psEditarPersona.setString(c++, bean.getCodigo());
+            psEditarPersona.setString(1, bean.getCodigo());
             resultPersona = psEditarPersona.executeUpdate();
 
             if (resultPersona == 1) {
-                response.setMensaje("Se actualizo correctamente");
-                response.setResultado("1"); // ACTUALIZO
+                response.setMessage("Se actualizo correctamente");
+                response.setStatus(true); // ACTUALIZO
             } else {
-                response.setMensaje("Error al actualizar");
-                response.setResultado("0"); // ERROR
+                response.setMessage("Error al actualizar");
+                response.setStatus(false); // ERROR
             }
 
-            jsonReturn = new JSONObject(response);
-
         } catch (SQLException e) {
-            response.setResultado("-1");
-            response.setMensaje("Error al actualizar -> " + e.getMessage() + "[" + e.getErrorCode() + "]");
-            jsonReturn = new JSONObject(response);
+            response.setStatus(false);
+            response.setMessage("Error al actualizar -> " + e.getMessage() + "[" + e.getErrorCode() + "]");
         } finally {
             try {
                 if (connection != null) {
@@ -407,10 +399,12 @@ public class PersonaSqlserverDAO implements PersonaDAO {
                     psEditarPersona.close();
                 }
             } catch (SQLException e) {
-                System.out.println("Error inesperado registrarPersona -> " + e.getMessage());
+                response.setStatus(false);
+                response.setMessage("Error al actualizar -> " + e.getMessage() + "[" + e.getErrorCode() + "]");
             }
         }
 
+        jsonReturn = new JSONObject(response);
         return jsonReturn;
     }
 

src/java/demojsoncrud/utils/ResponseHelper.java

@@ -13,9 +13,6 @@ import java.io.Serializable;
  */
 public class ResponseHelper implements Serializable {
 
-    private String resultado;
-    private String mensaje;
-
     private Object data;
     private boolean status;
     private String message;
@@ -40,22 +37,6 @@ public class ResponseHelper implements Serializable {
         this.message = message;
     }
 
-    public String getResultado() {
-        return resultado;
-    }
-
-    public void setResultado(String resultado) {
-        this.resultado = resultado;
-    }
-
-    public String getMensaje() {
-        return mensaje;
-    }
-
-    public void setMensaje(String mensaje) {
-        this.mensaje = mensaje;
-    }
-
     public Object getData() {
         return data;
     }

web/js/pages/persona.js

@@ -17,7 +17,7 @@ $(document).ready(function () {
     $('#btn_cerar_registrar_persona').on('click', function () {
         $("form_registrar_persona").validate().resetForm();
     });
-    
+
     filtrarPersona();
 });
 
@@ -240,7 +240,7 @@ function listarPersona() {
         var json = {
             codigo: data_row.codigo
         };
-
+        console.log(data_row.codigo);
         swal({
             title: "¿Estás seguro?",
             text: "¿Desea desactivar a " + data_row.apellidos + ", " + data_row.nombres + " ?",
@@ -277,7 +277,7 @@ function listarPersona() {
         var json = {
             codigo: data_row.codigo
         };
-
+        console.log(data_row.codigo);
         swal({
             title: "¿Estás seguro?",
             text: "¿Desea activar a " + data_row.apellidos + ", " + data_row.nombres + " ?",