Las revisiones de código son un paso clave en el ciclo de desarrollo para mejorar la calidad y mantenibilidad del código. Sin embargo, muchos equipos de ingeniería prescinden de ellas y se basan en pruebas automáticas o QA manual para validar el código antes del lanzamiento.

Las investigaciones demuestran que la inspección de diseño y código detecta muchos más defectos que las pruebas de software, como muestra el libro CodeComplete de Steve McConnel.

¿Cómo maximizar los beneficios de las revisiones de código, sobre todo en equipos distribuidos y asíncronos?

Primero, asegúrate de que tu equipo siga algunas prácticas probadas. Segundo, aprovecha herramientas modernas para facilitar la comunicación colaborativa asíncrona. Entremos en materia:

Cómo hacer una revisión de código: 5 consejos y buenas prácticas

No hace falta reinventar la rueda. Construye una base sólida empezando por estas buenas prácticas:

1. Elabora una lista de verificación para la revisión de código

Define criterios que tu equipo debe seguir en cada revisión de código. Debe incluir lo siguiente:

Elimina comentarios redundantes para mejorar la legibilidad.

• Sigue el principio KISS.
• Comprueba que el nuevo código cumpla tu guía de estilo.
• Aborda las vulnerabilidades recientes para garantizar la seguridad de la aplicación.
• Usa encapsulación y modularidad para separar responsabilidades.
• Asegúrate de que la cobertura de pruebas sea suficiente.
• Cuando tenga sentido, aplica el principio DRY: usa componentes, funciones y servicios reutilizables tanto como sea posible.

2. Establece límite de tiempo y de líneas de código (LoC)

Nuestra atención decae tras 60 min, por lo que es mejor hacer sesiones cortas de revisión de código con más frecuencia. Además, el tiempo reducido implica fijar un límite de LoC por sesión (p. ej., 200-400 líneas). De hecho, un informe de Cisco encontró que la capacidad de los programadores para detectar problemas disminuye después de 200 líneas.

Agrupar demasiados cambios en una sola revisión puede afectar la calidad del resultado. Investigación sobre revisión de código entre pares en desarrollo distribuido de software encontró que un aumento en el tamaño del parche reduce la efectividad del proceso de revisión. Los revisores participan menos, aportan menos feedback y tienden a pasar por alto casos límite.

Por último, no precipites el proceso. Un estudio encontró que los revisores que procesan menos de 400 líneas por hora tienen una habilidad superior a la media para encontrar errores. Mientras que quienes avanzan a más de 450 líneas/h alcanzan una densidad de defectos inferior a la media.

3. Mide el progreso con las métricas adecuadas

Lo que se mide, se hace: mediciones objetivas te ayudan a seguir la eficiencia y efectividad de las revisiones de código, analizar su aporte al proceso de desarrollo y estimar el tiempo necesario para un proyecto. Estas son las métricas clave y su significado:

Tasa de inspección: tiempo que tarda tu equipo en revisar una cantidad de código. Puede ayudarte a evaluar la legibilidad del código.

• Tasa de defectos: frecuencia con que tu equipo detecta un error. Muestra la eficacia de tu método de revisión.
• Densidad de defectos: número de errores en cierta cantidad de código. Ayuda a identificar componentes vulnerables y asignarles más recursos.

4. Ofrece feedback objetivo y constructivo

Las revisiones de código no solo sirven para señalar lo que hay que corregir. Los revisores deben crear un entorno colaborativo y de apoyo, dedicando tiempo a explicar el motivo de cada solicitud o sugerencia. También deben formular preguntas abiertas para fomentar la discusión y el intercambio de conocimientos.

Una explicación detallada agiliza el proceso al evitar idas y venidas innecesarias entre autor y revisor.

5. Configura pipelines de CI/CD robustos

Errar es humano. Incluso los mejores programadores cometen errores y los revisores más meticulosos pasan por alto fallos. Como parte de tu pipeline de integración continua/entrega continua (CI/CD), una revisión sistemática incluye pruebas automáticas y controles de seguridad para garantizar un producto de alta calidad.

Mejores revisiones de código para equipos distribuidos

Implementar buenas prácticas de revisión de código suele ser más difícil en equipos distribuidos por la falta de canales de comunicación efectivos:

Compartir conocimientos por escrito, por ejemplo por email, puede convertir algo fácil de señalar en pantalla en una novela. A menudo se pierden matices y las respuestas posteriores pasan desapercibidas entre hilos interminables de “responder a todos”.

• ¿Y las capturas de pantalla? Ahorran escribir párrafos al describir elementos visuales, pero la imagen estática no muestra cómo funcionan los componentes interactivos en todo el flujo.
• El chat o mensajería directa aporta interactividad y fomenta el debate, pero la falta de estructura puede generar mucho ir y venir. Además, en la mayoría de plataformas de mensajería directa es fácil perder mensajes o hilos, lo que dificulta el seguimiento.
• Las videollamadas permiten el intercambio en tiempo real, pero coordinar horarios puede ser un dolor. Las reuniones de video largas agotan al equipo e afectan su capacidad para detectar problemas.

Además, las reuniones pueden no ser la forma más rentable de revisar código. Un estudio encontró que los equipos de desarrollo pasan el 75 % de su tiempo en reuniones y el 25 % leyendo. Sin embargo, la lectura detecta el 80% de los defectos y los revisores son 12 veces más eficientes identificando problemas leyendo que en reuniones.

Cómo realizar revisiones de código efectivas en equipos asíncronos

Un proceso de revisión efectivo debe dar a los revisores más tiempo para examinar el código y herramientas para comunicar sus hallazgos de forma asíncrona. Así puedes recrear la inmediatez de las interacciones presenciales sin reuniones infinitas:

Aplica contexto a la conversación

Los comentarios escritos suelen centrarse en cambios individuales y no animan al equipo a dar un paso atrás para valorar el impacto en el sistema, tareas futuras y mantenimiento.

La naturaleza lineal de la mayoría de métodos asíncronos limita el alcance de las discusiones, dificultando integrar el contexto general y destacar cada punto. Si te desvías a otro tema, corres el riesgo de perder el hilo del asunto original.

Bubbles facilita añadir jerarquía y contexto a tu revisión de código guiando a tu equipo mediante compartición de pantalla durante tus comentarios. Puedes cambiar entre distintas pestañas para abordar detalles del código y mostrar cómo encaja en el panorama general sin perder la línea en cuestión.

Los miembros pueden añadir sus comentarios con marca de tiempo para indicar a qué se refieren y debatir aspectos del contexto general en un hilo relacionado, sin que todos se pregunten “¿cómo llegamos aquí y cómo se relaciona esto con esa línea de código?” tras unos intercambios.

Convierte una revisión unidireccional en una colaboración bidireccional

Con decenas de comentarios en cientos de líneas de código, ¿cómo gestionar varios hilos de conversación de forma eficaz y eficiente para cubrir todo?

Con Bubbles puedes dejar comentarios independientes, de modo que cada uno se refiera a una línea de código o problema concreto en el video. Los miembros pueden discutir cada punto y hacer seguimiento para resolverlo sin recorrer una larga lista de feedback en un email o documento.

Cada elemento puede abrir su propia discusión si es necesario para mantener los comentarios manejables. Puedes etiquetar a los miembros adecuados para que entren en la conversación, en lugar de que todos lean un extenso hilo de “responder a todos” que provoca confusión y frustración (seamos sinceros: ¿quién lee todo?).

Haz el proceso personal, respetuoso y conversacional

Los comentarios percibidos como hostiles o irrespetuosos pueden afectar cómo los miembros reciben el feedback y su moral. Creemos que nadie quiere ser agresivo o impulsivo a propósito, pero la mayoría de métodos de feedback no dan espacio para que los revisores sean reflexivos y empáticos.

El feedback asíncrono por herramienta de revisión, email o captura permite a los miembros revisar todo antes de comentar. Pero carecen de la inmediatez para transmitir matices gestuales y tonales, lo que facilita que la audiencia malinterprete el tono e intención del comentario.

En cambio, las reuniones sincrónicas (p. ej., videollamadas) ayudan a captar los matices de los comentarios. Pero la presión de responder al instante puede volver a las personas reactivas, de modo que la conversación se rija por el primer pensamiento en lugar del mejor.

¿Cómo obtener lo mejor de ambos mundos? Crea una burbuja para capturar contexto y matices compartiendo tu feedback con video en nuestros gestores. Mientras tanto, la comunicación asíncrona permite ver el video completo y asimilar los comentarios antes de responder, para compartir el mejor pensamiento en lugar de lo primero que venga a la mente.

Implementa buenas prácticas de revisión de código a escala

La comunicación asíncrona permite a los equipos revisar código de forma eficiente. Con las herramientas adecuadas, pueden revisar más código en menos tiempo y colaborar constructivamente para abordar el contexto general y mejorar la base de código con el tiempo.

Bubbles te ayuda a mejorar las revisiones asíncronas organizando los hilos, minimizando malentendidos y fomentando la colaboración bidireccional, para que tu equipo integre feedback de forma continua y cree un mejor producto.

No hay excusa para no crear una Bubble hoy: no necesitas registrarte para empezar y tu equipo solo requiere un navegador para verla. Solo tienes que hacer clic en el enlace y grabar tu primera Bubble. Comienza con Bubbles descargando nuestra extensión para Chrome y comprueba lo fácil que es realizar una revisión de código efectiva.

‍

‍