2

Buenas tengo una duda con un multi update en java: quiero lanzar varias sentencias update concatenadas en un stringBuilder.

El formato del stringBuilder es el siguiente:

"UPDATE morosos SET importe = importe + 80 WHERE dni = '33333333C';UPDATE morosos SET importe = importe + 100 WHERE dni = '44444444D'; etc, etc..."

Mi intención es ejecutar todas esas sentencias con un executeUpdate, pero me lanza el siguiente error:

'com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: You have an error in your SQL syntax;'

public void actualizarMorosos(String path, ConexionRecuperacion con) {
        try {
            BufferedReader br = new BufferedReader(new FileReader(file));
            String line;

            ArrayList<String> dniMorosos = new ArrayList<>();
            ResultSet rs = con.rs("SELECT * FROM morosos");
            while (rs.next()) {
                dniMorosos.add(rs.getString("dni"));
            }

            update = new StringBuilder();

            while ((line = br.readLine()) != null) {
                /* separacion de las lineas */
                String[] split = line.split("#");
                /* quitamos la primera linea */
                if (!line.contains("RECIBO") || !line.contains("PISO") || !line.contains("IMPORTE")) {
                    if (dniMorosos.contains(split[2])) {
                        update.append("UPDATE morosos SET ");
                        update.append("importe = importe + " + split[5]);
                        update.append(" WHERE dni = '" + split[2] + "';");
                    }
                }
            }

            /* imprimir query */
            System.out.println(update.toString());
            /* lanzar query */
            con.update(update.toString());
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
8
  • deberia existir un espacio entre el ; y tu siguiente sentencia: ";UPDATE ...." Commented el 10 ene. 2019 a las 15:59
  • ¿Podrías poner la cadena completa obtenida del StringBuilder y también la traza de error completa... ¿no dice algo como Near at en el mensaje de error?
    – A. Cedano
    Commented el 10 ene. 2019 a las 16:00
  • prueba dicha sentencia en tu gestor de base de datos mysql Commented el 10 ene. 2019 a las 16:00
  • probablemente no está agregando espacios entre el final de una sentencia y el inicio de la otra. Le sugiero editar la pregunta con la múltiples sentencias SQL armadas en el stringBuilder. También (para efectos de comprensión), cambie el título de esta pregunta. Commented el 10 ene. 2019 a las 16:00
  • Echa un vistazo a stackoverflow.com/questions/10797794/…
    – JDev
    Commented el 10 ene. 2019 a las 16:00

1 respuesta 1

5

En tu caso yo usaría consultas preparadas y una actualización por lotes usando transacciones.

Esto aporta seguridad al código (evitando ataques de inyección SQL) y rendimiento, ya que el plan de ejecución es preparado una sola vez. Además control, las actualizaciones se revierten si ocurre algún error (por eso el rollback en la parte del catch...

El proceso es simple:

  1. Creas una sola vez la instrucción SQL, cambiando los futuros valores por marcadores
  2. Indicas con setAutoCommit que vas a iniciar una transacción por lotes
  3. Dentro del while cuando se cumplan las condiciones, seteas cada valor y luego agregas al lote.
  4. Finalmente, al terminar el recorrido, ejecutas el lote de actualizaciones.
  5. Y mediante commit indicas que ya se puede ejecutar el lote
  6. Cuando se usa transacciones, en el catch se llama a rollback para que deshaga cualquier cambio operado por la transacción en caso de algún error

Sería algo así:

public void actualizarMorosos(String path, ConexionRecuperacion con) {
    try {
            BufferedReader br = new BufferedReader(new FileReader(file));
            String line;

            ArrayList<String> dniMorosos = new ArrayList<>();
            ResultSet rs = con.rs("SELECT dni FROM morosos");
            while (rs.next()) {
                dniMorosos.add(rs.getString("dni"));
            }


            String sql="UPDATE morosos SET importe = importe + ? WHERE dni = ?";
            PreparedStatement stmt = con.prepareStatement(sql);
            con.setAutoCommit(false);

            while ((line = br.readLine()) != null) {
                /* separacion de las lineas */
                String[] split = line.split("#");
                /* quitamos la primera linea */
                if (!line.contains("RECIBO") || !line.contains("PISO") || !line.contains("IMPORTE")) {
                    if (dniMorosos.contains(split[2])) {
                        stmt.setInt(1, Integer.parseInt(split[5]));
                        stmt.setString(2, split[2]);
                        stmt.addBatch(); 
                    }
                }
            }

            stmt.executeBatch(); 
            con.commit();

    } catch (Exception e) {
        e.printStackTrace();
        con.rollback();
    }
}

He asumido que importe es un número, si fuera del tipo long tendrías que convertir a ese tipo y usar setLong.

Espero te sirva.

5
  • muchísimas gracias. Commented el 10 ene. 2019 a las 16:40
  • muy concreta y fiable..!!! Commented el 10 ene. 2019 a las 16:40
  • Es una chorrada casi pero en la primera query es mejor solo traer el campo dni en vez de todos con SELECT dni FROM morosos
    – JDev
    Commented el 10 ene. 2019 a las 16:50
  • @JesusRomero había olvidado poner el código completo, como apagar autocommit, usar el rollback en caso de error, etc. Conviene hacerlo así cuando usamos transacciones. Revisa la respuesta editada.
    – A. Cedano
    Commented el 10 ene. 2019 a las 16:50
  • 2
    No @JDev no es ninguna chorrada. Yo he recomendado más de una vez evitar el uso de SELECT *, en mi caso nunca lo uso... No tiene sentido traer todas las columnas cuando sólo vas a usar una o algunas.
    – A. Cedano
    Commented el 10 ene. 2019 a las 16:52

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.