Saltar al contenido principal

Checklist universal de code smells

Checklist de Revisión

✅ Tamaño y Complejidad

  • ¿El método tiene más de 15-20 líneas?
  • ¿La clase tiene más de 200-300 líneas?
  • ¿Hay más de 3 niveles de anidamiento?
  • ¿Hay más de 4 parámetros?

✅ Claridad y Expresividad

  • ¿Los nombres reflejan el propósito?
  • ¿Hay comentarios explicando qué hace el código en vez de por qué?
  • ¿Hay números o strings mágicos?
  • ¿Es difícil entender el código en una primera lectura?

✅ Diseño y Estructura

  • ¿El código viola algún principio SOLID?
  • ¿Hay código duplicado o muy similar?
  • ¿Hay condicionales complejos que podrían reemplazarse con polimorfismo?
  • ¿Existen dependencias innecesarias entre componentes?

✅ Manejo de Errores

  • ¿Se usan códigos de retorno en lugar de excepciones/resultados?
  • ¿Hay lógica de error inconsistente?
  • ¿Se ignoran o suprimen errores?
  • ¿Se mezcla lógica de negocio con manejo de errores?