Discussion :

[Davboc]

Bonjour

J'ai vu que dans l'utilitaire de recherche dans le code (find, ctrl+f) on peut utiliser des expressions régulières.

J'ai approfondi un peu la chose et je suis tombé sur le détail des regexp qu'on peut utiliser :
http://www.eclipse.org/tptp/monitori...f/rregexp.html

Mais je n'arrive toujours pas à faire ce que je veux (faut dire que c'est un peu complexe ^^) :

Je cherche à écrire une regexp me permettant de trouver dans mon code les catch(SQLException xx) qui n'ont pas de finally associé, afin de chasser les connexions pas fermées...

J'ai créé ce petit fichier pour tester les regexp :

Code :

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
/******************SQLException AVEC FINALLY**********************/	
try {
			conn = ds.getConnection();
			ArrayList listeLien = new ArrayList();
			/***/
			switch (actionRecue) {
				case Actions.ANNULER:
					return JSPName.INDEX;
 
				case Actions.DOWNLOAD:
					List desLiens = (List) sm.getAttribute(LISTE_LIEN);
					Fichier unFic = (Fichier) desLiens.get(Integer.parseInt(req.getParameter("NOFIC")));
					break;
				default:
					break;
			}
		} catch (SQLException e1) {
			e1.printStackTrace();
		} finally {
			ConnectionUtil.fermetureConnectionSimple(conn, null, null, null);
		}
 
 
/******************SQLException SANS FINALLY**********************/	
try {
			conn = ds.getConnection();
			ArrayList listeLien = new ArrayList();
			/***/
 
			}
		} catch (SQLException e1) {
			e1.printStackTrace();

J'ai testé des tonnes de solutions, sans succès. La dernière tentative a donné l'expression suivante :

Code :

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

(?s)catch\s*\(SQLException.*\}.*(?!(finally)){0}.\s\{

(?s) me permet d'ignorer les sauts à la ligne, d'après ce que j'ai compris (?!(finally)) désigne l'absence du mot 'finally'.

Quand je ne met rien après ce (?!(finally)) ça me sélectionne tout depuis le catch jusque la fin de la page. J'aimerais que cette selection s'arrête a l'endroit ou je suis supposé trouver un finally, c'est à dire après le premier '}' trouvé...

Si quelqu'un sait comment s'y prendre j'apprécierais vraiment un ptit coup de pouce

Merci !

[CyberChouan]

Petite précision sur les expressions régulières: si tu ne précises rien, elles attrapent les groupes les plus grands possibles.

Si tu cherches "{.*}" (pour un replaceAll par exemple) dans la chaîne "{a} {b}", elle ne trouvera qu'un seul groupe où .* vaut "a} {b".

Pour palier à ce problème, il faut chercher "{.*?}". Cela t'aidera sans doute à ne pas attraper de groupes trop gros entre tes crochets

Cependant, dans ton cas, je ne suis pas certain que ça soit faisable par une simple expression régulière... Imagine le code:

Code :

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

1
2
3
4
5
6
7
8
9
 
try {
   // ...
} catch (SQLException e) {
   if(debugMode) { ecrireMsgDebug(); }
   else { faireAutreChose(); }
} finally {
   fermerLaConnexionDansTousLesCas();
}

Tu as des blocs imbriqués dans ton "catch" qui risquent fort de tromper ton expression. Si tu veux vraiment traiter tous les cas, tu dois faire plus compliqué qu'une simple expression régulière: à partir du catch, il faut compter les crochets pour trouver celui qui va fermer le catch et vérifier que le mot-clé qui suit ce dernier crochet est bien un "finally".

Bon courage

[kartben]

Peut-etre tout simplement regarder du côté d'outils d'analyse statique de code (type CheckStyle ou Findbugs) ; pour détecter tous les catch sans finally ? A moins d'avoir des milliers de fichiers (ce qui est peut-être ton cas ) ; ça me paraît pas la mort de faire next, next, next, ... pour parcourir tous les bouts de code sans finally

[Davboc]

Merci pour vos coups de main

J'ai pas pensé à utiliser l'outil d'analyse de code, c'est vrai que ça pourrait être une solution Il n'y a pas des milliers de fichiers (enfin je pense pas), mais il y en a quand même quelques-uns..

Je vous tiens au courant de l'avancée de ma problématique

[tnodev]

Salut

J'utilise QuickRex, un plugin eclipse gratuit très pratique pour tester les expressions régulières

http://www.bastian-bergerhoff.com/ec...ckREx/toc.html