Problème avec affichage contenu d'un tableau

Version imprimable

Bonjour,

Je voulais afficher le contenu de mon tableau,

voici le code
Code:

1 2 3 4 for(int i=0;i<lesclients.size();i++){ Client ccourant=(Client)lesclients.get(i); System.out.println (ccourant.numClt+"\t\t"+ccourant.prenomClt+"\t\t"+ccourant.nomClt+"\t\t"+ccourant.adresseClt); }
Client est une class
lesclients est un ArrayList de Client.

Le résultat me donne toujours le dernier élément du tableau et cela au tant de fois que d'éléments dans le tableau.

Au secours S'il vous plait.

07/08/2008, 14h45
link256

Citation:

Client est une class
lesclients est un ArrayList de Client.

Le résultat me donne toujours le dernier élément du tableau et cela au tant de fois que d'éléments dans le tableau.

Donc si j'ai bien compris si tu as 10 elements et que tu parcours ton ArraList

tu va afficher 10 fois ton dernier client.

Je pencherais plus pour un problème d'algo lors du remplissage de ton arraLyst
ou tu récuperais les données a chaque vois de chaque clients mais que tu enregistrerais toujours le même.

peux tu montrer le code utiliser pour son remplissage
07/08/2008, 16h28
demcoul

Voici le code de remplissage.
[CODE]
if (sortie.equals("1")) {
unclient.numClt="debut";int i=0;
while(!unclient.numClt.equals(" ")){
unclient.creer();
if(unclient.numClt.equals(" ")) break;
else {lesclients.add(i, unclient);i++;}
}
}

merci.
07/08/2008, 16h34
fnobb

Tu ne crée qu'une instance de client (unclient) puis tu change les valeurs et tu l'ajoute dans la liste => tu as donc dans la liste n fois le même objet avec les dernières modifications

dans ta boucle de remplissage commence par faire Client unClient = new Client();
07/08/2008, 16h37
fnobb

ou alors (je viens de voir le unClient.creer();) si cette méthode renvoie un bjet de type Client il faut remplacer la référence de unClient

Code:

unclient = unclient.creer()

Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 if (sortie.equals("1")) { unclient.numClt="debut"; int i=0; while(!unclient.numClt.equals(" ")) { unclient.creer(); if(unclient.numClt.equals(" ")) break; else { lesclients.add(i, unclient); i++; } } }
j'ai du mal à saisir ce que tu fais avec ce bout de code.
Pourrais tu donner plus d'explication car tu rentre toujours dans ta boucle While.
nous dire ce que tu aimerais que cette portion de code réalise ainsi que le code de ta classe client

Le code permet de lire un client et de le placer dans le tableau lesclients

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
if (sortie.equals("1")) {
    unclient.numClt="debut";int i=0;
    while(!unclient.numClt.equals(" ")){
         unclient.creer(); // lire un client au clavier
         if(unclient.numClt.equals(" ")) break;
        else {
                 // placer le client lu au clavier dans le tableau lesclients
                   lesclients.add(i, unclient);i++;
         }
   }
}

Le code de la classe Client est le suivant :

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
 
import java.io.*;
 
public class Client {
	//données membres
	String numClt;
	String nomClt;
	String prenomClt;
	String adresseClt;
	public Client(){}
	// fonctions membres
	void creer(){
		BufferedReader IN=null;
		try {
			IN=new BufferedReader(new InputStreamReader(System.in));
		} catch(Exception e){
			System.err.println("Erreur : "+e);
			System.exit(1);
		}
		try {
			System.out.print("Numero  : ");this.numClt=IN.readLine();
			System.out.print("Nom     : ");this.nomClt=IN.readLine();
			System.out.print("Prenom  : ");this.prenomClt=IN.readLine();
			System.out.print("Adresse : ");this.adresseClt=IN.readLine();
		} catch (IOException e) {
			// TODO Bloc catch auto-généré
			e.printStackTrace();
		}
	}
	void modifieradr(String adr){this.adresseClt =adr;}
	void afficher(){
		System.out.println("AFFICHE D'UN CLIENT");
		System.out.println("Numero  : "+this.numClt);
		System.out.println("Nom     : "+this.nomClt);
		System.out.println("Prenom  : "+this.prenomClt);
		System.out.println("Adresse : "+this.adresseClt);
	}
	void supprimer(){}
}

07/08/2008, 17h58
fnobb

Il faut donc dans la boucle qui fait "lesclients.add(i, unclient);"
(bien que "lesclients.add(unclient);" aura suffit) commencer par instancier un nouvel objet client.

Code:

Client unClient = new Client();
07/08/2008, 18h16
demcoul
Cela existe déjà en debut du code. Voici les lignes de définition des variables
Code:

1 2 3 String sortie=""; Client unclient=new Client(); List<Client> lesclients=new ArrayList<Client>();
08/08/2008, 09h23
link256

Oui mais comme fnobb te le dit il faut faire la déclaration dans la boucle

en dehors de celle-ci tu met

Code:

Client unClient

Et dans ta boucle for

Code:

unClient = new Client();

Si tu ne fais pas ca tu travail toujour sur le même objet client1 que tu modifies et que tu insert n fois dans ton tableau.

tu veux cree client1 tu appelles ta méthode creer pour renseigner les infos le concernant.
puis même chose pour le client 2 ect ...

la tu demande en boucle à ton client1 de redonner ces informations.

il faut faire quelque chose comme

Code:

1
2
3
4
5
6
7
8
9
10
String sortie="";
Client unclient;
List<Client> lesclients=new ArrayList<Client>();
.....
for(int i =0; i<......
{
	unclient = new Client();
	......
	lesclients.add(unclient);
	......