Diskussion: Unbenötigten Code löschen?

[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.

[GockelMehl]

Zitat von 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 !

Mache es fast genauso. An manchem - eventuell absolut blödsinnigen - von mir verbrochenem Quellcode kann ich meine damaligen Irrungen und Wirrungen noch nachvollziehen.

Alten Quellcode komplett löschen?

Dann gäbe es nur noch Klick+Run-Hochsprachenkiddies, die noch nie was von "echter Prozessoransprache mittels Assembler" gehört hätten.

Just my two cents.

[implementation]

Zitat von GockelMehl:

echter Prozessoransprache mittels Assembler

Also ich lösche die Abschnitte immer dann, wenn ich sie besser lösen konnte. Die müssen imho nicht ewig drinbleiben. Ganze Units lösche ich nie, auch wenn sie noch so schwachsinnig sind.

[Coffeecoder]

Zitat von 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.

Absolut stimme ich 100% zu. Was nicht mehr in den Code reingehört, soll gelöscht werden. Unbenötigter Code ist ein Code-Smell und kein "guter" Programmstyl. Also pflegt euren Code

Mfg Coffeecoder.

[s.h.a.r.k]

Was ist aber, wenn der bisherige Code eine coole Technik verwendet hat, die du wo anders auch noch brauchen könntest? Immer gleich eine Demo-Anwendung zu stricken ist da so eine Sache, vor allem, wenn man evtl. an etwas neuem dran ist und das probieren will. Zudem kann man durchaus von schnellen auf langsamen Code wechseln, da dieser vielleicht lesbarer ist. Aber man will den schnelleren Code vielleicht für andere Projekte nutzen. Pauschal lässt es sich nie sagen. Ich versuche so weit wie nur möglich meine Ideen in Komponenten/Libraries/Demos zu gießen, sodass ich jeden Code entsprechend verpackt habe. "Toten" Code gibts daher eigentlich kaum.

[Namenloser]

Manchmal schreibe ich erst recht „allgemeinen“ Code und kommentiere später aus Performancegründen Teile aus, die im Spezialfall nicht benötigt werden, die ich aber vielleicht später noch mal brauchen könnte. In solchen Fällen wäre es imo kontraproduktiv, den auskommentierten Code komplett zu löschen, zumal er oft zum Verständnis hilfreich ist.

[GockelMehl]

Weshalb sollte es auf die Frage des Threaderstellers denn eigentlich eine Antwort geben, die nicht genau so sinnlos/blöde ist, wie die Frage selbst?

[mschaefer]

Denke die IDE sollte eine Funktion haben um markierte Bereiche auf Tastendruck ein- und auszublenen. Wirklich obsoleter Code fliegt dann raus, aber kleine Hinweise und Ergänzungen könnten als ausblendbare Kommentare im Code vorhanden bleiben, stören aber nicht während des normalen Arbietens. Wünschen darf man sich ja so einiges....