[SOLUCIONADO] Excepciones y base de datos

[detective_jd]

Hola a todos, tengo la siguiente consulta, resulta que trato de no trabajar en memoria con la base de datos, es decir no tener una total dependencia a las colecciones, y hasta para crear y editar lo logro, estoy trabajando con una solución genérica, como ven:

Manejadora.java

Código Java:

public void add(T objeto)throws Exception {
        if(this.checkUniqueKey(objeto)){  
            throw new Exception(this.getFetchMessage());
        }
        else {
            FacadeBroker.getInstance().guardame(this.getDates(objeto), this.getTable());
        }
    }
    public void edit(T objeto)throws Exception {
        T aux = this.findById(Conversiones.CInt(getDates(objeto)[0]));
        if(!aux.equals(objeto)){
            if(this.checkUniqueKey(objeto)){  
                throw new Exception(this.getFetchMessage());
            }
        }
        FacadeBroker.getInstance().modificame(this.getDates(objeto), this.getTable());        
    }
 

pero cuando se trata de borrar, no sé cómo poner la excepción en esta solución genérica:

Código Java:

public void delete(T objeto)throws Exception {
        try {
            FacadeBroker.getInstance().eliminame(this.getTable(), this.getDeleteCondition(objeto)); 
        }
        catch(Exception ex){
            System.out.println(ex.getMessage());
        }
    }
 
protected boolean checkDelete(){ return false; }
protected String getDeleteCondition(T objeto) { return null; }
protected String getTable() { return null; }
 

ManCargos.java

Código Java:

@Override
    protected String getTable() {
        return "cargos";
    }
@Override
    protected boolean checkDelete() {
        return true;
    }
@Override
    protected String getDeleteCondition(Cargo objeto) {
        String query = " where carId =" +objeto.getId();
        if(this.checkDelete()){
            query +=" and carId not in (select distinct carId from personas)";
        }
        return query;
    }
 

La idea que las cargos no se borren si están asignados a personas, pero ¿cómo trato la excepción para el caso de las bajas?

Espero sus respuestas y saludos.

[detective_jd]

Hola, me pueden dar alguna de de cómo lanzar la excepción cuando el cargo esta siendo uso por una persona?? la consulta sql anda, pero el hacer andar el mensaje es el problema.

alguna ayuda? por favor.

Espero sus respuestas y saludos.

[Darkmarus]

mm no podrías consultar si el cargo esta usado en un persona no elimine y lance el mensaje dado que ahi te traería null? y cuando esta eliminando por alguna cuestión alguien lo uso en una persona, así que le pones en un trycatch para captar el error.
(no se si te sirve)

[detective_jd]

Hola Darkmarus, gracias por responder, verás en la clase MyBroker es que tengo la consulta:

MyBroker.java

Código Java:

@Override
    public void eliminame (String nombre, String condicion) {
        try {
            objBD.comando("delete from " + nombre + condicion);
        }
        catch (Exception e) {
           System.out.println(e.getMessage());
        }
    }
 

y en MyConex.java tengo lo del PreparedStatement

MyConex.java

Código Java:

public void comando(String comando) 
    {
    try 
        {
            query =objconex.conexion().prepareStatement(comando); 
            query.executeUpdate();
            query.close();
    } 
        catch (SQLException e) 
        {
            System.out.println(e.getMessage());
    }
    }
 

sólo que en Manejadora es que hago las validaciones, no sé en base a tu sugerencia como hago para crear la excepción como lo hago en add y edit??

Espero sus respuestas y saludos.

[Darkmarus]

mm voy a intentar hacer de esa forma XD dado que he trabajado mas usando plugin que de manera genérica, si me sale bien mi planteamiento te la respondo.

[Darkmarus]

mm no se como intentas eliminar dado que la base de datos te aria saltar la excepcion por intentar borrar una row utilizada en otra tabla, eso es integridad de la base de datos asi que con un catch capturas el error en esa ejecución

Código Java:

public void eliminar(long id) throws SQLException {
 
        String sql = "delete from encabezado_ingreso where id_ingreso=";
        sql += id;
        PreparedStatement ps = null;
        try {
            ps = conn.prepareStatement(sql);
            ps.executeUpdate();
        } catch (SQLException ex) {
            System.out.println("Error al borrar");
        }finally{
            if (ps!=null) {
                ps.close();
            }
        }
}
 

bueno aunque para obviar este tipo de problemas casi nunca se borra de la base de datos solo se hace una eliminación lógica con un tipo booleano como atributo
(me disculpo por no haber respondido ayer, me entretuve con un código en C# )

[Darkmarus]

y en la misma excepcion te dice el error

[detective_jd]

Ok, no te preocupes, pensando un poco en mi código lo que debería tener es algo que me indique si mi consulta me devuelve algo o no, debería ser una función booleana, y en el delete ponerlo antes del delete, pero cómo??

espero sus respuestas y saludos.

[detective_jd]

hola Darkmarus, finalmente pude arreglarlo, resulta que tenía que hacer esto:

En MyConex.java tenía que cambiar el método a una función que devuelva un entero:

Código Java:

public int comando(String comando) {
    try {
            query =objconex.conexion().prepareStatement(comando); 
            int i = query.executeUpdate();
            query.close();
            return i;
    } 
        catch (SQLException e) 
        {
            System.out.println(e.getMessage());
            return 0;
    }
    }
 

luego en MyBroker.java debía cambiar el método a una función booleana:

Código Java:

@Override
    public boolean eliminame (String nombre, String condicion) {
        try {
            return objBD.comando("delete from " + nombre + condicion) == 1;
        }
        catch (Exception e) {
           System.out.println(e.getMessage());
           return false;
        }
    }
 

al igual que en FacadeBroker.java

Código Java:

@Override
    public boolean eliminame(String nombre,String condicion) {
        return this.objB.eliminame(nombre, condicion); 
    }
 

y en IBroker.java

Código Java:

boolean eliminame (String nombre, String condicion);
 

Después en Manejadora.java tenía que cambiar el método y agregar un string para el mensaje de la excepción:

Código Java:

public void delete(T objeto)throws Exception {
        if(!FacadeBroker.getInstance().eliminame(this.getTable(), this.getDeleteCondition(objeto))){
            throw new Exception(this.getDeleteMessage());
        }
    }
 
protected String getDeleteMessage() { return null; }
 

Por último en ManCargos.java redefiní el mensaje de baja:

Código Java:

@Override
    protected String getDeleteMessage() {
        return "Tiene Personas usando ese Cargo";
    }
 

Y así me anduvo esto, muchas gracias, aunque este tema es Excepciones y Base de Datos.
Hay otra cosa que quiero preguntarte, pero en el sgte mensaje.

[detective_jd]

Mira lo otro que quiero consultar y preguntar es sobre suministro de valores para los parámetros del PreparedStatement: ¿esto es necesario o no? porque en php si usas pdo es necesario, pero te pregunto en java en que le aporta, te pongo el link de esto:

http://programacion.net/articulo/acc...ases_de_datos_[jdbc]_93/8#suminstrar

Espero sus respuestas y saludos.

PD: ¿cómo pones las imagenes en los mensajes de foros del web?

[Darkmarus]

creo que este articulo responde a tu pregunta

<a>http://www.tecnologiadigerida.com/2008/04/uso-correcto-del-preparedstatement</a>

[URL="http://www.tecnologiadigerida.com/2008/04/uso-correcto-del-preparedstatement"]http://www.tecnologiadigerida.com/2008/04/uso-correcto-del-preparedstatement[/URL]

[detective_jd]

Hola Darkmarus, gracias por responder verás intento hacer genérico la parte de los parámetros pero me da estos errores:

insert into cargos (carId,carNombre) values (?,?)
Parameter index out of range (0 < 1 ).
Parameter index out of range (1 > number of parameters, which is 0).

El código de eso está en la clase MyBroker.java

Código Java:

public void guardame(ArrayList<String> rows, String nombre) {
        try {
            this.objBD.getQuery().getConnection().prepareStatement(this.sqlInsert(nombre));
            this.doInsert(rows);
            this.objBD.getQuery().execute();
        }
        catch(Exception ex){
            System.out.println(ex.getMessage());
        }
    }
    private String sqlInsert(String nombre){
        ArrayList<String> columnas=objBD.obtenerColumnas(objBD.obtenerResultSet("select * from "+nombre));
        String Consulta = "insert into " + nombre + " (";
            for (String columna : columnas) {
                if (Consulta.endsWith("(")==false)
                    Consulta += ",";
                Consulta += columna;
            }
            Consulta += ") values (" ;
            for (String columna : columnas) {
                if (Consulta.endsWith("(")==false )
                    Consulta += ",";
                Consulta += "?";
            }
            Consulta += ")";
            System.out.println(Consulta);
            return Consulta;
    }
    private void doInsert(ArrayList<String> rows){
        for(int i =0 ; i < rows.size(); i++){
            try { 
                objBD.getQuery().setString(i, rows.get(i));                
            } 
            catch (SQLException ex) {
                System.out.println(ex.getMessage());
            }
        }
    }
 

Espero sus respuestas y saludos.

[detective_jd]

Hola a todos, después de unos de volver a tocar código pude resolver uno de los problemas de MyBroker:

Código Java:

@Override
    public void guardame(ArrayList<String> rows, String nombre) {
        try {
            this.objBD.getQuery().getConnection().prepareStatement(this.sqlInsert(nombre));
            this.doInsert(rows);
            this.objBD.getQuery().execute();
        }
        catch(Exception ex){
            System.out.println(ex.getMessage());
        }
    }
    private String sqlInsert(String nombre){
        ArrayList<String> columnas=objBD.obtenerColumnas(objBD.obtenerResultSet("select * from "+nombre));
        String Consulta = "insert into " + nombre + " (";
            for (String columna : columnas) {
                if (Consulta.endsWith("(") == false)
                    Consulta += ",";
                Consulta += columna;
            }
            Consulta += ") values (" ;
            for (String columna : columnas) {
                if (Consulta.endsWith("(") == false)
                    Consulta += ",";
                Consulta += "?";
            }
            Consulta += ")";
            System.out.println(Consulta);
            return Consulta;
    }
    private void doInsert(ArrayList<String> rows){
        System.out.println("Tamanio: " + rows.size());
        System.out.println("");
        for(int i = 1 ; i < rows.size(); i++){
            System.out.println("Element: " + i);
            try { 
                objBD.getQuery().setObject(i, rows.get(i));                
            } 
            catch (Exception ex) {
                System.out.println(ex.getMessage());
            }
            System.out.println("");
        }
    }
 

En MyConex.java hice un get del PreparedStatement:

Código Java:

private PreparedStatement query;
    public MyConex() { }
    
    public PreparedStatement getQuery() {
        return query;
    }
 

Y ahora me sale sólo este error:

insert into cargos (carId,carNombre) values (?,?)
Tamanio: 2

Element: 1
Parameter index out of range (1 > number of parameters, which is 0).

Estaba pensando que también puede darme ese error porque el carId es un entero y carNombre es un string, pero en esta línea: objBD.getQuery().setObject(i, rows.get(i)); cómo puedo hacer para lograr que este insert sea genérico de esta forma??

Espero sus respuestas y Saludos

[detective_jd]

Al final solucioné mi problema, mil gracias por las respuestas. Saludos!