Discussion :

[grunt2000]

Bonjour,

Je souhaite réaliser un test unitaire testant des lectures en base de données dans de nombreuses situations.

J'ai écrit cette fonction utilitaire pour m'aider :

Code :

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

1
2
3
4
5
private void checkAllRecordsMatchesCondition(String sql, Predicate<ResultSet> condition) throws SQLException {
	try(Connection connection = connect(); Statement stmt = connection.createStatement(); ResultSet rs = stmt.executeQuery(sql)) {
		while(rs.next()) assertTrue(sql, condition.test(rs));
	}
}

mais quand je l'utilise, elle me pose un problème :

Code :

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

1
2
3
@Test public void whereCondition_field_literal() throws SQLException {
	checkAllRecordsMatchesCondition("SELECT * FROM SignedBikeRoute WHERE FNODE_ < 2000", rs -> rs.getInt("FNODE_") < 2000);
}

Le compilateur désigne la partie en rouge d'un "Unhandled exception type SQLException".
Cette exception est dans la directive throws de ma fonction.

Quelque-chose m'échappe ?

Merci !

[Satch]

Tu es certain que le problème ne vient pas plutôt de ta méthode checkAllRecordsMatchesCondition, qui contient un try sans catch ni finally ?

[adiGuba]

Salut,

Tu es certain que le problème ne vient pas plutôt de ta méthode checkAllRecordsMatchesCondition, qui contient un try sans catch ni finally ?

Non : c'est le try-with-ressource de Java 7, qui permet de libérer proprement les ressources.

Le compilateur désigne la partie en rouge d'un "Unhandled exception type SQLException".
Cette exception est dans la directive throws de ma fonction.

Quelque-chose m'échappe ?

Le problème vient du concept même des "checked-exceptions" de Java : par défaut toute exception doit être soit traité soit déclarée.
Certe tu le fais bien au niveau de ta méthode, mais ce n'est pas fait dans la lambda !

Le problème c'est qu'il faut que cela soit fait au niveau de la lambda... ce qui n'est pas le cas.
Les lambdas "standards" du package java.util.function ne déclarent aucune exception, et ne peuvent donc pas les remonter.


On le voit mieux si on écrit cela de manière classique avec une classe anonyme :

Code :

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

1
2
3
4
5
6
7
8
	@Test public void whereCondition_field_literal() throws SQLException {
		checkAllRecordsMatchesCondition("SELECT * FROM SignedBikeRoute WHERE FNODE_ < 2000", new Predicate<ResultSet>() {
			@Override
			public boolean test(ResultSet rs) {
				 return rs.getInt("FNODE_") < 2000; // SQLException non-gérée
			}
		});
	}

Bref les checked-exceptions pose encore plus de problème avec les lambdas !



Du coup la meilleure solution consiste à se créer sa propre interface Predicate gérant les SQLException :

Code :

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

1
2
3
4
@FunctionalInterface
public interface SQLPredicate<T> {
	boolean test(T t) throws SQLException;
}

Voir même en paramétrant cela via les Generics :

Code :

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

1
2
3
4
@FunctionalInterface
public interface CheckedPredicate<T, E extends Throwable> {
	boolean test(T t) throws E;
}

a++

[Satch]

Non : c'est le try-with-ressource de Java 7, qui permet de libérer proprement les ressources.

Oops !

Faut vraiment que je me mette un peu plus à jour

[grunt2000]

Merci AdiGuba !
Je m'en vais essayer ta solution demain matin.

EDIT : Cela fonctionne très bien, merci !