Wann ist es richtig, den von einem Mitarbeiter geschriebenen Code zu bearbeiten? [Duplikat

Question

Diese Frage hat hier bereits eine Antwort:

• Ist Code -Besitz ein Code -Geruch? 5 Antworten

Für welche Themen ist es richtig, den von einem Kollegen geschriebenen Code zu bearbeiten? Für welche Themen ist es falsch?

Answer 1

Wenn es richtig ist

Wenn die Bearbeitung die Funktionalität des Programms verbessert oder den Code selbst lesbarer / wartbarer macht.

Wenn es falsch ist

Wenn die Bearbeitung die Funktionalität des Programms schadet oder keinen Zweck hat, außer dem Herausgeber mit Besetzungs zu versorgen.

Answer 2

Wer ist der Eigentümer des Code? Das Unternehmen oder der Entwickler?

Wenn der Eigentümer (wie üblich) das Unternehmen ist, sollte es keinen Unterschied in der Bearbeitungscode geben, die von Ihnen oder einem Kollegen verfasst wurde (es sei denn, eine bestimmte Code -Wartungsrichtlinie).

Answer 3

Das Bearbeiten eines anderen Code sollte stattfinden, wenn sich der Code entwickelt und als Praxis ermutigt werden, damit Sie eine kollektive Code -Besitzerschaft haben.

Der Code sollte dem gesamten Team gehören. Es ist eine schlechte Idee, Code zu haben, der nur von einem Entwickler bearbeitet wird (und daher nur wirklich verstanden wird). Wenn Sie nicht im Code sind und sich selbst ändern, verstehen Sie es wahrscheinlich nicht. Warten, bis der andere Entwickler das Unternehmen verlässt oder krank wird, ist die falsche Zeit, um den Code zu lernen.

Durch detaillierte Unit -Tests und die Verwendung der Quellvertretung sollten diese Praxis ein geringes Risiko bedeuten.

Ermutigen Sie das gesamte Team, Onkel Bobs "Boy Scout -Regel" zu folgen, und immer zu versuchen, den Code ein wenig sauberer einzuchecken, als Sie ihn ausgecheckt haben , unerbittliches Durcheinander.

Der einzige Nachteil dafür ist, wenn diese Praxis in Bearbeitungskriege verwandelt, bei denen die Entwickler sich ständig gegenseitig ändern. Dies ist wahrscheinlich ein Zeichen dafür, dass Sie sich nicht für einen Codierungsstandard für alles entschieden haben, was den Bearbeitungskrieg verursacht. Dies wäre also eine gute Gelegenheit, einen zu erstellen.

Answer 4

Ich würde sagen, es ist richtig, wenn das, woran Sie arbeiten, die Änderung erfordert. Es ist richtig, wenn Sie zugewiesen wurden, einen Fehler zu beheben, und der Fehler in diesem Code befindet.

Wenn der Code nichts mit dem zu tun hat, woran Sie arbeiten, aber repariert werden müssen, bringen Sie ihn an die Teamleiterin und lassen Sie ihn der Behebung einer Priorität zuweisen und eine Person (es ist möglicherweise nicht Sie), um zu reparieren es. Wenn er sagt, dass er Ihre Zeit nicht damit verbringen soll, dann wird es abgenutzt, um an diesem Punkt zu reparieren. Sie werden dafür bezahlt, an dem zu arbeiten, was Ihnen zugewiesen wurde, um nicht daran zu arbeiten, woran Sie arbeiten möchten. Auf jeden Fall geben Sie es an, dass es geändert werden muss, aber die Prioritätseinstellung ist die Aufgabe des Managers, nicht Ihre.

Es ist falsch, den Arbeitscode nur wegen einer persönlichen Präferenz zu ändern.

Es ist falsch, wenn Sie nicht verstehen, welches Problem der Code für die Behebung oder sogar verstehen, um den Code zu beheben oder sogar zu verstehen. Nur weil es für Ihre Bedürfnisse nicht funktioniert, heißt das nicht, dass Sie etwas nicht brechen, wenn Sie damit meiden. Wenn Sie es ändern müssen, um das zu machen, was Sie arbeiten, sprechen Sie mit dem Oreginal -Programmierer darüber, warum er getan hat, was er getan hat und was betroffen sein könnte, wenn Sie ihn ändern, um Ihre Bedürfnisse zu erfüllen. Ändern Sie niemals den Code, den Sie nicht verstehen, es sei denn, er funktioniert nicht. Dies bedeutet nicht, dass Sie den Code niemals beheben, sondern nur, dass Sie ihn nicht ändern, wenn Sie sich die Zeit nehmen, ihn gründlich zu verstehen, ihn nicht ändern, insbesondere wenn es sich bereits um Code in der Produktion handelt.

Answer 5

Wann ist es richtig, den von einem Mitarbeiter geschriebenen Code zu bearbeiten?

Dies hängt davon ab, in welcher Phase sich das Produkt befindet. Wenn das Produkt bereits stabil ist und bereits veröffentlicht wird, würde ich keine Änderungen vornehmen, die nicht benötigt werden, es sei denn, es wäre eine große Änderung für wirklich hässlichen Code erforderlich.

Für welche Themen ist es richtig, den von einem Kollegen geschriebenen Code zu bearbeiten?

Ich würde nicht über die Codierungsformatierung eingehen. Sie sollten nacharbeiten, wenn der Code übermäßig komplex ist und ohnehin geändert werden muss oder Probleme verursacht und nie richtig zu funktionieren scheint.

Answer 6

Wie Stackexchange oder Wikis sollten Sie bereit sein, Ihren Code zu teilen und jedem zu ermöglichen, ihn zu ändern, solange die Änderungen in das globale Design und die globalen Ziele passt. Aber stellen Sie sich eine Frage: Warum sollte jemand den Code eines anderen ändern müssen, wenn es nicht zum Debuggen ist? Haben Sie Ihr Projekt eindeutig ausreichend in Stücke geschnitten, um gleichzeitig eine Parallelisierung der Codierung durch mehrere Programmierer zu ermöglichen?

Wenn Sie gleichzeitig für unterschiedliche Funktionen gezwungen sind, denselben Code wie Ihr Mitarbeiter zu ändern, gibt es eine Code-Verstrickung und eine schlechte "Trennung von Bedenken". Sie sollten Ihr Design überarbeiten, damit jeder Programmierer an verschiedenen Teilen der Software -Code -Basis arbeiten kann.