Finden von "totem Code" in einer großen C++ Altanwendung

Question

Ich arbeite gerade an einer großen und alten C++ - Anwendung, die viele Entwickler vor mir hatte. Es gibt eine Menge "toten Code" im Projekt, Klassen und Funktionen, die von niemandem mehr benutzt werden.

Welche Tools stehen für C++ zur Verfügung, um eine Analyse großer Code-Basis durchzuführen, um toten Code zu erkennen und zu refaktorieren? Hinweis: Ich spreche nicht über Testabdeckung Tool wie GCOV.

Wie finden Sie toten Code in Ihrem Projekt?

Answer 1

Sie erhalten eine statische Analyse Tool verwenden wollen

• Stackoverflow: What open source C++ static analysis tools are available?
• Stackoverflow: C++ static code analysis tool on Windows
• Wikipedia: List of tools for static code analysis

Der Haupt Gotcha Ich habe laufen In ist, dass Sie darauf achten müssen, dass keine Bibliotheken von irgendwo verwendet werden, dass Sie nicht c ontrol/haben. Wenn Sie eine Funktion aus einer Klasse löschen, die verwendet wird, indem Sie auf eine Bibliothek in Ihrem Projekt verweisen, können Sie etwas zerstören, von dem Sie nicht wussten, dass es den Code verwendet hat.

Answer 2

für toten Code nicht speziell Obwohl, fand ich die Quelle Navigator

http://sourcenav.berlios.de/

sehr nützlich, obwohl umständlich einzurichten und ein bisschen Buggy. Das war vor einem Jahr auf Linux (Fedora).

Answer 3

Ein Ansatz besteht darin, den Kontextmenübefehl "Alle Referenzen suchen" für Klassen- und Funktionsnamen zu verwenden. Wenn eine Klasse/Funktion nur in sich selbst referenziert wird, ist es fast sicher ein toter Code.

Ein anderer Ansatz, der auf der gleichen Idee basiert, ist das Entfernen (Auskommentieren) von Dateien/Funktionen aus dem Projekt und die Anzeige von Fehlermeldungen, die Sie erhalten.

Answer 4

Ich denke, Ihre beste Wette wäre wahrscheinlich ein Coverage-Tool. Es gibt viel für beide * nix und Windows. Wenn Sie Komponententests haben, ist es einfach - wenn Sie eine geringe Testabdeckung haben, dann ist der aufgedeckte Code entweder tot oder noch nicht getestet (Sie wollen beide Teile dieser Information trotzdem). Wenn Sie keine Komponententests durchführen, erstellen Sie Ihre App mit der Instrumentierung, die von einem dieser Tools bereitgestellt wird, führen Sie sie durch einige (idealerweise alle) Ausführungspfade, und sehen Sie sich den Bericht an. Sie erhalten die gleichen Informationen wie bei Komponententests, es erfordert nur viel mehr Arbeit.

Da Sie Visual Studio verwenden, könnte ich Ihnen einige Links geben, die Sie verwenden in Erwägung ziehen könnte:

• Coverage meter
• BullseyeCoverage

Keiner von ihnen ist frei, auch nicht billig , aber das Ergebnis ist es normalerweise wert.

auf * nichts-ähnlichen Plattformen gcov gekoppelt mit Werkzeugen wie zcov oder lcov ist eine wirklich gute Wahl.

Answer 5

Nichts ist besser als die Vertrautheit mit dem Code.Außer vielleicht strengem Beschneiden, wie man weitergeht.

Manchmal wird das, was aussieht wie Totholz, als Gerüst für Komponententests usw. verwendet, oder es scheint nur am Leben zu sein, weil Legacy Unit Tests es ausüben, aber es wird nie außerhalb der Tests ausgeführt. Vor Kurzem habe ich über 1000 LOC entfernt, was externe CAD-Modellübersetzer unterstützte, wir hatten Tests, die diese externen Übersetzer aufrufen, aber diese Übersetzer wurden für 8+ Jahre nicht unterstützt und es gab keinen Weg, einen Benutzer der Anwendung zu verwenden, selbst wenn er es wollte könnte sie jemals aufrufen.

Wenn man das tote Holz nicht rigoros beseitigt, findet man sein Team jahrelang bei der Wartung.

Answer 6

Siehe unsere SD C++ Test Coverage.

Sie müssen eine Menge dynamischer Tests durchführen, um den Code auszuüben, um sicherzustellen, dass Sie die maximale Abdeckung erreichen. Code "nicht abgedeckt" kann oder kann nicht tot sein; vielleicht hattest du einfach keinen Testfall, um es auszuüben.

Answer 7

Caolán McNamara's callcatcher wird sehr effektiv innerhalb des LibreOffice-Projekts (~ 6 MLOC) verwendet, um toten Code zu finden.

Answer 8

können Sie Cppcheck für diesen Zweck verwenden:

$ cppcheck --enable=unusedFunction . 
Checking 2380153.c... 
1/2 files checked 0% done 
Checking main.c... 
2/2 files checked 0% done 
[2380153.c:1]: (style) The function '2380153' is never used.