Revisión por pares: Difference between revisions
Jump to navigation
Jump to search
| Line 26: | Line 26: | ||
== Centrarse en lo importante == | == Centrarse en lo importante == | ||
Para que la revisión sea eficaz, conviene centrarse en los aspectos de mayor impacto: | Para que la revisión sea eficaz, conviene centrarse en los aspectos de mayor impacto: | ||
* '''Lógica de negocio:''' ¿la implementación cumple lo que la historia de usuario o la spec pedían? | |||
* '''Casos extremos:''' ¿el código maneja correctamente los casos límite y los errores? | |||
* '''Legibilidad:''' ¿el código es comprensible para cualquier miembro del equipo? | |||
* '''Seguridad:''' ¿hay vulnerabilidades evidentes? | |||
* '''Duplicación:''' ¿se repite código que podría abstraerse? | |||
Las herramientas de análisis estático automatizado pueden encargarse de los aspectos más mecánicos (formato, estilo, métricas de complejidad), liberando al revisor para centrarse en los aspectos que requieren juicio. | |||
== Preparar la revisión == | == Preparar la revisión == | ||
La efectividad de la revisión depende en parte de la preparación: | La efectividad de la revisión depende en parte de la preparación: | ||