
Desde hace ya bastante tiempo comencé a utilizar conventional commits en mis proyectos y también en mis clases. El año pasadomientras corregía una entrega de los alumnos se me ocurrió utilizar una estrategia similar para las observaciones en las revisiones de código.
A mi parecer las observaciones más comunes que veo en el código de los estudiantes se pueden agrupar en las siguientes categorías:
- Naming (n): nombres inapropiados, poco representativos, confusos, etc
- Programación (p): métodos muy largos, números mágicos, indentación no uniforme, etc
- Encapsulamiento (e): elementos públicos que deberían ser privados, acceso a miembros de otros objetos, etc.
- Modelado (m): conceptos del dominio del problema representados en el código de forma inapropiada
- Separación de incumbencias (s): clases con varias responsabilidades, poco cohesivas, mezcla de incumbencias de dominio con incumbencias accidentales, etc
Entonces a partir de esto, es que cada comentario que hago en la revisión le agrego el prefijo correspondiente.
La idea de este experimento es poder extraer información sobre las observaciones más habituales en el código de los alumnos para luego poder accionar al respecto.

