Problème méthode recrutement

Version imprimable

Salut :mrgreen:
j'ai une question a vous poser car j'ai un petit probleme dans mon code, d'ailleurs le voici :

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
 
class Entreprise {
   Cellule chef;
 
   public Entreprise() {
      this.chef = null;
   }
 
   public Entreprise(Employe e) {
      this.chef = new Cellule(e);     
   }
 
   public void changerChef(Employe empi) {
      Cellule c1 = new Cellule(empi);
      c1.suivante = this.chef;
      this.chef = c1;
   }
 
   public void ajout(Employe empi) {
      Cellule c1 = new Cellule(empi);
      if(this.chef != null) {
         c1.suivante = this.chef.suivante;
         this.chef.suivante = c1;
      }
      else {
         this.chef = c1;
      }
   }
 
   public void affiche() {
      for(Cellule c = this.chef; c != null; c = c.suivante) {
         c.emp.affiche();
      }                   
   }
 
   public void recrute(Employe empi) {
      Cellule c1 = new Cellule(empi);
      for(Cellule c = this.chef; c != null; c = c.suivante) {
         if(c.suivante == null) {
            c.suivante = c1;
         }
      }
   }    
 
   /*public void augmentation() {
      for(Cellule c = this.Chef; c != null; c = c.suivante) {
   }*/
}

Code:

1
2
3
4
5
6
7
8
9
class Cellule {
   Employe emp;
   Cellule suivante;
 
   public Cellule(Employe e) {
      this.emp = e;
      this.suivante = null;
   }
}

Voilà, j'ai donc deux classes, entreprise et cellule (j'ai aussi une classe employé mais elle n'est pas utile ici je pense) et j'ai un probleme au niveau de la méthode recrute(), le probleme est qu'elle ne marche pas ! Je veux en fait ajouter un employer a mon entreprise, qui se situera à la FIN de celle - ci (c'est - a - dire derriere tous les autres employers). Quand j'execute, il me semble que la boucle boucle indéfiniment car rien ne s'affiche.

Voilà merci de votre aide :D

21/02/2014, 00h06
Neolex
tout d'abord je te conseille de mettre "suivante" en private dans la classe Cellule et de faire une methode getSuivante() et setSuivante() .
Pour ajouter à la fin de la liste je ferais ça :
Code:

1 2 3 4 5 6 7 8 9 public void recrute(Employe empi) { Cellule c1 = new Cellule(empi); Cellule c = this.chef; while(c.getSuivante()!=null) { c=c.getSuivante(); } c.setSuivante(c1); }
Et je n'ai pas trop compris ce que tu veux faire de différent dans la méthode ajout et la méthode recrute.
21/02/2014, 00h22
tamerla

Merci de ta réponse , ça fonctionne mais je ne comprends pas pourquoi mon for ne marche pas, c'est la même chose non?
En fait la méthode ajout() permet de rajouter une cellule employé juste après la cellule chef, tandis que recrute() la place à la fin.
Et je suis d'accord avec toi pour le private sur suivante mais on me demande de laisser l'attribut tel quel donc bon (de toute facon après il ne me sert plus puisque la fonction ajout() le remplace).

Code:

1 2 3 4 5 6 7 8 public void recrute(Employe empi) { Cellule c1 = new Cellule(empi); for(Cellule c = this.chef; c != null; c = c.suivante) { if(c.suivante == null) { c.suivante = c1; } } }
Je pense que ton problème est que "c != null " est toujours vrai puisque si c.suivante = null , tu rajoute une cellule , donc tu rajoute ta cellule à la derniere position mais retourne dans la boucle for et la ré ajoute à la fin .

Edit : pour régler ton problème tu peux utiliser soit la methode que je t'ai donnée , soit ajouter un "break" dans le if pour sortir de la boucle for

21/02/2014, 00h40
tamerla

Merci! ca me parait évident maintenant , j'avais pas fait attention, il me suffisait donc de rajouter un break; apres c.suivante = c1;

Merci de ton aide (rapide!). Problème résolu.