Discussion :

[Badshade23]

Bonjour,
J'utilise Spring boot pour un projet web ou il servira de back et j'ai un problème.

Je voudrais tester si mon optionnal est null ou vide.
J'ai commencé comme ça :

Code :

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

1
2
3
4
5
6
7
 
@GetMapping("/user/{id}")
	public ResponseEntity<User> getUserById(@PathVariable Long id) {
		return Optional.ofNullable(userService.findUserById(id)).map(user -> {
			return ResponseEntity.ok(user.get());
		}).orElseGet(() -> ResponseEntity.notFound().build());
	}

Mais même si je teste la nullité de l'optionnal grâce à Optional.ofNullable, si celui-ci est vide il passe quand même dans la map et créer une exception avec le "user.get()". Je sais qu'il existe d'autres moyens avec le ispresent , ifpresent ... mais je voulais éviter de faire un "if - else". Le but de cette question et simplement de savoir si il y a un autre moyen que le basique if-else.

Connaissez-vous un moyen de réussir ceci ?
J'ai essayé comme ça aussi :

Code :

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

1
2
3
4
5
@GetMapping("/user/{id}")
	public ResponseEntity<User> getUserById(@PathVariable Long id) {
		return Optional.ofNullable(userService.findUserById(id)).ifPresentOrElse(user -> ResponseEntity.ok(user.get()),
				() -> ResponseEntity.notFound().build());
	}

Mais ifPresentOrElse a un retour à void donc ....

Preneur d'idée, je ne sais pas si c'est réalisable

[gervais.b]

Bonjour,

C'est étrange; `Optional.ofNullable` crée un empty si la valeur est `null`. Il ne devrait donc pas passer dans le map.
Quel genre d'excpetion avez-vous ? Etes vous sur qu'elle provienne du déréferencement de `user` ?

[Badshade23]

Salut,
la valeur que retourne userService.findUserById(id); est un Optional<User>.
Optional.ofNullable vérifie juste que l'optional n'est pas null mais il ne vérifie pas si celui-ci est vide ou non ...
Donc on passe dans le map est celle-ci déclenche une exception au moment du user.get() car celui-ci n'existe pas.
Comme dit plus haut je peux très bien faire comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
 
@GetMapping("/user/{id}")
	public ResponseEntity<User> getUserById(@PathVariable Long id) {
		Optional<User> user = userService.findUserById(id);
		if (user.isPresent()) {
			return ResponseEntity.ok(user.get());
		}
		return ResponseEntity.notFound().build();
	}

Mais c'est pas ce que je recherche.

[gervais.b]

Mais si userService.findUserById(id) retourne déjà un Optional<User>, pourquoi encore le mettre dans un Optional ?

Code :

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

1
2
3
4
 
userService.findUserById(id)
.map( user -> ResponseEntity.ok(user)) // == .map(ResponseEntity::ok)
.orElseGet( () -> ResponseEntity.notFound().build() );

Edit Si votre userService.findUserById peux retourner un null le mieux serait de corriger ça. Sinon il faut utiliser Optional.flatMap

Code :

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

1
2
3
4
5
 
Optional.ofNullable( userService.findUserById(id) )
                .flatMap( opt -> opt )
                .map(ResponseEntity::ok)
                .orElseGet(()->ResponseEntity.notFound().build());

[Badshade23]

Edit Si votre userService.findUserById peux retourner un null le mieux serait de corriger ça. Sinon il faut utiliser Optional.flatMap

Code :

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

1
2
3
4
5
 
Optional.ofNullable( userService.findUserById(id) )
                .flatMap( opt -> opt )
                .map(ResponseEntity::ok)
                .orElseGet(()->ResponseEntity.notFound().build());

Ok merci, c'est exactement ce que je recherchais. Je ne comprend pas vraiment pour le map fait un Optional<Optional<U>> . Pour flatMap je n'y est pas pensé, je l'utilise surtout dans des streams pour récupérer tout les éléments des listes par exemple. Plus qu'a comprendre le fonctionnement ^^.

[gervais.b]

Ce n'est pas le map qui produit un Optional<Optional<User>> mais c'est le Optional.ofNullable( userService.findUserById(id) ). Etant donné que votre service retourne déjà un Optional c'est comme si vous faisiez Optional.ofNullable( Optional.ofNullable(..) ) .
`flatMap` est jsutement la pour ça, pour "merger" deux Optional en quelques sortes.

Comme je vous le conseillais, le plus propre serait d'éviter que userService.findUserById ne puisse retourner un null. Beaucoup d'IDE et d'autres systèmes de revue de code remontent un avertissement lorsqu'ils détectent qu'un Optional peut-être null. Ils sont justement la pour éviter d'utiliser des références null.




Pensez à marquer la discussion comme résolue si possible.