Quels outils d'analyse de code utilisez-vous pour vos projets Java?

Pour les outils d'analyse statique, j'utilise souvent de la FPC, PMD, FindBugs, et Checkstyle.

DPC est la DGM "Copier/Coller "Détecteur" de l'outil. J'ai été en utilisant PMD pour un peu de temps avant, j'ai remarqué la "Recherche de Code Dupliqué" lien sur le PMD page web.

Je tiens à souligner que ces outils peuvent parfois être prolongée au-delà de leur "out-of-the-box" ensemble de règles. Et pas seulement parce qu'ils sont open source, de sorte que vous pouvez réécrire. Certains de ces outils viennent avec des applications ou des "crochets" qui leur permettent d'être étendu. Par exemple, PMD est livré avec le "concepteur" de l'outil qui vous permet de créer de nouvelles règles. Aussi, Checkstyle a la DescendantToken vérifier qui a des propriétés qui permettent d'importants personnalisation.

- Je intégrer ces outils avec une Fourmi de génération basé sur le. Vous pouvez suivre le lien pour voir mon commenté de configuration.

En plus de la simple intégration dans le build, je trouve utile de configurer les outils pour être un peu plus "intégrée" dans un couple de d'autres façons. À savoir, la production de rapports et d'avertissement de suppression de l'homogénéité. Je tiens à ajouter ces aspects de cette discussion (qui doit probablement avoir l' "analyse statique" balise): comment sont les gens de la configuration de ces outils pour créer un "unifiée" de la solution? (J'ai posé cette question séparément ici)

Tout d'abord, pour des rapports d'alerte, je transforme la sortie de telle sorte que chaque avertissement a le format simple:

/absolute-path/filename:line-number:column-number: warning(tool-name): message

Ceci est souvent appelé le "Emacs format", mais même si vous n'êtes pas à l'aide d'Emacs, c'est une forme raisonnable pour l'homogénéisation des rapports. Par exemple:

/project/src/com/example/Foo.java:425:9: warning(Checkstyle):Missing a Javadoc comment.

Mon avertissement format transformations sont fait par mon script Ant avec Ant filterchains.

Le deuxième "intégration" que je fais, c'est de la mise en garde de la répression. Par défaut, chaque outil prend en charge des commentaires ou une annotation (ou les deux) que vous pouvez placer dans votre code pour le silence un avertissement que vous voulez ignorer. Mais ces diverses avertissement suppression des demandes n'ont pas un aspect qui semble un peu ridicule. Lorsque vous êtes à la suppression de l'avertissement, vous êtes à la suppression d'une alerte, alors pourquoi ne pas toujours écrire "SuppressWarning?"

Par exemple, PMD configuration par défaut supprime avertissement génération de lignes de code avec la chaîne "NOPMD" dans un commentaire. Aussi, PMD prend en charge Java @SuppressWarnings annotation. - Je configurer PMD à utiliser les commentaires contenant "SuppressWarning(PMD." au lieu de NOPMD de sorte que PMD suppressions ressemblent. - Je remplir la règle particulière qui est violé lorsque vous utilisez le style de commentaires suppression:

//SuppressWarnings(PMD.PreserveStackTrace) justification: (false positive) exceptions are chained

Seulement le "SuppressWarnings(PMD." la partie est importante pour un commentaire, mais il est compatible avec PMD de l'appui de la @SuppressWarning annotation qui ne reconnaissent individuels une violation des règles par nom:

@SuppressWarnings("PMD.CompareObjectsWithEquals") //justification: identity comparision intended

De même, Checkstyle supprime avertissement de génération entre les paires d'observations (aucune annotation soutien est fourni). Par défaut, les commentaires à son tour Checkstyle et s'éteint contiennent les chaînes CHECKSTYLE:OFF et CHECKSTYLE:ON, respectivement. La modification de cette configuration (avec Checkstyle est "SuppressionCommentFilter") d'utiliser les chaînes de caractères "BEGIN SuppressWarnings(CheckStyle." et "END SuppressWarnings(CheckStyle." rend les contrôles plus comme PMD:

//BEGIN SuppressWarnings(Checkstyle.HiddenField) justification: "Effective Java," 2nd ed., Bloch, Item 2 
//END SuppressWarnings(Checkstyle.HiddenField) 

Avec Checkstyle commentaires, l'enregistrement particulier de violation de laHiddenField) est important car chaque case a son propre "BEGIN/END" commentaire pair.

FindBugs prend également en charge d'avertissement génération de suppression avec un @SuppressWarnings annotation, donc aucune configuration supplémentaire n'est nécessaire pour atteindre un certain niveau d'uniformité avec d'autres outils. Malheureusement, Findbugs a pour support une coutume @SuppressWarnings annotation, car intégré dans Java @SuppressWarnings annotation a un SOURCE politique de rétention qui n'est pas assez forte pour conserver les annotations dans le fichier de classe où FindBugs en a besoin. Je suis entièrement d'qualifier FindBugs avertissements suppressions pour éviter des affrontements avec Java @SuppressWarnings annotation:

@edu.umd.cs.findbugs.annotations.SuppressWarnings("UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR")

Ces techniques rend les choses semblent relativement homogènes dans les divers outils. Notez que chaque avertissement suppression contiennent la chaîne "SuppressWarnings" le rend facile pour exécuter une recherche simple pour trouver toutes les occurrences de tous les outils à l'ensemble d'une base de code.

Tout ce qui suit, nous utiliser et à intégrer easiy dans nos deux Maven 2.x construit et Eclipse/RAD 7:

• De Test JUnit/TestNG
• D'analyse de Code - FindBugs, PMD
• De couverture de Code - Trèfle

En outre, dans nos builds Maven, nous avons:

• JDepend
• Tag checker (TODO, FIXME, etc)

En outre, si vous utilisez Maven 2.x, CodeHaus a une collection de pratique de plugins Maven dans leur Mojo projet.

Remarque: le Trèfle a out-of-the-box intégration avec le Bambou serveur CI (puisqu'ils sont tous les deux produits Atlassian). Il y a aussi le Bambou plugins pour FindBugs, PMD et CheckStyle mais, comme indiqué, la libre Hudson serveur CI a ceux qui sont trop.

Checkstyle est un autre que j'ai utilisé lors d'une précédente société... c'est surtout pour le style de vérification, mais il peut faire une analyse statique trop. Aussi, Trèfle de la couverture de code, mais être conscient qu'elle n'est pas un outil gratuit.

Nous utilisons FindBugs et JDepend intégré avec Ant. Nous utilisons JUnit, mais nous ne sommes pas à l'aide de tout outil de couverture.

Je ne suis pas à l'utiliser intégrée de Rational Application Developer (l'IDE que j'utilise pour développer des applications J2EE) parce que j'aime la façon dont soigné, il ressemble quand vous exécutez javac dans la console Windows. 😛

Notre équipe de l'utilisation PMD et Cobertura, en fait, nos projets sont des projets maven et il est très simple d'inclure des plug-ins pour l'analyse de code. La vraie question serait pour un projet spécifique qui analyse dont vous avez besoin pour utiliser, mon avis est que vous ne pouvez pas utiliser les mêmes plugins pour chaque projet.

Structure 101 est bon à l'analyse de code et de trouver le cyclique des dépendances de package.