Discussion :

[alex77722]

Bonjour,

Je suis un jeune stagiaire et aujourd'hui je me suis lancé dans une de mes dernières missions ! Repérer (avec un outil d'audit de code) l'utilisation d'une class qui a été remplacée !

Pour vous donner un peu plus de contexte, on utilise aujourd'hui un framework qui propose une class que l'on a remplacé par une nouvelle ! Les deux noms de class ont la même racine, exemple : Transaction et TransactionBalBla. J'ai décider d'utiliser CheckStyle qui me semble sympa ! J'ai rajouter ses dépendances maven et j'ai créé un fichier checkstyle.xml minimaliste qui ne relève aucune erreur (c'est volontaire):

Code :

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

1
2
3
4
5
6
7
8
9
<?xml version="1.0" ?>

<!DOCTYPE module PUBLIC
  "-//Puppy Crawl//DTD Check Configuration 1.2//EN"
  "http://www.puppycrawl.com/dtds/configuration_1_2.dtd">

<module name="Checker">

</module>

Est-ce que quelqu'un pourrai juste me donner des pistes pour pouvoir créer la règle ?

Merci d'avance !

[alex77722]

En fouillant un peu je suis tombé sur ce check : IllegalType.

Il semble correspondre à ce que je veux ! Du coup dans mon checkstyle minimaliste, j'ai ajouté (dans la balise "checker") :

Code :

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

1
2
3
4
5
<module name="TreeWalker">
	<module name="IllegalType">
		<property name="illegalClassNames" value="Transaction"/>
	</module>
</module>

La vérification ne me remonte que des classes commençants avec Abstract ...

J'ai l'impression qu'elle vérifie si les noms des classes sont "corrects" ou non (selon les standard) ... Et je ne sais pas comment retirer la vérification par défaut !

[alex77722]

Suis-je bête ! Interdire les Classes qui possède le mot clé Abstract, c'est la valeur par défaut de l'attribut format !!!!

Je me sens un peu bête de poster tout seul sur mon sujet avec un intervalle de 5 min . Et j'ai même pas pris de café

[alex77722]

Bon si j'ai bien compris, le fichier suivant permet d'interdire (=> checkstyle relève une error) toutes méthodes s'appellent Transaction :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<?xml version="1.0" ?>

<!DOCTYPE module PUBLIC
  "-//Puppy Crawl//DTD Check Configuration 1.2//EN"
  "http://www.puppycrawl.com/dtds/configuration_1_2.dtd">

<module name="Checker">
    <module name="TreeWalker">
		<module name="IllegalType">
			<property name="illegalClassNames" value="Transaction"/>
		</module>
	</module>
</module>

Pour prendre uniquement la méthode Transaction voulu, il suffit d'ajouter le chemin du package !

ce cette façon :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<?xml version="1.0" ?>

<!DOCTYPE module PUBLIC
  "-//Puppy Crawl//DTD Check Configuration 1.2//EN"
  "http://www.puppycrawl.com/dtds/configuration_1_2.dtd">

<module name="Checker">
    <module name="TreeWalker">
		<module name="IllegalType">
			<property name="illegalClassNames" value="fr.paris.lutece.util.sql.Transaction"/>
		</module>
	</module>
</module>