So bringen Sie Entwickler dazu, Codeüberprüfungen rechtzeitig durchzuführen

Für das Unternehmen, für das ich arbeite, muss der gesamte Code von anderen Entwicklern überprüft werden, bevor er festgeschrieben wird. Die Mitglieder meines Teams sind oft frustriert, weil die anderen Entwickler zu beschäftigt sind, um eine Überprüfung durchzuführen, insbesondere wenn diese sehr lang ist. Wie können Sie andere Entwickler dazu anregen, zeitnahe Codeüberprüfungen durchzuführen?

(Wir verwenden git-svn, damit wir weiter programmieren können, während wir auf eine Überprüfung warten. Ich finde es jedoch immer noch frustrierend, wenn ich lange warten muss, bis ich meinen Code festschreiben kann.)

Schauen Sie sich an, wie Facebook es mit seiner eigenen App namens phabricator macht: http://phabricator.org/

Sie verpflichten sich grundsätzlich pro Ausgabe, und für jede Ausgabe wird der Code angezeigt, der von jemandem überprüft werden muss. Der Code wird erst in das Haupt-Repository aufgenommen, wenn der Prüfer erklärt hat, dass dies in Ordnung ist.

Ich denke, es macht mehr Spaß.

Vielleicht sollte ein Code auch zwei Personen zugewiesen werden: einer, der ihn ausführt, und einer, der ihn überprüft.

Obwohl Ihre Teamkollegen vielleicht nicht an dieses Review-Ding glauben.

In Ermangelung von Gutachtern habe ich Unit-Tests für untergeordnete Funktionen und "den Hausmeistertest" für den Rest verwendet: Der Hausmeistertest wird so genannt, weil selbst der Hausmeister Ihren Code verstehen sollte.

Normalerweise entfernte ich einige kleinere Teile, wie z. B. Klammern für Block- / Funktionsbereiche, Sichtbarkeitsnotationen, manchmal sogar Typen, und zeigte sie Managern, Domain-Experten und Kollegen, die den Code angefordert hatten: "Ist dies das, was Sie wollen?"

Es hilft auch, persönlich dorthin zu gehen und erst zu gehen, wenn die Überprüfung abgeschlossen ist.

Oder falls Sie mit dem Team nicht gut zurechtkommen oder es Ihnen nicht gut geht, wissen Sie: "Wenn Sie die Firma ändern können, ändern Sie die Firma" ...

Ich werde dies auf ein paar Annahmen stützen:

1. Jeder scheint Code schreiben zu wollen (wenn nicht, haben Sie Leute, die gehen müssen.).
2. Jeder möchte, dass sein eigener Code eingecheckt wird.

Lassen Sie nur diejenigen, die ihre Bewertungen abschließen, ihren Code einchecken.

Möglicherweise kann ein bestimmter Zeitblock für Codeüberprüfungen verwendet werden, um das Problem der Unterbrechung zu vermeiden.

Das Ziel sollte sein, den Qualitätscode einzuchecken. Sie möchten die Bewertungen nicht bis zu dem Punkt bestrafen / erzwingen, an dem jeder nur eine "Stempel" -Zulassung erteilt.

Wenn Sie unterschiedliche Ebenen haben (jr., Sr. usw.), sollte die Beförderung und Pflege eines Titels von Ihrer Arbeit abhängen.

Bei einigen früheren Arbeitgebern haben wir gewechselt, wer täglich an der Codeüberprüfung teilnahm. Normalerweise teilten sich 3 Personen einen CR-Tag und jeder CR musste von zwei der Gutachter abgemeldet werden. Dies stellte sicher, dass Sie an Ihrem Tag wussten, dass CR von Ihnen erwartet wurde, unabhängig von Ihren anderen Projekten. Etwa alle fünf Tage waren Sie an der Reihe und konnten Ihre Entwicklungsaufgaben entsprechend anpassen.

Derzeit hat ein Teamleiter eine flüchtige CR für den Code seines Teams. Abhängig davon, welcher Bereich der Anwendung aktualisiert wurde, kann er nach Abschluss der CR an das Global Review Team weitergeleitet werden, wo wir nach Dingen suchen, die globale Auswirkungen auf die Anwendung (en) haben, anstatt nach Dingen, die dies sind in Bezug auf eine einzelne Funktion. Wenn eine große Überprüfung durchgeführt werden muss, wird diese normalerweise zwischen einigen wenigen Personen aufgeteilt, sodass sich niemand mit Änderungen in einer lächerlichen Anzahl von Dateien befassen muss.

Wir überprüfen jedoch immer nur Code, der für den aktuellen Dev-Zweig / die aktuelle Dev-Variante festgeschrieben wurde. Der Code muss Code Review, Global Review, DB Review und UI Review (jeweils nach Bedarf) bestehen, bevor er in die nächste (z. B. Alpha) Umgebung befördert werden kann.

Schließlich stimmen wir einer SLA zu, wie schnell Bewertungen umgedreht werden. Kaum etwas steht länger als 48 Stunden in der Warteschlange und die meisten Überprüfungen werden in weniger als 24 Stunden durchgeführt.

Für das Unternehmen, für das ich arbeite, muss der gesamte Code von anderen Entwicklern überprüft werden, bevor er festgeschrieben wird. Die Mitglieder meines Teams sind oft frustriert ...

Es gibt keine zwingenden Gründe, sich strikt an bestimmte Arten von Codeüberprüfungen zu halten, es sei denn, Sie führen unternehmenskritische Software oder kritische Patches für Code für Produktionsversionskandidaten durch.

• ^{Die Idee hinter Ihrer Unternehmensanforderung klingt auf einer Oberfläche vernünftig (100% überprüfter Code), aber die Mittel, für die sie sich entschieden haben, sind kontraproduktiv - denn wie Sie betonen, führen diese dazu, dass Entwickler frustriert sind.}

Wenn Sie in Ihre Fußstapfen treten, möchte ich das Management zunächst darauf hinweisen, dass Überprüfungen von Post-Commit- Codes genauso respektabel sind wie Pre-Commit- Überprüfungen . Diese Begriffe können im Web durchsucht werden. Falls erforderlich, finden Sie dort Verweise, um diese zu sichern. Es sind nicht unbedingt Pre-Commit- Überprüfungen erforderlich , um 100% überprüften Code zu erhalten.

Basierend auf dem oben Gesagten würde ich ihnen als nächstes vorschlagen, die Einstellung zum Mikromanagement aufzugeben und die Entwickler die Art und Weise ausprobieren zu lassen, die sich für sie bequemer anfühlt. Überprüfungen vor oder nach dem Festschreiben stehen den Programmierern am besten zur Auswahl. Wenn das Unternehmen es besser weiß als Programmierer, warum schreiben sie den Code nicht selbst?

Sie müssen eine Reihe von Problemen angehen - Sie müssen Herz und Verstand gewinnen und sicherstellen, dass Zeit für Codeüberprüfungen zur Verfügung steht.

Der zweite Teil ist wahrscheinlich am einfachsten - Sie stimmen zu (zusammen und das muss das Management einschließen), dass das erste, was ein Entwickler jeden Morgen tut, seine Codeüberprüfungen sind - dies ist einfach, verständlich, effektiv und gibt Ihnen einen schönen, klaren Stock, mit dem Sie Leute schlagen können wenn sie nicht entsprechen. Besser, Sie unterbrechen nichts, Sie fordern sie nicht auf, nicht mehr an ihrem Code zu arbeiten. Sie fordern die Leute nicht auf, etwas in ihre To-Do-Liste aufzunehmen ...

Der erste Teil ist das eigentliche Problem - die Teilnehmer des Überprüfungsprozesses müssen davon ausgehen, dass es einen Wert hat, sonst führen sie niemals eine Codeüberprüfung durch (die als nicht wertvoll angesehen wird), wenn sie Code schreiben oder Fehler beheben könnten (welche ist sicherlich wichtiger ...?).

Wenn Sie die beiden zusammenfügen können - erstens sicherstellen, dass jeder glaubt (oder versteht), dass die Codeüberprüfungen einen Wert haben -, sollte dies im Grunde genommen weniger Fehler bedeuten, was mehr neuen Code bedeutet, was normalerweise mehr Spaß macht - und zweitens arrangieren Dinge, so dass im Zeitplan ein freier Raum für die Codeüberprüfungen vorhanden ist, dann werden hoffentlich gute Dinge passieren ... es wird Teil der Kultur.

Sobald es Teil der Kultur ist, muss man vielleicht nicht mehr "jeden Tag als erstes" sagen - aber nachdem ich gesagt habe, dass es gut in das Muster passt, möchte man wahrscheinlich, dass ein Entwickler arbeitet.

Bei den meisten Unternehmen, für die ich gearbeitet habe, haben Sie 3 Tage Zeit, um eine Überprüfung abzuschließen. Es ist nicht akzeptabel, die Bewertungen nicht durchzuführen. Es ist Teil Ihres Jobs. Wenn Sie nicht rechtzeitig anständige Überprüfungen durchführen, wirkt sich dies auf Ihre Leistungsbeurteilung aus. Und ja, Bewertungen scheinen immer zu den ungünstigsten Zeiten zu erfolgen. Schade, lernen Sie, die Überprüfungszeit in Ihre Schätzungen einzubeziehen. Wenn das Management wirklich der Meinung ist, dass Überprüfungen wichtig sind (dh, dass der gesamte Code überprüft wird), würde es auf jeden Fall eine ähnliche Richtlinie verfolgen. Wenn Personen die Überprüfung nicht innerhalb der vorgegebenen Zeit abschließen, gilt dies auch als Akzeptanz des Materials.

Erwägen Sie die Verwendung eines Tools wie Review Board . Es ist sehr hilfreich, insbesondere für lange Bewertungen.

Sie können Ihre Diffs hochladen und warten, bis ein Prüfer seine Prüfung abgeschlossen hat. Wenn Sie offene Bewertungen haben, die Sie daran hindern, Ihre Arbeit fortzusetzen, können Sie dies während Ihrer täglichen Besprechungen melden (Ihr Team möchte, dass neue Funktionen eingecheckt werden, damit sie so schnell wie möglich getestet werden können, nicht wahr?).

Ein paar Punkte zum Hinzufügen, die nicht in den anderen Antworten enthalten sind.

Der zu überprüfende Code muss eingecheckt sein

• so dass Sie eine stabile Version überprüfen.
• Es kann sich im Hauptentwicklungszweig befinden, wenn ein Release weit genug entfernt ist
• Es kann auf Zweig sein, wenn es einen guten Grund gibt, Haupt nicht zu verschmutzen

Das Blockieren von Aufgaben hat Vorrang, daher sollten Codeüberprüfungen Vorrang vor anderen Arbeiten haben (aber versuchen, Ihren Fluss nicht zu unterbrechen). Als Entwickler sollten Sie möchten, dass andere Ihren Code überprüfen (um ihn zu verbessern). In diesem Wissen sollten Sie umgehend Überprüfungen für andere durchführen.

Die schwierigeren Fragen sind, wann und wie Codeüberprüfungen gut durchgeführt werden können.

Eine Regel, die für uns beim Zeitpunkt funktioniert hat, ist, dass gemeinsam genutzter Code überprüft werden muss, da er größere Auswirkungen hat, während er für Code für eine einzelne Anwendung (insbesondere angesichts der Verwendung einer testgetriebenen Entwicklung) weniger wichtig ist.