Discussion :

[miller31]

Bonjour tout le monde,
Je suis confronté à un petit souci de logique. En effet, il nous est demandé de réaliser une interface graphique dans laquelle on utilise un Vector que l'on doit charger avec des données extérieures. Pour cette partie tout se passe bien mais mon problème est que dans l'interface graphique il y'a un bouton qui permet d'avancer et un autre qui permet de reculer d'un pas dans le Vector. C'est à ce niveau que ma compréhension prend un coup : j'ai fait un code pour avancer mais il m'affiche le mme objet. Comment dois-je procéder s'il vous plait :(:(:(:(

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
 public Personne suivantVector(Vector<Personne> p){
        Personne personne=null;//ListIterator<Personne> itera=null;
        if(p.isEmpty()){
           JOptionPane.showMessageDialog(null,"La base est vide.");
        }else{
           //itera=p.listIterator();
            if(counter<p.size()){
                personne=new Personne(p.elementAt(nbre));
 
              System.out.println(personne.getNom());
            }else{
            }
        }
       return personne;
    }

[joel.drigo]

Salut,

Code :

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

1
2
3
4
5
6
7
   if(counter<p.size()){
                personne=new Personne(p.elementAt(nbre));
 
              System.out.println(personne.getNom());
            }else{
            }
    }

A chaque fois que tu appelles là méthode, ça va chercher l'élément dans le Vector à la même position nbre : il faudrait que tu incrémentes nbre à ce moment (et décrémente dans l'autre méthode qui va chercher le précédent).
Et utilise qu'une seule variable. Tu en utilises 2 : counter et nbre. Tu vas devoir incrémenter les 2. Travail en trop, risque d'oubli et donc d'incohérence.
Il faut modifier également la borne ici : si l'index est égal à p.size()-1, il est bien inférieur à p.size(). Si on l'incrémente, on sort du Vector et donc erreur quand on récupère l'élément à cette position.
D'autre part, pourquoi créer une nouvelle instance de Personne ?

Code :

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

1
2
3
4
5
6
7
8
9
if ( index < p.size()-1 ) {
    index++
    personne = p.elementAt( index );
    // ou à la place des deux lignes précédentes : personne = p.elementAt( ++index );
    System.out.println(personne.getNom());
} else {
    // rien
}
}

[miller31]

Merci beaucoup de me repondre, je m'y mets et je vois ce que cela peut donner,et je vous ferai part de l'avancement,DANKE