fonction for en java

bonsoir
j'ai un pb et je penses que c'est facile à résoudre ça mais je ne comprends pas pourquoi je n'arrives pas à le résoudre.
en fait je veux faire une fonction qui permet de trouver un objet qui est dans mon cas une page Facebook en lui donnant son identifiant.
voici le code que j'ai fait normalement il manque quelque chose

Code:

---

1 2 3 4 5 6 7 8 9

public PageFacebook trouverPage(PageFacebook p,int id){ for(int i=0;i<p.getAmis().size();i++){ PageFacebook amiDeMonAmi=p.getAmis().get(i); if(amiDeMonAmi.getIdPage()==id){   } } return amiDeMonAmi; }

---

Bonjour,

1- amiDeMonAmi n'est visible que dans la boucle for, donc quand tu faits return, il n'est plus visible --> erreur de compilation.
2- la méthode get n'est pas la meilleur façon de parcourir une List, utilise son iterator ou un for-each.

A+.

Ca, ca marche mais ça prend pas trop en compte le cas ou la page n'a pas été trouvée, pour ça, je te laisse chercher.

Code:

---

1 2 3 4 5 6 7 8 9 10

public PageFacebook trouverPage(PageFacebook p,int id){ PageFacebook amiDeMonAmi = null; for(int i=0;i<p.getAmis().size();i++){ amiDeMonAmi=p.getAmis().get(i); if(amiDeMonAmi.getIdPage()==id){   } } return amiDeMonAmi; }

---

Ben si getAmis() retourne un List, utiliser get() pour le parcourir n'est pas la meilleur façon.

Code:

---

1 2	for(PageFacebook pfb : p.getAmis()){ amiDeMonAmi=pfb;

---

ou

Code:

---

1 2	for(Iterator<PageFacebook> iterator = p.getAmis().iterator(); iterator.hasNext(); ){ amiDeMonAmi=iterator.next();

---

A+.

Code:

---

1 2	for(PageFacebook pfb : p.getAmis()){ amiDeMonAmi=pfb;

---

Pourquoi pas

Code:

---

for(PageFacebook amiDeMonAmi : p.getAmis()){

---

??

Citation:

---

Envoyé par BenWillard

Code:

---

1 2	for(PageFacebook pfb : p.getAmis()){ amiDeMonAmi=pfb;

---

Pourquoi pas

Code:

---

for(PageFacebook amiDeMonAmi : p.getAmis()){

---

??

---

Parce qu'il doit retourner un objet, et l'élément déclarer dans la boucle for n'est visible qu'à l'intérieur de for. Si amiDeMonAmi est déclaré avant la boucle for, tu ne peux la redéclarer avec for(PageFacebook amiDeMonAmi.



Tu auras une erreur de compilation.

A+.

Ah oui j'avais zappé le return

Mais il y a aussi

Code:

---

1 2	for(PageFacebook amiDeMonAmi : p.getAmis()) if(amiDeMonAmi.getIdPage()==id) return amiDeMonAmi;

---

Citation:

---

Envoyé par BenWillard

Ah oui j'avais zappé le return

Mais il y a aussi

Code:

---

1 2	for(PageFacebook amiDeMonAmi : p.getAmis()) if(amiDeMonAmi.getIdPage()==id) return amiDeMonAmi;

---

---

Dans ce cas il doit mettre un return null; après le boucle for. Mais je ne suis pas très fan de l'utilisation de plusieur return dans une fonction.

A+.

Citation:

---

Envoyé par BenWillard

Ah oui j'avais zappé le return

Mais il y a aussi

Code:

---

1 2	for(PageFacebook amiDeMonAmi : p.getAmis()) if(amiDeMonAmi.getIdPage()==id) return amiDeMonAmi;

---

---

Avec un return null après le for au cas où id n'a pas été trouvé. Il faudra que l'appelant de la méthode gère le cas où elle renvoie null

Citation:

---

Envoyé par Huitzilopochtli

Ca, ca marche mais ça prend pas trop en compte le cas ou la page n'a pas été trouvée, pour ça, je te laisse chercher.

Code:

---

1 2 3 4 5 6 7 8 9 10

public PageFacebook trouverPage(PageFacebook p,int id){ PageFacebook amiDeMonAmi = null; for(int i=0;i<p.getAmis().size();i++){ amiDeMonAmi=p.getAmis().get(i); if(amiDeMonAmi.getIdPage()==id){   } } return amiDeMonAmi; }

---

---

C'est faux, ça va toujours retourner le dernier ami de la liste.

A+.

Citation:

---

Mais je ne suis pas très fan de l'utilisation de plusieur return dans une fonction.

---

Non mais là ça simplifie les choses, tout en rendant le code plus performant, on aurait tort de se priver...

Code:

---

1 2 3

for(PageFacebook amiDeMonAmi : p.getAmis()) if(amiDeMonAmi.getIdPage()==id) return amiDeMonAmi; return null;

---

Citation:

---

Envoyé par BenWillard

Non mais là ça simplifie les choses, tout en rendant le code plus performant, on aurait tort de se priver...

---

en mettant seulement un break dans le boucle for, la différence de performance ne serait qu'un millième de seconde.
En fait, je n'utilise qu'un seul return quand on veut retourner un objet dans une fonction pour une meilleur lisibilité de code.

Citation:

---

Envoyé par andry.aime

En fait, je n'utilise qu'un seul return quand on veut retourner un objet dans une fonction pour une meilleur lisibilité de code.

---

Si ça te plaît à toi, c'est cool. Mais moi je vois juste qu'on a remplacé "plusieurs returns" par "une variable qui sera renvoyée par le return et qui est assignée à plusieurs endroits."
Du style Visual Basic. Je suis guère plus convaincu.

Citation:

---

Envoyé par thelvin

Mais moi je vois juste qu'on a remplacé "plusieurs returns" par "une variable qui sera renvoyée par le return et qui est assignée à plusieurs endroits."

---

Et oui :D.
Mais c'est une convention de coder suggérée par sun et je le trouve aussi pratique.

A+.

merci pour toutes ces réponses mais s'il vous si plait si quelqu'un peut m'expliquer encore plus le code proposé car j'ai pas bien compris.
vous m'avez proposez ce code là

Code:

---

1 2	for(PageFacebook pfb : p.getAmis()){ amiDeMonAmi=pfb;

---

s'il vous plait expliquez plus ce code là et ou il faut mettre le return car ça le pb.

Code:

---

1 2 3 4 5 6 7 8

PageFacebook amiDeMonAmi = null; for(PageFacebook pfb : p.getAmis()){ amiDeMonAmi=pfb; if(amiDeMonAmi.getIdPage()==id){ break } } return amiDeMonAmi;

---

merci pour votre réponse mais s'il vous plait qu'est ce qu'elle veut dire cette instruction

Code:

---

PageFacebook pfb : p.getAmis()

---

Une petite lecture te fera du bien ;).

Pour moi il est évident que la version sans break est plus claire et plus efficace : moins de mots, moins d'assignations, moins de variables... Je ne vois aucun avantage à l'autre version.

merci pour le lien j'ai bien compris et merci pour le code proposé aussi, ça va le pb est résolu.

Citation:

---

Pour moi il est évident que la version sans break est plus claire et plus efficace : moins de mots, moins d'assignations, moins de variables... Je ne vois aucun avantage à l'autre version.

---

sur quelle version vous parlez s'il vous plait?

Celle-là :

Code:

---

1 2 3

for(PageFacebook amiDeMonAmi : p.getAmis()) if(amiDeMonAmi.getIdPage()==id) return amiDeMonAmi; return null;

---

apparemment ce que vous avez proposé aussi est juste mais moi je ne sais pas j'ai utilisé la fonction proposé par andry.aime car j'ai cru que c'est faux.
en tout cas merci pour toi aussi

Citation:

---

Envoyé par boubounagh

merci pour le lien j'ai bien compris et merci pour le code proposé aussi, ça va le pb est résolu.

---

:ccool: Tant mieux, n'hésite pas à passer ton sujet en :resolu: dans ce cas.

Citation:

---

Envoyé par BenWillard

Pour moi il est évident que la version sans break est plus claire et plus efficace : moins de mots, moins d'assignations, moins de variables... Je ne vois aucun avantage à l'autre version.

---

Je suis entièrement d'accord avec toi.
J'ajouterai même, elle fonctionne! ( La version avec break retourne le dernier élément de la liste si rien n'a été trouvé...)

Citation:

---

Envoyé par Fred_34

Je suis entièrement d'accord avec toi.
J'ajouterai même, elle fonctionne! ( La version avec break retourne le dernier élément de la liste si rien n'a été trouvé...)

---

Ça ça se corrige, c'est pas la question.

... Par contre, le bug aurait eu nettement moins de chances d'arriver avec l'approche "return", et ça, c'est déjà un peu plus la question.

Citation:

---

Envoyé par Fred_34

Je suis entièrement d'accord avec toi.
J'ajouterai même, elle fonctionne! ( La version avec break retourne le dernier élément de la liste si rien n'a été trouvé...)

---

arf, une erreur de ma part

Code:

---

1 2 3 4 5 6 7 8

PageFacebook amiDeMonAmi = null; for(PageFacebook pfb : p.getAmis()){ if(pfb.getIdPage()==id){ amiDeMonAmi=pfb; break; } } return amiDeMonAmi;

---

s'il vous qu'elle la différence entre les deux versions ?
et normalement les deux fonctionnent et donnent le bon resultat non?

Citation:

---

arf, une erreur de ma part

---

c'est quoi la différence entre les deux si tu fais l'affectation avant le if ou bien aprés?

Citation:

---

Envoyé par boubounagh

s'il vous qu'elle la différence entre les deux versions ?
et normalement les deux fonctionnent et donnent le bon resultat non?
c'est quoi la différence entre les deux si tu fais l'affectation avant le if ou bien aprés?

---

Si c'est avant le if, ça va retourner le dernier ami s'il ne trouve pas pfb.getIdPage qui est égal à l'id passé en paramètre.

ok merci c'est compris mais dans mon cas sur il va trouver la page qui correspond à l'id passé en paramètre.