Frage

Nehmen wir an, Sie arbeiten an einem Ort, an dem jede Änderung am Quellcode mit einem Fehlerbericht oder einer Funktionsanforderung verbunden sein muss und es keine Möglichkeit gibt, diese Richtlinie zu reformieren.Wie geht man in einer solchen Umgebung am besten mit Code-Refactorings um (also Änderungen, die den Code verbessern, aber keinen Fehler beheben oder keine Funktion hinzufügen)?

  • Schreiben Sie einen Fehlerbericht und verknüpfen Sie das Refactoring damit.
  • Schreiben Sie eine Funktionsanfrage und verknüpfen Sie das Refactoring damit.
  • Machen Sie mit bei den Refactorings, während Sie an Code arbeiten, der mit einem Fehlerbericht/einer Funktionsanforderung verknüpft ist.
  • Führen Sie einfach kein Refactoring durch.
  • Andere

Beachten Sie, dass alle Fehlerberichte und Funktionsbeschreibungen für Manager und Kunden sichtbar sind.

War es hilfreich?

Lösung

Ich stimme für den „Sneak in Refactorings“ -Ansatz, das ist, glaube ich, ist die Art und Weise Refactoring in erster Linie getan werden soll. Es ist wahrscheinlich eine schlechte Idee, nur um von Refactoring „den Code aufzuräumen.“ Das bedeutet, dass Sie die Änderungen für keinen wirklichen Grund. Refactoring ist per definitionem das Modifizieren ohne die Absicht der Fehler zu beheben oder das Hinzufügen von Funktionen. Wenn Sie das KISS-Prinzip sind folgende, jedes neue Merkmal wird zumindest einige Refactoring brauchen, weil Sie nicht wirklich darüber nachzudenken, wie die meisten erweiterbares System möglich das erste Mal, um zu machen.

Andere Tipps

Wenn Sie auf einem Code-Block arbeiten, in den meisten Fällen das ist, weil es entweder ein Bug-Fix oder ein neue Funktion, die erfordert, dass Code-Block zu ändern, und das Refactoring ist entweder vor der Änderung, um es zu machen, einfacher, oder nach dem Wechsel das Ergebnis aufzuräumen. In jedem Fall können Sie den Refactoring mit der Bug-Fix oder Funktion in Verbindung bringen.

Die Art, wie wir es Arbeit ist: Es muss ein guter Grund sein, den Code Refactoring, warum sonst

Wenn der Grund ist ein weiteres Merkmal zu ermöglichen, den gleichen Code zu verwenden, verbinden Sie die Änderungen die Aufforderung der anderen Funktion.

Wenn es etwas schneller zu machen, ein Feature-Request für eine schnellere ‚xyz‘ erstellen und die Änderungen mit, dass assoziieren -. Dann werden die Kunden sehen Sie verbessern das Produkt

Wenn es einen Fehler entwerfen aus, melden Sie den Fehler.

Es ist erwähnenswert, dass in meiner Umgebung kann die Politik nicht durchgesetzt werden. Aber kluge Manager können Berichte über Änderungen erhalten und, wenn sie nicht einen Fehler \ Anfragereferenz in der Commit-Text haben wird es weiterverfolgt.

Hier kann bei jeder Option einen Blick:

  • Schreiben Sie einen Bug-Report und ordnen Sie den Refactoring mit.

Wenn Sie das Gefühl, dass, Ihrer Meinung nach, der ursprüngliche Code ein Sicherheitsrisiko oder Potenzial für Absturz oder Instabilität darstellt. Schreiben Sie einen kleinen Bericht Fehler umreißt die Gefahr, und dann beheben.

  • Schreiben Sie eine feature-Anfrage und assoziiert das Refactoring mit.

Es könnte schwieriger zu Reaktor-Code auf einer Feature-Anfrage basiert. Aber Sie könnten gültige Feature-Anfrage, dies zu tun, das führt mich auf den nächsten Punkt ...

  • Sneak in dem Refactorings, während auf Code arbeiten, die mit einer Bug-Report / Feature-Anforderung zugeordnet ist.

Wenn es eine gültige Bug oder Feature ist, Zustand, die x-Funktion hatte etwas ändern, um die Fehler zu beheben oder das Feature hinzuzufügen.

  • Nur tun Sie kein Refactoring.

Dies scheint die Selbstentwicklung zu deuten darauf hin, durch eine Anwendung zu verbessern ist nicht erlaubt. Entwickler sollten erlaubt sein, wenn nicht, ermutigen, neue Techniken und Technologien Explorer.

  • Andere

Vielleicht könnten Sie Ihre Verbesserung bei entsprechender Sitzung, so dass überzeugenden Gründe diskutieren, warum die Änderungen vorgenommen werden sollen. Dann zumindest werden Sie Management-Unterstützung auf die Änderung haben, ohne den Code durch eine andere Methode schleichen zu müssen.

  • Andere

Wenn Sie an einem Ort mit dieser Art von unflexibel (und lächerlich) Politik arbeiten, die beste Lösung ist, einen anderen Job zu finden!

Lizenziert unter: CC-BY-SA mit Zuschreibung
Nicht verbunden mit StackOverflow
scroll top