Diskussion: Unbenötigten Code löschen?

[nachti1505]

Bekannte Autoren (Fowler et.al.) empfehlen ja, ungenutzten oder gar auskommentierten Code aus der working-copy ins Nirvana zu schicken, mit der Begründung in der Quellcodeverwaltung sei ja alles noch vorhanden. Bis zu diesem Punkt korrekt!

Nun habe ich in unserem Unternehmen - spezieller in unserer Anwendung - viele Dutzend Stellen dieser Art entdeckt und mir überlegt, diese gemäß dem Clean-Code-Ansatz zu eleminieren. Sie sind ja tatsächlich im SVN noch vorhanden. Ich frage mich allerdings jetzt, wie beispielsweise jemand anderes in drei Jahren Funktionalitäten wiederentdecken möchte, die heute auskommentiert und morgen nur noch im SVN vorhanden sind?

Für einen Entwickler stellen sich ja naturgemäß später die Fragen:

1) Gab es da mal was?
2) Wo gab es das?

Wie kann man vorgehen, um den zukünftigen Entwicklern hier eine Brücke zu bauen und eventuell verschwundene Code-Schnipsel wieder entdecken?

[alfold]

Evtl and der Stelle eine Kommentarzeile einfügen, wo sich das original befindet?
So kann man auch später noch mal schauen, aha so sah dies mal aus, warum wurde er geändert usw

Ansonsten Dokumentation.

Gruss alfold

[s.h.a.r.k]

Mögliche Erwähnungen können im Commit-Kommentar, Changelog oder Ticketsystem positioniert werden. Mehr fällt mir spontan nicht ein

[himitsu]

Wenn die delphiinterne Dateisuche jetzt auch noch in den alten Repositories suchen würde, dann könnte man dort sowas auch wiederfinden.

[Coffeecoder]

Hallo,

Danke dass du dieses Thema ansprichst. Ich gebe mal meine Meinung dazu ab:

Zitat:

Mögliche Erwähnungen können im Commit-Kommentar, Changelog oder Ticketsystem positioniert werden. Mehr fällt mir spontan nicht ein

Einer der besten Idee. Quasi ein Bericht erstellen, auch von alten gelöschten Funktionen.
Toter Code, also nicht mehr gebrauchbarer Code sollte nicht im Quelltext stehen.

Mfg Coffeecoder

[s.h.a.r.k]

Naja, der Punkt ist ja, dass der Code ja mal in das System gewandert ist, ergo ist das schon mal dokumentiert worden und somit auch im SVN vorhanden. Wenn der dann wieder rausgeworfen wird, dann wird das ja auch dokumentiert, ebenso der Grund. Sollte mal jemand was nachschauen wollen, dass muss er halt entsprechend suchen, daher sollte die Doku gut ist

[himitsu]

Für die Zeit der Umstellung / für eine Weile kann man ja den alten Code noch drin lassen und als deprecated markieren, mit einem Hinweis auf den neuen Ort.

[s.h.a.r.k]

Vielleicht sollte man noch dazu sagen, dass man hier eine Art Regelwerk zusammenstellen sollte, wie man was zu machen hat. Wenn das jeder verinnerlicht, dann sollte die Suche später auch nicht schwer fallen.

[blauweiss]

Manchmal ist für mich toter (auskommentierter Code) noch nützlich, um zu dokumentieren, A) welcher alternative Ansatz schon mal versucht wurde und B) warum er nicht funktionierte.
Bin mir nicht sicher, ob es immer so einfach ist dafür einen guten Change-Text zu schreiben. Also so gut, daß der alte Code in der SVN verschwinden kann.

Wobei ich gestehen muß, daß bei mir B) / A) ~ 20%

Grüße,
blauweiss

[NamenLozer]

Also ich als Hobbyprogrammierer kommentiere alten Code meist erst mal nur aus, wenn ich eine alternative Lösung schreibe. Erst wenn ich dann nach einiger Zeit sicher bin, dass die neue Lösung stabil läuft, lösche ich den toten Code dann komplett.

@blauweiss Ja, mache ich auch so ähnlich.