Discussion :

[itachi9214]

Salut tout le monde j'ai fait une fonction qui permet de supprimer le dernier enregistrement dans une base de données

Mais la requete ne fonctionne pas car j'ai imbriqué un delete et un select J'ai tout essayé INNER JOIN, EXiST , IN mais rien ne fonctionne

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public void supprimerDernierElement() {
		String req = "DELETE from clients WHERE idClient=(SELECT Max(idClient) from clients)";
		try {
			pre = newCon.prepareStatement(req);
			pre.executeUpdate();
 
			System.out.println("ok");
		} catch (SQLException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
	}

[eulbobo]

Tu peux détailler ce qui "ne fonctionne pas" ?

ce que tu peux essayer, c'est de mettre un newCon.commit(); avant ton catch

Ensuite, dans les bonnes pratiques si tu ne veux pas avoir de soucis, ferme toujours tes objets de manipulation JDBC. Ton preparedStatement : ferme -le

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 void supprimerDernierElement() {
		String req = "DELETE from clients WHERE idClient=(SELECT Max(idClient) from clients)";
		try {
			pre = newCon.prepareStatement(req);
			pre.executeUpdate();
 
			System.out.println("ok");
                        newCon.commit();
		} catch (SQLException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		} finally {
                        if (pre != null){
                             try { pre.close(); } catch (Exception e){e.printStackTrace();}
                        }
                }
	}

Et si tu es en Java 7 ou plus, y'a plus simple

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public void supprimerDernierElement() throws SQLException {
		String req = "DELETE from clients WHERE idClient=(SELECT Max(idClient) from clients)";
		try (PreparedStatement pr = newCon.prepareStatement(req);){
			pr.executeUpdate();
 
			System.out.println("ok");
                        newCon.commit();
		} catch (SQLException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
                        newCon.rollBack();
		} 
	}

[jeffray03]

salut,
changes ta requete en:

Code :

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

String req = "DELETE from clients WHERE idClient IN (SELECT Max(idClient) from clients)";

Eric