message d'alerte de compilation

Version imprimable

Bonjour, j'ai le message suivant :
finally block does not complete normally

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
public static boolean VerifierIdentification(CIdentificationFlash ident) {
	ResultSet rs = null;
	Connection conn = null;
	PreparedStatement pstmt = null;
	boolean result = false;
        try {
		conn = getConnection();
		String query = "SELECT 1 FROM table_users u WHERE
                u.user_id="+ident.user_id+" AND u.user_passwd='"+ident.mdp+
                "' Limit 1;";
		pstmt = conn.prepareStatement(query);
		rs = pstmt.executeQuery();
		result=(rs.getRow()==1);
      } catch (Exception e) {
		return false;
     } finally {
	try {
		rs.close();
		pstmt.close();
	        conn.close();
	        return result;
        } catch (SQLException e) {
                return false;
       }
     }
}

Salut,

C'est normal car les bloc finally ne doivent pas contenir d'instruction return car cela peut masquer une exception.

D'une manière générale le code n'est pas très correcte.

Pour libérer proprement les ressources, il faut utiliser un try/finally par ressource, et le finally ne doit contenir que l'appel de la méthode close().
Catcher les exceptions pour retourner false c'est "tout pourri" : il n'y a rien de mieux pour perdre un temps fou à ne pas comprendre pourquoi le programme ne fonctionne pas comme on le souhaite :(
Il est préférable de laisser remonter l'exception (quitte à l'englober dans une RuntimeException).
Il faut faire un next() sur le ResultSet.
Il faut utiliser des PreparedStatement pour éviter les injections SQL
Et tant qu'à faire respecter les règles de nommages et d'encapsulation...

Bref cela devrait plutôt ressembler à ceci :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 public static boolean verifierIdentification(CIdentificationFlash ident) throws SQLException { Connection conn = getConnection(); try { PreparedStatement pstmt = conn .prepareStatement("SELECT 1 FROM table_users u WHERE u.user_id= ? AND u.user_passwd= ? Limit 1"); try { pstmt.setInt(1, ident.getUserId()); pstmt.setString(2, ident.getPassword()); ResultSet rs = pstmt.executeQuery(); try { if (rs.next()) return true; } finally { rs.close(); } } finally { pstmt.close(); } } finally { conn.close(); } return false; }
a++

15/04/2011, 23h56
majong

:aie:, merci pour la leçon de précision:ccool:, je vais essayer de programmer dans ce sens