Discussion :

[jacklafrip]

Bonjour,

Je récupère un code avec des espaces de partout, des indentations pourries, ...
D'autre part, je dois modifier le nom de certaines fonctions, car les fonctions appelées doivent changer (modification de la bibliothèque utilisée).
J'ai donc fait une "jolie" expression régulière, qui supprime les espaces en trop pour une fonction particulière ayant 3 arguments.
Malheureusement, cela ne va pas aussi loin que je le voudrais.

Code :

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

1
2
3
 
Regex MyRegex = new Regex(@"\s*([/]*)\s*FonctionAvant\s*\(\s*(.*)\s*,\s*(.*)\s*,\s*(.*)\s*\)\s*;\s*");
NewLine = MyRegex.Replace(Line,"$1FonctionApres($2, $3, $4);");

Cela marche bien lorsque le code est, par exemple :

Code :

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

1
2
3
4
5
 
// Avant
FonctionAvant  (PARA1       , PARA2        , PARA3     )   ;
// Après
FonctionApres(PARA1, PARA2, PARA3);

Mais si le code est

Code :

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

1
2
3
 
// Avant
FonctionAvant  (PARA1       , PARA2 | Truc        , PARA3     )  ;

Cela ne fonctionne pas, car il y a des espaces qui existent autrement qu'après / avant les virgules.

J'utilise vi, et remplacer le \s*(\S*)\s* par \s*(.*)\s* fonctionne très bien, pour résoudre le problème.
C#, pour le moment, considère le .* comme prioritaire sur \s, et ne supprime donc pas les espaces.

Code :

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

1
2
3
4
5
6
7
8
 
Regex MyRegex = new Regex(@"\s*([/]*)\s*FonctionAvant\s*\(\s*(\S*)\s*,\s*(\S*)\s*,\s*(\S*)\s*\)\s*;\s*");
NewLine = MyRegex.Replace(Line,"$1FonctionApres($2, $3, $4);");
 
// Avant
FonctionAvant  (PARA1       , PARA2 | Truc        , PARA3     )   ;
// Après
FonctionApres  (PARA1       , PARA2 | Truc        , PARA3     );

Avez-vous une idée ?

Merci

[Nathanael Marchand]

Il serait peut être opportun de prendre un plugin du genre Resharper qui permet de nettoyer et formater le code correctement

[jacklafrip]

En fait, je fais ce petit logiciel car cet outil devra être utilisé pour migrer du code C sur micro-contrôleur avec différents IDE (notamment Keil et Eclipse).
Donc pas de plugin envisageable...

[GuruuMeditation]

Resharper est un plugin (indispensable a mon sens) de l'editeur de visual studio. Ca t'aide à formatter ton code (entre autres). Mais ce n'est pas un plugin qui va être utilisé par ton appli lors de son exécution.

Ca va t'aider très fort pour la partie "Je récupère un code avec des espaces de partout, des indentations pourries"

[jacklafrip]

Je suis le seul à utiliser Visual Studio, puisque tous mes collègues utilisent Keil, Eclipse ou autres IDE.
J'utilise Visual Studio en C# pour écrire des utilitaires.
Je souhaite donc écrire un programme qui rend propre un code écrit en C, de façon à ce que les autres puissent aussi profiter de la migration pour récupérer du code propre, et ceci sans utiliser Visual Studio.

[Er3van]

Je ne comprends toujours pas...

Pourquoi ne pas simplement formater le code avec Visual Studio + ReSharper puis fournir les sources aux personnes utilisant les autres IDE pour qu'elles reparte de cette base "propre" ?

Ou alors tu suggères qu'elles ne travaillent pas sur les mêmes sources ? (versioning?)
Au pire, ça prendra sans doute moins de temps de formater une à une les différentes versions que de développer l'utilitaire qui le fera. A toi de voir.

Au passage, je trouve ça assez étrange que dans une même entreprise vous utilisiez autant d'IDE différents pour une seule et même techno.

[jacklafrip]

Plusieurs IDE sont utilisés, car plusieurs compilateurs pour des micro-contrôleurs différents.

Le but est plutôt de migrer progressivement tous les soft, en faisant participer les gens de façon aussi à ce qu'ils comprennent que coder moche, c'est mal.
Les forcer à indenter, espacer, accolader, ... correctement, cela pourrait être long dans des fichiers de 15000 lignes (oui, je sais !). S'ils doivent juste cliquer sur un bouton, c'est à la fois pas très compliqué, et en même temps, la démarche y est.

Merci

[neilbgr]

Je suggère de simplement supprimer les espaces avant certains caractères :
Motif de recherche :

Code :

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

(\s+)([(,;)])

Motif de remplacement :

Code :

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

$2

Donne pour :

Code :

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

FonctionAvant  (PARA1       , PARA2  | Truc        , PARA3     )   ;

Le résultat suivante :

Code :

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

FonctionAvant(PARA1, PARA2  | Truc, PARA3);

Et ce, quelque soit le nom de la fonction.
Cela te convient ?

[Er3van]

Plusieurs IDE sont utilisés, car plusieurs compilateurs pour des micro-contrôleurs différents.

Pour compiler ok, mais pour le développement ?

[jacklafrip]

Pour compiler ok, mais pour le développement ?

Mes collègues utilisent leur IDE (Keil, Eclipse, ...) pour développer, compiler, simuler puis tester sur cible avec points d'arrêts, ...
Personnellement, j'utilise vi comme IDE.
Et Visual Studio, c'est cher.

Code :

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

FonctionAvant(PARA1, PARA2  | Truc, PARA3);

Et ce, quelque soit le nom de la fonction.
Cela te convient ?

Oui, cela me convient, mais sous-entend deux passages. L'un pour enlever les espaces, l'autre pour remplacer le nom de la fonction, modifier le nombre de paramètres, l'ordre, ...

Néanmoins, j'ai trouvé une solution pour rester sur l'expression régulière de la mort qui tue:

Code :

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

1
2
3
 
Regex MyRegex = new Regex(@"\s*([/]*)\s*FonctionAvant\s*\(\s*(.*?)\s*,\s*(.*?)\s*,\s*(.*?)\s*,\s*(.*?)\s*\)\s*;\s*");
NewLine = MyRegex.Replace(Line, "$1FonctionApres($2, $3, $4, $5);");

Au lieu d'écrire (\S*), il suffit d'écrire : (.*?)
Le ? veut dire "le moins d'éléments possible". Donc les éléments de type \s s'agrandissent au maximum, en prenant le pas sur les autres, même si un "\s" est un "."
L'explication se trouve ici : http://msdn.microsoft.com/fr-fr/libr...4%28v=vs.80%29

Merci à tous.

[Nathanael Marchand]

Pour tester des expressions régulières, tu peux télécharger le super utilitaires "expresso" qui est gratuit en plus

[neilbgr]

Le ? veut dire "le moins d'éléments possible". Donc les éléments de type \s s'agrandissent au maximum, en prenant le pas sur les autres, même si un "\s" est un "."
L'explication se trouve ici : http://msdn.microsoft.com/fr-fr/libr...4%28v=vs.80%29

Je m'excuse d'avance auprès des modos qui n'aiment pas qu'on fasse des références externes, mais le site suivant est mon préféré en ce qui concerne les expression rationnelle : http://www.regular-expressions.info/reference.html

Et pour le coup, c'est la notion de "lazy" = "paresse" (pour pas dire féniasse).