[SOLUCIONADO] Automatizar SQL con DatabaseMetaData

[detective_jd]

Hola a todos, quería hacerles una consulta resulta que estoy tratando de automatizar las consultas sql con DatabaseMetaData digamos que con los insert, update, delete y selectxId no estoy teniendo problemas con un select genérico que quiero automatizar los inner join no sé cómo lo puedo encarar xq los campos y los where los puedo hacer, pero los inner me complican tanto que no sé cómo seguir.

Acá pongo el código:

Código Java:

public String select(String[] cols, String[] tables, String[] where){
String Consulta = "select ";
if(cols == null){
Consulta += "* ";
} else {
for(String c : cols){
Consulta += c;
if(!cols[cols.length -1].equals(c)){
Consulta += ",";
} else {
Consulta += " ";
}
}
}
Consulta += "from " + this.getTable();
if(tables != null){
char primera = this.getTable().charAt(0);
char prox;
Consulta += " " + primera + " ";
for(String t : tables){
Consulta += "inner join " + t + " on ";
if(!tables[tables.length -1].equals(t)){
Consulta += ",";
} else {
Consulta += " ";
}
}
}
if(where != null){
Consulta += " where ";
for(String w : where){
Consulta += w + " = ?";
if(!where[where.length -1].equals(w)){
Consulta += " and ";
}
}
}
return Consulta;
}
 

Espero sus respuestas y saludos.

[detective_jd]

Hola a todos, les cuento que estuve trabajando en el select genérico y este es el resultado:

Código Java:

public String select(String[] cols, String[] tables, String style, String[] where){
String Consulta = "select ";
if(cols == null){
Consulta += "* ";
} else {
Consulta += this.createColumns(cols, tables);
}
Consulta += "from " + this.getTable();
if(tables != null){
String[] tab = this.prepareTables(tables);
Consulta += " " + this.getTable().charAt(0); 
Consulta += this.createJoin(style,tab); 
}
if(where != null){
Consulta += " where ";
for(String w : where){
Consulta += w + " = ?";
if(!where[where.length -1].equals(w)){
Consulta += " and ";
}
}
}
return Consulta;
}
private String createJoin(String style,String[] tables){
String join = " ";
if(tables != null){
for(String t : tables){
if(!t.equals(this.getTable())){
char ps = t.charAt(0);
join += style + " join " + t + " " + ps + " on" + this.createCombos(tables);
}
}
}
return join;
}
private String[] prepareTables(String[] tables){
String[] tab = new String[tables.length +1];
tab[0] = this.getTable();
for(int i = 0; i < tables.length; i++){
tab[i+1] = tables[i];
}
return tab;
}
private String createCombos(String[] tables){
String combo = " ";
for(int i = 0; i < tables.length -1; i++){
char fk = tables[i+1].charAt(0);
char pk = tables[i].charAt(0);
LinkedList<String> keys = this.objbd.getPrimaryKeys(tables[i+1]);
LinkedList<String> fays = this.objbd.getForeignKeys(tables[i]);
for(String f : fays){
for(String k : keys){
if(k.equals(f)){
combo += fk +"."+f +" = "+pk +"."+k;
} 
}
} 
}
return combo;
}
private String createColumns(String[] cols,String[] tables){
String columns = "";
for(String c : cols){
if(tables == null){
columns += c;
} else {
for(String t : tables){
char s = t.charAt(0);
LinkedList<String>temp = this.objbd.getColumns(t);
for(String c1 : temp){
if(c1.equals(c)){
columns += s + "." + c;
}
}
} 
} 
if(!cols[cols.length -1].equals(c)){
columns += ",";
} else {
columns += " ";
}
}
return columns;
}
 

lo que me faltaría es ver bien las condiciones, los order y group by, además debería probar con los renombramientos de sql xq de momento esta sólo obteniendo la primer letra cosa que no es muy buena idea, ya que si hay otra tabla con la misma letra inicial, la consulta queda fastidiada, no sé que me sugieren?

[detective_jd]

Mejoré un poco la cosa:

Código Java:

public String select(String[] cols, String[] tables, String[] style, String[] where, String[] conditions,String[] parts){
        String Consulta = "select ";
        if(cols == null){
            Consulta += "* ";
        } else {
            Consulta += this.createColumns(cols, tables);
        }
        Consulta += "from " + this.getTable();        
        if(tables != null){
            String[] tab = this.prepareTables(tables);
            Consulta += " " + this.getTable().charAt(0);            
            Consulta += this.createJoin(style,tab);
            if(where != null && conditions != null){
                Consulta += " " + this.createConditions(tab,where, conditions, parts);
            }
        }         
        return Consulta;
    }
    private String createJoin(String[] style,String[] tables){
        String join = " ";
        if(tables != null && style != null){
            for(int i = 0; i < tables.length; i++){
                if(!tables[i].equals(this.getTable())){
                    String ps = "" + tables[i].charAt(0);
                    String pp = "" + tables[i -1 ].charAt(0);
                    if(ps.equals(pp)){
                        ps += tables[i].charAt(1);
                    }
                    join += style[i-1] + " join " + tables[i] + " " + ps + " on" + this.createCombos(tables[i],tables[i -1]);
                    if(!tables[tables.length -1].equals(tables[i])){
                        join += " ";
                    }
                }
            }
        }
        return join;
    }
    private String createCombos(String emisor, String receptor){
        String combo = " ";
        String fk = "" + emisor.charAt(0);
        String pk = "" + receptor.charAt(0);
        if(fk.equals(pk)){
            fk += receptor.charAt(1);
        }
        LinkedList<String> keys = this.objbd.getPrimaryKeys(emisor);
        LinkedList<String> fays = this.objbd.getForeignKeys(receptor);
        for(String f : fays){
            for(String k : keys){
                if(k.equals(f)){
                    combo += pk +"."+k +" = "+fk +"."+f;
                    break;
                }                                    
            }
        }                        
        return combo;
    }
    private String[] prepareTables(String[] tables){
        String[] tab = new String[tables.length +1];
        tab[0] = this.getTable();
        for(int i = 0; i < tables.length; i++){
            tab[i+1] = tables[i];
        }
        return tab;
    }   
    private String createColumns(String[] cols,String[] tables){
        String columns = "";
        for(String c : cols){
            if(tables == null){
                columns += c;
            } else {
                for(int i = 0; i < tables.length; i++){
                    String ps = "" + tables[i].charAt(0);
                    if(tables.length > 1 && i > 0){
                        String pp = "" + tables[i-1].charAt(0);
                         if(ps.equals(pp)){
                            ps += tables[i].charAt(1);
                        }
                    }                    
                    LinkedList<String>temp = this.objbd.getColumns(tables[i]);
                    for(String c1 : temp){
                        if(c1.equals(c)){
                            columns += ps + "." + c;
                        }
                    }
                }                
            }            
            if(!cols[cols.length -1].equals(c)){
                columns += ",";
            } else {
                columns += " ";
            }
        }
        return columns;
    }
    private String createConditions(String[] tables, String[] where, String[] conditions, String[] parts){
        String condition = "where ";
        for(int i = 0 ; i < where.length; i++){
            if(tables != null){
                for(int j = 0; j < tables.length; j++){
                    String ps = "" + tables[j].charAt(0);
                    if(tables.length > 1 && j > 0){
                        String pp = "" + tables[j-1].charAt(0);
                         if(ps.equals(pp)){
                            ps += tables[j].charAt(1);
                        }
                    }
                    LinkedList<String>temp = this.objbd.getColumns(tables[j]);
                    for(String c1 : temp){
                        if(c1.equals(where[i])){
                            where[i] = ps + "." + where[i];
                        }
                    }
                }                
            }
        }
        for(int i = 0 ; i < where.length; i++){
            if(conditions[i].equals("between") || conditions[i].equals("not between")){
                condition += where[i] + " " + conditions[i] + " ? and ?";
            }else if(conditions[i].equals("not null") || conditions[i].equals("null")){
                condition += where[i] + " " + conditions[i];
            } else {
                condition += where[i] + " " + conditions[i] + " ?";            
            }
            if(!where[where.length -1].equals(where[i])){
                condition += " " + parts[i] + " ";
            }
        }        
        return condition;
    }
 

pero no me pasa que para el group by y order by es parecido a lo anterior, ¿quería preguntar cómo mejorar los renombramientos? ya que hice que para que no me duplique que en tabla que le haga el join me concatene su segunda letra pero eso no funcionará por mucho, además me quedé pensando cómo hacer si tengo subconsultas?? ahí no sé que hacer.

espero sus respuestas y saludos.

[detective_jd]

Hola a todos, les comento que estuve tocando código y la verdad es q logré darle buenos avances a la automatización de los select pero tengo problemas en cómo armar el array del having y cómo armar la función del having necesito sugerencias, luego falta pulir el createColumns para que pueda aceptar renombramientos y funciones agregadas, por otro lado necesito ayuda para automatizar las subconsultas que ahí no tengo ni una idea de cómo llevarlo a cabo.

Pongo el pastebin: http://pastebin.com/6ghUPA3g.

Espero sus respuestas y Saludos

[agleiva]

Borrá todo eso y usa un ORM.

http://hibernate.org/

[detective_jd]

agleiva gracias por responder, verás más adelante pienso usar JPA y hibernate....

La cuestión es que quería hacer algo a mano automatizando SQL ya que la clase Model.java es una clase abstracta que otros modelos la heredarán pero quería primero automatizar todo SQL.

Con respecto a lo que pregunté, me puedes dar una idea de cómo llevar a cabo lo preguntado???

Espero sus respuestas y Saludos.

[agleiva]

Iniciado por detective_jd

agleiva gracias por responder, verás más adelante pienso usar JPA y hibernate....

La cuestión es que quería hacer algo a mano automatizando SQL ya que la clase Model.java es una clase abstracta que otros modelos la heredarán pero quería primero automatizar todo SQL.

Con respecto a lo que pregunté, me puedes dar una idea de cómo llevar a cabo lo preguntado???

Espero sus respuestas y Saludos.

No, no te puedo dar una idea. Gracias a Dios no he tenido que tocar una sola linea de java en mi vida, y no pienso empezar ahora. De hecho ni siquiera puedo leer detenidamente tu codigo sin que me duela la cabeza por lo horrendo, retrógrado, dinosaurio y vomitivo que es el lenguaje java, sobre todo si uno está acostumbrado a lenguajes modernos y bien diseñados.

Dicho esto, entré a este foro solamente para ayudarte enormemente con el comentario anterior y que te ahorres esfuerzo en implementar algo que no tiene ningun sentido.

Lamento pincharte el globo (*), pero todo tu codigo es un error enorme. Concatenar strings para generar SQL es una pésima idea y un gran agujero de seguridad a menos que sanitices todos tus inputs. Para el momento que llegues a eso, felicidades, reinventaste la rueda (una rueda bastante ineficiente) en lugar de usar Hibernate. Tu rueda es mas bien cuadrada, no se adapta a ningun modelo de auto y tiene fallas que pueden provocar accidentes fatales.

Te sugiero que desistas de esta pésima idea y utilices o bien un ORM, un Micro ORM, o al menos prepared statements como para que tu codigo no sea inmediatamente vulnerable a inyeccion SQL.

(*) - No, la verdad que no lo lamento.

[detective_jd]

créeme quería ver cómo sería todo a mano pero bueno, haré un poco cómo antes, veré q sale. Saludos