"Reseñas de código"

Consejos de programación para #programadores.

-----------------------------------------------------------------------

LIBRO: 97 COSAS QUE TODO PROGRAMADOR NECESITA SABER.

POR: KEVLIN HENNEY.

-----------------------------------------------------------------------

14° "RESEÑAS DE CÓDIGO":

Deberías hacer revisiones de código. ¿Por qué? Porque aumentan la calidad del código y reducen la tasa de defectos. Pero, no necesariamente por las razones que podrías pensar, porque, es posible que anteriormente hayan tenido algunas malas experiencias con las revisiones de código. Muchos programadores tienden a no gustarles, he visto organizaciones que requieren que todo el código pase una revisión formal antes de implementarse en producción.

A menudo, es el arquitecto o un desarrollador principal quien realiza ésta revisión, una práctica que se puede describir como que el arquitecto revisa todo. Ésto se afirma en el manual de proceso de desarrollo de software de la empresa, por lo que los programadores debe cumplir.

Puede haber algunas organizaciones que necesiten un proceso tan rígido y formal, pero la mayoría no. En la mayoría de las organizaciones, éste enfoque es contraproducente.

Los revisores, pueden sentir que están siendo juzgados por una junta de libertad condicional. Los revisores necesitan tanto el tiempo para leer el código como el tiempo para mantenerse al día con todos los detalles del sistema; pueden convertirse rápidamente en el cuello de botella en este proceso, y el proceso pronto degenera.

En lugar de simplemente corregir errores en el código, el propósito de las revisiones de código debería ser compartir conocimientos y establecer pautas de codificación comunes. Compartir su código con otros programadores permite la propiedad colectiva del código. Deja que un miembro del equipo al azar analice el código con el resto del equipo en lugar de buscar errores. Debes revisar el código tratando de aprender y entenderlo.

Se amable durante las revisiones de código. Asegúrate de que los comentarios sean constructivos, y no cáusticos. Introduce diferentes roles para la reunión de revisión y evitar que la antigüedad en la organización entre los miembros del equipo afecte la revisión del código.

Ejemplos de funciones podría incluir que un revisor se centre en la documentación, otro en excepciones, y un tercero para ver la funcionalidad. Éste enfoque ayuda a distribuir la carga de la revisión entre los miembros del equipo.

Tenga un día de revisión de código regular cada semana. Pasar un par de horas en una revisión, "reunión". Rota al evaluado en cada reunión en un patrón rotatorio simple.

Recuerda cambiar los roles entre los miembros del equipo también en cada reunión de revisión. Involucra a los novatos en las revisiones de código. Pueden ser inexpertos, pero sus nuevos conocimientos universitarios pueden proporcionar una perspectiva diferente. Involucrar a expertos para su experiencia y conocimiento, identificarán el código propenso a errores más rápido y con más precisión.

Las revisiones de código fluirán más fácilmente si el equipo tiene convenciones de codificación que son verificadas por las herramientas. De esa manera, el formato del código nunca se discutirá durante la reunión de revisión del código.

Hacer que las revisiones de código sean divertidas es quizás el contribuyente más importante para el éxito. Las reseñas se refieren a las personas que las reseñan. Si la reunión de revisión es dolorosa o aburrida, será difícil motivar a alguien. Conviértalo en una revisión de código informal cuyo objetivo principal es compartir conocimientos entre los miembros del equipo. Deja comentarios sarcásticos afuera, y trae un pastel o un almuerzo en una bolsa marrón en su lugar.

-Mattias Karlsson-