Code review en Java: qué revisar y cómo dar retroalimentación a principiantes

Code review en Java: qué revisar y cómo dar retroalimentación

El code review (revisión de código) no es solo una formalidad antes de fusionar un pull request. Es una herramienta poderosa para mejorar la calidad del código, compartir conocimientos y prevenir errores. Para los desarrolladores Java principiantes, participar en una revisión a menudo genera miedo: "¿Y si me critican?", "¿Y si paso por alto un error crítico?". En este artículo, analizaremos en qué fijarse al revisar código Java y cómo dar retroalimentación que ayude, no que desmotive.

1. Qué revisar en código Java: lista de verificación para principiantes

El primer paso, y el más importante, es saber qué buscar. No intentes encontrar todo de una vez. Concéntrate en las áreas clave. Aquí tienes una lista de verificación básica para el code review en Java:

• Lógica y corrección: ¿El código resuelve la tarea planteada? ¿Hay errores lógicos evidentes? ¿Se manejan correctamente los casos límite?
• Naming (nombres): ¿Los nombres de variables, métodos y clases cumplen con las convenciones de Java (camelCase, PascalCase)? ¿Son descriptivos? int a es malo, int numberOfUsers es bueno.
• Manejo de errores: ¿Hay bloques try-catch? ¿Se usan tipos de excepción específicos en lugar del genérico Exception? ¿No se tragan las excepciones en silencio (catch vacío)?
• Rendimiento: ¿Hay problemas evidentes, como crear objetos dentro de bucles, usar concatenación de cadenas con + en un bucle (mejor usar StringBuilder), o llamadas a métodos innecesarias?
• Pruebas: ¿El código está cubierto por pruebas unitarias? ¿Prueban solo escenarios positivos o también negativos y casos límite?

Ejemplo de código malo que debería notarse en la revisión:

public void processData(List data) {    for (Object item : data) {        String s = "";        for (int i = 0; i < 1000; i++) {            s += item.toString() + i; // Problema potencial de rendimiento        }        System.out.println(s);    }}

¿Qué está mal aquí? Usar + en un bucle crea muchos objetos String intermedios. Es mejor reemplazarlo por StringBuilder. También falta una verificación de data para null.

2. Cómo dar retroalimentación constructiva

El aspecto más difícil del code review no es la revisión técnica, sino la comunicación. Tu objetivo no es demostrar quién es más inteligente, sino ayudar al equipo a crecer. Aquí tienes algunas reglas:

• Enfócate en el código, no en el autor. En lugar de "Escribiste un código horrible", di "Este método se puede simplificar usando Stream API".
• Usa la forma interrogativa. En lugar de afirmaciones, haz preguntas. Ejemplo: "¿Por qué se eligió ArrayList aquí y no LinkedList? Dado que hay inserciones frecuentes al inicio de la lista, LinkedList podría ser más eficiente".
• Explica el "por qué". Un simple "Corrige esto" no funciona. Escribe: "Cambiemos == por .equals() para comparar cadenas, porque == compara referencias, no el contenido".
• Elogia el buen código. Si ves una solución elegante, menciónalo. "Excelente uso del patrón Builder aquí: el código se ha vuelto mucho más legible".

Ejemplo de mal comentario: "Este código es una porquería, reescríbelo". Ejemplo de buen comentario: "Me parece que este bloque se podría extraer a un método privado separado para mejorar la legibilidad del método principal. ¿Qué opinas?"

3. Errores típicos de principiantes en Java y cómo señalarlos

Conociendo los errores típicos, podrás encontrarlos más rápido y dar consejos más relevantes. Aquí están los 3 errores principales que suelen aparecer en las revisiones de desarrolladores Java principiantes:

Error 1: Ignorar los contratos de equals() y hashCode()