Discussion :

[Narizuke]

Bonjour tout le monde,

Je viens vous demander votre aide. Pour être plus compréhensible, je vais mettre un exemple.
J'ai une classe voiture avec comme attribut, l'immatriculation de la voiture, la marque de la voiture, année, modèle.
J'ai une ArrayList<Voiture> qui ait au préalable rempli. Dans mon écran, j'ai un arbre qui se présente comme suit :

Renault, Clio :
- 2014
- 2015

Renault, Mégane :
- 2000
- 2010

Ford, Fiesta:
- 1999
- 2008

J'aimerais grouper mon ArrayList par Marque_de_voiture,modèle, et que pour chaque Couple(Marque, Modèle), je récupère la liste des années correspondantes, comme sur mon arbre au dessus.

Je vous remercie d'avance pour votre aide.

Nari.

[Népomucène]

Pour pouvoir trier / regrouper les véhicules, il faut pouvoir dire :
"ce véhicule A doit être classé avant ce véhicule B"
ta classe Voiture doit donc implémenter l'interface comparable
Dans la méthode CompareTo, tu dois donner les critères qui font qu'un objet Voiture sera classé avant un autre objet Voiture.

Après, pour classer ton ArrayList, tu utiliseras la méthode statique Collections.sort
voir : https://docs.oracle.com/javase/tutor...ces/order.html

[Narizuke]

Merci pour votre réponse.

J'avais faire un truc dans le genre :

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
 
 
                List<Voiture> listVoitureGroup = new ArrayList<Voiture>();
		List<String> listAnnee= new ArrayList<String>();
		List<String> listGroupFinal = new ArrayList<String>();
 
		for(Voiture v : listeVoiture){
			if(!listAnnee.contains(v.getAnnee())){
				listAnnee.add((v.getAnnee());
				if(!listGroupFinal.contains(v.getMarque()+ " " + tire.getModele()){
					listGroupFinal.add(v.getMarque()+ " " + tire.getModele());
					listVoitureGroup .add(v);
				}
			}
		}

J'arrive bien à coupler par Marque,Modele (y'a surement mieux en code). Par contre, la liste des années pour chaque couple est HS. Je m'explique, je me retrouve avec 2014,2015,2000,2010,1999,2008 pour chaque couple :/
Juste pour confirmer, le compareTo() à le même comportement que le contains() ?

[eulbobo]

Juste pour confirmer, le compareTo() à le même comportement que le contains() ?

Non, l'un sert à comparer les éléments entre eux, l'autre à savoir s'ils sont égaux.
De base, deux objets pour être égaux doivent pointer vers la même instance d'objet, mais si tu implémentes les méthodes hashCode et equals, tu peux faire en sorte que deux instance différentes d'objet soient considérées égales en fonction de leur propriétés internes


Sinon, dans ton code, tu as une seule liste pour TOUS les groupes, donc tous les groupes auront forcement toutes les années ^^
Il faut créer une liste par groupe

[Népomucène]

compareTo te permet de définir ta propre règle de classement.

On compare toujours un objet avec un autre objet
Pour toi, ce sera de comparer un objet Voiture A avec un autre objet Voiture B

Pour classer les Voitures par marque et par année tu vas créer ta propre méthode compareTo
qui indiquera si A.getBrand() est avant B.getBrand() et A.getYear() est avant B.getYear()

Ta classe doit implémenter l'interface comparable

[Narizuke]

Merci pour vos réponses, je vais essayer avec un compareTo()

[joel.drigo]

Salut,

Regarde les streams et les fonctions de groupement. Pas besoin d'implémenter Comparable.

Si tu as une classe Voiture, avec une méthode getMarque() et une méthode getAnnee() :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
List<Voiture> voitures = ...;
 
Map<Marque, List<Voiture>> groupeeParMarque = voitures.stream()
                                                       .collect(Collectors.groupingBy(Voiture::getMarque));
 
Map<Integer, List<Voiture>> groupeeParAnnee = voitures.stream()
                                                      .collect(Collectors.groupingBy(Voiture::getAnnee));
 
Map<Marque, Map<Integer, List<Voiture>>> groupeeParMarqueEtAnnee = voitures.stream()
                                                                           .collect(Collectors.groupingBy(Voiture::getMarque,
							                            Collectors.groupingBy(Voiture::getAnnee)));

[Narizuke]

Quand je fais ma liste, il ne me propose pas le .stream(), je suppose que ça fonctionne qu'avec les dernières versions de java ?

[joel.drigo]

Quand je fais ma liste, il ne me propose pas le .stream(), je suppose que ça fonctionne qu'avec les dernières versions de java ?

Java 8, oui. El il est temps d'y passer (Java 7 n'est plus supporté depuis avril — ne parlons même pas des versions antérieures).

[Narizuke]

Malheureusement, j'ai pas le choix que de pas y passer sur la version 8. ça m'arrangerait bien sinon ^^

Du coup, je suppose que je n'ai pas le choix que de passer par un compareTo() ?
Si j'ai des données dans ma liste du genre :

123XL11,Renault,Clio,2014
352SK22,Renault,Clio,2015
....

Dans mon arbre, je ne prends pas en compte mon immatriculation, mais si je fais un compareTo(), ça va me mettre grouper par immatriculation, et non par Marque,Modèle,Année ?

[joel.drigo]

Du coup, je suppose que je n'ai pas le choix que de passer par un compareTo() ?

Pour le groupement, je ne vois pas trop pourquoi : des maps, des listes, et containsKey() suffisent, et bien implémenter hashcode() et equals() dans les classes spécifiques que tu utilises pour les attributs (comme la marque par exemple)

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
List<Voiture> voitures = ...
Map<Marque, Map<Integer, List<Voiture>>> mapParMarque = new HashMap<>();
for(Voiture voiture : voitures) {
 
     Map<Integer, List<Voiture>>> mapParAnnee = mapParMarque.get( voiture.getMarque() ); 
     if ( mapParAnnee == null ) {
        mapParAnnee = new HashMap<>();
        mapParMarque.put( voiture.getMarque(), mapParAnnee);
     }
 
    List<Voiture> voitureParAnneeParMarque = mapParAnnee.get( voiture.getAnnee() );
    if ( voitureParAnneeParMarque  == null ) {
       voitureParAnneeParMarque  = new ArrayList<>();
       mapParAnnee.put( voiture.getAnnee(), voitureParAnneeParMarque );
    }
 
    voitureParAnneeParMarque.add(voiture);
 
}

Ensuite, si tu as besoin de trier l'une des Map, tu devras soit implémenter Comparable, soit implémenter un Comparator, et utiliser des TreeMap au lieu de HashMap.

[Narizuke]

En utilisant des maps et des listes, j'ai pu me dépatouiller pour obtenir ce que je voulais.
Je vous remercie tous pour votre aide