Discussion :

[momjunior]

Bonjour

J'ai une requête HQL qui retourne l'identifiant d'un client à partir de son nom:

Code :

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

1
2
@Query("SELECT COALESCE(a.idClient,0) from Client a WHERE a.nomClient = :nomClient")
	long findIdClientByNomClient(@Param("nomClient") String nomClient);

Code :

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

1
2
3
4
@RequestMapping(value="/entrepot", method=RequestMethod.POST)
	public void getAllMagasinage(ModelMap model, @RequestParam(name = "nomClientRecherche") String nomClient, HttpServletResponse response) throws ServletException, IOException {
 
	long  idClient = clientService.findIdClientByNomClient(nomClient);

Sauf que quand le nom du client n'existe pas, j'aimerai retourner 0 comme identifiant. Pour ça, j'ai utilisé la fonction COALESCE mais je reçois un message d'erreur lorsque le nom du client n'existe pas:

Null return value from advice does not match primitive return type for: public abstract long com.nitraentrepot.repository.ClientRepository.findIdClientByNomClient(java.lang.String)

ça me retourne une valeur null au lieu de 0, et comme le type long n'accepte pas de valeur null, j'ai ce message d'erreur.

Donc comment retourner 0 lorsque le résultat est null.

Merci

[BugFactory]

Bonjour,

Tout d'abord coalesce ne vous aidera pas ici. Si a.idClient était null, il le transformerait en 0. Mais ce n'est pas le problème : si aucun client n'est trouvé la requête retourne zéro ligne, et il n'y a aucun idClient sur lequel coalesce pourrait être exécuté!

La meilleure solution serait de remplacer long par Long (ou Optional<Long>) et de tester les null. Null est censé représenter les valeurs manquantes, utiliser 0 est une "valeur magique" et c'est toujours une source de problèmes plus tard.

Si vous tenez absolument à utiliser 0 pour les résultats de recherche vides, le plus simple est d'utiliser les méthodes par défaut des interfaces.

Code :

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

1
2
3
4
5
6
7
8
 
@Query("SELECT COALESCE(a.idClient,0) from Client a WHERE a.nomClient = :nomClient")
	Long findIdClientByNomClientNullEmpty(@Param("nomClient") String nomClient);
 
        default long findIdClientByNomClientZeroEmpty(String nomClient) {
                Long idClient = findIdClientByNomClientNullEmpty(nomClient);
                return idClient == null ? 0L : idClient.longValue();
        }

Mais encore une fois, en Java on utilise null (ou Optional) pour les valeurs vides, ce code à rallonge n'est que le début de vos ennuis si vous persistez.

[PhilippeGibault]

Très franchement, je suis entièrement d'accord avec le commentaire au dessus, à savoir utiliser un Long plutôt qu'un long, et si ça n'existe pas, retourner un null.

Dans la pratique, c'est comme ça que l'on fait.

Remarque à part: nomClient n'est pas forcement unique, ce qui implique que idClient n'est pas forcement unique si on a 2 dupond dans laBDD.

[BugFactory]

Bonne remarque, mieux vaut retourner une liste au cas où deux personnes auraient le même nom.

[momjunior]

Merci beaucoup, j'ai compris.