Discussion :

[majong]

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

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
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;
       }
     }
}

[adiGuba]

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 :

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
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++

[majong]

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