Discussion :

[SuperJoker]

Bonjour à tous,

Je suis en train de développer un programme qui analyse des fichiers selon des regex. j'ai créé une classe par type de fichier et une classe modèle de donnéee.

Mais je pense que j'ai abordé le problème sous le mauvais angle car ma classe AnalyseFichier comporte un switch avec tous les cas possibles

Un collègue m'a parlé des factories, j'ai regardé un peu en pensant stocker dans une map (Hashtable) les différents types de lignes et classes associées (j'aurais donc une classe par type de ligne).

Ex :
"MSC.Nastran.*Linux" -> classe Entete
"OUT=.*" -> classe Version
etc....

Est-ce la bonne approche ? Si oui, comment gérer proproment la map ?

Merci pour vos conseils !

[thelvin]

Disons qu'une map c'est un peu bizarre. L'idée d'une map c'est que les clés soient des indexes qui permettent de déterminer immédiatement s'il y a une valeur associée à cette clé ou non. Or une regex c'est pas un index. Tu serais forcé de parcourir toutes les clés disponibles et de les tester dans l'ordre pour voir s'il y a une valeur associée. C'est toujours mieux qu'une suite de if, mais c'est bizarre d'utiliser une Map.

Il serait plus logique que tes classes fournissent chacune la regex qui permet de la reconnaître, et garder une List de tes classes. Ainsi tu boucles sur chacune, tu vérifies si sa regex se vérifie et si oui tu l'appliques, sinon tu passes à la suivante.

Code :

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

1
2
3
4
5
for(ParseurDeLigne parseur : parseurs) {
  if(ligne.matches(parseur.getRegex())) {
    return parseur.parse(ligne);
  }
}

Ou quelque chose comme ça.

[SuperJoker]

Code :

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

1
2
3
4
5
for(ParseurDeLigne parseur : parseurs) {
  if(ligne.matches(parseur.getRegex())) {
    return parseur.parse(ligne);
  }
}

Merci pour ton retour. Par contre je ne comprends pas la boucle for(ParseurDeLigne parseur : parseurs) : tu parcours quoi ?

[thelvin]

Bah, une instance de Entete, une de Version, et une de tous les autres.

[tchize_]

Merci pour ton retour. Par contre je ne comprends pas la boucle for(ParseurDeLigne parseur : parseurs) : tu parcours quoi ?

Tous les parseurs (que tu dois créer, c'est l'idée), chacun capable de supporter une regexp particulière (getRegex) et, en réponse, de générer l'une ou l'autre classe dont tu as besoin (méthode parse())

Exemple:

Code :

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

1
2
3
4
5
6
7
8
9
 
public abstract class Parseur {
   private Pattern regex;
   public Parseur(String regex){
      this.regex = Pattern.compile(regex);
   }
   public Pattern getRegex(){ return regex; }
   public abstract Object parse(String ligne);
}

Code :

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

1
2
3
4
5
6
7
8
9
 
public class ParseurEntete extends Parseur{
    public ParseurEntete(){
        super("MSC.Nastran.*Linux");
    }
    public Object parse(String ligne){
        return new Entete(......);
    }
}

Code :

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

1
2
3
 
List<Parseur> parseurs = new ArrayList<Parseur>();
parseurs.add(new ParseurEntete());

[SuperJoker]

OK, merci à vous deux, je pense que j'ai saisi le principe

Juste une question : je vais donc avoir autant de parseurs.add que je lignes à tester (environ 20 ou 30 !) donc quelque chose comme ça :

Code :

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

1
2
3
4
5
6
 
List<Parseur> parseurs = new ArrayList<Parseur>();
parseurs.add(new ParseurEntete());
parseurs.add(new ParseurNom());
parseurs.add(new ParseurVersion());
......

Existe-il une solution pour "balayer" tous les classes de type Parseurs et faire un add dans une booucle par exemple ?

[thelvin]

Rien de bien pratique, non, une question récurrente.

Mais tu peux utiliser par exemple google-code-reflections ou une autre bibliothèque du même genre. Elles utilisent des bricolages, mais ça marche.

[tchize_]

Il existe plueisuers manières de le faire. Par exemple mettre une annotation précise sur chaque parseur, et scanner à la recherche de cette annotation. Faire un fichier de config avec tous les noms à charger, ...

Mais toutes ces méthodes vont vraisemblablement amener plus de lignes de code dans ton programme que les 20/30 lignes de add Sans compter qu'avec les 20 lignes de add, au moins, tout l'information sur 'quels sont les parseurs utilisés par l'application' se trouve au même endroit.

[gloax29]

le bute est d'injecté la ligne dans un Object ou une list suivant la regex ?

[thelvin]

Il a très clairement décrit son but. Par contre, on n'a aucune idée de ce que tu demandes.

[gloax29]

sont but, est d ouvrir un fichier et de le parcer en une class qui correspond a ce fichier?

[SuperJoker]

salut,

Je dois parser des fichiers et stocker les données dont j'ai besoin dans une classe, sachant qu'après je veux générer un fichier XML (je sépare la partie traitement fichier, parser des lignes, modèle de données et mise en forme). Donc je suis parti sur la solution proposée par thelvin et tchize_ (encore merci au passage !).

Mes fichiers auraient été structurés, je serais bien parti avec l'API Easy Batch.

Je passe le sujet en résolu afin de ne pas encombrer le forum avec un sujet ouvert.

[gloax29]

sault,
mon but est juste de savoir si c'est tu veux parcer un fichier avec son contenu.

[gloax29]

car j'ai fait un fram de type JAXB qui parse les fichiers plat en Object, et qui fonction par annotation.
Object structurer comme JAXB et avec la posibilitée de test Regex pour determiné a quel objet il appartient.

[SuperJoker]

sault,
mon but est juste de savoir si c'est tu veux parcer un fichier avec son contenu.

Oui, je dois analyser des lignes et récupérer des infos de ces lignes, infos qui seront stockées dans les attributs d'une classe ModeledeDonnees.

[gloax29]

donc j'ai ce qui te faut.
car si ton object a une structure jpa ou Jaxb tu peut y ajouté les annotations.

[thelvin]

Tout ça c'est bien mignon, mais on comprend rien à ce que tu racontes.

Arrête de parler autour du pot et montre-nous comment fonctionne ton machin.

[gloax29]

mon machin fonctionne bien.
je parse juste 220 Mo en 10sc pour une injection en base en moin d'une minute.
pas grave.