Discussion :

[boubounagh]

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 :

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

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;
	}

[andry.aime]

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+.

[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 :

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

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;
	}

[andry.aime]

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

Code :

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

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

ou

Code :

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

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

A+.

[JoeChip]

Code :

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

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

Pourquoi pas

Code :

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

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

??

[andry.aime]

Code :

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

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

Pourquoi pas

Code :

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

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+.

[JoeChip]

Ah oui j'avais zappé le return

Mais il y a aussi

Code :

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

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

[andry.aime]

Ah oui j'avais zappé le return

Mais il y a aussi

Code :

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

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+.

[oneagaindoguys]

Ah oui j'avais zappé le return

Mais il y a aussi

Code :

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

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

[andry.aime]

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 :

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

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+.

[JoeChip]

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 :

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

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

[andry.aime]

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.

[thelvin]

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.

[andry.aime]

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 .
Mais c'est une convention de coder suggérée par sun et je le trouve aussi pratique.

A+.

[boubounagh]

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 :

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

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.

[andry.aime]

Code :

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

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;

[boubounagh]

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

Code :

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

PageFacebook pfb : p.getAmis()

[andry.aime]

Une petite lecture te fera du bien .

[JoeChip]

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.

[boubounagh]

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