Gibt es einen gültigen Grund für die Code-Duplizierung?

Question

Ich überprüfe gerade ein sehr altes C++ Projekt und sehe dort viel Code Duplikation.

Zum Beispiel gibt es eine Klasse mit 5 MFC-Nachrichtenhandlern, die jeweils 10 identische Codezeilen enthalten. Oder es gibt ein 5-zeiliges Snippet für eine sehr spezifische String-Transformation hier und da. Die Reduzierung von Code-Duplikaten ist in diesen Fällen überhaupt kein Problem.

Aber ich habe das seltsame Gefühl, dass ich etwas missverstehen könnte und dass es ursprünglich einen Grund für diese Vervielfältigung gab.

Was könnte ein gültiger Grund sein, Code zu duplizieren?

Answer 1

Als ich anfing zu programmieren, schrieb ich eine App, wo ich eine Reihe ähnlicher Funktionen hatte, die ich in eine nette kleine 20-30 Zeilenfunktion einwickelte ... Ich war sehr stolz auf mich selbst für das Schreiben solch eines eleganten Stückes von Code.

Kurz danach änderte der Client den Prozess in ganz bestimmten Fällen, dann wieder, dann wieder, dann wieder, und wieder und wieder .... (viele, viele weitere Male) Mein eleganter Code wurde zu einem sehr schwierigen, hackish, buggy, & wartungsarm.

Ein Jahr später, als ich aufgefordert wurde, etwas sehr ähnliches zu tun, entschied ich mich bewusst, DRY zu ignorieren.Ich stellte den grundlegenden Prozess zusammen und generierte alle doppelten Code. Der doppelte Code wurde dokumentiert und ich speicherte die Vorlage, die zum Generieren des Codes verwendet wurde. Wenn der Klient nach bestimmten bedingten Änderungen fragte (wie, wenn x == y^z + b dann 1 + 2 == 3.42), war es ein Stück Kuchen. Es war unglaublich einfach zu pflegen & Änderung.

Rückblickend hätte ich wahrscheinlich viele dieser Probleme mit Funktionszeigern und Prädikaten lösen können, aber mit dem Wissen, das ich damals hatte, glaube ich immer noch an diesen speziellen Fall, das war die beste Entscheidung.

Answer 2

Faulheit, das ist der einzige Grund, den ich mir vorstellen kann.

Auf eine ernstere Notiz. Der einzige gültige Grund, an den ich denken kann, sind Änderungen am Ende des Produktzyklus. Diese neigen dazu, viel genauer untersucht zu werden, und die kleinste Veränderung hat tendenziell die höchste Erfolgsrate. In diesem eingeschränkten Fall ist es einfacher, eine Code-Duplizierungsänderung durchzusetzen, als eine kleinere Änderung umzuformen.

Noch hinterlässt einen schlechten Geschmack in meinem Mund.

Answer 3

Die einzige "gültige" Sache, die ich daraus sehen kann, ist, wenn diese Codezeilen unterschiedlich waren, konvergierte dann durch spätere Bearbeitungen zur selben Sache. Ich hatte das schon einmal mit mir, aber nicht allzu oft.

Dies ist natürlich, wenn es Zeit ist, dieses gemeinsame Code-Segment in neue Funktionalität zu zerlegen.

Das gesagt, ich kann mir keinen vernünftigen Weg vorstellen, um doppelten Code zu rechtfertigen. Schau, warum es schlecht ist.

Es ist schlecht, weil eine Änderung an einem Ort eine Änderung an mehreren Stellen erfordert. Dies ist eine erhöhte Zeit mit einer Chance auf Fehler. Wenn Sie dies ausschließen, pflegen Sie den Code an einem einzigen Arbeitsplatz. Wenn Sie ein Programm schreiben, schreiben Sie es nicht zweimal, warum sollte eine Funktion anders sein?

Answer 4

Es gibt keinen guten Grund für die Code-Duplizierung.

Siehe das Refactor Mercilessly Entwurfsmuster.

Der ursprüngliche Programmierer war entweder in Eile, einen Termin einzuhalten oder faul. Fühlen Sie sich frei, den Code zu überarbeiten und zu verbessern.

Answer 5

Für diese Art von Code-Duplizierung (viele Zeilen dupliziert viele Male), würde ich sagen:

• entweder Faulheit (Sie nur einige Code hier und dort einfügen, ohne sich um irgendwelche Gedanken machen zu müssen Auswirkungen könnte es auf andere Teile der Anwendung haben - während des Schreibens einer neuen Funktion und die Verwendung an zwei Stellen könnte, nehme ich an, einige Auswirkungen haben)
• oder keine gute Praxis kennen (Code wiederverwenden, trennen verschiedene Aufgaben in verschiedenen Funktionen/Methoden)

Wahrscheinlich ist die erste Lösung, aber von dem, was habe ich in der Regel :-(

gesehen

beste Lösung, die ich gegen das gesehen habe: haben Ihre developpers durch die Aufrechterhaltung einige alte Anwendung zu starten, wenn sie eingestellt werden - - das wird ihnen beibringen, dass diese Art von Sache nicht gut ist ... Und sie werden verstehen, warum, das ist der wichtigste Teil.

Splitting Code in mehrere Funktionen, die Wiederverwendung von Code der richtige Weg, und alles, was oft mit Erfahrung kommen - oder Sie haben nicht die richtigen Leute eingestellt ;-)

Answer 6

vor langer Zeit, als ich früher Wenn Sie Grafiken programmieren, würden Sie in einigen speziellen Fällen doppelten Code verwenden, um JMP-Anweisungen auf niedriger Ebene zu vermeiden, die im Code generiert werden (dies würde die Leistung verbessern, indem der Sprung zur Beschriftung/Funktion vermieden wird). Es war eine Möglichkeit zu optimieren und ein Pseudo "Inlining" zu machen.

In diesem Fall glaube ich nicht, dass sie es deshalb taten, heh.

Answer 7

Klingt wie der ursprüngliche Autor war entweder unerfahren und/oder wurde hart auf Zeit gedrückt. Die meisten erfahrenen Programmierer bündeln Dinge, die wiederverwendet werden, weil später weniger Wartung benötigt wird - eine Form der Faulheit.

Die einzige Sache, die Sie überprüfen sollten, ist, wenn es irgendwelche Nebenwirkungen gibt, wenn der kopierte Code auf einige globale Daten zugreift, ein bisschen Refactoring möglicherweise benötigt wird.

edit: Damals, als Compiler noch beschissen und Optimierer waren, konnte es passieren, dass man aufgrund eines Fehlers im Compiler einen solchen Trick machen musste, um einen Fehler zu umgehen. Vielleicht ist es so etwas? Wie alt ist alt?

Answer 8

Sie können dies tun, um sicherzustellen, dass zukünftige Änderungen in einem Teil nicht unbeabsichtigt den anderen Teil ändern. zum Beispiel

Do_A_Policy() 
{ 
    printf("%d",1); 
    printf("%d",2); 
} 

Do_B_Policy() 
{ 
    printf("%d",1); 
    printf("%d",2); 
} 

Sie können jetzt „Code-Duplizierung“ mit Funktion wie diese betrachten verhindern:

first_policy() 
{ 
printf("%d",1); 
printf("%d",2); 
} 

Do_A_Policy() 
{ 
first_policy() 
} 

Do_B_Policy() 
{ 
first_policy() 
} 

Es besteht jedoch die Gefahr, dass einige andere Programmierer ändern möchten Do_A_Policy() und tun Wenn Sie also first_policy() ändern, wird der Nebeneffekt der Änderung von Do_B_Policy() auftreten, ein Nebeneffekt, den der Programmierer möglicherweise nicht bemerkt. so kann diese Art von "Code-Duplikation" als ein Sicherheitsmechanismus gegen diese Art von zukünftigen Änderungen im Programm dienen.

Answer 9

Manchmal haben Methoden und Klassen, die domänenweise nichts gemeinsam haben, sich aber in der Implementierung sehr ähnlich.In diesen Fällen ist es oft besser, Code-Duplikation als zukünftige Änderungen durchzuführen, die diese Implementierungen nicht in etwas verzweigen, das nicht das Gleiche ist.

Answer 10

Wenn verschiedene Aufgaben aus Versehen ähnlich sind, ist das Wiederholen derselben Aktionen an zwei Stellen nicht unbedingt eine Duplizierung. Wenn sich die Handlungen an einem Ort ändern, ist es wahrscheinlich, dass sie sich auch an anderen Orten ändern? Dann ist dies eine Duplizierung, die Sie vermeiden oder umstrukturieren sollten.

Auch manchmal - auch wenn Logik doppelt vorhanden ist - sind die Kosten für die Reduzierung von Duplikaten zu hoch. Dies kann besonders dann passieren, wenn es nicht nur um die Duplizierung von Code geht: Wenn Sie beispielsweise Datensätze mit bestimmten Feldern haben, die sich an verschiedenen Stellen wiederholen (DB-Tabellendefinition, C++ - Klasse, textbasierte Eingabe), können Sie dies normalerweise reduzieren Duplizierung erfolgt mit Code-Generierung. Dies erhöht die Komplexität Ihrer Lösung. Fast immer zahlt sich diese Komplexität aus, manchmal aber auch nicht - es ist Ihr Kompromiss.

Answer 11

Bei großen Projekten (die mit einer Code-Base so groß wie ein GB) ist es möglich bestehende API zu verlieren. Dies liegt in der Regel an unzureichender Dokumentation oder daran, dass der Programmierer den ursprünglichen Code nicht finden kann. daher doppelter Code.

Abkochung zu Faulheit oder schlechte Review-Praxis.

EDIT:

Eine weitere Möglichkeit besteht darin, dass es in diesen Verfahren zusätzlicher Code haben kann, die auf dem Weg entfernt wurden.

Haben Sie sich den Überarbeitungsverlauf in der Datei angesehen?

Answer 12

Neben unerfahren zu sein, ist es, warum duplizierten Code Ereignisse auftauchen könnte:

Keine Zeit richtig

Refactoring Die meisten von uns sind in einer realen Welt zu arbeiten, wo reale Zwänge uns zwingen, sich schnell zu bewegen zu echten Problemen, anstatt über die Nettigkeit des Codes nachzudenken. Also kopieren wir & einfügen und weitermachen. Wenn ich später sehe, dass der Code mehrere Male dupliziert wird, ist es das Zeichen, dass ich etwas mehr Zeit damit verbringen muss und alle Instanzen zu einem konvergieren muss.

Generalisierung des Codes nicht möglich/nicht ‚ziemlich‘ aufgrund von Sprach Zwänge

Lassen Sie uns sagen, dass tief im Inneren einer Funktion, die Sie mehrere Aussagen haben, die sich stark von Instanz zu Instanz gleichen duplizierten Code unterscheiden. Zum Beispiel: Ich habe eine Funktion, die 2d Miniaturbilder für das Video zeichnet, und es ist mit der Berechnung jeder Miniaturbildposition eingebettet. Um den Treffer-Test zu berechnen (Index-Index aus Klick-Position berechnen) benutze ich denselben Code, aber ohne zu malen.

Sie sind nicht sicher, dass es Generalisierung bei allen auf den ersten

doppelten Code sein, und später beobachten, wie es sich entwickeln wird. Da wir Software schreiben, können wir "so spät wie möglich" Änderungen an der Software vornehmen, da alles "weich" und veränderbar ist.

Ich füge mehr hinzu, wenn ich mich an etwas anderes erinnere.

---

Später hinzugefügt ...

Schleife

In der Zeit Abrollen vor Compiler intelligent waren wie Einstein und Hawking kombiniert, hatten Sie die Schlaufen oder Inline-Code entrollen, schneller zu sein. Schleifen-Abrollung wird dazu führen, dass Ihr Code dupliziert wird, und wahrscheinlich um ein paar Prozent schneller, hat der Compiler das sowieso nicht gemacht.

Answer 13

Der gültige Grund, den ich mir vorstellen kann: Wenn der Code viel komplexer wird, um die Duplizierung zu vermeiden. Im Grunde ist das der Ort, an dem man in mehreren Methoden fast das Gleiche macht - aber eben nicht ganz dasselbe. Natürlich können Sie spezielle Parameter wie Zeiger auf verschiedene Elemente, die geändert werden müssen, refaktorieren und hinzufügen. Aber die neue, refactored Methode kann zu kompliziert werden.

Beispiel (Pseudo-Code):

procedure setPropertyStart(adress, mode, value) 
begin 
    d:=getObject(adress) 
    case mode do 
    begin 
    single: 
     d.setStart(SingleMode, value); 
    delta: 
     //do some calculations 
     d.setStart(DeltaSingle, calculatedValue); 
    ... 
end; 

procedure setPropertyStop(adress, mode, value) 
begin 
    d:=getObject(adress) 
    case mode do 
    begin 
    single: 
     d.setStop(SingleMode, value); 
    delta: 
     //do some calculations 
     d.setStop(DeltaSingle, calculatedValue); 
    ... 
end; 

Sie könnten den Methodenaufruf Refactoring out (setXXX) irgendwie - aber abhängig von der Sprache könnte es schwierig sein (vor allem mit Vererbung). Es ist Code-Duplizierung, da der größte Teil des Körpers für jede Eigenschaft gleich ist, aber es kann schwierig sein, die gemeinsamen Teile umzuformen.

Kurz gesagt - wenn die Refaktor-Methode Faktoren komplizierter ist, würde ich mit Code-Duplizierung gehen, obwohl es "böse" ist (und böse bleiben wird).

Answer 14

Alle Antworten sieht richtig aus, aber ich denke, es gibt eine andere Möglichkeit. Vielleicht gibt es Überlegungen zur Leistung, da die Dinge, die Sie sagen, erinnert mich "Inline-Code". Inline-Funktionen, die sie aufrufen, sind immer schneller. Vielleicht wurde der Code, den Sie betrachten, zuerst vorverarbeitet?

Answer 15

Eine gute Lektüre darüber ist large scale c++ software design von John Lakos.

Er hat viele gute Punkte über Code-Duplikation, wo es ein Projekt helfen oder behindern könnte.

Der wichtigste Punkt zu fragen, wenn Doppelarbeit zu entfernen entscheiden oder Code kopieren:

Wenn diese Methode in Zukunft ändert, will ich das Verhalten in der duplizierten Methode ändern, oder es muss die bleiben wie es ist?

Schließlich enthalten Methoden (Geschäfts) Logik, und manchmal wollen Sie die Logik für jeden Anrufer ändern, manchmal nicht. Hängt von den Umständen ab.

Am Ende dreht sich alles um Wartung, nicht um hübsche Quelle.

Answer 16

Ich habe keine Probleme mit dupliziertem Code, wenn es von einem Quellcode-Generator erzeugt wird.

Answer 17

Ich kenne nicht viele gute Gründe für Code-Duplikation, aber anstatt in Fuß zuerst zu refactoring springen, ist es wahrscheinlich besser, nur die Teile des Codes, die Sie tatsächlich ändern, Refactoring, anstatt eine große Codebase zu ändern Du verstehst das noch nicht ganz.

Answer 18

Etwas, das uns dazu brachte, Code zu duplizieren, war unser Pixelmanipulationscode. Wir arbeiten mit sehr großen Bildern und der Overhead des Funktionsaufrufs verzehrte in der Größenordnung von 30% unserer pro-Pixel-Zeit.

Die Duplizierung des Pixelmanipulationscodes ermöglichte uns eine um 20% schnellere Bilddurchquerung auf Kosten der Komplexität des Codes.

Dies ist offensichtlich ein sehr seltener Fall, und am Ende hat es unsere Quelle erheblich aufgebläht (eine 300 Zeilen Funktion ist jetzt 1200 Zeilen).

Answer 19

meiner bescheidenen Meinung nach gibt es keinen Platz für Code-Duplizierung. haben Sie einen Blick, zum Beispiel bei this wikipedia article

oder lassen Sie uns zu Larry Wall Zitat beziehen:

„Wir ermutigen Sie die drei großen Tugenden eines Programmierers zu entwickeln: Faulheit, Ungeduld und Hybris. "

Es ist ziemlich klar, dass Code-Duplizierung hat nichts mit "Faulheit" zu tun. haha;)

Answer 20

Da es das "Strategie-Muster" gibt, gibt es keinen gültigen Grund für doppelten Code. Nicht eine einzige Codezeile muss dupliziert werden, alles andere ist episch fehlgeschlagen.