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.