Ajout de données dans une ArrayList

Version imprimable

02/11/2010, 11h15
McArthur

Ajout de données dans une ArrayList
Bonjour, j'ai un soucis avec la gestion d'un ArrayList.
Il s'agit d'une gestion de commande de fournitures. Une "ligne" de ma list doit au final contenir un nom, un article, et une quantité.
En utilisant un constructeur de type (String, Article, int) je n'ai pas de soucis tout est bien ajouté.
Seulement je voudrais créer une commande avec seulement le nom et plus tard lui ajouter articles et quantité.

Mon main ressemble à ça :
Code:

1 2 3 4 5 6 Commande uneCommande = new Commande("Alex"); Stylo unStylo = new Stylo(....); // Création d'un article uneCommande.addLigne(unStylo, 3); // on commande 3 stylos "unStylo"
Mon problème c'est donc le "addLigne". Je ne vois pas comment "ajouter" les informations à un objet déjà créé.
Merci de votre aide.

Je ferais plutôt une map des articles avec en regard la quantité :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
class Commande  {
 
   String nom;
   Map<Article, Quantite> mapArticles = new HashMap<Article, Quantite>();
 
   public void addLigne(Article unArticle, int enQuantite) {
      Quantite quantite = mapArticles.get(unArticle);
      if (quantite == null) {
         quantite = new Quantite();
         mapArticles.put(unArticle, quantite);
      }
      quantite.n += enQuantite;
   }
}
 
class Quantite {
   int n;
}

02/11/2010, 12h57
McArthur

N'y a t il pas un moyen d'appeller un constructeur de Commande depuis la méthode addLigne ?
J'aurais ainsi dans commande deux constructeurs

Commande(String nom); // Pour la premiere création de commande
Commande(String nom, Articles a, int q); // Pour lorsque je veux ajouter

Et dans addLigne je voudrais faire un appel au second.

(je suis sûr que cela doit fonctionner avec Map comme tu me le conseil, mais je n'ai pas encore vu ça, et l'exercice était faisable sans, donc soit je cherche trop compliqué, soit je loupe un truc evident :? )
02/11/2010, 13h04
herve91

addLigne est bien une méthode de Commande ? quelle est sa signature ?
02/11/2010, 13h09
McArthur

Oui addLigne est bien une méthode de Commande. Qu'appelles-tu "sa signature" ?
02/11/2010, 13h14
herve91

Par exemple dans un message plus haut, on trouve :

Code:

public void addLigne(Article unArticle, int enQuantite)

c'est la signature de la méthode addLigne

Je ne comprends pas bien, une commande est-elle composée de plusieurs articles, chacun dans une quantité donnée ?
Si c'est la cas je ne vois pas en quoi la méthode addLigne devrait appeler un constructeur de Commande puisque addLigne est une méthode de Commande !
02/11/2010, 13h23
McArthur

Je ne comprends pas bien, une commande est-elle composée de plusieurs articles, chacun dans une quantité donnée ?

Oui! je crée une commande au debut de mon main, et je dois pouvoir lui ajouter des articles au fur et à mesure.

uneCommande.addLigne(unStylo, 3)
uneCommande.addLigne(unFeutre, 5)
...

Et à la fin je dois pouvoir afficher tout ce que j'ai ajouté à uneCommande.

Si c'est la cas je ne vois pas en quoi la méthode addLigne devrait appeler un constructeur de Commande puisque addLigne est une méthode de Commande !

Ma question était un peu désespérée en fait :aie:. Je ne vois pas comment ajouter les commandes au fur et à mesure, dans une même commande...
02/11/2010, 13h38
herve91

Comment as-tu associé la quantité à un article dans l'ArrayList de Commande ?

Justement ! C'est la tout mon problème :)

Je crée une commande avec le nom de la personne :

Commande uneCommande = new Commande("Alex");

Je crée un Article :

Article unStylo = new Article(.......);

Je veux ajouter cet article à ma commande uneCommande (et donc l'associer à "Alex") via la méthode addLigne !

Ma classe commande ressemble à ça :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 public class Commande { private String nom; private Articles article; private int quantite; private static ArrayList<Commande> toutesLesCommandes = new ArrayList<Commande>(); Commande(String name){ nom = name; } public void addLigne(Articles art, int nb){ // Je fais quoi la dedans ?? ;) }
Je précise qu'en première intention, j'avais fait ceci :
Code:

1 2 3 4 5 6 7 8 9 public void addLigne(Articles art, int nb){ article = art; quantite = nb; toutesLesCommandes.add(this); }
Mais la seconde commande effacait toujours la première...

02/11/2010, 14h02
herve91
toutesLesCommandes n'est pas une ArrayList de Command mais d'Articles !
Comment doit réagir ton application si on fait :
Code:

1 2 uneCommande.addLigne(unStylo, 3); uneCommande.addLigne(unStylo, 5);
Doit-on avoir deux lignes correspondant à :

Citation:

unStylo 3
unStylo 5

ou une seule ligne correspondant à :

Citation:

unStylo 8

?
02/11/2010, 14h11
McArthur
C'est une très bonne question... :D

Je dois afficher un résumé de ma commande (via System.Out.println(uneCommande)).

Je pense que je peux me contenter de
Code:

1 2 3 unStylo 3 unStylo 5
pour le moment !

Ok alors ceci devrait suffire :

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
49
50
51
52
53
54
	public class Article {
 
		private String nom;
 
		public Article(String nom) {
			this.nom = nom;
		}
 
		public String getNom() {
			return nom;
		}
	}
 
	public class Ligne {
 
		private Article article;
		private int quantite;
 
		public Ligne(Article article, int quantite) {
			this.article = article;
			this.quantite = quantite;
		}
 
		public Article getArticle() {
			return article;
		}
 
		public int getQuantite() {
			return quantite;
		}
	}
 
	public class Commande {
 
		private String nom;
		private List<Ligne> listeLignes = new ArrayList<Ligne>();
 
		public Commande(String nom) {
			this.nom = nom;
		}
 
		public String getNom() {
			return nom;
		}
 
		public void addLigne(Article article, int quantite) {
			Ligne ligne = new Ligne(article, quantite);
			listeLignes.add(ligne);
		}
 
		public List<Ligne> getlisteLignes() {
			return listeLignes;
		}
	}

J'ai "simplifier" le problème en recommencant de zero.
Mon main se résume à ça:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
public class Appli {
 
	public static void main(String[] args) {
 
		Liste pers1 = new Liste("Mouse", "Mickey");
 
		Pouvoir pow1 = new Pouvoir("Voler");
		Pouvoir pow2 = new Pouvoir("Teleporter");
 
		pers1.addPower(pow1,10);
		pers1.addPower(pow2,50);
 
 
		Liste.affAll();
 
	}
 
}

En sortie j'obtiens :

Code:

1
2
3
 
 Mickey Mouse Teleporter 50
 Mickey Mouse Teleporter 50

Alors que je voudrais :

Code:

1
2
3
 
 Mickey Mouse Voler 10
 Mickey Mouse Teleporter 50

Le code de la classe Liste :

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
 
import java.util.ArrayList;
 
 
public class Liste {
 
	private String nom;
	private String prenom;
	private Pouvoir pouvoir;
	private int puissance;
 
	private static ArrayList<Liste> toutesLesPersonnes = new ArrayList<Liste>();
 
 
	Liste(String fName, String lName){
 
		nom = lName;
		prenom = fName;
 
	}
 
 
 
	public void addPower(Pouvoir pow, int power){
 
		pouvoir = pow;
		puissance = power;
		toutesLesPersonnes.add(this);
 
	}
 
	public static void affAll(){
 
		for(int i = 0; i < toutesLesPersonnes.size(); ++i)
			System.out.println(toutesLesPersonnes.get(i));
 
	}
 
	public String toString(){
 
		return(nom + " " + prenom + " " + pouvoir + puissance);
	}
 
 
}

Et celui de la classe pouvoir :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
 
 
public class Pouvoir {
 
	private String nom;
	private int puissance;
 
	Pouvoir(String name){
 
		nom = name;
 
 
	}
 
	public String toString(){
 
		return nom;
 
	}
}

Ca me semble plus clair, mais le pb (et donc le résultat) est le même !

Merci de ton aide, en attendant, je regarde la solution que tu me propose.

En fait, tu veux gérer (dans la dernière version) une liste de pouvoir dans ta classe Liste. Alors qu'avec le code source que tu fournis, tu ne gères qu'un seul et unique pouvoir par "Liste" (le nom de classe est très mal choisi).

En me basant sur les sources de ton dernier post, voici une solution :
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 49 50 51 52 53 54 55 import java.util.ArrayList; import java.util.Hashtable; public class Pouvoir { private String nom; Pouvoir(String name){ nom = name; } public String toString(){ return nom; } } public class Liste { private String nom; private String prenom; private int puissance; private Hashtable pouvoirs; private static ArrayList<Liste> toutesLesPersonnes = new ArrayList<Liste>(); Liste(String fName, String lName){ nom = lName; prenom = fName; toutesLesPersonnes.add(this); pouvoirs = = new Hashtable(); } public void addPower(Pouvoir pow, int power){ pouvoirs.put(pow, power); } public static void affAll(){ for(int i = 0; i < toutesLesPersonnes.size(); ++i){ System.out.println(toutesLesPersonnes.get(i)); } } public String toString(){ String buffer = nom + " " + prenom + " : "; for (Pouvoir pow : pouvoirs.keys()) { buffer += "\r\n\t" + pow.toString() + " "+ pouvoirs.get(pow); } return(buffer + "\r\n"); } }
Par contre dans le toString de la classe Liste il manque peut être un cast (pour le foreach) et surtout il faut revoir ton découpage objet avec par exemple :
Une classe Personnage avec nom, prénom et liste de pouvoir
une classe Pouvoir avec nom et puissance
une classe ListePersonnage avec une liste de personnage (ou mieu héritant de ArratList<Personnage> pour la manipuler plus facilement

Alors la réponse simpliste que je cherchais était celle là:
Simplement rajouter une ArrayList dans ma classe Liste (qui oui à un très mauvais nom), et dans addPower, ajouter le pouvoir à cette liste.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
public class Appli {
 
	public static void main(String[] args) {
 
		Liste pers1 = new Liste("Mouse", "Mickey");
 
		Pouvoir pow1 = new Pouvoir("Voler");
		Pouvoir pow2 = new Pouvoir("Teleporter");
 
		pers1.addPower(pow1);
		pers1.addPower(pow2);
 
 
		Liste.affAll();
 
	}
 
}

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
 
 
import java.util.ArrayList;
 
 
public class Liste {
 
	private String nom;
	private String prenom;
	private Pouvoir pouvoir;
	private int puissance;
	private ArrayList mesPouvoirs = new ArrayList();
	private static ArrayList<Liste> toutesLesPersonnes = new ArrayList<Liste>();
 
 
 
	Liste(String fName, String lName){
 
		nom = lName;
		prenom = fName;
 
	}
 
 
 
	public void addPower(Pouvoir pow){
 
		mesPouvoirs.add(pow);
 
	}
 
	public static void affAll(){
 
		for(int i = 0; i < mesPouvoirs.size(); ++i)
			System.out.println(mesPouvoirs.get(i));
 
	}
 
	public String toString(){
 
		return(nom + " " + prenom + " " + pouvoir + puissance);
	}
 
 
}

Voilà, merci à tous pour vos idées et solutions éventuelles. :ccool: