Diskussion: Unbenötigten Code löschen?

[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

[Namenloser]

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.

[FredlFesl]

Ich frag mich: was mag das für Code sein, den man auskommentiert, und irgendwann, viel später, wieder gebrauchen könnte.
Da muss in der Planung schon Einiges schief gelaufen sein.

Wenn ich Code auskommentiere, dann während einer Iteration, also dem Test eines neuen Features. Ist das dann validiert, schmeisse ich den alten Code weg.

Ich kann mir auch nicht vorstellen, das ich eine Option erst hinzufüge, dann wieder wegnehme, um sie viel später wieder hinzuzufügen.

Alles eine Sache der Planung, würd ich mal sagen.

[Hansa]

Bei mir ist alter Sourcecode nie weg. Allerdings : sag niemals Nie ! Absoluten Quatsch muss man nicht unbedingt aufheben. Es lassen sich meistens immer auch nach langer Zeit noch Rückschlüsse aus altem QUelltext ziehen. Das gilt insbesondere, falls jemand sich einarbeiten muss und gerade an einer Stelle dran ist, die irgendwann mal in Bearbeitung war. Das gilt auch für den/die, die da mal rumgefummelt hat. Insofern : auskommentieren. Aber nie ohne Kommentar !

[Phoenix]

Ich habe in all den Jahren in denen ich jetzt programmiere nur ein einziges mal alten Code gesucht - und nach 2 Minuten in der Versionierung gefunden.

Wer toten Code in seinen Files rumliegen lässt müllt defakto seinen Arbeitsplatz, nämlich seinen Quellcode, zu und ist ein Messie. Nicht nur, dass der Kommentar einen von der Arbeit ablenkt - woher soll man wissen, ob der Code auskommentiert wurde, weil er nicht funktionierte, weil er unnötig geworden ist aufgrund einer anderen Anforderung oder ob es gar noch nicht fertiger Code ist, der nur auskommentiert wurde weil der kollege damit nicht fertig geworden ist und das nur zur Sicherung commiten wollte bevor er in Urlaub gefahren ist (dafür gibts eigentlich Branches, aber sei's drum).

Code der keine Funktion hat oder haben soll gehört komplett weg. Falls man ihn wider erwarten doch mal brauchen sollte, ist der in der Versionierung. Und das Argument das ein Kollege den brauchen könnte wenn er was anderes macht ist nicht valide.

Wenn er es heute machen sollte, dann braucht er zusätzliche Zeit sich in fremde Codes einzulesen und hätte es wahrscheinlich schneller - und möglicherweise besser - neu geschrieben. Will er (oder man selber) es in einem halben Jahr machen, hat man zwangsläufig in der Zwischenzeit so viel dazu gelernt, dass man es ohne den Einlese- und Ablenkungsfaktor nicht nur schneller, sondern auch definitiv besser neu schreibt, als es damals war. Und vielleicht hat man unbeeinflusst von der alten Implementierung sogar einen genialeren Einfall wie man das ganze Zeug besser implementieren kann und im besten Fall kann man dadurch noch mehr alten Code loswerden.