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

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