¿Qué es una revisión de código (code review)?

Revisión de código: definición y finalidad

En una revisión de código, un segundo desarrollador lee los cambios de su compañero antes de que lleguen a la versión principal. Es el principio de los cuatro ojos aplicado al software. Se examinan la función, la comprensibilidad, la seguridad y la coherencia con las normas del proyecto: ¿resuelve el código la tarea y podrá alguien entenderlo y modificarlo dentro de un año?

Importa el efecto secundario: las revisiones reparten conocimiento. Si varias personas conocen cada cambio, el proyecto no depende de un único desarrollador. Además, son una ocasión para imponer estándares en común y orientar a los recién llegados. Los clientes se benefician indirectamente, con menos errores en producción.

Cómo se desarrolla una revisión de código

Lo habitual es el flujo mediante pull requests o merge requests en plataformas Git como GitHub o GitLab. El desarrollador describe su cambio, los compañeros comentan línea por línea, piden ajustes o aprueban el cambio. Solo tras la aprobación se incorpora a la rama principal.

Como complemento se ejecutan comprobaciones automáticas: los linters verifican el estilo y patrones de error típicos, las pruebas aseguran el comportamiento y los análisis estáticos buscan problemas de seguridad. Así, las personas no tienen que discutir sobre sangrías y espacios y pueden centrarse en la lógica, la arquitectura, la seguridad y los riesgos, donde su criterio es realmente necesario.

Ejemplo práctico: el cálculo de descuentos en una tienda online

Un comerciante hace incorporar a su tienda descuentos por volumen para clientes empresariales. El desarrollador implementa el cálculo y, en la revisión, a su compañero le llama la atención que con determinadas cantidades se redondea mal y que el descuento se aplica por error también a los gastos de envío. Ninguna de las dos cosas figura en la descripción de la tarea.

Ambas habrían pasado probablemente desapercibidas con comprobaciones por muestreo y solo se habrían notado por las reclamaciones de clientes. Así, la corrección se hace antes de la publicación y ningún cliente llega a ver un total erróneo. También en funciones de pago, comprobaciones de permisos e interfaces con sistemas de contabilidad merece la pena una segunda mirada, porque allí los errores salen caros.

Ventajas, límites y errores habituales

Las revisiones de código mejoran la calidad y la seguridad y la colaboración. Pero cuestan tiempo y valen tanto como el cuidado de los revisores; no sustituyen ni las pruebas ni la aceptación funcional. Además, pueden frenar a los equipos si las aprobaciones quedan días sin tramitar. Errores típicos:

• Revisar cambios enormes de golpe, de modo que solo se leen por encima
• Dar el visto bueno sin revisión real para cumplir el calendario
• Discutir solo cuestiones de estilo en lugar de lógica y seguridad
• Criticar a la persona en lugar del código, lo que daña la colaboración
• No tener reglas sobre qué debe comprobar una revisión

Qué debe tener en cuenta su empresa

Pregunte a los proveedores si cada cambio lo comprueba una segunda persona y qué comprobaciones automáticas se ejecutan además. Las áreas críticas, como el pago, el inicio de sesión y el acceso a datos, merecen una atención especial. En Nano Concepts, cada cambio pasa por una revisión y por comprobaciones automatizadas antes de llegar a su sistema.