Utilisation du Stream - Votre avis sur cette méthode

Version imprimable

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:

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

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:

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:

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:

1 2 stream.flatMap(ligne -> Arrays.stream(ligne.split(";")).skip(1)) .forEach(System.out::println);
pour avoir le même résultat

Ou
Code:

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

04/10/2017, 12h11
Kasko

Salut,

Citation:

Envoyé par 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

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

Salut,
Citation:
Envoyé par joel.drigo
ou encore (et pour gagner en performance, au lieu de String split(), compiler le pattern (une fois), puis l'utilise)
Code:

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().
:arrow: 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++

04/10/2017, 21h02
joel.drigo

Citation:

Envoyé par adiGuba

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().
:arrow: 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.