Discussion :

[demcoul]

Bonjour,

Je voulais afficher le contenu de mon tableau,

voici le code

Code :

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

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.

[link256]

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

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

[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();

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

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

unclient = unclient.creer()

[link256]

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

[demcoul]

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

Code :

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

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 :

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

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(){}
}

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

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

Client unClient = new Client();

[demcoul]

Cela existe déjà en debut du code. Voici les lignes de définition des variables

Code :

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

1
2
3
String sortie="";
Client unclient=new Client();
List<Client> lesclients=new ArrayList<Client>();

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

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

Client unClient

Et dans ta boucle for

Code :

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

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.

[fnobb]

il faut faire quelque chose comme

Code :

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

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);
	......