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

Introducción: ¿por qué es necesario el code review en TypeScript?

Code review (revisión de código) es el proceso de verificar el código escrito por otros desarrolladores. Para los desarrolladores principiantes de TypeScript, esto no es solo una formalidad, sino una poderosa herramienta de aprendizaje. Una revisión adecuada ayuda a detectar errores en etapas tempranas, mejorar la arquitectura y mantener un estilo uniforme en el equipo.

En este artículo, analizaremos en qué aspectos específicos hay que fijarse durante un code review en TypeScript, cómo dar retroalimentación para que sea útil, y proporcionaremos ejemplos de código. El material está orientado a principiantes que recién comienzan a trabajar con TypeScript y quieren mejorar sus habilidades de revisión.

1. Tipado: la base de TypeScript

Lo primero que hay que revisar en TypeScript es la corrección de los tipos. TypeScript añade tipado estático, lo que reduce la cantidad de errores, pero solo si los tipos se usan correctamente.

Qué revisar:

• Tipos explícitos para parámetros y valores de retorno. Evite any en todo lugar donde sea posible.
• Uso de interfaces y tipos en lugar de primitivos para estructuras complejas.
• Uso correcto de tipos Union e Intersection.
• Ausencia de aserciones de tipo (type assertions) innecesarias.

// Mal: tipo implícito y anyfunction processData(data: any) {  return data.name;}

// Bien: interfaz explícitainterface User {  name: string;  age: number;}

function processData(data: User): string {  return data.name;}

Al revisar, preste atención a los lugares donde el tipo any podría ser reemplazado por uno más específico. Esto mejora inmediatamente la legibilidad y seguridad del código.

2. Verificación de null y undefined

TypeScript ayuda a evitar errores con null y undefined mediante las comprobaciones estrictas de null (strict null checks). Pero los desarrolladores a menudo olvidan manejar los valores opcionales.

Qué revisar:

• Uso de ? para propiedades y parámetros opcionales.
• Verificación de null antes de acceder a propiedades.
• Uso de los operadores ?? y ?..

// Mal: sin protección contra undefinedfunction getLength(str: string | undefined): number {  return str.length; // Error: str puede ser undefined}

// Bien: usando optional chaining y nullish coalescingfunction getLength(str: string | undefined): number {  return str?.length ?? 0;}

En el code review, verifique que todos los valores potencialmente vacíos estén manejados. Esta es una causa frecuente de errores en producción.

3. Limpieza del código y legibilidad

TypeScript es una extensión de JavaScript, por lo que los principios de código limpio siguen siendo relevantes. Un buen code review incluye la verificación del estilo y la estructura.

Qué revisar:

• Nombres de variables y funciones. Los nombres deben ser significativos.
• Longitud de las funciones. Si una función hace más de una cosa, divídala.
• Comentarios. Deben explicar el "por qué", no el "qué".
• Ausencia de números mágicos. Extraiga las constantes a variables separadas.

// Mal: números mágicos y nombre poco clarofunction calc(a: number) {  return a * 1.2;}

// Bien: constante y nombre comprensibleconst TAX_RATE = 0.2;

function calculateTotalWithTax(amount: number): number {  return amount * (1 + TAX_RATE);}

Al revisar, pregúntese: "¿Podrá un nuevo desarrollador entender este código sin explicaciones?" Si no es así, solicite mejorarlo.

4. Cómo dar retroalimentación: con suavidad y de forma constructiva

El code review no es un examen, sino un diálogo. Para los principiantes, es especialmente importante recibir retroalimentación sin críticas personales. Aquí hay algunas reglas que ayudarán a dar comentarios útiles.