Boucle infini dans mon affichage

Version imprimable

Bonjour à tous,

Comme indiqué dans le titre, mon affichage me met une boucle infini sur le 1er élement de ma liste :( C'est incompréhensible!

Code:

1
2
3
4
5
6
7
8
9
10
11
12
public class ListeChainee {
            Personne contenu;
            ListeChainee suivant;
 
         public void afficher(){
		ListeChainee l = this;
		while(l!=null){
			l.getContenu().afficher();
			l = l.suivant;
		 }
	   }
}

Code:

1
2
3
4
5
6
7
8
public class Personne {
	Cle cle;
	Info info;
 
              public void afficher(){
		System.out.println(this.getCle().getName()+" : 0"+this.getInfo().getNumber());
	}
}

Code:

1
2
3
4
5
6
7
8
public class Test {
	public static void main(String[]args){
                    ListeChainee l = new ListeChainee();
		l.ajout("Morty", 82);
		l.ajout("JD", 82);
		l.afficher();
	}
}

Et il me répète en boucle "JD : 082"
Comment faire?

Je vous remercie :)

17/11/2010, 20h08
Rei Ichido

Sans plus de code, et en particulier la méthode ajout, ça va être dur de répondre.

A priori, c'est mal codé, et JL a pour suivant lui même.

Mille excuses Rei :)

Code:

1
2
3
4
5
6
public void ajout (String qui, int uneInfo){
		ListeChainee l = this;
		this.setContenu(new Personne(new Cle(qui), new Info(uneInfo)));
		this.setSuivant(l);
		System.out.println(qui+" est maintenant dans l'annuaire. Le numero associe a "+qui+" est 0"+this.getContenu().getInfo().getNumber());
	}

Qu'entends-tu par "JL a pour suivant lui même"?

17/11/2010, 21h28
adiGuba
Salut,

Citation:

Envoyé par Cellendhyll82

Qu'entends-tu par "JL a pour suivant lui même"?

La réponse est dans ton code :
Code:

1 2 ListeChainee l = this; this.setSuivant(l);
Ce qui revient à faire ceci :

Code:

this.setSuivant(this);

a++
18/11/2010, 00h28
Cellendhyll82

Je viens de "comprendre" mon erreur. Je dis "comprendre" car j'arrive à faire tourner le programme mais ne comprend pas pourquoi.

J'ai remarqué que le programme tournait si j'écrivais "ListeChainee l = new ListeChainee(this)" au lieu de "ListeChainee l = this" et je ne comprends pas la différence.

De même, si j'écris "while(this.getContenu()!=null)" au lieu de "while(l!=null)", dans la methode d'affichage, je n'ai pas de nullPointerException. Pourriez vous m'expliquer pourquoi svp?

Je vous remercie pour votre aide :)

Mais pourquoi est-ce que tu veux que le suivant de l'élément ajouté soit lui même, ou une copie de lui même ? Ce que tu veux faire, c'est ajouter un _nouvel_ élément. Cela dit il faudrait voir les constructeurs disponibles :mrgreen:
Code:

1 2 3 4 5 6 7 public void ajout (String qui, int uneInfo){ ListeChainee l = new ListChainee(); l.setContenu(new Personne(new Cle(qui), new Info(uneInfo))); this.setSuivant(l); System.out.println(qui+" est maintenant dans l'annuaire. Le numero associe a "+qui+" est 0"+this.getContenu().getInfo().getNumber()); }
Edit : cela dit, ce code donne une "liste chainée" avec au plus deux éléments ... Il faudrait ou bien stocker le dernier élément, ou bien une méthode qui parcourt toute la chaine pour renvoyer le dernier élément, ou bien stocker le dernier en premier, mais dans ce cas la méthode ajout doit renvoyer la nouvelle instance.