Discussion :

[Cellendhyll82]

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 :

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

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 :

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

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 :

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

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

[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.

[Cellendhyll82]

Mille excuses Rei

Code :

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

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"?

[adiGuba]

Salut,

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

La réponse est dans ton code :

Code :

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

1
2
ListeChainee l = this;
this.setSuivant(l);

Ce qui revient à faire ceci :

Code :

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

this.setSuivant(this);

a++

[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

[Rei Ichido]

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

Code :

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

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.