Discussion :

[Reisors]

Bonjour,

J'ai un souci d'égalité d'Integer dans une expression lambda qui ne fonctionne pas et je ne comprend pas pourquoi, pourriez vous m'aidez ?

D'abord j'ai ce code complètement fonctionnel :

Type :
this.intervention = un objet Intervention
this.getInterventions() = une liste d'Intervention
i.getIdIntervention() = l'Id de l'intervention (Integer)
this.navigationBean.getIdIntervention() = retourne l'Id de l'intervention de l'url (Integer)

Code :

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

1
2
 
this.intervention = this.getInterventions().stream().filter(i -> i.getIdIntervention().equals(this.navigationBean.getIdIntervention())).findFirst().get();

Ce code fonctionne parfaitement le lambda me renvoie bien une Intervention, pas de souci, c'est la suite qui se complique :

Je fait la même chose pour un Objectif :

Type :
this.objectif = un objet Objectif
this.intervention.getObjectifs() = liste d'Objectif dans l'Intervention
i.getIdTache() = l'id de l'Objectif (Integer) (Objectif hérite de Tache)
this.navigationBean.getIdObjectif() = l'Id de l'objectif dans l'url (Integer)

Code :

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

1
2
 
this.objectif = this.intervention.getObjectifs().stream().filter(i -> i.getIdTache().equals(this.navigationBean.getIdObjectif())).findFirst().get();

Et cette fois ça ne marche pas, il me dit que qu'il ne trouve pas d'objet correspondant alors que :

Code :

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

1
2
3
4
5
6
 
for(Objectif o : this.intervention.getObjectifs()){
          if(o.getIdTache().equals(this.navigationBean.getIdObjectif())){
                  this.objectif = o;
              }
}

Fonctionne parfaitement !

Si vous avez une idée je suis preneur, je ne comprend vraiment pas ....

[tchize_]

Que donne

Code :

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

1
2
3
4
5
6
7
this.objectif = this.intervention.getObjectifs().stream().filter(i -> {
                  System.out.println(
                      i.getIdTache()+" = "+this.navigationBean.getIdObjectif()+"?"+
                      i.getIdTache().equals(this.navigationBean.getIdObjectif())
                      );
                  i.getIdTache().equals(this.navigationBean.getIdObjectif())
      }).findFirst().get();

[Reisors]

Il aime pas le System.out.println au milieu et je vois pas comment corriger l'erreur

J'ai essayer en convertissant automatiquement (via netbean) le foreach en Functional Operation :

Code :

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

1
2
3
this.intervention.getObjectifs().stream().filter((o) -> (o.getIdTache().equals(this.navigationBean.getIdObjectif()))).forEach((o) -> {
                this.objectif = o;
            });

Ça ne marche pas, à mon avis il y a un bug dans les lambda c'est pas possible ....

[tchize_]

J'ai juste oublié le return dans le code que je t'ai donné

Code :

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

1
2
3
4
5
6
7
this.objectif = this.intervention.getObjectifs().stream().filter(i -> {
                  System.out.println(
                      i.getIdTache()+" = "+this.navigationBean.getIdObjectif()+"?"+
                      i.getIdTache().equals(this.navigationBean.getIdObjectif())
                      );
                  return i.getIdTache().equals(this.navigationBean.getIdObjectif());
      }).findFirst().get();

[Reisors]

ok dsl je débute avec les lambda

Du coup j'ai essayer ton code, il a rien le temps de m'afficher il me met toujours la même erreur :

WARNING: StandardWrapperValve[Faces Servlet]: Servlet.service() for servlet Faces Servlet threw exception
java.util.NoSuchElementException: No value present

J'ai passé le code :

Code :

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

1
2
 
this.objectif = this.intervention.getObjectifs().stream().filter(i -> i.getIdTache().equals(this.navigationBean.getIdObjectif())).findFirst().get();

en DEBUG et j'ai mis this.navigationBean.getIdObjectif() dans une variable x pour vérifier la valeur
=> this.intervention.getObjectifs() contient un seul élément de type Objectif ou idTache = 11562
=> x = 11562

Franchement c'est incompréhensible cette histoire ...

[tchize_]

Le system.out.println est dans la console de ton serveur. Si la lambda n'affiche rien dans la console, c'est que ta liste est vide. A mon avis, en debug, tu ne regarde pas les mêmes objet que ceux que tu utilise dans ta lambda.
Tu peux aussi afficher ça avant la lambda si tu veux.

Code :

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

System.out.println("Objectfis: "+this.intervention.getObjectifs());

Je ne sais pas quel "x" tu regarde en debug, mais il n'y a pas de X dans ta lambda.

[joel.drigo]

C'est quoi le type de this.intervention.getObjectifs() ?

Et comme ça aussi, ça t'affiches l'exception avant autre chose :

Code :

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

1
2
3
4
5
6
7
8
this.intervention.getObjectifs().stream()
		.peek(i-> 
				System.out.println(
	                      i.getIdTache()+" = "+this.navigationBean.getIdObjectif()+"?"+
	                      i.getIdTache().equals(this.navigationBean.getIdObjectif())
	                      )
		)
		.filter(i -> i.getIdTache().equals(this.navigationBean.getIdObjectif())).findFirst().get();

Fais aussi un essai en décomposant en plusieurs instructions, pour voir où se situe l'exception (à priori c'est sur le get(), mais c'est pour être sûr).

[Reisors]

tchize_ :
En fait j'ai fait sa :

Integer x = this.navigationBean.getIdObjectif();
this.objectif = this.intervention.getObjectifs().stream().filter(i -> i.getIdTache().equals(x)).findFirst().get();

x = 11562
this.intervention.getObjectifs().get(0).getIdTache() = 11562

Pourtant je me prend une erreur.

joel.drigo :
J'ai essayer ton code, même problème, affichage de l'erreur uniquement

Et oui c'est au niveau du get() ou du findFirst() car en réalité "this.intervention.getObjectifs().stream().filter(i -> i.getIdTache().equals(this.navigationBean.getIdObjectif()))" renvoie une liste ne contenant rien

Pour moi c'est un problème dans le lambda car le foreach marche mais bon ...

[joel.drigo]

Tu fais le forEach exactement avec le même "x" (même type, même valeur) et la même instance de liste ?

Parce que si le peek() n'affiche rien, je rejoins l'avis de @Tchize_ : le stream est vide, avant même l'application du filtre.

Un System.out.prinln( this.intervention.getObjectifs().stream().count() ) dit quoi ?


Je ne vois pas trop quel problème il pourrait y avoir dans le lambda...

[Reisors]

Bon, merci à vous deux je viens de trouver pourquoi

En fait c'est le mode debug qui m'a trompé, je m'explique :

Le bug est du soit à JPA soit au traitement que fait la lambda, de base les listes d'objets sont chargé en mode LAZY par JPA c'est à dire uniquement à la demande, c'est pour ça que je voyais bien mon objectif dans la liste en debug, par contre je viens de m’apercevoir que la lambda ne la charge pas , du coup j'ai passé la liste d'Objectif en EAGER dans l'objet Intervention, et ça fonctionne.

Voila voila ...

[joel.drigo]

J'y connais rien en JPA, et de comment fonctionne le mode LAZY, mais à priori, je dirais que c'est iterator() qui doit charger la liste, alors que splitIterator() ne doit pas le faire. Du coup ça marche avec un foreach et pas un stream. tu pourrais le vérifier en appelant .iterator() sur getObjectifs() juste avant l'appel de stream(). En tout cas, ce n'est pas le lambda.

[EDIT] pour info : https://bugs.eclipse.org/bugs/show_bug.cgi?id=433075