Discussion :

[BobbyWeb]

Hello !

Je viens d'écrire une fonction qui supprime tous les champs sélectionner d'une table Mysql, mais j'ai une erreur sur la suppression :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
public static void cleanDatabase() throws Exception
{
        Connection conn = getConnection();
        Statement stmt = conn.createStatement();
        try {
 
            ResultSet rs = stmt.executeQuery("SELECT * FROM MaTable WHERE MonChamp='critere'");
            while (rs.next()) {
                  stmt.executeUpdate("DELETE FROM MaTable WHERE MonChamp = '"+rs.getString("MonChamp")+"'");
                }
            }
 
        } catch(SQLException e) {
            e.printStackTrace();
        } 
        stmt.close();
        conn.close();
}

Normalement ce code devrait bien supprimer toute les lignes ramenées par ma requête, mais voici ce que la console me renvoie :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

java.sql.SQLException: Operation not allowed after ResultSet closed

Pourtant je n'ai pas fermé mon resultset...

[SucreGlace]

C'est parce que tu ré-utilises le même Statement.

A ResultSet object is automatically closed when the Statement object that generated it is closed, re-executed, or used to retrieve the next result from a sequence of multiple results.

[sinok]

Et puis accessoirement ce que tu cherches à faire est jouable en une seule requête surtout...

Code sql :

Sélectionner tout - Visualiser dans une fenêtre à part

delete from MaTable where monchamp='critere';

Ou si tu as deux tables:

Code sql :

Sélectionner tout - Visualiser dans une fenêtre à part

delete from MaTable1 where monchamp1 in (select monchamp2 from MaTable2 where monchamp3='critere')

[BobbyWeb]

Oui c'est sûr mais je voudrais faire certains tests avant de supprimer. Du genre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
ResultSet rs = stmt.executeQuery("SELECT * FROM MaTable WHERE MonChamp='critere'");
while (rs.next()) {
         if(rs.getString("MonChamp")= monTest) {
                  stmt.executeUpdate("DELETE FROM MaTable WHERE MonChamp = '"+rs.getString("MonChamp")+"'");
         }
}

Je veux juste savoir ce qui pose problème dans le delete...

[adiGuba]

Salut,

Oui c'est sûr mais je voudrais faire certains tests avant de supprimer. Du genre :

Heu... Là tu revérifies la même chose...
Il y a de très forte chance que tu puisses faire cela via du SQL, ce qui serait surement bien plus performant.

Je veux juste savoir ce qui pose problème dans le delete...

Ben cela a été dit : Le fait d'exécuter une requête sur le même Statement ferme ton ResultSet...

Donc il faut soit utiliser un autre Statement, soit utiliser un ResultSet "updatable" et d'utiliser la méthode deleteRow()


a++

PS : Et au passage, préfères aussi l'utilisation de PreparedStatement à la place de concaténation de chaines qui peuvent entrainer des bugs voir des failles (SQL injection)

[BobbyWeb]

c'est parfait adiGuba : je créer un PreparedStatementdans mon While et je l’exécute à la fin, la j'ai plus d'erreur et ça fonctionne correctement