Wie kann ich eine Codebasis umgestalten, während andere sich schnell darauf festlegen?


18

Ich bin in einem privaten Projekt, das irgendwann zu Open Source wird. Wir haben ein paar Teammitglieder, die mit den Technologien zum Erstellen von Apps talentiert genug sind, aber keine engagierten Entwickler, die sauberen / schönen und vor allem langfristig wartbaren Code schreiben können.

Ich habe mich vorgenommen, die Codebasis umzugestalten, aber es ist ein bisschen unhandlich, da jemand im Team in einem anderen Land, mit dem ich nicht regelmäßig in Kontakt stehe, diese völlig separate Sache aktualisieren könnte.

Ich weiß, dass eine Lösung darin besteht, schnell zu kommunizieren oder bessere PM-Praktiken anzuwenden, aber wir sind noch nicht so groß. Ich möchte nur den Code bereinigen und gut mit dem zusammenführen, was er aktualisiert hat. Wäre die Verwendung einer Filiale ein geeigneter Plan? Eine Best-Effort-Fusion? Etwas anderes?

Antworten:


35

Eine Sache , die Menschen oft nicht zu berücksichtigen ist , dass eine saubere Architektur nicht nur langfristige Wartung beschleunigen, es beschleunigt auch die Entwicklung bis jetzt . Versuchen Sie nicht, Ihre Änderungen vor Ihren Kollegen zu isolieren, bis diese "fertig" sind. Ihre Änderungen tragen dazu bei, dass sie produktiver und weniger fehleranfällig sind.

Der häufigste Fehler, den die Leute machen, wenn sie einen großen Refactor durchführen, ist, nicht oft genug zu fusionieren, sondern zu versuchen, dies in einem einzigen "Urknall" zu tun. Der richtige Weg, dies zu tun, besteht darin, den kleinstmöglichen Refactor zu erstellen, ihn zu testen, dann in die Filiale Ihres Kollegen zu integrieren und ihn über die Änderung zu unterrichten, damit er sie in Zukunft integrieren kann. Idealerweise führen Sie eine Zusammenführung pro Tag oder mindestens eine pro Woche durch.


17
Ja Ja Ja. Widerstehen Sie dem Drang, eine einmonatige Solo-Tour-de-Force zu machen, um festzustellen, dass sich die Codebasis, die Sie umgestalten sollen, vollständig geändert hat und Sie von vorne beginnen müssen. Mach es besser einen Schritt nach dem anderen.
Tdammers

Genau richtig! Große Refactorings gehen nirgendwo hin (siehe Netscape 6 oder die Projektpyramide )
Andomar

8

Sie sind nie "nicht groß genug, um zu kommunizieren". Wenn Sie mit den Fingern tippen können, können auch Ihre Lippen sprechen. Letztendlich besteht die Verbesserung der Technologie zu 85% aus Kommunikation und zu 15% aus Technik. Nur weil Sie lieber da sitzen und nicht mit jemandem reden, heißt das noch lange nicht, dass es eine gute Idee ist. Kommunikation ist eigentlich das Schwierigste an dem, was Sie versuchen, aber vermeiden Sie es nicht.


Es ist nicht wirklich die Schwierigkeit zu kommunizieren, es ist, dass ich nicht möchte, dass der gegenwärtige Entwickler langsamer wird. Eigentlich bin ich mir nicht mal sicher, ob er den richtigen Weg lernen muss, solange er überarbeitet werden kann. Er ist kein Programmierer, er ist ein Wissenschaftler auf einem anderen Gebiet.
Inkognito

+1. Sie können eine Codebasis nicht mit jemandem teilen, ohne zu kommunizieren
MarkJ

4

Ja, eine Filiale ist dafür eine gute Lösung.

Ich würde vorschlagen, dass Sie damit beginnen, an einem Zweig zu arbeiten, und sicherstellen, dass dieser HEADin der Zwischenzeit sauber auf Ihren aktuellen angewendet wird (dh, dass Sie in regelmäßigen Abständen Test-Rebases durchführen und zusammenführen, um sicherzustellen, dass Sie Ihre Änderungen problemlos anwenden können und Ihre Tests weiterhin erfolgreich sind - - Suchen Sie auch hier nachgit rerere Hilfe git. Sobald Sie fertig sind, führen Sie Ihre Änderungen wieder in Ihrem zusammen HEAD.

Je früher Sie damit beginnen, desto besser, da die Änderung der Architektur umso mehr funktioniert, je kälter der Code wird. Es kann auch viele Fälle von handcodiertem Code geben, die über die gesamte Codebasis verstreut sind, z.


1
-1: Nein. Siehe @ Karl Bielefeldts Antwort.
Jim G.

Ja? Ich bin nicht anderer Meinung als Karl, deshalb habe ich darauf geachtet, schnell anzufangen.
Benjamin Bannier

Und ich sage: "Verzweige dich nicht und füge dich dann wieder zusammen". Bestenfalls ist es vergebliche Mühe. Im schlimmsten Fall werden Sie ein großes Durcheinander machen.
Jim G.

3

Haben Sie die Option "Noch nicht machen" in Betracht gezogen?

Während es wahrscheinlich am besten ist, diese Arbeit in einem separaten Zweig zu erledigen, bereiten Sie sich auf eine massive schmerzhafte Verschmelzung vor.

Die anderen Typen fügen vermutlich viele neue Funktionen hinzu, ändern vorhandene Funktionen und entfernen möglicherweise einige Funktionen.

Wenn sich die Entwicklung von Mainstream-Entwicklern irgendwann in der Zukunft etwas verlangsamt hat, können Sie möglicherweise leichter umgestalten.


+1. Wenn sich Ihre Codebasis in einem massiven Wandel befindet, ist dies wahrscheinlich nicht der beste Zeitpunkt, um ein umfassendes Umschreiben zu versuchen. Wählen Sie eine Zeit in Ihrem Entwicklungszyklus, in der es ruhiger wird.
23.03.12

2

tl; dr - Hört sich an, als wäre es an der Zeit, zu den großen Ligen aufzusteigen. Wenn Sie einem Schwein Lippenstift geben, wird es nicht schöner, es sei denn, Sie mögen so etwas ...

Das Menschenproblem

Das erste Problem ist die Festschreibungssynchronisierung. WENN mehrere Personen gleichzeitig an demselben Code arbeiten, benötigen Sie nur eine Regel, um Probleme zu vermeiden:

Rule 1: Always pull before you merge/rebase

Wenn es um DVCS geht, ist es schwierig, Änderungen an einem entfernten Zweig (dh dem Haupt-Repository) vorzunehmen, und es ist sehr einfach, Änderungen am lokalen Zweig vorzunehmen. Jede Person ist dafür verantwortlich, dass ihre eigenen Code-Ergänzungen problemlos in das Gesamtbild passen. Es sei denn, 2 Personen verpflichten sich genau zur gleichen Zeit, sollten Sie nicht erfahren. Der Commit-Zugriff auf den Ursprungs- / Remote-Master sollte auf wenige Entwickler beschränkt sein, und sie sollten Änderungen von den anderen Entwicklern über Remote-Tracking-Zweige abrufen.

Das Code-Problem

Woher wissen Sie, dass die vorgenommenen Änderungen den Code nicht beschädigen?

Einfache Antwort ... Schreiben Sie Tests, um zu beweisen, dass dies nicht der Fall ist. Wenn Sie die TDD (Test Driven Design) -Schule ignorieren, besteht der springende Punkt der Tests darin, eine Überprüfungsebene hinzuzufügen, mit der Sie Code ändern können, ohne ihn zu beschädigen.

Rule 2: Don't make assumptions, write proofs (ie tests).

Darüber hinaus sollte die gesamte Bandbreite der Tests ausgeführt werden, bevor Sie einen Push an den Origin / Remote-Master senden.

Halten Sie Ihre Commits so klein und prägnant wie möglich. Auf diese Weise ersparen Sie sich die Neuimplementierung der Teile, die den Code nicht beschädigt haben, wenn Sie eine Änderung rückgängig machen müssen, die später zu Problemen geführt hat.

Möglicherweise müssen Sie zuerst die Organisation neu strukturieren

Wenn die obigen Lösungen nicht einfach angewendet werden können, gibt es wahrscheinlich einige Probleme mit der Entwicklungsstruktur, die zuerst behoben werden müssen.

Der Eigentümer des Projekts sollte der Gatekeeper sein. Bei Problemen mit der Commit-Synchronisierung sind wahrscheinlich zu viele Personen mit Commit-Zugriff beschäftigt. Selbst bei umfangreichen Projekten wie dem Linux-Kernel haben nur eine Handvoll Entwickler Zugriff auf das Origin / Remote-Master-Repository. Es gibt tatsächlich mehrere Ebenen von Repositorys, um Commits zu verwalten. Anstelle eines einschichtigen Festschreibungsmodells, bei dem alle ihre Änderungen an den Ursprung verschieben, verfügt das hierarchische Modell über Gatekeeper, die Änderungen abrufen und deren Qualität überprüfen, bevor sie in das Projekt aufgenommen werden. Das hierarchische Festschreibungsmodell kann wesentlich größer und effektiver skaliert werden als das Einzelschichtmodell, ohne dass die Qualität beeinträchtigt wird.

Entwickler, die keinen Commit-Zugriff erhalten, sollten lernen, ihre eigenen Remote-Tracking-Zweige zu erstellen (Git und Gitorious sind dafür gut), damit Entwickler, die über Commit-Zugriff verfügen, problemlos Zweige in den Ursprung ziehen / integrieren können. Wenn die Änderungen gering sind, funktionieren Patches genauso gut.

Die Möglichkeit, Änderungen vor dem Zusammenführen / Wiederherstellen abzurufen, setzt voraus, dass Sie nicht in Ihrer lokalen Hauptniederlassung entwickeln. Die einfache Möglichkeit, dies zu handhaben, besteht darin, einen ersten Pull durchzuführen, bevor Sie mit dem Code beginnen, und dann Ihre gesamte Arbeit an diesem Zweig auszuführen. Der schwierige Weg ist, es kurz vor dem Zusammenführen zu verzweigen und den Master zurückzusetzen.

Definieren Sie den Codierungsstil für das Projekt insgesamt und lassen Sie die Entwickler ihm folgen. Mitwirkende Entwickler sollten Code schreiben, der den Standards / Normen des Projekts entspricht, um die Bereinigung zu minimieren. Coding-Stil kann eine große Ego-Barriere in einem offenen Projekt sein. Wenn kein Standard festgelegt ist, wird jeder in seinem bevorzugten Stil codieren und die Codebasis wird sehr schnell sehr hässlich.

Der Mythos von "The Mythical Man Month"

Ob Sie es glauben oder nicht, die größeren / erfolgreicheren Open Source-Projekte laufen nicht wie eine Demokratie. Sie werden als Hierarchie geführt. Es ist naiv zu behaupten, dass ein Projekt nicht effektiv über 8-10 Entwickler hinauswachsen kann. Wenn das wahr wäre, gäbe es keine Megaprojekte wie den Linux-Kernel. Das tiefere Problem ist, dass es zu schwierig ist, eine effektive Kommunikation zu handhaben, wenn jedem ein Commit-Zugriff gewährt wird.

Das Problem des mythischen Mannmonats kann tatsächlich überwunden werden. Sie müssen Ihr Projekt nur wie das Militär betreiben. Es gibt viele Ebenen innerhalb der Hierarchie, da es allgemein bekannt ist, dass einzelne Personen nur mit einer Handvoll von Personen effektiv kommunizieren können. Solange keine einzelne Person für die Verwaltung der Arbeit von mehr als 5-7 Personen verantwortlich ist, kann das System unbegrenzt skaliert werden.

Möglicherweise beschränken sich die besten / erfahrenen Entwickler darauf, mehr Integration und übergeordnetes Design / Planung durchzuführen, aber das ist keine schlechte Sache. Ein Teil der Skalierung besteht darin, zu entscheiden, dass das Projekt einen langfristigen Plan benötigt. Die Menschen auf höchster Ebene, die die größte Investition (Zeit ist auch eine Ressource) in die Zukunft des Projekts haben, sollten mit den großen Entscheidungen beauftragt werden.

Es ist schön zu hören, dass ein Open-Source-Projekt immer mehr Probleme hat. Herzlichen Glückwunsch und viel Glück.


-1

sauberer / schöner und vor allem langfristig wartbarer Code.

Meiner Erfahrung nach ist sauber / schön der Feind von wartbar. Schöner Code oft:

  • Verfügt über eine Ebene im Framework, die eine höhere Abstraktionsebene einführt
  • Optimiert die Wiederverwendung von Code, was zu vielen Abhängigkeiten führt
  • Versucht, das generische Problem anstelle des spezifischen zu lösen

Andererseits wartbarer Code:

  • Wird direkt auf das Framework geschrieben, damit alle Entwickler es lesen können
  • Optimiert für eine geringe Anzahl von Abhängigkeiten, sodass sich eine Änderung in einem Bereich nicht auf einen anderen auswirkt
  • Versucht nicht, mehr Probleme zu lösen als nötig

Ihre schöne Codebeschreibung kann auch mit wartbarem Code einhergehen, denn wenn Sie eine höhere Abstraktionsebene einführen und Ihren Code für die Wiederverwendung optimieren, ist dies ohnehin einfacher zu pflegen.
Karthik Sreenivasan

Abgesehen davon, dass die Abstraktion den Test der Zeit nicht bestehen wird. Und jedes Problem mit der Abstraktion hebt einen lokalen Fix in einen Fix, der möglicherweise eine anwendungsweite Auswirkung hat.
Andomar
Durch die Nutzung unserer Website bestätigen Sie, dass Sie unsere Cookie-Richtlinie und Datenschutzrichtlinie gelesen und verstanden haben.
Licensed under cc by-sa 3.0 with attribution required.