Récupération d'une collection

Version imprimable

Bonjour

Voila mon souci; j'ai un programme qui sert à récupérer des fichiers XML correspondants à des mairies de villes. Ces fichiers contiennent pleins d'infos sur les mairies mais on ne se contente que du nom, du code insee, du code postal, du téléphone, de l'email et du site internet.

Dans une interface graphique je dois afficher un annuaire complet (je l'ai fait grâce à une collection) et pouvoir rechercher une mairie par nom.

Le soucis c'est que je n'arrive pas à prendre en compte la collection que j'ai créé pour l'annuaire afin de faire la recherche. A chaque fois que je fais une recherche (via un champ de saisie puis un clique sur un bouton "search"), pleins d'erreurs s'affichent dans la console.

Voici quelques bouts de code:

Dans la classe Collection (qui ajoute les villes à une collection et qui gère la recherche):
Code:

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 public CollecVille () { } private ArrayList<Ville> vil = new ArrayList<Ville>(); public void add(Ville commune) { if (!vil.contains(commune)) { //si la collection ne contient pas la commune vil.add(commune); //alors ajouter la commune à la collection } } public String searchNom(String name) throws NoSuchElementException{ Iterator<Ville> iterator = vil.iterator(); System.out.println("****************"); while (iterator.hasNext()) { Ville commune = iterator.next(); if (commune.getNom().contains(name)) { return commune.toString(); } } throw new NoSuchElementException("Aucune ville dont le nom contient <<" + name + ">> n'a été trouvée"); }
Une partie de la classe de l'interface graphique (le bouton4, c'est le bouton "search")

Code:

private CollecVille inter = new CollecVille();
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 //Bouton Search par nom try { bouton4.addActionListener( new ActionListener () { public void actionPerformed (ActionEvent e) { JFrame fnom = new JFrame("Recherche par nom"); fnom.setBounds(0, 210, 465, 600); JTextArea textArea = new JTextArea(inter.searchNom(nameField.getText())); System.out.println("Nom = " + inter.searchNom(nameField.getText())); textArea.setBackground(new Color(253,191,183)); textArea.setEditable(false); fnom.add(textArea); fnom.setContentPane(new JScrollPane(textArea)); fnom.setVisible(true); } } );
Le Main dans lequel on créé la collection

Code:

CollecVille inter = new CollecVille();
Code:

1 2 v = new Ville(nom,codeInsee,codePostal,telephone,email, url); inter.add(v);
Les messages d'erreur:

Exception in thread "AWT-EventQueue-0" java.util.NoSuchElementException: Aucune ville dont le nom contient <<Recherche par nom>> n'a été trouvée

at collection.CollecVille.searchNom(CollecVille.java:35)

at collection.AnnuGUI$4.actionPerformed(AnnuGUI.java:131)

at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)

at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)

at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)

at javax.swing.DefaultButtonModel.setPressed(Unknown Source)

at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source)

at java.awt.Component.processMouseEvent(Unknown Source)

at javax.swing.JComponent.processMouseEvent(Unknown Source)

at java.awt.Component.processEvent(Unknown Source)

at java.awt.Container.processEvent(Unknown Source)

at java.awt.Component.dispatchEventImpl(Unknown Source)

at java.awt.Container.dispatchEventImpl(Unknown Source)

at java.awt.Component.dispatchEvent(Unknown Source)

at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)

at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)

at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)

at java.awt.Container.dispatchEventImpl(Unknown Source)

at java.awt.Window.dispatchEventImpl(Unknown Source)

at java.awt.Component.dispatchEvent(Unknown Source)

at java.awt.EventQueue.dispatchEventImpl(Unknown Source)

at java.awt.EventQueue.access$200(Unknown Source)

at java.awt.EventQueue$3.run(Unknown Source)

at java.awt.EventQueue$3.run(Unknown Source)

at java.security.AccessController.doPrivileged(Native Method)

at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)

at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)

at java.awt.EventQueue$4.run(Unknown Source)

at java.awt.EventQueue$4.run(Unknown Source)

at java.security.AccessController.doPrivileged(Native Method)

at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)

at java.awt.EventQueue.dispatchEvent(Unknown Source)

at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)

at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)

at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)

at java.awt.EventDispatchThread.pumpEvents(Unknown Source)

at java.awt.EventDispatchThread.pumpEvents(Unknown Source)

at java.awt.EventDispatchThread.run(Unknown Source)

28/12/2013, 12h36
joel.drigo
Salut,

Le message
```
Exception in thread "AWT-EventQueue-0" java.util.NoSuchElementException: Aucune ville dont le nom contient <<Recherche par nom>> n'a été trouvée
```
devrait te mettre sur la voie...

Ton JTextArea, dans ton dialogue, je suppose que c'est pour afficher le résultat de la recherche d'une ville dont le nom a été tapé dans le champ nameField.

Or tu initialises ton JTextArea par :
Code:

1 2 3 ... JTextArea textArea = new JTextArea(inter.searchNom(nameField.getText())); ...
Le problème est que quand la recherche ne trouve rien, tu soulèves une exception.
Ce qui n'est pas si mal, bien que chercher quelque chose qui n’existe pas n'est pas vraiment une erreur (en plus, l'exception choisie n'est pas adéquate : elle signifie qu'il n'y a plus d'éléments à récupérer, et non pas qu'aucun ne correspond), mais ce serait plus simple à gérer en amont de retourner tout simplement null (et de retourner la ville trouvée, le cas échéant, plutôt qu'une String).
Bref, avec l'exception, la création de textArea va planter lorsque le texte contenu dans nameField ne correspond pas à une ville existante.

Il faudrait que tu fasses, par exemple :
Code:

1 2 3 4 5 6 7 8 9 10 ... String text; try { text=inter.searchNom(nameField.getText()); } catch(NoSuchElementException e) { text=e.getMessage(); } JTextArea textArea = new JTextArea(text); ...
28/12/2013, 12h49
uluk7

Merci pour cette première réponse. En effet, ça permet au moins de virer toutes les erreurs dans la console.

En revanche, je n'ai toujours pas accès à la collection que je créé dans le Main :?
Du coup, peu import ma recherche, il va constamment m'envoyer le message "Aucune ville dont le nom contient <<...>> n'a été trouvée" dans ma fenêtre de réponses à la recherche.

Tu fais ta recherche dans la liste de villes représentée par la variable inter : c'est cette liste que tu dois remplir. C'est sûr que si tu remplis une autre variable, ça ne fonctionnera pas.

Tu dois avoir qu'une seule variable de type CollecVille, et pouvoir y accéder depuis les différentes classes qui en ont besoin. Je n'ai pas toute la structure de ton programme, donc je ne peux te donner de directives précises. Mais on peut imaginer, que si tu la créés dans ton main, tu peux la passer en argument du constructeurs des différentes classes qui en ont besoin.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
public class Program {
 
   public static void main(String[] args) {
 
       CollecVille inter= ...
 
 
       JFrame frame = new AnnuGUI(inter);
 
   }
 
}

Code:

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
public class AnnuGUI extends JFrame {
 
    private CollecVille inter;
 
    public AnnuGUI(CollecVille inter) {
 
         ...
         this.inter= inter;
         ...
 
        bouton4.addActionListener(
            new ActionListener ()
            {
                public void actionPerformed (ActionEvent e)
                {
                    JFrame fnom = new JFrame("Recherche par nom"); 
                    fnom.setBounds(0, 210, 465, 600);
                    JTextArea textArea = new JTextArea(inter.searchNom(nameField.getText()));      
                    System.out.println("Nom = " + inter.searchNom(nameField.getText()));   
                    textArea.setBackground(new Color(253,191,183));                
                    textArea.setEditable(false);               
                    fnom.add(textArea);                    
                    fnom.setContentPane(new JScrollPane(textArea));                    
                    fnom.setVisible(true);
                }
            }
        );
 
 
 
}

[code

28/12/2013, 13h27
uluk7

Pu****
C'est tout con :cry:

ça marche !

Dernier petit soucis: lorsque je fais une recherche, il ne m'affiche qu'un résultat, alors que plusieurs sont possibles
par exemple, je tape "Villiers", il devrait me renvoyer Villers-en-Arthies, Villiers-Adam, Villiers-le-Bel et Villiers-le-Sec. Or il ne me renvoie que Villiers-Adam.

Citation:

Envoyé par uluk7

Dernier petit soucis: lorsque je fais une recherche, il ne m'affiche qu'un résultat, alors que plusieurs sont possibles
par exemple, je tape "Villiers", il devrait me renvoyer Villers-en-Arthies, Villiers-Adam, Villiers-le-Bel et Villiers-le-Sec. Or il ne me renvoie que Villiers-Adam.

Bah, ta méthode retourne une valeur, dès qu'elle trouve une ville qui correspond :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public String searchNom(String name) throws NoSuchElementException{
        Iterator<Ville> iterator = vil.iterator();
        System.out.println("****************");
        while (iterator.hasNext()) {
            Ville commune = iterator.next();
            if (commune.getNom().contains(name)) {
 
                return commune.toString();
            }
 
        }
        throw new NoSuchElementException("Aucune ville dont le nom contient <<" + name + ">> n'a été trouvée");
 
    }

Si tu veux retourner toutes les villes qui correspondent, il faudra les cumuler dans un résultat, et retourner le résultat à la fin. Par exemple :

Code:

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
public CollecVille () {
 
    }
 
    private List<Ville> villes = new ArrayList<>();
 
    public void add(Ville ville) {
        if (!villes.contains(ville)) { //si la collection ne contient pas la commune
            villes.add(ville);   //alors ajouter la commune à la collection
        }
    }  
 
 
public List<Ville> searchNom(String name) throws NoSuchElementException{
 
 
        List<Ville> villesTrouvees = new ArrayList<>();
 
        for(Ville ville : villes ) {
            if (ville.getNom().contains(name)) {
 
                villesTrouvees.add(ville);
 
            }
 
        }
        if ( villesTrouvees.isEmpty() ) {
        throw new NoSuchElementException("Aucune ville dont le nom contient <<" + name + ">> n'a été trouvée");
        }
        return villesTrouvees.
    }

Et gérer l'affichage :

Code:

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
bouton4.addActionListener(
            new ActionListener ()
            {
                public void actionPerformed (ActionEvent e)
                {
                    JFrame fnom = new JFrame("Recherche par nom"); 
                    fnom.setBounds(0, 210, 465, 600);
                    
                    String resultat;
                    try {
                           List<Ville> villes = inter.searchNom(nameField.getText());
                           StringBuilder sb=new StringBuilder();
                           boolean first=true;
                           for(Ville ville : villes) {
                                   if  (first) first=false; else sb.append("\n");
                                   sb.append(ville);
                           }
                           resultat=sb.toString();
                    } catch( NoSuchElementException e ) {
                           resultat = e.getMessage();
                    }

                    JTextArea textArea = new JTextArea(resultat);      
                    textArea.setBackground(new Color(253,191,183));                
                    textArea.setEditable(false);               
                    fnom.add(textArea);                    
                    fnom.setContentPane(new JScrollPane(textArea));                    
                    fnom.setVisible(true);
                }
            }
        );

28/12/2013, 13h51
uluk7

Et bien je te remercie pour ton aide. ça marche super bien.
:ccool: