Discussion :

[Kasko]

Bonjour à tous,
En fait j'ai écrit une méthode qui se base sur un fichier(chaque ligne est composée du nom du film et l'ensemble des acteurs séparés par ";") et ajouté chaque ligne dans une table de hachage avec comme clef le nom du film et valeur la liste des acteurs en utilisant Collectors.toMap.

voici le code : ma question est en commentaire couleur rouge

Code Java :

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

public class ActeursFilm {
	
	private final String nameFilm;
	private final List<String> acteurs;
	
	public ActeursFilm(String nameFilm, List<String> acteurs){
		this.nameFilm = nameFilm;
		this.acteurs = acteurs;
	}
	public static void traiterActeur(String chemin) throws IOException{
		Objects.requireNonNull(chemin);
		Path path = Paths.get(chemin);
		Map<String, List<String>> acteurs = new HashMap<>();
		
		try(Stream<String> stream = Files.lines(path,StandardCharsets.ISO_8859_1)){
			acteurs = stream
					.map(ligne -> Arrays.asList(ligne.split(";")))
                          /*je me demande s'il n'y a pas un autre moyen de faire sans faire ce nouveau appel à map dans le but de 
              créer une instance pour chaque ligne.D'ailleurs est-ce que c'est nécessaire de créer une instance pour chaque ligne?*/
					.map(list -> new ActeursFilm(list.get(0), list.subList(1, list.size())))
					.collect(Collectors.toMap(ActeursFilm::getNameFilm, ActeursFilm::getActeursFilm));		

			/*stream.forEach(ligne -> {
				List<String> listeActeur = Arrays.asList(ligne.split(";"));
				acteurs.put(listeActeur.get(0),listeActeur.subList(1, listeActeur.size()));
			});*/
		}
		
		acteurs.values().stream().forEach(System.out::println);
	}
}

[joel.drigo]

Salut,

La ligne est inutile puisque la classe ActeursFilm ne fait aucun traitement sur les données, que tu extrais juste après, en perdant l'instance. Donc, non seulement la ligne est inutile, mais tu éviterais de créer toutes ces instances pour rien :

Code :

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

1
2
3
		Map<String, List<String>> acteurs = stream
				.map(ligne -> Arrays.asList(ligne.split(";")))
				.collect(Collectors.toMap(list->list.get(0), list-> list.subList(1, list.size())));

ou encore (et pour gagner en performance, au lieu de String split(), compiler le pattern (une fois), puis l'utilise)

Code :

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

1
2
3
4
5
6
7
Pattern pattern=Pattern.compile(";");
Map<String, List<String>> acteurs = stream.map(ligne -> pattern.split(ligne))
				.collect(Collectors.toMap(
						ligne->ligne[0],
						ligne-> Arrays.stream(ligne)
						              .skip(1)
						              .collect(Collectors.toList())));

Si le but n'est que d'afficher la liste des acteurs (ce que tu fais juste après), la map elle-même est inutile, puisque tu peux directement écrire :

Code :

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

1
2
stream.flatMap(ligne -> Arrays.stream(ligne.split(";")).skip(1))
	.forEach(System.out::println);

pour avoir le même résultat

Ou

Code :

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

1
2
3
Pattern pattern=Pattern.compile(";");
stream.flatMap(ligne->pattern.splitAsStream(ligne).skip(1))
          .forEach(System.out::println);

[Kasko]

Salut,

Salut,
La ligne est inutile puisque la classe ActeursFilm ne fait aucun traitement sur les données, que tu extrais juste après, en perdant l'instance. Donc, non seulement la ligne est inutile, mais tu éviterais de créer toutes ces instances pour rien

je m'en doutais et merci pour les autres précisions.

[adiGuba]

Salut,

ou encore (et pour gagner en performance, au lieu de String split(), compiler le pattern (une fois), puis l'utilise)

Code :

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

1
2
3
4
5
6
7
Pattern pattern=Pattern.compile(";");
Map<String, List<String>> acteurs = stream.map(ligne -> pattern.split(ligne))
				.collect(Collectors.toMap(
						ligne->ligne[0],
						ligne-> Arrays.stream(ligne)
						              .skip(1)
						              .collect(Collectors.toList())));

Attention si ceci est vrai d'une manière générale pour l'utilisation des Patterns, ce ne l'est pas forcément pour String.split().
Lorsqu'on l'utilise avec un simple caractère, elle utilise une implémentation basé sur des indexOf() à la place des expressions régulières...


a++

[joel.drigo]

Attention si ceci est vrai d'une manière générale pour l'utilisation des Patterns, ce ne l'est pas forcément pour String.split().
Lorsqu'on l'utilise avec un simple caractère, elle utilise une implémentation basé sur des indexOf() à la place des expressions régulières...

Effectivement. J'oublie toujours cette optimisation introduite en Java 7.