Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

  1. #1
    Membre averti
    Homme Profil pro
    Développeur informatique
    Inscrit en
    octobre 2014
    Messages
    347
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France

    Informations professionnelles :
    Activité : Développeur informatique

    Informations forums :
    Inscription : octobre 2014
    Messages : 347
    Points : 341
    Points
    341

    Par défaut Utilisation du Stream - Votre avis sur cette méthode

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

  2. #2
    Modérateur
    Avatar de joel.drigo
    Homme Profil pro
    Ingénieur R&D - Développeur Java
    Inscrit en
    septembre 2009
    Messages
    11 667
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 50
    Localisation : France, Paris (Île de France)

    Informations professionnelles :
    Activité : Ingénieur R&D - Développeur Java
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : septembre 2009
    Messages : 11 667
    Points : 27 469
    Points
    27 469
    Billets dans le blog
    2

    Par défaut

    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);
    L'expression "ça marche pas" ne veut rien dire. Indiquez l'erreur, et/ou les comportements attendus et obtenus, et donnez un Exemple Complet Minimal qui permet de reproduire le problème.
    La plupart des réponses à vos questions sont déjà dans les FAQs ou les Tutoriels, ou peut-être dans une autre discussion : utilisez la recherche interne.
    Des questions sur Java : consultez le Forum Java. Des questions sur l'EDI Eclipse ou la plateforme Eclipse RCP : consultez le Forum Eclipse.
    Une question correctement posée et rédigée et vous aurez plus de chances de réponses adaptées et rapides.
    N'oubliez pas de mettre vos extraits de code entre balises CODE (Voir Mode d'emploi de l'éditeur de messages).
    Nouveau sur le forum ? Consultez Les Règles du Club.

  3. #3
    Membre averti
    Homme Profil pro
    Développeur informatique
    Inscrit en
    octobre 2014
    Messages
    347
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France

    Informations professionnelles :
    Activité : Développeur informatique

    Informations forums :
    Inscription : octobre 2014
    Messages : 347
    Points : 341
    Points
    341

    Par défaut

    Salut,

    Citation Envoyé par joel.drigo Voir le message
    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.

  4. #4
    Expert éminent sénior
    Avatar de adiGuba
    Homme Profil pro
    Développeur Java/Web
    Inscrit en
    avril 2002
    Messages
    13 933
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France

    Informations professionnelles :
    Activité : Développeur Java/Web
    Secteur : Transports

    Informations forums :
    Inscription : avril 2002
    Messages : 13 933
    Points : 22 986
    Points
    22 986
    Billets dans le blog
    1

    Par défaut

    Salut,

    Citation Envoyé par joel.drigo Voir le message
    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++

  5. #5
    Modérateur
    Avatar de joel.drigo
    Homme Profil pro
    Ingénieur R&D - Développeur Java
    Inscrit en
    septembre 2009
    Messages
    11 667
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 50
    Localisation : France, Paris (Île de France)

    Informations professionnelles :
    Activité : Ingénieur R&D - Développeur Java
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : septembre 2009
    Messages : 11 667
    Points : 27 469
    Points
    27 469
    Billets dans le blog
    2

    Par défaut

    Citation Envoyé par adiGuba Voir le message
    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.
    L'expression "ça marche pas" ne veut rien dire. Indiquez l'erreur, et/ou les comportements attendus et obtenus, et donnez un Exemple Complet Minimal qui permet de reproduire le problème.
    La plupart des réponses à vos questions sont déjà dans les FAQs ou les Tutoriels, ou peut-être dans une autre discussion : utilisez la recherche interne.
    Des questions sur Java : consultez le Forum Java. Des questions sur l'EDI Eclipse ou la plateforme Eclipse RCP : consultez le Forum Eclipse.
    Une question correctement posée et rédigée et vous aurez plus de chances de réponses adaptées et rapides.
    N'oubliez pas de mettre vos extraits de code entre balises CODE (Voir Mode d'emploi de l'éditeur de messages).
    Nouveau sur le forum ? Consultez Les Règles du Club.

+ Répondre à la discussion
Cette discussion est résolue.

Discussions similaires

  1. [Sequoia] Votre avis sur la Méthode
    Par Zorro dans le forum Autres
    Réponses: 5
    Dernier message: 21/05/2008, 16h02
  2. Votre avis sur cette maquette : dzfoot.webdispo.com
    Par stanley dans le forum Mon site
    Réponses: 3
    Dernier message: 09/03/2008, 12h06
  3. Votre avis sur cette demo (source et uml)
    Par TheDrev dans le forum SDL
    Réponses: 3
    Dernier message: 05/10/2007, 15h42
  4. [Conception] Votre avis sur une méthode
    Par AIexis dans le forum PHP & Base de données
    Réponses: 7
    Dernier message: 17/04/2007, 18h08
  5. Votre avis sur cette page.
    Par robotractor dans le forum Flash
    Réponses: 3
    Dernier message: 04/04/2007, 11h17

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo