Ich habe eine Klasse, die Teil eines Codebibliotheksprojekts ist, das für einen bestimmten Zweck geschrieben wurde, der nicht mehr benötigt wird. Die Frage ist also, was machen Sie mit Code wie diesem? Löschen Sie es einfach, oder belassen Sie es im Hinblick darauf, dass zukünftige Entwickler auf es stoßen und nicht erkennen, dass sie es ignorieren können oder haben Sie ein Archivsystem, gibt es ein anerkanntes "Muster", das ist benutze ...
Wie Neil sagte, lösche es. Wenn ich angeheuert werde, um dein Projekt zu erhalten, Jahre nachdem du damit fertig bist und es immer noch voller totem Code ist .. Ich werde dich verfolgen. Und nicht die ooooohhhhh nett irgendwie eindringlich .. aber die ARRRRWWWGGGGGRRRR !!!!! nervige Art von Spuk.
Es kommt darauf an.
Wenn es nicht verwendet wird, weil es veraltet ist, lösche ich es von der aktuellen Codebasis, indem ich es lösche. Wenn sich herausstellt, dass es tatsächlich benötigt wird, können Sie es immer aus der Quellcodeverwaltung abrufen.
Wenn es momentan nicht verwendet wird, aber in naher Zukunft verwendet werden kann, würde ich es in der aktuellen Codebasis beibehalten, da ich nicht erwarte, dass andere Entwickler die Quellcodeverwaltung für Features durchsuchen . Mit anderen Worten: Wenn Sie etwas löschen, das eine hohe Wahrscheinlichkeit hat, verwendet zu werden, besteht die Möglichkeit, dass jemand es erneut implementiert.
Wenn es nirgendwo verwendet wird und nicht mehr benötigt wird, sollten Sie es löschen, um Verwechslungen zu vermeiden.
Sie haben nicht gesagt, welchen Code Sie verwenden, aber in C # / VisualStudio können Sie das Obsolete-Attribut verwenden, um anderen Entwicklern mitzuteilen, dass sie den Code nicht verwenden sollen. Sie können das Argument errors auf true setzen dass der Code verwendet wird.
Ich würde damit beginnen, die veralteten Codeelemente mit der Obsolete zu versehen Attribut. Auf diese Weise werden Sie in der Lage sein, jeden Code zu finden, der auf die veralteten Elemente verweist, und Ihnen so die Möglichkeit gibt, diese Teile zu aktualisieren. Wenn Sie keine Compilerwarnungen mehr erhalten, dass Sie veralteten Code verwenden, fahren Sie fort und löschen Sie ihn.
Update: OK, jetzt habe ich an .NET und C # gedacht, aber ich bin mir sicher, dass viele andere Sprachen ähnliche Funktionen haben ...
Ich stimme Neil vollkommen zu. Verwenden Sie SVN oder ein anderes Versionskontrollsystem, um Ihren Code zu verfolgen und alles zu löschen, was überflüssig ist. Zu viel kommentierter Code macht den Code nur schwer lesbar und in einigen Fällen ist das Debugging unmöglich.
Die beste Option ist, den Code zu entfernen, damit Sie ein saubereres Repository haben. Die meiste Zeit ist es nur eine kurze Zeit, die Sie tun, löschen Sie etwas von potenziell enormem Wert. Auf svn zu zählen, wenn der Programmierer es später braucht, wird nicht wirklich funktionieren. Weil du den Code kennen musst, der vorher existiert und dann einige durch den SVN scannen muss. Wenn ich wirklich denke, dass ich diesen Code behalten möchte, mache ich normalerweise ein Archiv aus den Dateien und füge sie mit einer Beschreibung in unser Wiki ein und lösche dann den Code. Über die Suche im Wiki kann jemand den Code finden. Scannen Sie es mit dem Archiv und da die Beschreibung Repository und Revisionsnummer enthält, können sie sogar die Teile, die sie benötigen, einfach wiederherstellen.
Installieren Sie GIT dann:
%Vor%... Refactor den Code ...
%Vor%... Erstellen Sie ein Konto auf GitHub oder richten Sie einen GitServer ein
%Vor%Und du bist fertig ...:)
Tags und Links refactoring