Skip to main content
se añadieron 4 caracteres en el cuerpo
Origen Enlace
Eduardo Jiménez
  • 2.2k
  • 2
  • 8
  • 29
public void createComicFilter(String name, String number, String variant,
              String sign, String editorial, String format,
              String procedence, String year, String writer,
              String sketcher) throws SQLException {

    ArrayList<String> strFilter = new ArrayList<>();

    String connector = " WHERE ";
    StringBuilder sql = new StringBuilder();
    sql.append("SELECT * FROM comicsbbdd");
    if (name != null) {
        sql.append(connector).append("name = ?");
        connector = " AND ";
        strFilter.add(name);
    }
    if (number != null) {
        sql.append(connector).append("number = ?");
        connector = " AND ";
        strFilter.add(number);
    }
    if (variant != null) {
        sql.append(connector).append("variant = ?");
        connector = " AND ";
        strFilter.add(variant);
    }

    // Así por cada campo

    PreparedStatement ps = con.prepareStatement(sql.toString());
    for (int i = 0; i < strFilter.size(); ) {
        ps.setString(++i, strFilter.get(i - 1));
    }
    ResultSet rs = ps.executeQuery();
//  Aquí ya haces el manejo de tus datos
    while(rs.next()){
        break;
    }
}
public void createComicFilter(String name, String number, String variant,
              String sign, String editorial, String format,
              String procedence, String year, String writer,
              String sketcher) throws SQLException {

    ArrayList<String> strFilter = new ArrayList<>();

    String connector = " WHERE ";
    StringBuilder sql = new StringBuilder();
    sql.append("SELECT * FROM comicsbbdd");
    if (name != null) {
        sql.append(connector).append("name = ?");
        connector = " AND ";
        strFilter.add(name);
    }
    if (number != null) {
        sql.append(connector).append("number = ?");
        connector = " AND ";
        strFilter.add(number);
    }
    if (variant != null) {
        sql.append(connector).append("variant = ?");
        connector = " AND ";
        strFilter.add(variant);
    }

    // Así por cada campo

    PreparedStatement ps = con.prepareStatement(sql.toString());
    for (int i = 0; i < strFilter.size(); ) {
        ps.setString(++i, strFilter.get(i));
    }
    ResultSet rs = ps.executeQuery();
//  Aquí ya haces el manejo de tus datos
    while(rs.next()){
        break;
    }
}
public void createComicFilter(String name, String number, String variant,
              String sign, String editorial, String format,
              String procedence, String year, String writer,
              String sketcher) throws SQLException {

    ArrayList<String> strFilter = new ArrayList<>();

    String connector = " WHERE ";
    StringBuilder sql = new StringBuilder();
    sql.append("SELECT * FROM comicsbbdd");
    if (name != null) {
        sql.append(connector).append("name = ?");
        connector = " AND ";
        strFilter.add(name);
    }
    if (number != null) {
        sql.append(connector).append("number = ?");
        connector = " AND ";
        strFilter.add(number);
    }
    if (variant != null) {
        sql.append(connector).append("variant = ?");
        connector = " AND ";
        strFilter.add(variant);
    }

    // Así por cada campo

    PreparedStatement ps = con.prepareStatement(sql.toString());
    for (int i = 0; i < strFilter.size(); ) {
        ps.setString(++i, strFilter.get(i - 1));
    }
    ResultSet rs = ps.executeQuery();
//  Aquí ya haces el manejo de tus datos
    while(rs.next()){
        break;
    }
}
Origen Enlace
Eduardo Jiménez
  • 2.2k
  • 2
  • 8
  • 29

Anotaciones

  • Tu código es súper vulnerable a inyecciones de datos, te indicaría que hagas uso de PreparedStatement, aparte de que brinda mayor seguridad es más rápido en ejecución (según recuerdo de los resultados de pruebas en nanosegundos de un vídeo que vi hace años)
  • Ya que todos tus campos son tipo String se simplifica el asunto. y sólo hacemos uso de PreparedStatement#setString, de lo contrario una solución podría ser el uso de if's y del operador instanceof
  • Si tienes duda del uso de PreparedStatement, te recomiendo leer el tutorial oficial de Oracle de JDBC
  • La comparación de los filtros se hace por capas, se indica que un filtro no se debe aplicar si el campo es null, es decir, si name == null entonces ese filtro no se aplica
  • Cambié el español por inglés, así que obviamente no va a funcionar con tu código, será cosa de que adaptes el nombre de las columnas y variables
public void createComicFilter(String name, String number, String variant,
              String sign, String editorial, String format,
              String procedence, String year, String writer,
              String sketcher) throws SQLException {

    ArrayList<String> strFilter = new ArrayList<>();

    String connector = " WHERE ";
    StringBuilder sql = new StringBuilder();
    sql.append("SELECT * FROM comicsbbdd");
    if (name != null) {
        sql.append(connector).append("name = ?");
        connector = " AND ";
        strFilter.add(name);
    }
    if (number != null) {
        sql.append(connector).append("number = ?");
        connector = " AND ";
        strFilter.add(number);
    }
    if (variant != null) {
        sql.append(connector).append("variant = ?");
        connector = " AND ";
        strFilter.add(variant);
    }

    // Así por cada campo

    PreparedStatement ps = con.prepareStatement(sql.toString());
    for (int i = 0; i < strFilter.size(); ) {
        ps.setString(++i, strFilter.get(i));
    }
    ResultSet rs = ps.executeQuery();
//  Aquí ya haces el manejo de tus datos
    while(rs.next()){
        break;
    }
}