Discussion :

[pingoui]

Bonjour,
Après avoir changé mon fusils d'épaule et travaillé sur un code plus simple, j'aimerai créer une fonction recherche pour mon programme.

Pour le moment, j'arrive sans aucun souci à faire une recherche mais celle-ci s'arrête lorsque le prog trouve un résultat. J'aimerai que celle--ci continu à rechercher dans le fichier xml pour trouver d'autre résultats.

En gros, j'ai un un fichier.xml qui contient des canaris. Chaque canari à un nom. J'aimerai trouvé tous les canaris qui on le nom "titi" par exemple !
Je sais que ça semble idiot d'avoir des canaris de même nom mais je compte utilisé cette fonction pour rechercher tous les canaris de m^me sexe par exemple.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
 
public class Annee { 
	  private List listeCanaris = new ArrayList();  
	  private String file;
	  private String canariMale;
 
 
	   public Annee() { 
	      super(); 
	   } 
 
	   public Annee(String file) { 
	      this(); 
	      this.file=file;
	   } 
 
	   public Canari getCanari(String nomCanari) { 
           for(Iterator ite = listeCanaris.iterator();ite.hasNext();){ 
               Canari unCanari = (Canari) ite.next(); 
               if(nomCanari.equals(unCanari.getNom())){ 
               return unCanari; // canari trouvé on le renvoie 
           } 
        } 
        // si on trouve pas ??? 
        System.out.println("Pas de canari du nom de "+nomCanari+" trouvé dans le fichier "+file+ "!!!"); 
        return null; 
       } 
 
 
	   public void removeCanari(String nomCanari){ 
	      Canari canariASupprimer = getCanari(nomCanari); 
	      if(canariASupprimer!=null) listeCanaris.remove(canariASupprimer); 
	   }  
 
       public void setCanari(String nomCanari, String nouveauNom){
       	  Canari canariAModifier = getCanari(nomCanari);
       	  if(canariAModifier!=null) canariAModifier.setNom(nouveauNom);
       }
 
	   public void addCanari(Canari unCanari) { 
	   	listeCanaris.add(unCanari);  
	   } 
 
	   public List getAnnee() { 
	      return listeCanaris; 
	   }  
 
	   public void setAnnee(List listeCanaris) { 
	      this.listeCanaris = listeCanaris; 
	   } 
 
	   public String getNomCanariMale(){ 
	    return canariMale; // ce qui est dans le flux 
	   } 
 
	 public Canari getNomCanariMale(String file){ 
	    //fait le lien entre le nom et les infos du canari 
	    return Annee.lire(file).getCanari(getNomCanariMale()); 
	   }

Pour le moment, j'arrive à retrouvé un canari en faisant:

Code :

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

1
2
3
 
Annee lecture = Annee.lire(nomDuFichier);
Canari unCanari = lecture.getCanari(nomDuCanari);

Dès que le porgramme trouve un canari, il arrête les recherches!


Je pense donc mettre les canaris dans une arrayList pour stocker tous le canari qui ont le nom "titi" par exemple.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
public ArrayList getPlusieursCanari(String nomCanari) { 
	       ArrayList array = new ArrayList(); 
	            for(Iterator ite = listeCanaris.iterator();ite.hasNext();){ 
	                Canari unCanari = (Canari) ite.next(); 
	                if(nomCanari.equals(unCanari.getNom())){ 
	      array.add(unCanari); 
	            } 
	      return array; 
	         } 
	 }

Avec cette fonction, je pense arriver à mes fins.... Mais j'ai cette erreur :

Code :

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

1
2
 
This method must return a result of type ArrayList

Help please

[adiGuba]

Salut,

Ton code est mal indenté, return est à l'intérieur du for...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public ArrayList getPlusieursCanari(String nomCanari) {
	ArrayList array = new ArrayList();
	for(Iterator ite = listeCanaris.iterator();ite.hasNext();) {
		Canari unCanari = (Canari) ite.next();
		if(nomCanari.equals(unCanari.getNom())) {
			array.add(unCanari);
		}
		return array;
	}
	/* le return devrait être ICI */
}

L'erreur vient du fait que dans certains cas (si lIterator est vide) tu ne rentre pas dans le for et du coups tu arrive à la fin de la méthode sans avoir fait de return ce qui provoque l'erreur à la compilation...

Un conseil : indente bien ton code...
En plus la plupart des editeurs Java propose une indentation automatique...

a++

[pingoui]

Super... merci beaucoup pour tes explications

Comment récupérer et afficher le résultat, maintenant?

Code :

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

1
2
3
4
 
Annee lecture = Annee.lire(nomDuFichier);
listeDeCanaris desCanari = lecture.getPlusieursCanari("titi");
System.out.println("Canaris trouvé:"+listeDeCanaris);

pour le moment, j'ai cette erreur :

Code :

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

1
2
 
listeDeCanaris cannot be resolved or is not a type

[adiGuba]

Ta méthode renvoit une ArrayList et non pas une listeDeCanaris !!!
D'ailleurs listeDeCanaris n'est pas un type mais le nom de ton attribut...

Essayce comme cela :

Code :

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

1
2
3
Annee lecture = Annee.lire(nomDuFichier);
List desCanari = lecture.getPlusieursCanari("titi");
System.out.println("Canaris trouvé:"+listeDeCanaris);

D'ailleurs il serait plus propre de retourner une List au lieu d'une ArrayList dans ta méthode getPlusieursCanari().
Ca te permet d'utiliser éventuellement un autre type de liste sans avoir à modifier autre chose que ta méthode getPlusieursCanari()

Plus d'explications dans les les cours et tutoriels pour apprendre Java : http://java.developpez.com/cours/

a++

[pingoui]

merci beaucoup

juste un petit truc....

j'ai ce résultat dans la console:

Code :

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

1
2
 
Canaris trouvé:[Canari@9f0d]

avec

Code :

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

1
2
 
System.out.println("Canaris trouvé:"+desCanari);

[adiGuba]

Normal tu stockes des objet Canari dans ta List.

Lorsque tu affiches une liste les éléments de la liste sont affiché de la manière suivante :

Code :

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

[element1, element2, element3, ...]

Dans ton cas il n'y a qu'un element...
Il t'affiche donc ton objet Canari.
Si tu veux un affichage + clair il faut surcharger toString()

a++

[pingoui]

ok.... je suis désolé mais je vais,encore , te poser une question c**

Si je fais :

Code :

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

1
2
 
System.out.println("Canaris trouvés:"+desCanari.toString());

je n'obtiens rien.... je pense que c'est normale...car il faut afficher la bonne structure

Un canari =

Code :

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

1
2
 
Canari(String nom,String sexe,int stam,int bague, int year)

Dans la class, j'ai des méthodes "setBidule et "getBidule"

comment lui indiquer la bonne structure poyur l'affichage?

Code :

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

System.out.println("Canaris trouvés:"+desCanari.getNom().toString() ,desCanari.getSexe().toString(), desCanari.getStam().toString(),desCanari.getBague().toString(),desCanari.getYear().toString()    );

[adiGuba]

Tu retournes un objet List et pas un objet Canari !

Tu dois donc parcourir ta liste :

Code :

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

1
2
3
4
5
6
Iterator iterator = desCanari.iterator();
 
while ( iterator.hasNext() ) {
	Canari unCanari = (Canari) iterator.next ();
	System.out.println ( unCanari.getNom() );
}

[pingoui]

Merci beaucoup adiGuba !

Ca marche !