Bienvenue à Blogs CodeS-SourceS Identification | Inscription | Aide

StyleCop et FxCop

J’ai activé StyleCop et FxCop  pour prendre des bonnes habitudes de développement !!!

Mais il y a des conflits entre GuideLines et StyleCop !!

Par exemple :

1.         

                GuideLine : Les variables membres sont préfixées par _

              StyleCop :  Warning:SA1306: Variable names must start with a lower-case letter: _Pages.

(StyleCop ne précise pas les différences entre les variables membres et les autres variables.)

2.         

Je dois utiliser le “this.” Pour tous les variables membres et toutes les méthodes !!

SA1101: The call to AddPage must begin with the 'this.' prefix to indicate that the item is a member of the class.

3.         

Sur un Event généré par VS il donne des warnings :

SA1300: method names begin with an upper-case letter: btnCancel_Click.

 

Bon je désactive les certains règles de StyleCop. Avez-vous l’autre solution ??

Par contre FxCop est très bien  J

 

 

Ce post vous a plu ? Ajoutez le dans vos favoris pour ne pas perdre de temps à le retrouver le jour où vous en aurez besoin :
Posted: vendredi 9 janvier 2009 15:12 par Mona7170
Classé sous :

Commentaires

Danuz a dit :

Bonjour,

Vous êtes qui ?

Sinon, je reste utilisateur de FxCop (je ne connaissais pas StyleCop), et utilise généralement les règles de base...

# janvier 9, 2009 16:32

RaptorXP a dit :

Je ne peux pas supporter Style Cop, il impose des règles qui sont complètement arbitraire.

# janvier 9, 2009 17:47

KooKiz a dit :

Le but est justement d'imposer des règles, afin que tous les programmeurs sur un même projet utilisent le même style, pas qu'elles plaisent à tout le monde :p

De toutes façons, il est prévu de pouvoir les personnaliser, si ce n'est pas déjà fait (pas trop suivi l'avancement du projet depuis les premières versions publiques).

# janvier 12, 2009 09:50

TheSaib a dit :

Moi j'utilise les deux, et j'en suis très satisfait. Je trouve que le code "styleCopisé" est très clair.

1. On ne préfixe plus par _, ca n'a strictement aucun intérêt aujourd'hui, la casse et le this te permettent de distinguer tes membres.

2. Oui, c'est du bon sens ! Tu te repères beaucoup plus vite dans le code !

3. Oui parceque VS ne génère pas du code propre, il ne respecte pas les notations usuelles!

Ce sont deux très bon outils qui facilitent grandement la maintenance d'application. Le fait qu'il soit ouvert à l'extensibilité permet de définir ses propres règles. Le tout couplé à TFS, et c'est magique.

# janvier 12, 2009 13:56

Mona7170 a dit :

Auteur : lgmorand

plus que des règles de bonne écriture, fxCop donne aussi des règles de développement à suivre. Bon, c'est souvent plus facile à dire qu'à faire et par exemple, il m'embete si mon enum ne commence par zéro SAUF si les valeurs sont des bitflags (et donc faut mettre l'attribut [flags]).

autre exemple cité plus haut, surement la faute à VS en effet, j'utilise aussi la notation controle+nom (btnOK, btnCancel). les méthodes commencent donc par une minuscule et fxCop nous peste dessus.

Concernant le "this", je suis pas trop d'accord avec toi TheSaib. le "this" existe depuis des années et bien avant le .Net et on utilisait déjà la notation avec le "_" pour les membres (ou "m_" quand une variable pouvait pas commencer par un underscore). Je ne pense pas qu'on puisse parler de "sans intérêt" tant les deux choses veulent dire la même chose. Je pense qu'on peut très bien s'en sortir avec des membres privés avec le underscore. De plus, si j'utilise les "this", plusieurs outils comme Resharper nous les enlève car jugés redondants). De plus Microsoft utilise le underscore dans son code source et décourage le "m_" MAIS ils l'utilisent aussi dans leur code :D

Bref, this ou "_" même si la plupart des documentations de convention de nommage parlent de "_", ca reste une question de préférence personnelle et tant que c'est compréhensible, l'un ou l'autre ne gêne pas

pour toi, mona7170, tu n'es pas obligé(e?) de suivre à la lettre les guidelines que propose fxCop ou StyleCop, ca aide à nettoyer ton code et avoir de bonnes habitudes mais c'est pas ca qui rendra du code "propre" donc perd pas trop de temps dessus à essayer de faire disparaitre tous les warnings. ;)

______

Mona7170 :

Desolé , je ne voulais pas supprimé votre comment, je me suis trompé.

# janvier 12, 2009 15:32

Mona7170 a dit :

Merci à tous pour toutes ces informations :)

# janvier 12, 2009 15:54

lgmorand a dit :

Petit ajout, ce matin j'ai installé StyleCop et lui, préconise l'utilisation du "this".

# janvier 13, 2009 11:53
Les commentaires anonymes sont désactivés

Les 10 derniers blogs postés

- Etendre le Team Web Access de TFS 2012 – Step 0 par Philippe Didiergeorges Aka Philess le il y a 17 heures et 41 minutes

- Simuler facilement l’envoi de mail par Blog de Jérémy Jeanson le 05-22-2013, 12:52

- ProcDump 6.0 : support du filtrage sur messages d'exceptions .NET, des filtres multiples et du ciblage par nom de service par CoqBlog le 05-20-2013, 14:50

- Votez pour le TOP 10 des influenceurs SharePoint francophones ! par Le blog de Patrick [MVP SharePoint] le 05-20-2013, 12:59

- [Conf’SharePoint] Dernier rappel ! :-) par Le blog de Patrick [MVP SharePoint] le 05-20-2013, 09:09

- [ #SharePoint 2013 ] les modèles de sites standards… par Le blog de Patrick [MVP SharePoint] le 05-20-2013, 09:03

- 10 erreurs de compréhension concernant SharePoint… par Le blog de Patrick [MVP SharePoint] le 05-20-2013, 08:27

- Conf’SharePoint : 10 bonnes raisons pour ne pas la rater par Le petit blog de Pierre / Pierre's little blog le 05-14-2013, 02:24

- [Event] Soirée de lancement Agile .NET France à Lyon par Blog Agile/ALM de Vincent THAVONEKHAM le 05-13-2013, 01:29

- .NET / Debug : inspection de la mémoire d'applications .NET (dump ou processus live) : première livraison d'une librairie .NET par Microsoft par CoqBlog le 05-11-2013, 22:21