Verwenden Sie FxCop zur Codeprüfung und -optimierung

Ich habe vor einiger Zeit ein Projekt mit FxCop erstellt, einem kostenlosen Testtool von Microsoft. Nach der Verwendung fand ich es ziemlich gut. Es kann auch einige automatische Tests von Codes durchführen und die Codeausführung verbessern Effizienz und verbessert dadurch die Gesamtqualität des Codes.

Die Orte, die dieses Tool testen kann, sind: Design, Globalisierung, Interoperabilität, Benennung, Leistung (Leistungsregeln), Sie können es auch selbst hinzufügen. Regeln werden getestet;

Das Tool ist sehr einfach zu bedienen und ich habe schnell angefangen
Ein paar Tipps
1 Sie können die falschen Regeln anpassen, es wird empfohlen, Globalisierungsregeln, Sicherheitsregeln und andere Regeln aufzuheben

2. Klicken Sie auf den Link zum Problem, und dann können Sie sich direkt mit dem Code-Problem verbinden

3 Sie können sich die Info-Kommentare ansehen, die bessere Lösungen finden


Der geänderte Code nach der Inspektion umfasst:

1 Klassenbenennungsregeln

2 Bestimmen Sie, ob das Zeichen leer ist, indem Sie
if (strStartValue.ToString() == " ")
Ändern zu
if (strStartValue.Length() == 0))


3 Beim Vergleich von Zeichenfolgen
dr["PrefixM"].ToString().ToLower () == "mm" Verwenden Sie String.Compare(strPrefix, "yy", true) == 0 Methode


4 Löschen Sie die in der Methode definierten nicht verwendeten Parameter

5 Löschen ungültige Zuweisungen

5 Nicht in DataReader geschlossen (in anderen Programmen)

6 Aufforderung zum Löschen nicht verwendeter Methoden im Programm

7 Aufforderung, dass Catch angegeben werden muss Ausnahme, dies kann beim Abfangen von Ausnahmen Speicherverbrauch sparen

8 Überflüssiges mithilfe von Referenzen löschen

Die aufgetretenen Probleme sind:
1 Es gibt viele Globalisierungsanforderungen, aber ich achte normalerweise nicht darauf Ja, obwohl das entwickelte Programm mehrsprachige Unterstützung erfordert
strMonth = StartDateValue.Month.ToString();

Dies wird darauf hinweisen, dass mehrsprachige Unterstützung erforderlich ist

2 Die Für das Programm ist AssembliesShouldDeclareMinimumSecurity nicht definiert.
Eine Signatur ist erforderlich, aber dies wird normalerweise nicht bemerkt.

3 Die von CLR geschriebene gespeicherte Prozedur muss Out-Parameter haben, wird aber auch dazu aufgefordert, also detaillierter Regelwerk muss formuliert werden

4 Überall dort, wo es String-Ergänzungen in der Schleife gibt, wird man aufgefordert, Stringbuilder zu verwenden. Diese Änderung ist ziemlich mühsam, es sei denn, man achtet vorher darauf,