Pregunta

Esta pregunta ya tiene una respuesta aquí:

Para qué temas es lo correcto para editar el código escrito por un compañero de trabajo? Para lo cual los problemas ¿Está mal?

¿Fue útil?

Solución

Cuando correcto

Cuando la edición mejora la funcionalidad del programa o hace que el código en sí mismo / mantenible más legible.

Cuando está mal

Cuando la edición perjudica la funcionalidad del programa o no sirve para nada aparte de proporcionar el director con el trabajo no productivo.

Otros consejos

¿Quién es el propietario del código? La empresa o el desarrollador?

Si el propietario es (como siempre) la empresa, no debería haber ninguna diferencia en la edición de código escrito por usted o por un compañero de trabajo (a menos que una política de mantenimiento código específico).

Edición uno del otro código debe estar sucediendo a medida que evoluciona de código y debe fomentarse como una práctica para que tenga la propiedad colectiva del código.

El código debe ser propiedad de todo el equipo; tener código que sólo está editado por (y por lo tanto sólo es realmente entendido por) un desarrollador es una mala idea. Si no está en el código de hacer cambios a sí mismo, es probable que no entiende. Esperar hasta que las otras hojas de desarrolladores de la empresa o se enferma es un mal momento para aprender el código.

Tener pruebas de unidad que se detallan en el lugar y el uso de control de la fuente debe significar esta práctica es de bajo riesgo.

Fomento de todo el equipo para seguir "regla de Boy Scout" de tío Bob y siempre tratando de comprobar en el código un poco más limpia de lo que comprobamos es una buena manera de asegurar que las estancias código limpio con el tiempo y no lo hace incumbencia en un lío ilegible, imposible de mantener.

La única desventaja de esto es que si esta práctica se convierte en guerras de edición, donde los desarrolladores están constantemente revirtiendo los unos las modificaciones. Esta es probablemente una señal de que no haya decidido sobre un estándar de codificación de lo que está causando la guerra de ediciones, por lo que esta sería una oportunidad buena para crear una.

Yo diría que es correcto cuando lo que se está trabajando requiere el cambio. Es justo cuando se le ha asignado para corregir un error, y el error es en ese código.

Si el código no tiene nada que ver con lo que se está trabajando, pero tiene que ser fijo, y luego llevarlo hasta el líder del equipo y dejar que le asigne una prioridad para su fijación y asignar a una persona (que puede no estar ) arreglarlo. Si dice que no gastar su tiempo en él, entonces se worng para fijar en ese punto. Que se les paga a trabajar en lo que le ha sido asignado para trabajar en lo que no quiere trabajar. Por todos los medios sacar el tema que necesita ser cambiado, pero el establecimiento de prioridades es el trabajo del administrador no la suya.

Es un error el código de trabajo el cambio sólo a causa de una preferencia personal.

Está mal cuando no entiende cuál es el problema del código fue diseñado para fijar o incluso entender el código. El hecho de que no funciona para sus necesidades no significa que no va a romper algo si lodo con él. Si necesita cambiar para hacer lo que está haciendo el trabajo, a continuación, ir a hablar con el programador oreginal acerca de por qué hizo lo que hizo y lo que podría verse afectada si cambia para satisfacer sus necesidades. NUNCA código de cambio que no entiende a menos que no está funcionando. Esto no quiere decir que no siempre arreglar el código, sólo significa que hasta que se tome el tiempo para entender a fondo, no lo cambie, sobre todo si se trata de código ya está en producción.

¿Cuándo es adecuado para editar el código escrito por un compañero de trabajo?

Esto depende de en qué fase se encuentra en el producto. Si el producto ya es estable y ya liberado no haría ningún cambio que no son necesarios a menos que hubiera un gran cambio necesitaba código muy feo.

Para qué temas es lo correcto para editar el código escrito por un compañero de trabajo?

Me no nitpick acerca de la codificación de formatear. Usted debe refactorizar cuando el código es demasiado complejo y necesita ser cambiado o de todas formas está causando problemas y nunca parece correcto trabajo.

Al igual que StackExchange o wikis, usted debe estar dispuesto a compartir su código y permitir que cualquiera pueda modificarlo, siempre y cuando los ataques cambios en el diseño y los objetivos globales. Pero hacerse una pregunta: ¿por qué alguien tiene que modificar el código de otra persona si no es para la depuración? Tiene cortar claramente su proyecto en trozos suficientemente para permitir paralelización de codificación a través de múltiples programadores al mismo tiempo?

Si, al mismo tiempo, para diferentes funcionalidades, que se ven obligados a modificar el mismo código que su compañero de trabajo, hay un entrelazamiento de código y malo "separación de intereses". Usted debe revisar su diseño para permitir que cada programador para trabajar en diferentes partes de la base de código de software.

Licenciado bajo: CC-BY-SA con atribución
scroll top