2012-02-15 3 views
0

Quelle différence entre ces méthodes, pourquoi SortDropDown est correct mais Page_load et GetCases ne le sont pas?StyleCop SA1630

Je ne comprends pas.
Et comment résoudre ce problème?

enter image description here

Modifier ne peux pas bien voir sur la photo

code:

 /// <summary> 
     /// Sort items in drop down list 
     /// </summary> 
     /// <param name="dropDown">Drop down list</param> 
     internal static void SortDropDown(ref DropDownList dropDown) 
     { 

     } 

     /// <summary> 
     /// PageLoad event handler 
     /// </summary> 
     /// <param name="sender">Sender</param> 
     /// <param name="e">Event Args</param> 
     protected void Page_Load(object sender, EventArgs e) 
     { 

     } 

     /// <summary> 
     /// Get all cases by authority and ShopNo 
     /// </summary> 
     /// <param name="authority">Authority</param> 
     /// <param name="shopNo">Shop No</param> 
     /// <returns>Cases list</returns> 
     private static IEnumerable<CaseSummary> GetCases(string authority, string shopNo) 
     { 

     } 


Merci!

+2

S'il vous plaît code postal, pas des captures d'écran de code. C'est très difficile à lire et impossible à éditer. – Yuck

Répondre

6

Certains de vos paramètres ont un seul mot comme documentation, ce qui n'est évidemment pas suffisant (il faut au moins 10 caractères + au moins un espace).

  • Sender
  • Authority

Ecrire une description utile sur ce qui est le but de ces paramètres.

En outre, pour les gestionnaires d'événements, vous devriez vraiment adopter le texte de la documentation utilisée par Microsoft:

/// <summary> 
/// Handles the XXXXX event of YYYY. 
/// </summary> 
/// <param name="sender">The source of the event.</param> 
/// <param name="e">The <see cref="EventArgs"/> instance containing the event data.</param> 
+0

Merci, ça marche! Mais est-il nécessaire de mettre "." en fin de description? ou c'est comme "bon style" de la documentation? –

+0

@andronz AFAIK, no. Mais au moins 10 caractères sont requis, alors peut-être qu'il n'y en a pas assez sans votre "." ... – ken2k

+0

Je suggère que vous utilisiez une orthographe, une majuscule et une ponctuation correctes. – Bernard

1

La différence est que la documentation utilise un mot pour décrire l'un des paramètres de l'événement Page_Load() et GetCases() méthode , alors que la documentation de la méthode SortDropDown() utilise plus d'un mot pour décrire son paramètre. Soyez plus descriptif et vous éviterez cette violation de règle.