Quand est-il droit de modifier le code écrit par un collègue? [dupliquer]

Question

    

Cette question a déjà une réponse ici:

    

        
•              Est Code de la propriété une odeur de code?                                      5 réponses                          
    

    

Pour quelles questions est-il droit de modifier le code écrit par un collègue? Pour ce qui est des questions mal?

Answer 1

Quand il est juste

Lorsque la modification améliore la fonctionnalité du programme ou rend le code plus lisible lui-même / maintenable.

Quand il est faux

Lorsque la modification nuit à la fonctionnalité du programme ou ne sert à rien à part de fournir l'éditeur avec busywork.

Answer 2

Qui est le propriétaire du code? L'entreprise ou le développeur?

Si le propriétaire est (comme d'habitude) la société, il devrait y avoir aucune différence dans l'édition du code écrit par vous ou par un collègue de travail (sauf si une politique de maintenance du code spécifique).

Answer 3

Modification d'un code de l'autre devrait se produire selon l'évolution du code et devrait être encouragée comme une pratique afin que vous ayez la propriété collective du code.

Le code devrait appartenir à toute l'équipe; ayant un code qui n'édité par (et donc seulement vraiment compris par) un développeur est une mauvaise idée. Si vous n'êtes pas dans le code prise vous change, vous ne probablement comprenez pas. Attendre que les autres feuilles de développement de l'entreprise ou tombe malade est le mauvais moment pour apprendre le code.

Avoir des tests unitaires détaillés en place et en utilisant le contrôle de la source devrait signifier cette pratique est faible risque.

Encourager toute l'équipe à suivre « la règle de boy-scout » Oncle Bob et essayer de toujours vérifier dans le code un peu plus propre que son extraction est une bonne façon de faire en sorte que les séjours de code propre au fil du temps et ne est pas dévolu dans un désordre illisible, ingérable.

Le seul inconvénient est que cette pratique tourne dans les guerres d'édition, où les développeurs sont constamment revenir mutuellement à des changements. Ceci est probablement un signe que vous ne l'avez pas décidé d'une norme de codage pour tout ce qui est à l'origine de la guerre d'édition, alors ce serait une bonne occasion de créer un.

Answer 4

Je dirais qu'il est juste quand ce que vous travaillez nécessite le changement. Il est juste quand vous avez été affecté pour corriger un bug et le bug est dans ce code.

Si le code n'a rien à voir avec ce que vous travaillez, mais doit être fixé, puis la mettre à la tête de l'équipe et lui permettent d'attribuer une priorité à fixer et assigner une personne (il ne peut pas être vous ) réparer. S'il dit de ne pas passer votre temps là-dessus, il est worng de fixer à ce moment-là. Vous êtes payé pour travailler sur ce que vous avez été affecté à travailler sur pas ce que vous voulez travailler. Par tous les moyens mettre en place qu'il doit être changé, mais l'établissement des priorités est le travail du gestionnaire ne vous appartient pas.

Il est erroné de changer le code de travail juste à cause de la préférence personnelle.

Il est faux quand vous ne comprenez pas ce problème, le code a été conçu pour fixer ou même comprendre le code. Tout simplement parce que cela ne fonctionne pas pour vos besoins ne signifie pas que vous ne serez pas casser quelque chose si vous muck avec elle. Si vous avez besoin de changer pour faire ce que vous faites le travail, puis allez parler au programmeur oreginal pourquoi il a fait ce qu'il a fait et ce qui pourrait être affecté si vous changez pour répondre à vos besoins. NE JAMAIS code de changement que vous ne comprenez pas à moins que cela ne fonctionne pas. Cela ne signifie pas que vous ne jamais corriger le code, cela signifie simplement que jusqu'à vous prenez le temps de le comprendre à fond, vous ne changez pas, surtout si elle est un code déjà en production.

Answer 5

  

Quand est-il droit de modifier le code écrit par un collègue de travail?

Cela dépend de quelle phase le produit est. Si le produit est déjà stable et déjà sorti, je ne ferais pas de changements qui ne sont pas nécessaires, à moins qu'il y avait un grand changement nécessaire au code vraiment moche.

  

Pour quelles questions est-il droit de modifier le code écrit par un collègue?

Je ne pinailler sur le codage du formatage. Vous devez refactoring lorsque le code est trop complexe et doit être changé de toute façon ou est à l'origine des problèmes et ne semble fonctionner à droite.

Answer 6

Comme StackExchange ou les wikis, vous devriez être prêt à partager votre code et permettre à quiconque de le modifier tant que les changements dans la conception unique globale et les objectifs. Mais vous poser une question: pourquoi quelqu'un devrait avoir à modifier le code de quelqu'un d'autre si ce n'est pas pour le débogage? Avez-vous coupé clairement votre projet en morceaux suffisamment pour permettre parallélisation de codage par plusieurs programmeurs en même temps?

Si, en même temps, pour des fonctionnalités différentes, vous êtes obligé de modifier le même code que votre collègue, il y a un enchevêtrement de code et mauvais « séparation des préoccupations ». Vous devriez revoir votre conception pour permettre à chaque programmeur de travailler sur différentes parties de la base de code logiciel.